[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