zeromq / libzmq

ZeroMQ core engine in C++, implements ZMTP/3.1
https://www.zeromq.org
Mozilla Public License 2.0
9.46k stars 2.34k forks source link

"configure --help" output doesn't match actual behaviour #4613

Open ellert opened 8 months ago

ellert commented 8 months ago

What is the intended behaviour here?

https://github.com/zeromq/libzmq/blob/f0c471aaabb7f619fc0569317e300f28d00a22ec/configure.ac#L1029-L1042

I would prefer to change the implemented behaviour to match the documented behaviour, i.e.

AC_ARG_ENABLE([drafts], 
    AS_HELP_STRING([--enable-drafts], 
        [Build and install draft classes and methods [default=yes]]), 
    [enable_drafts=$enableval], 
    [enable_drafts=yes]) 

(and remove the definition of the now unused defaultval).

But if this is not acceptible, change the documentation to match the implementation:

AC_ARG_ENABLE([drafts], 
    AS_HELP_STRING([--enable-drafts], 
        [Build and install draft classes and methods [default=$defaultval]]), 
    [enable_drafts=$enableval], 
    [enable_drafts=$defaultval]) 

As it is now, users reading the output of "configure --help" think they have drafts enabled when they actually have not.