[MPlayer-dev-eng] [PATCH] configure: much faster configure --help
Alexander Strasser
eclipse7 at gmx.net
Wed Feb 13 00:16:16 CET 2013
Erik Auerswald wrote:
> On 02/12/2013 10:19 AM, Carl Eugen Hoyos wrote:
> >Erik Auerswald<auerswal<at> unix-ag.uni-kl.de> writes:
> >
> >>Thanks for reviewing, maybe someone finds this useful.
> >
> >Can't you remove the show_help call below
> >"for ac_option do"?
>
> No, because then e.g. ./configure --enable-debug --help would not
> work any more.
While I think about it: As the number of arguments for useful configure
invocations is even in the worst case quite small, we could do two passes.
Early iteration of the arguments only checks for the help option. Then
we do not need to check for it in the 2nd pass. That would remove the
duplication of the help check and provide early exit even if help is not
the 1st argument.
Caveat:
./configure -cfv --enable-debug --help
would work too. (Just printing help and exit).
OTOH I do not know if that is actually worse as currently this already
works anyway:
./configure --help --enable-debug -cvz
ATM I would tend towards the 2-pass solution. Opinions welcome!
Alexander
P.S.
No new patch necessary I have done the modification locally.
More information about the MPlayer-dev-eng
mailing list