[MPlayer-cvslog] CVS: main configure,1.1029,1.1030

[The Wanderer]

Michael Niedermayer wrote:

> Hi
> 
> On Friday 22 July 2005 13:25, Reimar Döffinger wrote:
> 
>> Hi,
>> 
>> On Fri, Jul 22, 2005 at 06:28:58AM -0400, The Wanderer wrote:
>> 
>>> I would much rather address this with an explanatory note
>>> somewhere (perhaps in the output of 'configure --help'), at least
>>> until such time
>> 
>> I think in --help we have that already, but better check
>> yourself... We could of course also add a variable _evil_enable to
>> configure and set it whenever an enable option that should be
>> autodetected is set. In that case we can print that big evil
>> warning message Rich suggested

Turns out yes, there is; I hadn't read the entire --help output in so
long I had no idea that was there. I think the warning as he worded it
is overly strong, at the least because there *are* a *few* such options
which don't merely override autodetection - --enable-gui and the like -
but if that problem can be addressed then I could support such a thing.

> what about removing all --enable options where theres autodetect code
> and fix the later if it fails to detect something

I was getting towards suggesting this earlier, when I asked if there was
any situation in which most of the --enable options actually served any
functional purpose (i.e., in which the main code would compile but
autodetection would fail). If the answer to that question is no, then
I'd say that this seems like the only sensible course of action - though
I agree with Rich's caveats.

-- 
       The Wanderer

Warning: Simply because I argue an issue does not mean I agree with any
side of it.

A government exists to serve its citizens, not to control them.