This is an automated email from Gerrit.

Andreas Fritiofson ([email protected]) just uploaded a new patch set 
to Gerrit, which you can find at http://openocd.zylin.com/3927

-- gerrit

commit 0b5e1dbcc498ad784a9ffc1ad1581c5ba7428637
Author: Andreas Fritiofson <[email protected]>
Date:   Tue Dec 27 01:47:23 2016 +0100

    helper,rtos,server: Restore missing warning flags
    
    These libraries override the used CFLAGS without adding the
    defaults. This didn't have any effect until change
    http://openocd.zylin.com/3870 (ef4c139). Restore by adding
    AM_CLAGS to the per-target CFLAGS.
    
    Interestingly, automake seems to clear the CFLAGS for the target
    even if the override variable is only mentioned within a non-active
    conditional branch, such as the IS_MINGW for the affected libraries.
    
    Change-Id: I805206865e59e3fa33a7ea3c0d3472e51219351c
    Signed-off-by: Andreas Fritiofson <[email protected]>

diff --git a/src/helper/Makefile.am b/src/helper/Makefile.am
index 3623894..e0f7f49 100644
--- a/src/helper/Makefile.am
+++ b/src/helper/Makefile.am
@@ -37,7 +37,7 @@ else
 %C%_libhelper_la_SOURCES += %D%/ioutil_stubs.c
 endif
 
-%C%_libhelper_la_CFLAGS =
+%C%_libhelper_la_CFLAGS = $(AM_CFLAGS)
 if IS_MINGW
 # FD_* macros are sloppy with their signs on MinGW32 platform
 %C%_libhelper_la_CFLAGS += -Wno-sign-compare
diff --git a/src/rtos/Makefile.am b/src/rtos/Makefile.am
index aca266e..c59ee3f 100644
--- a/src/rtos/Makefile.am
+++ b/src/rtos/Makefile.am
@@ -24,7 +24,7 @@ noinst_LTLIBRARIES += %D%/librtos.la
        %D%/rtos_mqx_stackings.h \
        %D%/rtos_ucos_iii_stackings.h
 
-%C%_librtos_la_CFLAGS =
+%C%_librtos_la_CFLAGS = $(AM_CFLAGS)
 
 if IS_MINGW
 # FD_* macros are sloppy with their signs on MinGW32 platform
diff --git a/src/server/Makefile.am b/src/server/Makefile.am
index 5970d03..804efac 100644
--- a/src/server/Makefile.am
+++ b/src/server/Makefile.am
@@ -10,7 +10,7 @@ noinst_LTLIBRARIES += %D%/libserver.la
        %D%/tcl_server.c \
        %D%/tcl_server.h
 
-%C%_libserver_la_CFLAGS =
+%C%_libserver_la_CFLAGS = $(AM_CFLAGS)
 if IS_MINGW
 # FD_* macros are sloppy with their signs on MinGW32 platform
 %C%_libserver_la_CFLAGS += -Wno-sign-compare

-- 

------------------------------------------------------------------------------
Check out the vibrant tech community on one of the world's most 
engaging tech sites, SlashDot.org! http://sdm.link/slashdot
_______________________________________________
OpenOCD-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/openocd-devel

Reply via email to