[MPlayer-cvslog] r29043 - trunk/configure

[Diego Biurrun]

On Tue, Mar 24, 2009 at 03:55:02PM +0300, Andrew Savchenko wrote:
> 
> On Tuesday 24 March 2009 15:43, Diego Biurrun wrote:
> > On Tue, Mar 24, 2009 at 02:55:21PM +0300, Andrew Savchenko wrote:
> > > On Tuesday 24 March 2009 02:37, diego wrote:
> > > > Log:
> > > > Rename --with-extraincdir option to --extra-cflags.
> > >
> > > [...]
> > >
> > > > @@ -765,9 +765,6 @@ for ac_option do
> > > > -  --with-extraincdir=*)
> > > > -    _inc_extra=-I`echo $ac_option | cut -d '=' -f 2 | sed
> > > > 's,:, -I,g'` -    ;;
> > > > -L,g'` ;;
> > > > @@ -801,6 +798,9 @@ for ac_option do
> > > >
> > > > +  --extra-cflags=*)
> > > > +    _extra_cflags=-`echo $ac_option | cut -d '=' -f 2`
> > > > +    ;;
> > >
> > > This brokes multiple paths specification, previously it was
> > > possible using ':' separator. Please fix it.
> >
> > This is intended behavior.  Just pass whatever CFLAGS you need
> > to the new option, surrounded by quotes.
> 
> This is easy to do on plain command line, but it will produce an 
> escape hell for nested bash variables commonly used in different 
> packaging systems. While this problem is still solvable, it 
> creates a lot of headache and imho is not desirable.

This is standard behavior for most other configure programs I know, so
packaging systems must already have found a way to deal with this.

Diego