diff options
| author | Nico Huber <nico.huber@secunet.com> | 2016-09-20 12:59:53 +0200 |
|---|---|---|
| committer | Patrick Georgi <pgeorgi@google.com> | 2016-09-21 17:03:23 +0200 |
| commit | 156d87c024697161333087e155e656efeb070bcc (patch) | |
| tree | 80ff14067b76af2643b2bf3186735463c0fc12bb /util/crossgcc/buildgcc | |
| parent | aee7f46cd3177e469f008a4a48a7b253cb9b7c2a (diff) | |
buildgcc: Warn when building GCC with a different major version
GCC build instruction recommend to bootstrap a native compiler first.
Not sure, when that is really necessary. A major version change seems
reasonable.
Change-Id: I80a9ec25739b7d33a1d1c7b4b2140d19d89a99ae
Signed-off-by: Nico Huber <nico.huber@secunet.com>
Reviewed-on: https://review.coreboot.org/16675
Tested-by: build bot (Jenkins)
Reviewed-by: Patrick Georgi <pgeorgi@google.com>
Diffstat (limited to 'util/crossgcc/buildgcc')
| -rwxr-xr-x | util/crossgcc/buildgcc | 14 |
1 files changed, 13 insertions, 1 deletions
diff --git a/util/crossgcc/buildgcc b/util/crossgcc/buildgcc index 07425921af..2907c098b9 100755 --- a/util/crossgcc/buildgcc +++ b/util/crossgcc/buildgcc @@ -225,6 +225,16 @@ hostcc_version() { printf "%d.%d" "$(hostcc_major)" "$(hostcc_minor)" } +check_cc() { + if is_package_enabled "GCC"; then + if [ "$(hostcc_major)" != "$(buildcc_major)" -a "${BOOTSTRAP}" != "1" ]; then + printf "\n${red}warning: Building GCC $(buildcc_version) with a different major " + printf "version ($(hostcc_version)).\n" + printf " Bootstrapping (-b) is recommended.${NC}\n\n" + fi + fi +} + check_sum() { test -z "$CHECKSUM" || \ test "$(cat sum/$1.cksum 2>/dev/null | sed -e 's@.*\([0-9a-f]\{40,\}\).*@\1@')" = \ @@ -836,6 +846,9 @@ searchtool bzip2 "bzip2," > /dev/null check_for_library "-lz" "zlib (zlib1g-dev or zlib-devel)" +CC=cc +check_cc + if [ "$HALT_FOR_TOOLS" -ne 0 ]; then exit 1 fi @@ -848,7 +861,6 @@ fi # Set up host compiler and flags needed for various OSes -CC=cc if is_package_enabled "GCC"; then if [ $UNAME = "Darwin" ]; then #GCC_OPTIONS="$GCC_OPTIONS --enable-threads=posix" |