[MPlayer-cvslog] r26639 - trunk/configure

[Michael Niedermayer]

On Thu, May 01, 2008 at 09:18:07PM +0300, Ivan Kalvachev wrote:
> On Thu, May 1, 2008 at 8:47 PM, Reimar Döffinger
> <Reimar.Doeffinger at stud.uni-karlsruhe.de> wrote:
> > On Thu, May 01, 2008 at 07:45:20PM +0200, Diego Biurrun wrote:
> >  > On Thu, May 01, 2008 at 07:41:55PM +0200, Reimar Döffinger wrote:
> >  > > On Thu, May 01, 2008 at 07:32:52PM +0200, diego wrote:
> >  > > >
> >  > > > Log:
> >  > > > Remove --disable-gcc-check option and related code.
> >  > >
> >  > > It is not a big deal, but I am not too happy about having to hack
> >  > > configure for compile-testing with e.g. Sun's C compiler.
> >  >
> >  > I'll look into it.  The idea of rejecting unknown compilers is braindead
> >  > IMO.  The logic should accept, not reject, by default.

No not any more braindead than the obvious alternative.
There really are 3 kinds (at least)
1. the versions we know do work
2. the versions we know do not work
3. the versions we do not know

Its clear 1. should be accepted and 2. be rejected by default
having 3. require an explicit parameter to configure might reduce silly
bugreports. Its a question of slight inconvenience to the users vs. to
the developers.
Personally i really dont care either way, it seems the extra option was
preferred in the past and iam all for removing it if consensus is that
its better without it. But somehow the "braindead" sounds like the
original reasons for this are not understood. And that sounds all to
familar, changes being done because the original reasons, original
designs or original code is plain not understood instead of anything
being wrong with it.



[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

If a bugfix only changes things apparently unrelated to the bug with no
further explanation, that is a good sign that the bugfix is wrong.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/mplayer-cvslog/attachments/20080502/eeed20ad/attachment.pgp>