Buildroot Archive on lore.kernel.org
 help / color / mirror / Atom feed
* [Buildroot] [PATCH v2 1/2] toolchain-external: support gcc < 4.3
@ 2016-11-23 23:40 Arnout Vandecappelle
  2016-11-23 23:40 ` [Buildroot] [PATCH v2 2/2] toolchain: kill ADI Blackfin toolchain Arnout Vandecappelle
  2016-11-25 22:00 ` [Buildroot] [PATCH v2 1/2] toolchain-external: support gcc < 4.3 Thomas Petazzoni
  0 siblings, 2 replies; 5+ messages in thread
From: Arnout Vandecappelle @ 2016-11-23 23:40 UTC (permalink / raw)
  To: buildroot

We currently support gcc as old as 4.3. However, Buildroot works
perfectly well with even older gcc versions (tested with 4.1). So we
can add an option BR2_TOOLCHAIN_EXTERNAL_GCC_OLD to support that. The
help text of this option is written with plenty of discouragement.

We use _OLD and not something like _PRE_4_3, because at some point we
will likely remove the 4.3 option and what would then require a name
change.

We don't set any _AT_LEAST option in this case because it's no use -
there is no lower bound on the version in this case. We therefore leave
BR2_TOOLCHAIN_GCC_AT_LEAST empty (the implicit default). When it is
empty, we don't do a version check at all in check_gcc_version
(previously we errored out when it was empty).

Signed-off-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be>
---
v2: rebase on next, slightly reword commit message

We should also remove the 4.3 and 4.4 options, because they are
currently unused and (after the next patch) untested in the autobuilders.
---
 toolchain/helpers.mk                                          |  3 +--
 .../toolchain-external-custom/Config.in.options               | 11 +++++++++++
 2 files changed, 12 insertions(+), 2 deletions(-)

diff --git a/toolchain/helpers.mk b/toolchain/helpers.mk
index 3991bc1..72e7292 100644
--- a/toolchain/helpers.mk
+++ b/toolchain/helpers.mk
@@ -145,8 +145,7 @@ check_kernel_headers_version = \
 check_gcc_version = \
 	expected_version="$(strip $2)" ; \
 	if [ -z "$${expected_version}" ]; then \
-		printf "Internal error, gcc version unknown (no GCC_AT_LEAST_X_Y selected)\n"; \
-		exit 1 ; \
+		exit 0 ; \
 	fi; \
 	real_version=`$(1) --version | sed -r -e '1!d; s/^[^)]+\) ([^[:space:]]+).*/\1/;'` ; \
 	if [[ ! "$${real_version}" =~ ^$${expected_version}\. ]] ; then \
diff --git a/toolchain/toolchain-external/toolchain-external-custom/Config.in.options b/toolchain/toolchain-external/toolchain-external-custom/Config.in.options
index cdc2555..5ff2491 100644
--- a/toolchain/toolchain-external/toolchain-external-custom/Config.in.options
+++ b/toolchain/toolchain-external/toolchain-external-custom/Config.in.options
@@ -59,6 +59,17 @@ config BR2_TOOLCHAIN_EXTERNAL_GCC_4_3
 	bool "4.3.x"
 	select BR2_TOOLCHAIN_GCC_AT_LEAST_4_3
 
+config BR2_TOOLCHAIN_EXTERNAL_GCC_OLD
+	bool "older"
+	help
+	  Use this option if your GCC version is older than any of the
+	  above.
+
+	  Note that the Buildroot community doesn't do any testing with
+	  such old toolchains. Some packages may fail to build in
+	  surprising ways, or the generated root filesystem may not
+	  work at all. Use such old toolchains at your own risk.
+
 endchoice
 
 choice
-- 
2.10.2

^ permalink raw reply related	[flat|nested] 5+ messages in thread
end of thread, other threads:[~2016-11-25 22:01 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2016-11-23 23:40 [Buildroot] [PATCH v2 1/2] toolchain-external: support gcc < 4.3 Arnout Vandecappelle
2016-11-23 23:40 ` [Buildroot] [PATCH v2 2/2] toolchain: kill ADI Blackfin toolchain Arnout Vandecappelle
2016-11-24 15:21   ` Waldemar Brodkorb
2016-11-25 22:01   ` Thomas Petazzoni
2016-11-25 22:00 ` [Buildroot] [PATCH v2 1/2] toolchain-external: support gcc < 4.3 Thomas Petazzoni

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox