diff mbox series

package/pcm-tools: fix build with GCC 4.8.2

Message ID 20181215013813.451-1-casantos@datacom.com.br
State Accepted
Headers show
Series package/pcm-tools: fix build with GCC 4.8.2 | expand

Commit Message

Carlos Santos Dec. 15, 2018, 1:38 a.m. UTC
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/

Signed-off-by: Carlos Santos <casantos@datacom.com.br>
---
 package/pcm-tools/pcm-tools.mk | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

Comments

Thomas Petazzoni Dec. 16, 2018, 3:39 p.m. UTC | #1
Hello,

On Fri, 14 Dec 2018 23:38:13 -0200, Carlos Santos wrote:
> 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.

> 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
diff mbox series

Patch

diff --git a/package/pcm-tools/pcm-tools.mk b/package/pcm-tools/pcm-tools.mk
index c0d6d8bc52..98b562eac4 100644
--- a/package/pcm-tools/pcm-tools.mk
+++ b/package/pcm-tools/pcm-tools.mk
@@ -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