Pkg xen devel - Dec 2020 - [PATCH 14/19] debian/rules: Set CC/LD to enable cross-building

Always set $(CC) and $(LD) when building.  This has no effect on native
builds, but is needed for cross-builds to use the right compiler.

Signed-off-by: Elliott Mitchell <ehem+debian at m5p.com>
---
 debian/rules | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/debian/rules b/debian/rules
index 95f3cc4311..dffdef9a9d 100755
--- a/debian/rules
+++ b/debian/rules
@@ -135,6 +135,8 @@ dpkg_CPPFLAGS := $(shell $(dbmo) dpkg-buildflags --get
CPPFLAGS)
 dpkg_LDFLAGS  := $(shell $(dbmo) dpkg-buildflags --get LDFLAGS)
 
 make_args_common := \
+	CC=$(DEB_HOST_MULTIARCH)-gcc \
+	LD=$(DEB_HOST_MULTIARCH)-ld \
 	XEN_COMPILE_ARCH=$(xen_arch_$(DEB_BUILD_ARCH))
 
 make_args_xen := $(make_args_common) \
-- 


-- 
(\___(\___(\______          --=> 8-) EHM <=--          ______/)___/)___/)
 \BS (    |         ehem+sigmsg at m5p.com  PGP 87145445         |    )   /
  \_CS\   |  _____  -O #include <stddisclaimer.h> O-   _____  |   /  _/
8A19\___\_|_/58D2 7E3D DDF4 7BA6 <-PGP-> 41D1 B375 37D0 8714\_|_/___/5445

On 7/17/20 4:07 AM, Elliott Mitchell wrote:
> Always set $(CC) and $(LD) when building.  This has no effect on native
> builds, but is needed for cross-builds to use the right compiler.
> 
> Signed-off-by: Elliott Mitchell <ehem+debian at m5p.com>
Acked-by: Hans van Kranenburg <hans at knorrie.org>

Added to knorrie/sid, thanks.
Hans
> ---
>  debian/rules | 2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/debian/rules b/debian/rules
> index 95f3cc4311..dffdef9a9d 100755
> --- a/debian/rules
> +++ b/debian/rules
> @@ -135,6 +135,8 @@ dpkg_CPPFLAGS := $(shell $(dbmo) dpkg-buildflags --get
CPPFLAGS)
>  dpkg_LDFLAGS  := $(shell $(dbmo) dpkg-buildflags --get LDFLAGS)
>  
>  make_args_common := \
> +	CC=$(DEB_HOST_MULTIARCH)-gcc \
> +	LD=$(DEB_HOST_MULTIARCH)-ld \
>  	XEN_COMPILE_ARCH=$(xen_arch_$(DEB_BUILD_ARCH))
>  
>  make_args_xen := $(make_args_common) \
>

Hi,

On 7/17/20 4:07 AM, Elliott Mitchell wrote:
> Always set $(CC) and $(LD) when building.  This has no effect on native
> builds, but is needed for cross-builds to use the right compiler.
> 
> Signed-off-by: Elliott Mitchell <ehem+debian at m5p.com>
> ---
>  debian/rules | 2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/debian/rules b/debian/rules
> index 95f3cc4311..dffdef9a9d 100755
> --- a/debian/rules
> +++ b/debian/rules
> @@ -135,6 +135,8 @@ dpkg_CPPFLAGS := $(shell $(dbmo) dpkg-buildflags --get
CPPFLAGS)
>  dpkg_LDFLAGS  := $(shell $(dbmo) dpkg-buildflags --get LDFLAGS)
>  
>  make_args_common := \
> +	CC=$(DEB_HOST_MULTIARCH)-gcc \
> +	LD=$(DEB_HOST_MULTIARCH)-ld \
>  	XEN_COMPILE_ARCH=$(xen_arch_$(DEB_BUILD_ARCH))
>  
>  make_args_xen := $(make_args_common) \
> 
Hm, this causes an FTBFS on i386 :(

https://buildd.debian.org/status/fetch.php?pkg=xen&arch=i386&ver=4.14.0%2B88-g1d1d1f5391-1&stamp=1608036439&raw=0

scripts/Kconfig.include:39: compiler 'i386-linux-gnu-gcc' not found

Oh, yay, on i386, the thing is named /usr/bin/i686-linux-gnu-gcc, sigh.

Since it's most important *right now* to get the security patches out, I
will re-upload with this change reverted. Afterwards we can see what the
best solution will be.

Hans