Skip to contentSkip to conversation

omxplayer: fix compilation with GCC 6

PATCH Applied Jonathan Liu 2 participants 2 messages Jun 7, 2016 – Jun 14, 2016

lore
Share & tools
lore
#0 lore raw

PATCH 1/1 omxplayer: fix compilation with GCC 6

Specifying -isystem${STAGING_DIR_HOST}/usr/include in INCLUDES gives:

In file included from utils/PCMRemap.cpp:26:0:
.../build/tmp/sysroots/raspberrypi2/usr/include/c++/6.1.1/cstdlib:75:25: fatal error: stdlib.h: No such file or directory
#include_next <stdlib.h>
^
compilation terminated.
Makefile:44: recipe for target 'utils/PCMRemap.o' failed

To resolve this, /usr/include shouldn't be specified as it is already a default include path relative to the sysroot.

See https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70129.

---
 recipes-multimedia/omxplayer/omxplayer_git.bb | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)
recipes-multimedia/omxplayer/omxplayer_git.bb +1 -2
index 7d3b464..d9460de 100644
@@ -40,8 +40,7 @@ export LDFLAGS = "-L${S}/ffmpeg_compiled/usr/lib \
                   -L${STAGING_DIR_HOST}/lib \
                   -L${STAGING_DIR_HOST}/usr/lib \
                  "
-export INCLUDES = "-isystem${STAGING_DIR_HOST}/usr/include \
-                   -isystem${STAGING_DIR_HOST}/usr/include/interface/vcos/pthreads \
+export INCLUDES = "-isystem${STAGING_DIR_HOST}/usr/include/interface/vcos/pthreads \
                    -isystem${STAGING_DIR_HOST}/usr/include/freetype2 \
                    -isystem${STAGING_DIR_HOST}/usr/include/interface/vmcs_host/linux \
                    -isystem${STAGING_DIR_HOST}/usr/include/dbus-1.0 \
Signature · 1 line
2.8.3
#1 lore raw

Hi,

Quoted 9 lines · Jonathan Liu

Specifying -isystem${STAGING_DIR_HOST}/usr/include in INCLUDES gives:

In file included from utils/PCMRemap.cpp:26:0:
.../build/tmp/sysroots/raspberrypi2/usr/include/c++/6.1.1/cstdlib:75:25: fatal error: stdlib.h: No such file or directory
#include_next <stdlib.h>
^
compilation terminated.
Makefile:44: recipe for target 'utils/PCMRemap.o' failed

Quoted 6 lines

To resolve this, /usr/include shouldn't be specified as it is already a default include path relative to the sysroot.

See https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70129.

Pushed to master. Thanks.

--
Andrei Gherzan