[MPlayer-cvslog] CVS: main configure,1.985,1.986

[Alexander Strasser]

Rich Felker wrote:
> On Tue, May 03, 2005 at 06:32:16PM +0200, Nicolas Plourde CVS wrote:
> > CVS change done by Nicolas Plourde CVS
> > 
> > Update of /cvsroot/mplayer/main
> > In directory mail:/var2/tmp/cvs-serv20615
> > 
> > Modified Files:
> > 	configure 
> > Log Message:
> > proper list of libav codec
> > 
> > Index: configure
> > ===================================================================
> > RCS file: /cvsroot/mplayer/main/configure,v
> > retrieving revision 1.985
> > retrieving revision 1.986
> > diff -u -r1.985 -r1.986
> > --- configure	3 May 2005 11:16:35 -0000	1.985
> > +++ configure	3 May 2005 16:32:13 -0000	1.986
> > @@ -6899,7 +6899,7 @@
> >  AMR_NB=$_amr_nb
> >  AMR_NB_FIXED=$_amr_nb_fixed
> >  AMR_WB=$_amr_wb
> > -`echo $_libavcodecs | awk '{$0=toupper($0)}1' | sed 's/ \?\([-_a-zA-Z0-9]*\)/CONFIG_\1=yes\n/g'`
> > +$(echo -e `echo $_libavcodecs | awk '{$0=toupper($0)}1' | sed 's/\([-_a-zA-Z0-9]*\)/CONFIG_\1=yes\\\n/g'`)
> 
> Rejected! Please fix this: $() syntax is not compatible with
> traditional bourne shell. If you need nested `` use some hideous shell
> quoting with \ and ", or better yet find a simpler way to do this.

I just wanted to speak up. It should work the other way too if \? caused
the prob and won't break for traditional shells.
Nicolas could you test with your changes and `` as before?

  Alex (beastd)