gnucash maint: Multiple changes pushed

Geert Janssens janssens-geert at telenet.be
Fri May 2 10:01:01 EDT 2014


On Friday 02 May 2014 15:19:06 Geert Janssens wrote:
> 
> This commit breaks the build on my system (ironically a Fedora
> system).
> 
> The _FORTIFY_SOURCE addition results in the following compile error:
> /usr/include/features.h:327:4: error: #warning _FORTIFY_SOURCE
> requires compiling with optimization (-O) [-Werror=cpp]
>  #  warning _FORTIFY_SOURCE requires compiling with optimization (-O)
> 
> Due to the incorrect logic the part adding _FORTIFY_SOURCE was never
> reached before this patch. It now is but it looks like it shouldn't
> be added unconditionally because it needs to build with optimization
> enabled.
> 
I have pushed a commit that only adds _FORTIFY_SOURCE if compilation with 
optimization is detected.

Feel free to improve it if you have a cleaner solution.

Geert


More information about the gnucash-devel mailing list