> 
> On 14-07-2008 16:05:53 +0200, Markus Duft wrote:
> > Haubi suggested this small enhancement:
> >
> > [[ " ${CFLAGS} " != *" -D_ALL_SOURCE "* ]] && .......
> 
> ok with me, smarter actually ;)
> 

Hm. Thinking about it, i guess i like this one better than defining _ALL_SOURCE 
inside gcc or the wrapper. Using strip-flags in ebuilds seems more readable, 
than append-flags -U_ALL_SOURCE... I think this is the way we should go.

If you agree, I'll bootstrap a prefix on interix 5.2, and also patch this (and 
all ebuilds)...

Cheers, Markus

> > Another thought from haubi:
> >
> > What if we just patch gcc to define _ALL_SOURCE as built-in. if we
> > require it to be undefined, we can still use -U on the command line,
> > right?
> 
> That last thing is a requirement of course (hence I put the define
> first
> thing, so it can be overridden).
> 
> Further thoughts include:
> - messing with gcc in general is no fun, alternative is the wrapper,
>   but could do it in the gcc spec language too, I guess
> - how about the bootstrap phase, is setting -D_ALL_SOURCE easy there (I
>   think for my method it is)
> - patched gcc (invocation) is nice since we always did all the work to
>   have a working compiler setup in/with Prefix without having to do
>   anything special (think of library paths to Prefix' libs)
> 
> Looking at the random thoughts above, I guess it means haubi is going
> to
> fix it ;)
> 
> 
> --
> Fabian Groffen
> Gentoo on a different level
> --
> [email protected] mailing list


--
[email protected] mailing list

Reply via email to