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