[MPlayer-cvslog] r26111 - trunk/mp_msg.h

Rich Felker dalias at aerifal.cx
Sun Mar 2 05:35:08 CET 2008


On Sat, Mar 01, 2008 at 11:38:23AM +0100, Diego Biurrun wrote:
> On Thu, Feb 28, 2008 at 10:53:06PM -0500, Rich Felker wrote:
> > On Thu, Feb 28, 2008 at 07:31:35PM +0100, diego wrote:
> > > 
> > > Log:
> > > TARGET_OS2 is never set, use __OS2__ instead.
> > > 
> > > --- trunk/mp_msg.h	(original)
> > > +++ trunk/mp_msg.h	Thu Feb 28 19:31:35 2008
> > > @@ -111,7 +111,7 @@ int mp_msg_test(int mod, int lev);
> > >  
> > > -#ifdef TARGET_OS2
> > > +#ifdef __OS2__
> > 
> > Again, I'm against this general class of change. I've explained the
> > reasoning many times! Would you please honor that or at least discuss
> > it rather than continuing these bad commits?
> 
> I've discussed it before, I get tired of repeating it: I have little
> interest in discussing your theories.  There is no practical alternative
> to what I committed.
> 
> If you disagree: Patches welcome.

I could write the code to detect OS2 and add #define TARGET_OS2 in
configure, but I figure you're more qualified as the configure script
maintainer and someone familiar with it... While you're fixing the
problem described in this commit, you might as well fix it right..

Rich



More information about the MPlayer-cvslog mailing list