helper,rtos,server: Restore missing warning flags 27/3927/2
authorAndreas Fritiofson <andreas.fritiofson@gmail.com>
Tue, 27 Dec 2016 00:47:23 +0000 (01:47 +0100)
committerPaul Fertser <fercerpav@gmail.com>
Sun, 15 Jan 2017 10:16:16 +0000 (10:16 +0000)
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 <andreas.fritiofson@gmail.com>
Reviewed-on: http://openocd.zylin.com/3927
Tested-by: jenkins
Reviewed-by: Paul Fertser <fercerpav@gmail.com>
src/helper/Makefile.am
src/rtos/Makefile.am
src/server/Makefile.am

index 362389436c07cea1a17d5c69c034a3a591d23c93..e0f7f49bb0f4ffbc9183314a9e0726472999b10a 100644 (file)
@@ -37,7 +37,7 @@ else
 %C%_libhelper_la_SOURCES += %D%/ioutil_stubs.c
 endif
 
 %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
 if IS_MINGW
 # FD_* macros are sloppy with their signs on MinGW32 platform
 %C%_libhelper_la_CFLAGS += -Wno-sign-compare
index aca266e18de246159f133b13743f6704435180b6..c59ee3f8c7b430952202375a561abce0326c6516 100644 (file)
@@ -24,7 +24,7 @@ noinst_LTLIBRARIES += %D%/librtos.la
        %D%/rtos_mqx_stackings.h \
        %D%/rtos_ucos_iii_stackings.h
 
        %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
 
 if IS_MINGW
 # FD_* macros are sloppy with their signs on MinGW32 platform
index 5970d0314638c4ead77c3f0a842f1f302a1e7c08..804efac16dc2a9035a58e6c595ec3f837df4972f 100644 (file)
@@ -10,7 +10,7 @@ noinst_LTLIBRARIES += %D%/libserver.la
        %D%/tcl_server.c \
        %D%/tcl_server.h
 
        %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
 if IS_MINGW
 # FD_* macros are sloppy with their signs on MinGW32 platform
 %C%_libserver_la_CFLAGS += -Wno-sign-compare

Linking to existing account procedure

If you already have an account and want to add another login method you MUST first sign in with your existing account and then change URL to read https://review.openocd.org/login/?link to get to this page again but this time it'll work for linking. Thank you.

SSH host keys fingerprints

1024 SHA256:YKx8b7u5ZWdcbp7/4AeXNaqElP49m6QrwfXaqQGJAOk gerrit-code-review@openocd.zylin.com (DSA)
384 SHA256:jHIbSQa4REvwCFG4cq5LBlBLxmxSqelQPem/EXIrxjk gerrit-code-review@openocd.org (ECDSA)
521 SHA256:UAOPYkU9Fjtcao0Ul/Rrlnj/OsQvt+pgdYSZ4jOYdgs gerrit-code-review@openocd.org (ECDSA)
256 SHA256:A13M5QlnozFOvTllybRZH6vm7iSt0XLxbA48yfc2yfY gerrit-code-review@openocd.org (ECDSA)
256 SHA256:spYMBqEYoAOtK7yZBrcwE8ZpYt6b68Cfh9yEVetvbXg gerrit-code-review@openocd.org (ED25519)
+--[ED25519 256]--+
|=..              |
|+o..   .         |
|*.o   . .        |
|+B . . .         |
|Bo. = o S        |
|Oo.+ + =         |
|oB=.* = . o      |
| =+=.+   + E     |
|. .=o   . o      |
+----[SHA256]-----+
2048 SHA256:0Onrb7/PHjpo6iVZ7xQX2riKN83FJ3KGU0TvI0TaFG4 gerrit-code-review@openocd.zylin.com (RSA)