diff mbox series

[1/1] package/apitrace: force Release mode

Message ID 20200517140515.2665046-1-fontaine.fabrice@gmail.com
State Superseded
Headers show
Series [1/1] package/apitrace: force Release mode | expand

Commit Message

Fabrice Fontaine May 17, 2020, 2:05 p.m. UTC
Force Release mode otherwise the toolchain workarounds added by commit
6f5fcb43296ffc54b2b67d0b3723405b56d8d439 will be ignored when debug is
enabled as CMAKE_C_FLAGS_RELWITHDEBINFO is appended to CFLAGS:

output/build/apitrace-9.0/thirdparty/brotli/CMakeFiles/brotli_dec.dir/flags.make:C_FLAGS = -D_LARGEFILE_SOURCE -D_LARGEFILE64_SOURCE -D_FILE_OFFSET_BITS=64  -Os -g2  -O0 -fvisibility=hidden -Wall -Wno-sign-compare -fno-strict-aliasing -fmessage-length=0 -fno-omit-frame-pointer -O2 -g -DNDEBUG -fPIC

Fixes:
 - http://autobuild.buildroot.org/results/2e31abd7b115d4c29117ca82007fd4f87f853ff7

Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com>
---
 package/apitrace/apitrace.mk | 2 ++
 1 file changed, 2 insertions(+)

Comments

Thomas Petazzoni Oct. 17, 2020, 12:54 p.m. UTC | #1
On Sun, 17 May 2020 16:05:15 +0200
Fabrice Fontaine <fontaine.fabrice@gmail.com> wrote:

> Force Release mode otherwise the toolchain workarounds added by commit
> 6f5fcb43296ffc54b2b67d0b3723405b56d8d439 will be ignored when debug is
> enabled as CMAKE_C_FLAGS_RELWITHDEBINFO is appended to CFLAGS:
> 
> output/build/apitrace-9.0/thirdparty/brotli/CMakeFiles/brotli_dec.dir/flags.make:C_FLAGS = -D_LARGEFILE_SOURCE -D_LARGEFILE64_SOURCE -D_FILE_OFFSET_BITS=64  -Os -g2  -O0 -fvisibility=hidden -Wall -Wno-sign-compare -fno-strict-aliasing -fmessage-length=0 -fno-omit-frame-pointer -O2 -g -DNDEBUG -fPIC
> 
> Fixes:
>  - http://autobuild.buildroot.org/results/2e31abd7b115d4c29117ca82007fd4f87f853ff7
> 
> Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com>
> ---
>  package/apitrace/apitrace.mk | 2 ++
>  1 file changed, 2 insertions(+)

This was fixed in a different way in commit
ce053a2004d089b3080a750496b3b3b21bcc224d.

I've marked this patch as Superseded then.

Thanks!

Thomas
diff mbox series

Patch

diff --git a/package/apitrace/apitrace.mk b/package/apitrace/apitrace.mk
index e4e6c41fed..71a3929fe0 100644
--- a/package/apitrace/apitrace.mk
+++ b/package/apitrace/apitrace.mk
@@ -34,7 +34,9 @@  ifeq ($(BR2_TOOLCHAIN_HAS_GCC_BUG_85180),y)
 APITRACE_CXXFLAGS += -O0
 endif
 
+# Force Release mode otherwise the above workarounds will be ignored
 APITRACE_CONF_OPTS += \
+	-DCMAKE_BUILD_TYPE=Release \
 	-DCMAKE_C_FLAGS="$(APITRACE_CFLAGS)" \
 	-DCMAKE_CXX_FLAGS="$(APITRACE_CXXFLAGS)"