toolchain: Add support for Sourcery Codebench Standard
PATCH 1/1 toolchain: Add support for Sourcery Codebench Standard
---
Buildroot configuration:
Since these toolchain can't be downloaded, use the Custom toolchain backend.
Select:
Toolchain type (External toolchain)
Toolchain (Custom toolchain)
Toolchain origin (Pre-installed toolchain)
or
Toolchain origin (Toolchain to be downloaded and installed)*
Then set all toolchain settings (kernel headers, libc, rpc, c++ etc...)
Last but not least, set the Target Optimizations with -msgxx-glibc
*Just copy the toolchain archive to dl directory.
--- toolchain/helpers.mk | 23 ++++++++++++++-------- toolchain/toolchain-external/toolchain-external.mk | 11 +++++++++-- 2 files changed, 24 insertions(+), 10 deletions(-)
toolchain/helpers.mk +15 -8
@@ -193,16 +193,22 @@ check_glibc_feature = \ # Check the availability of RPC support in a glibc toolchain # # $1: sysroot directory +# $2: arch sysroot directory # check_glibc_rpc_feature = \ IS_IN_LIBC=`test -f $(1)/usr/include/rpc/rpc.h && echo y` ; \ - if [ "$(BR2_TOOLCHAIN_HAS_NATIVE_RPC)" != "y" -a "$${IS_IN_LIBC}" = "y" ] ; then \ - echo "RPC support available in C library, please enable BR2_TOOLCHAIN_HAS_NATIVE_RPC" ; \ - exit 1 ; \ + IS_IN_ARCH_LIBC=`test -f $(2)/usr/include/rpc/rpc.h && echo y` ; \ + if [ "$(BR2_TOOLCHAIN_HAS_NATIVE_RPC)" != "y" ] ; then \ + if [ "$${IS_IN_ARCH_LIBC}" = "y" -o "$${IS_IN_LIBC}" = "y" ] ; then \ + echo "RPC support available in C library, please enable BR2_TOOLCHAIN_HAS_NATIVE_RPC" ; \ + exit 1 ; \ + fi ; \ fi ; \ - if [ "$(BR2_TOOLCHAIN_HAS_NATIVE_RPC)" = "y" -a "$${IS_IN_LIBC}" != "y" ] ; then \ - echo "RPC support not available in C library, please disable BR2_TOOLCHAIN_HAS_NATIVE_RPC" ; \ - exit 1 ; \ + if [ "$(BR2_TOOLCHAIN_HAS_NATIVE_RPC)" = "y" ] ; then \ + if [ "$${IS_IN_ARCH_LIBC}" != "y" -a "$${IS_IN_LIBC}" != "y" ] ; then \ + echo "RPC support not available in C library, please disable BR2_TOOLCHAIN_HAS_NATIVE_RPC" ; \ + exit 1 ; \ + fi ; \ fi #
@@ -216,12 +222,13 @@ check_glibc_rpc_feature = \ # check_glibc = \ SYSROOT_DIR="$(strip $1)"; \ - if test `find $${SYSROOT_DIR}/ -maxdepth 2 -name 'ld-linux*.so.*' -o -name 'ld.so.*' -o -name 'ld64.so.*' | wc -l` -eq 0 ; then \ + ARCH_SYSROOT_DIR="$(strip $2)"; \ + if test `find $${SYSROOT_DIR}/ -maxdepth 3 -name 'ld-linux*.so.*' -o -name 'ld.so.*' -o -name 'ld64.so.*' | wc -l` -eq 0 ; then \ echo "Incorrect selection of the C library"; \ exit -1; \ fi; \ $(call check_glibc_feature,BR2_USE_MMU,MMU support) ;\ - $(call check_glibc_rpc_feature,$${SYSROOT_DIR}) + $(call check_glibc_rpc_feature,$${SYSROOT_DIR},$${ARCH_SYSROOT_DIR}) # # Check that the selected C library really is musl
toolchain/toolchain-external/toolchain-external.mk +9 -2
@@ -458,8 +458,12 @@ define TOOLCHAIN_EXTERNAL_CONFIGURE_CMDS @echo "External toolchain doesn't support --sysroot. Cannot use." ; \ exit 1 ; \ fi ; \ + ARCH_SYSROOT_DIR="$(call toolchain_find_sysroot,$(TOOLCHAIN_EXTERNAL_CC) $(TOOLCHAIN_EXTERNAL_CFLAGS))" ; \ + if test ! -d "$${SYSROOT_DIR}" ; then \ + SYSROOT_DIR=$$(echo -n "$${ARCH_SYSROOT_DIR}" | sed -r 's:(/libc/)([^&]*):\1:') ; \ + fi ; \ $(call check_kernel_headers_version,\ - $(call toolchain_find_sysroot,$(TOOLCHAIN_EXTERNAL_CC)),\ + $(call toolchain_find_sysroot,$(TOOLCHAIN_EXTERNAL_CC) $(TOOLCHAIN_EXTERNAL_CFLAGS)),\ $(call qstrip,$(BR2_TOOLCHAIN_HEADERS_AT_LEAST))); \ if test "$(BR2_arm)" = "y" ; then \ $(call check_arm_abi,\
@@ -474,7 +478,7 @@ define TOOLCHAIN_EXTERNAL_CONFIGURE_CMDS elif test "$(BR2_TOOLCHAIN_EXTERNAL_MUSL)" = "y" ; then \ $(call check_musl,$${SYSROOT_DIR}) ; \ else \ - $(call check_glibc,$${SYSROOT_DIR}) ; \ + $(call check_glibc,$${SYSROOT_DIR},$${ARCH_SYSROOT_DIR}) ; \ fi endef
@@ -548,6 +552,9 @@ define TOOLCHAIN_EXTERNAL_INSTALL_CORE exit 1 ; \ fi ; \ ARCH_SYSROOT_DIR="$(call toolchain_find_sysroot,$(TOOLCHAIN_EXTERNAL_CC) $(TOOLCHAIN_EXTERNAL_CFLAGS))" ; \ + if test ! -d "$${SYSROOT_DIR}" ; then \ + SYSROOT_DIR=$$(echo -n "$${ARCH_SYSROOT_DIR}" | sed -r 's:(/libc/)([^&]*):\1:') ; \ + fi ; \ ARCH_LIB_DIR="$(call toolchain_find_libdir,$(TOOLCHAIN_EXTERNAL_CC) $(TOOLCHAIN_EXTERNAL_CFLAGS))" ; \ SUPPORT_LIB_DIR="" ; \ if test `find $${ARCH_SYSROOT_DIR} -name 'libstdc++.a' | wc -l` -eq 0 ; then \
Signature · 1 line
1.9.3
PATCH 1/1 toolchain: Add support for Sourcery Codebench Standard
Dear Romain Naour,
Quoted 1 line · Romain Naour
Signed-off-by Romain Naour <[email protected]>
Thanks. Could you improve a bit the commit log to explain *why* those changes are needed? I don't have access to the Sourcery Codebench Standard toolchains (though I'm trying to arrange that), so I can't review the patch since I have no idea what are the differences between the Standard toolchains and the Lite toolchains.
Thanks,
Thomas
Signature · 3 lines
Thomas Petazzoni, CTO, Free Electrons Embedded Linux, Kernel and Android engineering http://free-electrons.com
PATCH 1/1 toolchain: Add support for Sourcery Codebench Standard
Hi Thomas,
Le 19/06/2014 08:27, Thomas Petazzoni a ?crit :
Quoted 11 lines
Dear Romain Naour,
Quoted 1 line · Romain Naour
Signed-off-by Romain Naour <[email protected]>Thanks. Could you improve a bit the commit log to explain *why* those changes are needed? I don't have access to the Sourcery Codebench Standard toolchains (though I'm trying to arrange that), so I can't review the patch since I have no idea what are the differences between the Standard toolchains and the Lite toolchains.
Quoted 4 lines
Thanks,
Thomas
See my reply in this thread for some explanations:
http://lists.busybox.net/pipermail/buildroot/2014-June/099767.html
The main difference (for the toolchain wrapper) is that standard edition has two sysroot sub-directories:
$./i686-pc-linux-gnu-gcc -msgxx-glibc -print-sysroot "i686-pc-linux-gnu/libc/sgxx-glibc": Intel Pentium Pro - Included Glibc 32bits
The command used to find the ARCH_SYSROOT_DIR works. $./i686-pc-linux-gnu-gcc -msgxx-glibc -print-file-name=libc.a Full/path/to/i686-pc-linux-gnu/libc/sgxx-glibc/usr/lib/libc.a
$./i686-pc-linux-gnu-gcc -print-sysroot i686-pc-linux-gnu/libc/system32: use native system libraries (host)
The "i686-pc-linux-gnu/libc/system32" directory does not exist, there is only "i686-pc-linux-gnu/libc/sgxx-glibc"
Here is the problem, the command used to find the SYSROOT_DIR don't work. $./i686-pc-linux-gnu-gcc -print-file-name=libc.a
libc.a
it is normal since i686-pc-linux-gnu/libc/system32/usr/lib/libc.a is missing.
The toolchain wrapper does not handle the case where SYSROOT_DIR is not null and it's not a directory.
That why we need to check if SYSROOT_DIR is a directory, if not ARCH_SYSROOT_DIR is used to find it (using sed).
In the copy_toolchain_sysroot, we need to "Create a symbolic link that matches the name of the subdirectory for the architecture variant in the original sysroot". For that, SYSROOT_DIR and ARCH_SUBDIR must be different.
After that, toolchain_find_sysroot need the -msgxx-glibc flag to work.
Maybe check_glibc need to use ARCH_SYSROOT_DIR instead of SYSROOT_DIR ? For now, I added ARCH_SYSROOT_DIR as second argument, but it's only used for that case...
Tell me if you need more details, I know it's not easy for you to review this patch without reproducing the issue.
It would be nice if you get a license too ;-)
Ok, the commit log need to be *very* more verbose.
I don't expect that this patch to be commited as is without any review since it may break the toolchain wrapper...
PS: I'm using Sourcery CodeBench 2014.05
Best regards,
Romain