On 09/06/2013 10:59 PM, Dunrong Huang wrote:
If the _FORTIFY_SOURCE has been already defined, we shouldn't redefine it, or it will raise a build error as below: In file included from spice-audio.c:36:0: ../config.h:12:0: error: "_FORTIFY_SOURCE" redefined [-Werror] spice-audio.c:1:0: note: this is the location of the previous definition Suggested-by: Christophe Fergeau <cfergeau@xxxxxxxxxx> Signed-off-by: Dunrong Huang <riegamaths@xxxxxxxxx> --- m4/spice-compile-warnings.m4 | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/m4/spice-compile-warnings.m4 b/m4/spice-compile-warnings.m4 index 31fc0fc..533bac6 100644 --- a/m4/spice-compile-warnings.m4 +++ b/m4/spice-compile-warnings.m4 @@ -104,7 +104,7 @@ AC_DEFUN([SPICE_COMPILE_WARNINGS],[ # Use improved glibc headers AH_VERBATIM([FORTIFY_SOURCE], [/* Enable compile-time and run-time bounds-checking, and some warnings. */ - #if defined __OPTIMIZE__ && __OPTIMIZE__ + #if !defined _FORTIFY_SOURCE && defined __OPTIMIZE__ && __OPTIMIZE__ # define _FORTIFY_SOURCE 2 #endif ])
Ack. _______________________________________________ Spice-devel mailing list Spice-devel@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/spice-devel