diff options
author | John Baldwin <jhb@FreeBSD.org> | 2018-03-24 07:53:52 +0800 |
---|---|---|
committer | John Baldwin <jhb@FreeBSD.org> | 2018-03-24 07:53:52 +0800 |
commit | 9b6b838a93bdf6151f71653e0f5e9e8af6e4155b (patch) | |
tree | 62bd4cf82565587402a460797bb74349cff050d6 | |
parent | abd7dc4985633d6a068ef0f6597f02ac81e77454 (diff) | |
download | freebsd-ports-9b6b838a93bdf6151f71653e0f5e9e8af6e4155b.tar.gz freebsd-ports-9b6b838a93bdf6151f71653e0f5e9e8af6e4155b.tar.zst freebsd-ports-9b6b838a93bdf6151f71653e0f5e9e8af6e4155b.zip |
Fix --sysroot for cross-toolchain GCC packages
By default, GCC assumes that a cross compiler should not have any valid
initial include or library paths aside from /usr/local/lib/gcc/<mumble>/.
This means that one cannot use --sysroot to point to a system root for
another architecture and have GCC automatically look for headers in
${sysroot}/usr/include, etc. Currently we workaround this in FreeBSD's
build system with explicit -isystem, -B, and -L directives. However,
this is cumbersome compared to clang (where a bare --sysroot DTRT)
especially when using the compiler to build other software (such as test
programs, etc.).
One can override GCC's assumption and force it to assume that it should
honor --sysroot by setting the '--with-sysroot' option to force GCC to
assume a specific system root. By setting this to '/', this means that
the cross-compiler will attempt to use the host's headers by default
if --sysroot is not specified, but if --sysroot is specified then
it is fully honored including for include paths and library paths. With
this change I can now cross-compile both C and C++ binaries simply by
using --sysroot without the need for -isystem, -B, or -L directives. Note
that the base/gcc and devel/riscv64-gcc ports both use --with-sysroot='/'
already.
By default, GCC looks for headers in /usr/local/include (under the
sysroot) before /usr/include. To disable this and only look for headers
in /usr/include, patch gcc/Makefile.in to not define LOCAL_INCLUDE_DIR.
Once -nostdinc is no longer required, the headers installed along with the
compiler are now used for compiling worlds and test programs. The "fixed"
headers in include-fixed are generally not helpful and are also derived
from the host's headers which might not match the target --sysroot, so
just delete them entirely. Even the stub limits.h headers GCC ships when
using an empty build sysroot are not helpful and need to be removed.
On a related note, I also fixed the name of the C++ include directory
option to configure. By my reading it is 'with-gxx-include-dir' rather
than 'with-gcc-include-dir'.
Reviewed by: bapt, brooks, kan
Sponsored by: DARPA / AFRL
Differential Revision: https://reviews.freebsd.org/D14627
Notes
Notes:
svn path=/head/; revision=465416
-rw-r--r-- | devel/aarch64-gcc/pkg-plist | 3 | ||||
-rw-r--r-- | devel/amd64-gcc/pkg-plist | 15 | ||||
-rw-r--r-- | devel/mips-gcc/pkg-plist | 3 | ||||
-rw-r--r-- | devel/mips64-gcc/pkg-plist | 3 | ||||
-rw-r--r-- | devel/powerpc64-gcc/Makefile | 8 | ||||
-rw-r--r-- | devel/powerpc64-gcc/pkg-plist | 3 | ||||
-rw-r--r-- | devel/sparc64-gcc/pkg-plist | 3 |
7 files changed, 6 insertions, 32 deletions
diff --git a/devel/aarch64-gcc/pkg-plist b/devel/aarch64-gcc/pkg-plist index 3e513ce8990c..985237f38594 100644 --- a/devel/aarch64-gcc/pkg-plist +++ b/devel/aarch64-gcc/pkg-plist @@ -8,9 +8,6 @@ bin/%%GCC_TARGET%%-gcc-nm bin/%%GCC_TARGET%%-gcc-ranlib bin/%%GCC_TARGET%%-gcov bin/%%GCC_TARGET%%-gcov-tool -lib/gcc/%%GCC_TARGET%%/%%GCC_VERSION%%/include-fixed/README -lib/gcc/%%GCC_TARGET%%/%%GCC_VERSION%%/include-fixed/limits.h -lib/gcc/%%GCC_TARGET%%/%%GCC_VERSION%%/include-fixed/syslimits.h lib/gcc/%%GCC_TARGET%%/%%GCC_VERSION%%/include/arm_acle.h lib/gcc/%%GCC_TARGET%%/%%GCC_VERSION%%/include/arm_neon.h lib/gcc/%%GCC_TARGET%%/%%GCC_VERSION%%/include/float.h diff --git a/devel/amd64-gcc/pkg-plist b/devel/amd64-gcc/pkg-plist index 75fb56212022..673a6965c9b9 100644 --- a/devel/amd64-gcc/pkg-plist +++ b/devel/amd64-gcc/pkg-plist @@ -8,21 +8,6 @@ bin/%%GCC_TARGET%%-gcc-nm bin/%%GCC_TARGET%%-gcc-ranlib bin/%%GCC_TARGET%%-gcov bin/%%GCC_TARGET%%-gcov-tool -lib/gcc/%%GCC_TARGET%%/%%GCC_VERSION%%/include-fixed/README -lib/gcc/%%GCC_TARGET%%/%%GCC_VERSION%%/include-fixed/limits.h -lib/gcc/%%GCC_TARGET%%/%%GCC_VERSION%%/include-fixed/netinet/ip_fil.h -lib/gcc/%%GCC_TARGET%%/%%GCC_VERSION%%/include-fixed/netinet/ip_lookup.h -lib/gcc/%%GCC_TARGET%%/%%GCC_VERSION%%/include-fixed/netinet/ip_nat.h -lib/gcc/%%GCC_TARGET%%/%%GCC_VERSION%%/include-fixed/netinet/ip_proxy.h -lib/gcc/%%GCC_TARGET%%/%%GCC_VERSION%%/include-fixed/netinet/ip_scan.h -lib/gcc/%%GCC_TARGET%%/%%GCC_VERSION%%/include-fixed/netinet/ip_state.h -lib/gcc/%%GCC_TARGET%%/%%GCC_VERSION%%/include-fixed/stddef.h -lib/gcc/%%GCC_TARGET%%/%%GCC_VERSION%%/include-fixed/stdio.h -lib/gcc/%%GCC_TARGET%%/%%GCC_VERSION%%/include-fixed/stdlib.h -lib/gcc/%%GCC_TARGET%%/%%GCC_VERSION%%/include-fixed/sys/types.h -lib/gcc/%%GCC_TARGET%%/%%GCC_VERSION%%/include-fixed/syslimits.h -lib/gcc/%%GCC_TARGET%%/%%GCC_VERSION%%/include-fixed/unistd.h -lib/gcc/%%GCC_TARGET%%/%%GCC_VERSION%%/include-fixed/wchar.h lib/gcc/%%GCC_TARGET%%/%%GCC_VERSION%%/include/adxintrin.h lib/gcc/%%GCC_TARGET%%/%%GCC_VERSION%%/include/ammintrin.h lib/gcc/%%GCC_TARGET%%/%%GCC_VERSION%%/include/avx2intrin.h diff --git a/devel/mips-gcc/pkg-plist b/devel/mips-gcc/pkg-plist index 8015851e862e..e72f4ea9bd88 100644 --- a/devel/mips-gcc/pkg-plist +++ b/devel/mips-gcc/pkg-plist @@ -8,9 +8,6 @@ bin/%%GCC_TARGET%%-gcc-nm bin/%%GCC_TARGET%%-gcc-ranlib bin/%%GCC_TARGET%%-gcov bin/%%GCC_TARGET%%-gcov-tool -lib/gcc/%%GCC_TARGET%%/%%GCC_VERSION%%/include-fixed/README -lib/gcc/%%GCC_TARGET%%/%%GCC_VERSION%%/include-fixed/limits.h -lib/gcc/%%GCC_TARGET%%/%%GCC_VERSION%%/include-fixed/syslimits.h lib/gcc/%%GCC_TARGET%%/%%GCC_VERSION%%/include/float.h lib/gcc/%%GCC_TARGET%%/%%GCC_VERSION%%/include/iso646.h lib/gcc/%%GCC_TARGET%%/%%GCC_VERSION%%/include/loongson.h diff --git a/devel/mips64-gcc/pkg-plist b/devel/mips64-gcc/pkg-plist index c971453d6fad..4782d0921fee 100644 --- a/devel/mips64-gcc/pkg-plist +++ b/devel/mips64-gcc/pkg-plist @@ -8,9 +8,6 @@ bin/%%GCC_TARGET%%-gcc-nm bin/%%GCC_TARGET%%-gcc-ranlib bin/%%GCC_TARGET%%-gcov bin/%%GCC_TARGET%%-gcov-tool -lib/gcc/%%GCC_TARGET%%/%%GCC_VERSION%%/include-fixed/README -lib/gcc/%%GCC_TARGET%%/%%GCC_VERSION%%/include-fixed/limits.h -lib/gcc/%%GCC_TARGET%%/%%GCC_VERSION%%/include-fixed/syslimits.h lib/gcc/%%GCC_TARGET%%/%%GCC_VERSION%%/include/float.h lib/gcc/%%GCC_TARGET%%/%%GCC_VERSION%%/include/iso646.h lib/gcc/%%GCC_TARGET%%/%%GCC_VERSION%%/include/loongson.h diff --git a/devel/powerpc64-gcc/Makefile b/devel/powerpc64-gcc/Makefile index ca6c702a1f9a..65774a932da3 100644 --- a/devel/powerpc64-gcc/Makefile +++ b/devel/powerpc64-gcc/Makefile @@ -2,7 +2,7 @@ PORTNAME= gcc PORTVERSION= 6.3.0 -PORTREVISION= 1 +PORTREVISION= 2 CATEGORIES= devel MASTER_SITES= GCC/releases/gcc-${DISTVERSION} PKGNAMEPREFIX?= powerpc64- @@ -48,7 +48,8 @@ CONFIGURE_ARGS+=--target=${GCC_TARGET} --disable-nls --enable-languages=c,c++ \ --with-gmp=${LOCALBASE} \ --with-pkgversion="FreeBSD Ports Collection for ${PKGNAMEPREFIX:C/-//g}" \ --with-system-zlib \ - --with-gcc-include-dir=/usr/include/c++/v1/ \ + --with-gxx-include-dir=/usr/include/c++/v1/ \ + --with-sysroot="/" \ --with-as=${LOCALBASE}/bin/${BU_PREFIX}-as \ --with-ld=${LOCALBASE}/bin/${BU_PREFIX}-ld @@ -61,6 +62,8 @@ INSTALL_TARGET?= install-gcc post-patch: @${REINPLACE_CMD} -e '/LIBSTDCXX/ s/stdc\+\+/c\+\+/g ' \ ${WRKSRC}//gcc/cp/g++spec.c + @${REINPLACE_CMD} -e '/LOCAL_INCLUDE_DIR/ d ' \ + ${WRKSRC}//gcc/Makefile.in .if ${TARGETARCH} == "amd64" CONFIGURE_TARGET= x86_64-unknown-${OPSYS:tl}${OSREL} @@ -94,6 +97,7 @@ post-install: .endfor @${RM} ${STAGEDIR}${PREFIX}/info/* @${RM} ${STAGEDIR}${PREFIX}/man/man7/* + @${RM} -r ${STAGEDIR}${PREFIX}/lib/gcc/${GCC_TARGET}/${PORTVERSION}/include-fixed # This port and all its slave ports fail on aarch64 -- but not all # the same way. Provide this definition for the master. diff --git a/devel/powerpc64-gcc/pkg-plist b/devel/powerpc64-gcc/pkg-plist index 3b1dc5f26556..f1a7d3ff9fbc 100644 --- a/devel/powerpc64-gcc/pkg-plist +++ b/devel/powerpc64-gcc/pkg-plist @@ -8,9 +8,6 @@ bin/%%GCC_TARGET%%-gcc-nm bin/%%GCC_TARGET%%-gcc-ranlib bin/%%GCC_TARGET%%-gcov bin/%%GCC_TARGET%%-gcov-tool -lib/gcc/%%GCC_TARGET%%/%%GCC_VERSION%%/include-fixed/README -lib/gcc/%%GCC_TARGET%%/%%GCC_VERSION%%/include-fixed/limits.h -lib/gcc/%%GCC_TARGET%%/%%GCC_VERSION%%/include-fixed/syslimits.h lib/gcc/%%GCC_TARGET%%/%%GCC_VERSION%%/include/altivec.h lib/gcc/%%GCC_TARGET%%/%%GCC_VERSION%%/include/float.h lib/gcc/%%GCC_TARGET%%/%%GCC_VERSION%%/include/htmintrin.h diff --git a/devel/sparc64-gcc/pkg-plist b/devel/sparc64-gcc/pkg-plist index a3c059b2af46..06c835490a50 100644 --- a/devel/sparc64-gcc/pkg-plist +++ b/devel/sparc64-gcc/pkg-plist @@ -8,9 +8,6 @@ bin/%%GCC_TARGET%%-gcc-nm bin/%%GCC_TARGET%%-gcc-ranlib bin/%%GCC_TARGET%%-gcov bin/%%GCC_TARGET%%-gcov-tool -lib/gcc/%%GCC_TARGET%%/%%GCC_VERSION%%/include-fixed/README -lib/gcc/%%GCC_TARGET%%/%%GCC_VERSION%%/include-fixed/limits.h -lib/gcc/%%GCC_TARGET%%/%%GCC_VERSION%%/include-fixed/syslimits.h lib/gcc/%%GCC_TARGET%%/%%GCC_VERSION%%/include/float.h lib/gcc/%%GCC_TARGET%%/%%GCC_VERSION%%/include/iso646.h lib/gcc/%%GCC_TARGET%%/%%GCC_VERSION%%/include/stdalign.h |