[Mplayer-cvslog] CVS: main configure,1.932,1.933

Diego Biurrun diego at biurrun.de
Fri Nov 5 14:37:44 CET 2004


On Fri, Nov 05, 2004 at 02:21:58PM +0100, Moritz Bunkus wrote:
> 
> > -  cc_name=`( $_cc -V ) 2>&1 | head -1 | cut -d ',' -f 1`
> > -  cc_version=`( $_cc -V ) 2>&1 | head -1 | cut -d ',' -f 2 | cut -d ' ' -f 3`
> > +  cc_name=`( $_cc -V ) 2>&1 | head -n 1 | cut -d ',' -f 1`
> > +  cc_version=`( $_cc -V ) 2>&1 | head -n 1 | cut -d ',' -f 2 | cut -d
> > -  ' ' -f 3`
> 
> Such patches have been rejected in the past, and I guess that there will
> be some flaming now, too. The "proper" solution would IMHO be to check
> whether head and tail want -n or not and to use that information.

I'll add it to the FAQ later today, this has come up far too often now.

Is Colin even on the cvslog mailing list?  He hasn't committed in ages...

Diego




More information about the MPlayer-cvslog mailing list