[MPlayer-cvslog] r29043 - trunk/configure

[Andrew Savchenko]

Hi,

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.

-- 
Best regards,
Andrew
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/mplayer-cvslog/attachments/20090324/9a7b4808/attachment.pgp>