Skip to contentSkip to conversation

package/pcm-tools: fix build with GCC 4.8.2

PATCH New Carlos Santos 2 participants 2 messages Dec 15, 2018 – Dec 16, 2018

lore
Share & tools
lore
#0 lore raw

PATCH 1/1 package/pcm-tools: fix build with GCC 4.8.2

Pass $(TARGET_CONFIGURE_OPTS) as arguments to make and add "-std=c++11" to CXXFLAGS, forcing C++ 2011 standard (it's experimental in GCC 4.8.2 but goot enough to build pcm-tools).

Fixes:
http://autobuild.buildroot.net/results/cf3c79f0c94be8a184d532570bdb1893090316a3/

---
 package/pcm-tools/pcm-tools.mk | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)
package/pcm-tools/pcm-tools.mk +2 -1
index c0d6d8bc52..98b562eac4 100644
@@ -15,7 +15,8 @@ PCM_TOOLS_EXE_FILES = \
 
 define PCM_TOOLS_BUILD_CMDS
 	touch $(@D)/daemon-binaries
-	$(TARGET_MAKE_ENV) $(MAKE) $(TARGET_CONFIGURE_OPTS) -C $(@D) \
+	$(TARGET_MAKE_ENV) $(MAKE) -C $(@D) $(TARGET_CONFIGURE_OPTS) \
+		CXXFLAGS="$(TARGET_CXXFLAGS) -std=c++11" \
 		UNAME=Linux HOST=_LINUX
 endef
 
Signature · 1 line
2.19.2
#1 lore raw

PATCH 1/1 package/pcm-tools: fix build with GCC 4.8.2

Hello,

Quoted 1 line · Carlos Santos

Pass $(TARGET_CONFIGURE_OPTS) as arguments to make and add "-std=c++11"

TARGET_CONFIGURE_OPTS were already passed as argument: they are already passed to the right hand-side of $(MAKE), so I dropped this particular change.

Quoted 2 lines

to CXXFLAGS, forcing C++ 2011 standard (it's experimental in GCC 4.8.2 but goot enough to build pcm-tools).

This was enough to fix the build failure.

Applied with this change. Thanks!

Thomas

Signature · 3 lines
Thomas Petazzoni, CTO, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com