mirror of
https://github.com/xen-project/xen.git
synced 2026-10-28 02:01:24 +01:00
CI: Fix build script when CROSS_COMPILE is in use
Some testcases use a cross compiler. Presently it's only arm32 and due to
previous cleanup the only thing which is now wrong is printing the compiler
version at the start of day.
Construct $cc to match what `make` will eventually choose given CROSS_COMPILE,
taking care not to modify $CC. Use $cc throughout the rest of the script.
Also correct the compiler detection logic. Plain "gcc" was wrong, and
"clang"* was a bodge highlighting the issue, but neither survive the
CROSS_COMPILE correction. Instead, construct cc_is_{gcc,clang} booleans like
we do elsewhere in the build system, by querying the --version text for gcc or
clang.
While making this change, adjust cc_ver to be calculated once at the same time
as cc_is_* are calculated.
Signed-off-by: Andrew Cooper <andrew.cooper3@citrix.com>
Reviewed-by: Stefano Stabellini <sstabellini@kernel.org>
This commit is contained in:
1 parent
d329ca6baf
commit
df57a2c8da
1 file changed
+14
-8
@@ -2,13 +2,12 @@
|
||||
|
||||
test -f /etc/os-release && cat "$_"
|
||||
|
||||
$CC --version
|
||||
# Construct $cc such that it matches what `make` will chose when taking
|
||||
# CROSS_COMPILE into account. Do not modify $CC directly, as that will cause
|
||||
# `make` to double-account CROSS_COMPILE.
|
||||
cc="${CROSS_COMPILE}${CC}"
|
||||
|
||||
# Express the compiler version as an integer. e.g. GCC 4.9.2 => 0x040902
|
||||
cc-ver()
|
||||
{
|
||||
$CC -dumpversion | awk -F. '{ printf "0x%02x%02x%02x", $1, $2, $3 }'
|
||||
}
|
||||
$cc --version
|
||||
|
||||
# random config or default config
|
||||
if [[ "${RANDCONFIG}" == "y" ]]; then
|
||||
@@ -50,7 +49,14 @@ else
|
||||
cfgargs=()
|
||||
cfgargs+=("--enable-docs")
|
||||
|
||||
if [[ "${CC}" == "clang"* ]]; then
|
||||
# booleans for which compiler is in use
|
||||
cc_is_gcc="$($cc --version | grep -q gcc && echo "y" || :)"
|
||||
cc_is_clang="$($cc --version | grep -q clang && echo "y" || :)"
|
||||
|
||||
# The compiler version as an integer. e.g. GCC 4.9.2 => 0x040902
|
||||
cc_ver="$($cc -dumpversion | awk -F. '{ printf "0x%02x%02x%02x", $1, $2, $3 }')"
|
||||
|
||||
if [[ "${cc_is_clang}" == "y" ]]; then
|
||||
# SeaBIOS cannot be built with clang
|
||||
cfgargs+=("--with-system-seabios=/usr/share/seabios/bios.bin")
|
||||
# iPXE cannot be built with clang
|
||||
@@ -73,7 +79,7 @@ else
|
||||
fi
|
||||
|
||||
# SeaBIOS requires GCC 4.6 or later
|
||||
if [[ "${CC}" == "gcc" && `cc-ver` -lt 0x040600 ]]; then
|
||||
if [[ "${cc_is_gcc}" == "y" && "${cc_ver}" -lt 0x040600 ]]; then
|
||||
cfgargs+=("--with-system-seabios=/bin/false")
|
||||
fi
|
||||
|
||||
|
||||