From: Yann E. MORIN <yann.morin.1998@free.fr>
To: buildroot@busybox.net
Subject: [Buildroot] [PATCH 03/14] toolchain: remove CodeSourcery sh toolchain
Date: Sun, 18 Jun 2017 11:09:40 +0200 [thread overview]
Message-ID: <20170618090940.GC2918@scaer> (raw)
In-Reply-To: <0fa55d92e86e034d134945a6950229963712102c.1497583686.git.baruch@tkos.co.il>
Baruch, All,
On 2017-06-16 06:32 +0300, Baruch Siach spake thusly:
> Since glibc 2.17, executable link command need not include the -lrt option for
> clock_* system calls. As a result, over time less and less software packages
> bother to check whether to toolchain needs -lrt. We are now at a point where
> maintainers refuse to add this complexity into their build system. This
> requires Buildroot to carry patches fixing this issue indefinitely.
>
> glibc 2.17 is now 4.5 years old. There is no reason to use an older version
> with current software.
>
> This commit removes the predefined profile for CodeSourcery sh toolchain that
> is based on glibc 2.16. One may still use the custom external toolchain
> support in Buildroot to get this toolchain back, and deal with any build
> issues that this toolchain causes.
>
> Signed-off-by: Baruch Siach <baruch@tkos.co.il>
Reviewed-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
Regards,
Yann E. MORIN.
> ---
> Config.in.legacy | 9 ++++++++
> toolchain/toolchain-external/Config.in | 6 ------
> .../toolchain-external-codesourcery-sh/Config.in | 25 ----------------------
> .../Config.in.options | 9 --------
> .../toolchain-external-codesourcery-sh.hash | 3 ---
> .../toolchain-external-codesourcery-sh.mk | 13 -----------
> 6 files changed, 9 insertions(+), 56 deletions(-)
> delete mode 100644 toolchain/toolchain-external/toolchain-external-codesourcery-sh/Config.in
> delete mode 100644 toolchain/toolchain-external/toolchain-external-codesourcery-sh/Config.in.options
> delete mode 100644 toolchain/toolchain-external/toolchain-external-codesourcery-sh/toolchain-external-codesourcery-sh.hash
> delete mode 100644 toolchain/toolchain-external/toolchain-external-codesourcery-sh/toolchain-external-codesourcery-sh.mk
>
> diff --git a/Config.in.legacy b/Config.in.legacy
> index b851af2017b5..9af0e1516ce8 100644
> --- a/Config.in.legacy
> +++ b/Config.in.legacy
> @@ -145,6 +145,15 @@ endif
> ###############################################################################
> comment "Legacy options removed in 2017.08"
>
> +config BR2_TOOLCHAIN_EXTERNAL_CODESOURCERY_SH
> + bool "SH Sourcery toolchain has been removed"
> + select BR2_LEGACY
> + help
> + The Sourcery CodeBench toolchain for the sh architecture has
> + been removed, since it uses glibc older than 2.17 that requires
> + -lrt to link executables using clock_* system calls. This makes
> + this toolchain difficult to maintain over time.
> +
> config BR2_TOOLCHAIN_EXTERNAL_CODESOURCERY_X86
> bool "x86 Sourcery toolchain has been removed"
> select BR2_LEGACY
> diff --git a/toolchain/toolchain-external/Config.in b/toolchain/toolchain-external/Config.in
> index 75229bbddf47..4cd9ee4752cd 100644
> --- a/toolchain/toolchain-external/Config.in
> +++ b/toolchain/toolchain-external/Config.in
> @@ -33,9 +33,6 @@ source "toolchain/toolchain-external/toolchain-external-codescape-mti-mips/Confi
> # NIOSII
> source "toolchain/toolchain-external/toolchain-external-codesourcery-niosII/Config.in"
>
> -# SH4a
> -source "toolchain/toolchain-external/toolchain-external-codesourcery-sh/Config.in"
> -
> # x86_64
> source "toolchain/toolchain-external/toolchain-external-codesourcery-amd64/Config.in"
>
> @@ -126,9 +123,6 @@ source "toolchain/toolchain-external/toolchain-external-codescape-mti-mips/Confi
> # NIOSII
> source "toolchain/toolchain-external/toolchain-external-codesourcery-niosII/Config.in.options"
>
> -# SH4a
> -source "toolchain/toolchain-external/toolchain-external-codesourcery-sh/Config.in.options"
> -
> # x86_64
> source "toolchain/toolchain-external/toolchain-external-codesourcery-amd64/Config.in.options"
>
> diff --git a/toolchain/toolchain-external/toolchain-external-codesourcery-sh/Config.in b/toolchain/toolchain-external/toolchain-external-codesourcery-sh/Config.in
> deleted file mode 100644
> index a47dab7611f9..000000000000
> --- a/toolchain/toolchain-external/toolchain-external-codesourcery-sh/Config.in
> +++ /dev/null
> @@ -1,25 +0,0 @@
> -config BR2_TOOLCHAIN_EXTERNAL_CODESOURCERY_SH
> - bool "Sourcery CodeBench SH 2012.09"
> - depends on BR2_sh4a || BR2_sh4aeb
> - depends on BR2_HOSTARCH = "x86_64" || BR2_HOSTARCH = "x86"
> - depends on !BR2_STATIC_LIBS
> - select BR2_TOOLCHAIN_EXTERNAL_GLIBC
> - select BR2_TOOLCHAIN_HAS_SSP
> - select BR2_TOOLCHAIN_HAS_NATIVE_RPC
> - select BR2_INSTALL_LIBSTDCPP
> - select BR2_HOSTARCH_NEEDS_IA32_LIBS
> - select BR2_TOOLCHAIN_HEADERS_AT_LEAST_3_5
> - select BR2_TOOLCHAIN_GCC_AT_LEAST_4_7
> - help
> - Sourcery CodeBench toolchain for the SuperH architecture,
> - from Mentor Graphics. It uses gcc 4.7.2, binutils 2.23.51,
> - glibc 2.16, uClibc 0.9.30, gdb 7.4.50 and kernel headers
> - 3.5.4. It has support for the following variants:
> - - SH4A, glibc, little endian
> - Default.
> - - SH4A, glibc, big endian
> - Add -mb to BR2_TARGET_OPTIMIZATION
> - - SH4A, uClibc, little endian
> - Not usable in Buildroot yet.
> - - SH4A, uClibc, big endian
> - Not usable in Buildroot yet.
> diff --git a/toolchain/toolchain-external/toolchain-external-codesourcery-sh/Config.in.options b/toolchain/toolchain-external/toolchain-external-codesourcery-sh/Config.in.options
> deleted file mode 100644
> index 90061cef49fd..000000000000
> --- a/toolchain/toolchain-external/toolchain-external-codesourcery-sh/Config.in.options
> +++ /dev/null
> @@ -1,9 +0,0 @@
> -if BR2_TOOLCHAIN_EXTERNAL_CODESOURCERY_SH
> -
> -config BR2_TOOLCHAIN_EXTERNAL_PREFIX
> - default "sh-linux-gnu"
> -
> -config BR2_PACKAGE_PROVIDES_TOOLCHAIN_EXTERNAL
> - default "toolchain-external-codesourcery-sh"
> -
> -endif
> diff --git a/toolchain/toolchain-external/toolchain-external-codesourcery-sh/toolchain-external-codesourcery-sh.hash b/toolchain/toolchain-external/toolchain-external-codesourcery-sh/toolchain-external-codesourcery-sh.hash
> deleted file mode 100644
> index ff2ba5f473c4..000000000000
> --- a/toolchain/toolchain-external/toolchain-external-codesourcery-sh/toolchain-external-codesourcery-sh.hash
> +++ /dev/null
> @@ -1,3 +0,0 @@
> -# Locally calculated
> -sha256 59d6766fde244931aa52db01433d5acd051998762a931121c5fc109536a1a802 renesas-2012.09-61-sh-linux-gnu-i686-pc-linux-gnu.tar.bz2
> -sha256 e2e58c10e52395d5d35157e35f85233f713c6f9223a652dfc56194cfd2eed004 renesas-2012.09-61-sh-linux-gnu.src.tar.bz2
> diff --git a/toolchain/toolchain-external/toolchain-external-codesourcery-sh/toolchain-external-codesourcery-sh.mk b/toolchain/toolchain-external/toolchain-external-codesourcery-sh/toolchain-external-codesourcery-sh.mk
> deleted file mode 100644
> index 5370bb22a820..000000000000
> --- a/toolchain/toolchain-external/toolchain-external-codesourcery-sh/toolchain-external-codesourcery-sh.mk
> +++ /dev/null
> @@ -1,13 +0,0 @@
> -################################################################################
> -#
> -# toolchain-external-sourcery-sh
> -#
> -################################################################################
> -
> -TOOLCHAIN_EXTERNAL_CODESOURCERY_SH_VERSION = 2012.09-61
> -
> -TOOLCHAIN_EXTERNAL_CODESOURCERY_SH_SITE = https://sourcery.mentor.com/public/gnu_toolchain/$(TOOLCHAIN_EXTERNAL_PREFIX)
> -TOOLCHAIN_EXTERNAL_CODESOURCERY_SH_SOURCE = renesas-$(TOOLCHAIN_EXTERNAL_CODESOURCERY_SH_VERSION)-$(TOOLCHAIN_EXTERNAL_PREFIX)-i686-pc-linux-gnu.tar.bz2
> -TOOLCHAIN_EXTERNAL_CODESOURCERY_SH_ACTUAL_SOURCE_TARBALL = renesas-$(TOOLCHAIN_EXTERNAL_CODESOURCERY_SH_VERSION)-$(TOOLCHAIN_EXTERNAL_PREFIX).src.tar.bz2
> -
> -$(eval $(toolchain-external-package))
> --
> 2.11.0
>
> _______________________________________________
> buildroot mailing list
> buildroot at busybox.net
> http://lists.busybox.net/mailman/listinfo/buildroot
--
.-----------------.--------------------.------------------.--------------------.
| Yann E. MORIN | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
| +33 662 376 056 | Software Designer | \ / CAMPAIGN | ___ |
| +33 223 225 172 `------------.-------: X AGAINST | \e/ There is no |
| http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL | v conspiracy. |
'------------------------------^-------^------------------^--------------------'
next prev parent reply other threads:[~2017-06-18 9:09 UTC|newest]
Thread overview: 41+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-06-16 3:32 [Buildroot] [PATCH 00/14] toolchain-external: remove support for glibc before 2.17 Baruch Siach
2017-06-16 3:32 ` [Buildroot] [PATCH 01/14] docs/manual: update external toolchain advantages list Baruch Siach
2017-06-18 9:02 ` Yann E. MORIN
2017-06-16 3:32 ` [Buildroot] [PATCH 02/14] toolchain: remove CodeSourcery x86 toolchain Baruch Siach
2017-06-18 9:08 ` Yann E. MORIN
2017-06-18 9:14 ` Baruch Siach
2017-06-18 9:19 ` Yann E. MORIN
2017-06-16 3:32 ` [Buildroot] [PATCH 03/14] toolchain: remove CodeSourcery sh toolchain Baruch Siach
2017-06-18 9:09 ` Yann E. MORIN [this message]
2017-06-16 3:32 ` [Buildroot] [PATCH 04/14] package: remove CodeSourcery sh dependencies Baruch Siach
2017-06-18 9:12 ` Yann E. MORIN
2017-06-16 3:32 ` [Buildroot] [PATCH 05/14] toolchain-external: update list of toolchains Baruch Siach
2017-06-18 9:21 ` Yann E. MORIN
2017-06-16 3:32 ` [Buildroot] [PATCH 06/14] docs/manual: drop mention of removed external toolchains Baruch Siach
2017-06-18 9:23 ` Yann E. MORIN
2017-06-16 3:32 ` [Buildroot] [PATCH 07/14] swupdate: don't link with librt Baruch Siach
2017-06-18 9:26 ` Yann E. MORIN
2017-06-16 3:32 ` [Buildroot] [PATCH 08/14] aiccu: " Baruch Siach
2017-06-18 9:27 ` Yann E. MORIN
2017-06-16 3:32 ` [Buildroot] [PATCH 09/14] libv4l: " Baruch Siach
2017-06-18 9:31 ` Yann E. MORIN
2017-06-18 9:34 ` Baruch Siach
2017-06-18 9:36 ` Yann E. MORIN
2017-06-16 3:32 ` [Buildroot] [PATCH 10/14] e2fsprogs: " Baruch Siach
2017-06-18 9:33 ` Yann E. MORIN
2017-06-18 10:13 ` Baruch Siach
2017-06-18 11:55 ` Yann E. MORIN
2017-06-19 19:14 ` Baruch Siach
2017-06-16 3:32 ` [Buildroot] [PATCH 11/14] liboping: no need to check for clock_gettime Baruch Siach
2017-06-18 9:37 ` Yann E. MORIN
2017-06-16 3:32 ` [Buildroot] [PATCH 12/14] openipmi: " Baruch Siach
2017-06-18 9:42 ` Yann E. MORIN
2017-06-16 3:32 ` [Buildroot] [PATCH 13/14] lirc-tools: " Baruch Siach
2017-06-18 9:45 ` Yann E. MORIN
2017-06-16 3:32 ` [Buildroot] [PATCH 14/14] softether: " Baruch Siach
2017-06-18 9:47 ` Yann E. MORIN
2017-06-20 20:23 ` [Buildroot] [PATCH 00/14] toolchain-external: remove support for glibc before 2.17 Thomas Petazzoni
2017-06-22 7:57 ` Arnout Vandecappelle
2017-06-22 17:01 ` Baruch Siach
2017-06-23 21:33 ` Waldemar Brodkorb
2017-06-25 4:57 ` Baruch Siach
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20170618090940.GC2918@scaer \
--to=yann.morin.1998@free.fr \
--cc=buildroot@busybox.net \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.