Den 23/01/2013 kl. 11.05 skrev Konstantin Belousov <kostik...@gmail.com>:

> On Wed, Jan 23, 2013 at 09:23:41AM +0100, Erik Cederstrand wrote:
>> [...]
>> 
>> -CFLAGS+=    -g
>> +CFLAGS+=    $(DEBUG_FLAGS)
>> CFLAGS+=     -DGENERATOR_FILE -DHAVE_CONFIG_H
>> 
>> # Override LIBIBERTY set by Makefile.inc, We use our own for
> 
> This fragment catched my eye, and it is typical for the patch.
> Why do you  need to add DEBUG_FLAGS to the CFLAGS manually ?
> Wouldn't share/mk magic do this automatically ?

Quite possibly. My only reason for this approach was to change as little as 
possible. I'll test if I can just remove the line entirely.

Thanks,
Erik
_______________________________________________
freebsd-toolchain@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/freebsd-toolchain
To unsubscribe, send any mail to "freebsd-toolchain-unsubscr...@freebsd.org"

Reply via email to