Skip to content

Some build suggestions #61

Closed
Closed
Some build suggestions#61

Description

@kuraga
kuraga
opened on Jan 30, 2015

Good day, @etr !

Thanks for your work! I would use libhttpserver in my ARM-uClibc-Linux embedded system.

I haven't used the library yet but I have compiled it.
And have some suggestions and notes.

  • 1. Your last released Git tag is v0.7.1 (on 21 Mar 2014). Release new tags, please because of 0.8.0 version is pointed in docs.
  • 2. Suggestion: to introduce --disable-doc flag as a shortcut for --disable-doxygen-doc --disable-doxygen-dot --disable-doxygen-man --disable-doxygen-html flags collection.
  • 3. Suggestion: to introduce --disable-examples flag to disable examples compilation.
  • 4. Suggestion: to introduce --disable-tests (and --disable-unit-tests, --disable-integ-tests?) flag to disable tests compilation.
  • 5. You use PKG_CHECK_MODULES autoconf's macros and -lmicrohttpd linker's flag for libmicrohttpd using. But what if I want to use a static version of libmicrohttpd library or its source code instead?
  • 6. libmicrohttpd library can be compiled without some features. dauth and bauth is just examples. But your library references to these functions in code. I think we should use conditional compilation (and configuration flag) instead. There are some lines of code for dauth and bauth only.
  • 7. ./configure checks C++11 but it seems that USE_CPP_11 flag isn't used.

I would appreciate an answer for each of them.
Thanks very much.

Activity

  1. etr commented on Feb 1, 2015

    @etr
    etr
    on Feb 1, 2015
    Owner
    1. Definitely right - I am working on a refactor of some of the api. A new tag will come out pretty soon then.
      2),3),4) Good Ideas. I like in general the approach to allow people to disable features from compilation that they are not interested into.
    2. The current version comes from my habit of dynamic link everything I build - get the point that it might be useful to be able to compile the lib against static libmicrohttpd versions or source code.
    3. As per the compilation options, I believe that people should not pay for features they do not want to use - so I think this is a good idea.

    This said, my current priority for the library (and what I am spending my time on) is to improve the readability of the code, eliminating some over-complicated code and simplifying logical branches and secondary to improve the test coverage.

    All you suggested seems really useful though - would you be happy to implement it?

  2. kuraga commented on Feb 2, 2015

    @kuraga
    kuraga
    on Feb 2, 2015
    Author
    1. The problem is not to pay for features. The problem is: I have undefined reference errors and can't compile libhttpserver at all if libmicrohttpd has been compiled using --disable-bauth or --disable-dauth. Note: I has been switched to static linking of libmicrohttpd in makefile manually.

  3. kuraga commented on Feb 2, 2015

    @kuraga
    kuraga
    on Feb 2, 2015
    Author

    @etr , thanks for the replies! Some things are 'really useful' but needless for my project (well I really can write four flags instead of one 😄 ). Sure I would be happy to implement this (and other) but if you will discuss and merge.

  4. etr commented on Feb 3, 2015

    @etr
    etr
    on Feb 3, 2015
    Owner

    I will be more than happy to discuss and merge :)

  5. LeSpocky commented on Jan 19, 2022

    @LeSpocky
    LeSpocky
    on Jan 19, 2022
    Contributor
    1. The problem is not to pay for features. The problem is: I have undefined reference errors and can't compile libhttpserver at all if libmicrohttpd has been compiled using --disable-bauth or --disable-dauth. Note: I has been switched to static linking of libmicrohttpd in makefile manually.

    Looks like I rediscovered that problem years later and reported it with #232? 😄

  6. etr commented on Feb 19, 2026

    @etr
    etr
    on Feb 19, 2026
    Owner

    Last few issues addressed in: #367

  7. etr
    closed this as completedon Feb 19, 2026
  8. etr
    added 3 commits that reference this issue on May 28, 2026

    TASK-051 Tier 3: address 2nd-pass review findings (fix/task-051-2nd-r…

    d5dd7fd

    review backlog: Pass 1 — disposition all unworked items in 22 review …

    9dbc291

    review backlog: Pass 2A — examples security fixes + REGRESSION.md drift

    fbfefde
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions