[PATCH] gcc5: Add back g++ sysroot patch

[PATCH] gcc5: Add back g++ sysroot patch

[Richard Purdie]

Without this, g++/c++ compilation doesn't work on target due to missing
header files. Automated sanity tests fail. Add back the gcc4 patch to
address this.

Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>

diff --git a/meta/recipes-devtools/gcc/gcc-5.1.inc b/meta/recipes-devtools/gcc/gcc-5.1.inc
index 305736b..1d64f9e 100644
--- a/meta/recipes-devtools/gcc/gcc-5.1.inc
+++ b/meta/recipes-devtools/gcc/gcc-5.1.inc
@@ -69,6 +69,7 @@ SRC_URI = "\
            file://0035-Dont-link-the-plugins-with-libgomp-explicitly.patch \
            file://0036-Use-SYSTEMLIBS_DIR-replacement-instead-of-hardcoding.patch \
            file://0037-pr65779.patch \
+           file://0038-fix-g++-sysroot.patch \
           "
 
 #S = "${TMPDIR}/work-shared/gcc-${PV}-${PR}/gcc-${SNAP}"
diff --git a/meta/recipes-devtools/gcc/gcc-5.1/0038-fix-g++-sysroot.patch b/meta/recipes-devtools/gcc/gcc-5.1/0038-fix-g++-sysroot.patch
new file mode 100644
index 0000000..f024dd5
--- /dev/null
+++ b/meta/recipes-devtools/gcc/gcc-5.1/0038-fix-g++-sysroot.patch
@@ -0,0 +1,44 @@
+Portions of
+
+http://www.mail-archive.com/gcc-patches@gcc.gnu.org/msg26013.html
+
+are not upstreamed yet. So lets keep missing pieces.
+
+Upstream-Status: Pending
+
+Without this, compiling something simple like #include <limits> on target
+with c++ test.cpp fails unable to find the header. strace shows it looking in
+usr/include/xxxx rather than /usr/include/xxxx
+
+Signed-off-by: Khem Raj <raj.khem@gmail.com>
+
+Index: gcc-4.8.1/gcc/configure.ac
+===================================================================
+--- gcc-4.8.1.orig/gcc/configure.ac	2013-07-15 15:55:49.488399132 -0700
++++ gcc-4.8.1/gcc/configure.ac	2013-07-15 16:02:31.772406679 -0700
+@@ -148,7 +148,9 @@
+ if test "${with_sysroot+set}" = set; then
+   gcc_gxx_without_sysroot=`expr "${gcc_gxx_include_dir}" : "${with_sysroot}"'\(.*\)'`
+   if test "${gcc_gxx_without_sysroot}"; then
+-    gcc_gxx_include_dir="${gcc_gxx_without_sysroot}"
++    if test x${with_sysroot} != x/; then
++      gcc_gxx_include_dir="${gcc_gxx_without_sysroot}"
++    fi
+     gcc_gxx_include_dir_add_sysroot=1
+   fi
+ fi
+Index: gcc-4.8.1/gcc/configure
+===================================================================
+--- gcc-4.8.1.orig/gcc/configure	2013-07-15 15:55:49.472399132 -0700
++++ gcc-4.8.1/gcc/configure	2013-07-15 16:02:31.780406680 -0700
+@@ -3325,7 +3325,9 @@
+ if test "${with_sysroot+set}" = set; then
+   gcc_gxx_without_sysroot=`expr "${gcc_gxx_include_dir}" : "${with_sysroot}"'\(.*\)'`
+   if test "${gcc_gxx_without_sysroot}"; then
+-    gcc_gxx_include_dir="${gcc_gxx_without_sysroot}"
++    if test x${with_sysroot} != x/; then
++      gcc_gxx_include_dir="${gcc_gxx_without_sysroot}"
++    fi
+     gcc_gxx_include_dir_add_sysroot=1
+   fi
+ fi

Re: [PATCH] gcc5: Add back g++ sysroot patch

[Khem Raj]

[-- Attachment #1: Type: text/plain, Size: 3171 bytes --]


> On May 12, 2015, at 5:02 AM, Richard Purdie <richard.purdie@linuxfoundation.org> wrote:
> 
> Without this, g++/c++ compilation doesn't work on target due to missing
> header files. Automated sanity tests fail. Add back the gcc4 patch to
> address this.


ah thanks for this. I was about to start looking at it.

> 
> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
> 
> diff --git a/meta/recipes-devtools/gcc/gcc-5.1.inc b/meta/recipes-devtools/gcc/gcc-5.1.inc
> index 305736b..1d64f9e 100644
> --- a/meta/recipes-devtools/gcc/gcc-5.1.inc
> +++ b/meta/recipes-devtools/gcc/gcc-5.1.inc
> @@ -69,6 +69,7 @@ SRC_URI = "\
>            file://0035-Dont-link-the-plugins-with-libgomp-explicitly.patch \
>            file://0036-Use-SYSTEMLIBS_DIR-replacement-instead-of-hardcoding.patch \
>            file://0037-pr65779.patch \
> +           file://0038-fix-g++-sysroot.patch \
>           "
> 
> #S = "${TMPDIR}/work-shared/gcc-${PV}-${PR}/gcc-${SNAP}"
> diff --git a/meta/recipes-devtools/gcc/gcc-5.1/0038-fix-g++-sysroot.patch b/meta/recipes-devtools/gcc/gcc-5.1/0038-fix-g++-sysroot.patch
> new file mode 100644
> index 0000000..f024dd5
> --- /dev/null
> +++ b/meta/recipes-devtools/gcc/gcc-5.1/0038-fix-g++-sysroot.patch
> @@ -0,0 +1,44 @@
> +Portions of
> +
> +http://www.mail-archive.com/gcc-patches@gcc.gnu.org/msg26013.html
> +
> +are not upstreamed yet. So lets keep missing pieces.
> +
> +Upstream-Status: Pending
> +
> +Without this, compiling something simple like #include <limits> on target
> +with c++ test.cpp fails unable to find the header. strace shows it looking in
> +usr/include/xxxx rather than /usr/include/xxxx
> +
> +Signed-off-by: Khem Raj <raj.khem@gmail.com>
> +
> +Index: gcc-4.8.1/gcc/configure.ac
> +===================================================================
> +--- gcc-4.8.1.orig/gcc/configure.ac	2013-07-15 15:55:49.488399132 -0700
> ++++ gcc-4.8.1/gcc/configure.ac	2013-07-15 16:02:31.772406679 -0700
> +@@ -148,7 +148,9 @@
> + if test "${with_sysroot+set}" = set; then
> +   gcc_gxx_without_sysroot=`expr "${gcc_gxx_include_dir}" : "${with_sysroot}"'\(.*\)'`
> +   if test "${gcc_gxx_without_sysroot}"; then
> +-    gcc_gxx_include_dir="${gcc_gxx_without_sysroot}"
> ++    if test x${with_sysroot} != x/; then
> ++      gcc_gxx_include_dir="${gcc_gxx_without_sysroot}"
> ++    fi
> +     gcc_gxx_include_dir_add_sysroot=1
> +   fi
> + fi
> +Index: gcc-4.8.1/gcc/configure
> +===================================================================
> +--- gcc-4.8.1.orig/gcc/configure	2013-07-15 15:55:49.472399132 -0700
> ++++ gcc-4.8.1/gcc/configure	2013-07-15 16:02:31.780406680 -0700
> +@@ -3325,7 +3325,9 @@
> + if test "${with_sysroot+set}" = set; then
> +   gcc_gxx_without_sysroot=`expr "${gcc_gxx_include_dir}" : "${with_sysroot}"'\(.*\)'`
> +   if test "${gcc_gxx_without_sysroot}"; then
> +-    gcc_gxx_include_dir="${gcc_gxx_without_sysroot}"
> ++    if test x${with_sysroot} != x/; then
> ++      gcc_gxx_include_dir="${gcc_gxx_without_sysroot}"
> ++    fi
> +     gcc_gxx_include_dir_add_sysroot=1
> +   fi
> + fi
> 
> 


[-- Attachment #2: Message signed with OpenPGP using GPGMail --]
[-- Type: application/pgp-signature, Size: 211 bytes --]