On 4/26/19 10:23 PM, Julien Grall wrote:
Hi Ian,

Thank you for looking into this.

On 4/26/19 5:39 PM, Ian Jackson wrote:
Signed-off-by: Ian Jackson <ian.jack...@eu.citrix.com>
CC: Julien Grall <julien.gr...@arm.com>
CC: Stefano Stabellini <sstabell...@kernel.org>
---
  ts-kernel-build | 19 ++++++++++++++++++-
  1 file changed, 18 insertions(+), 1 deletion(-)

diff --git a/ts-kernel-build b/ts-kernel-build
index 72ca98a1..29c6c43d 100755
--- a/ts-kernel-build
+++ b/ts-kernel-build
@@ -21,6 +21,7 @@ BEGIN { unshift @INC, qw(.); }
  use Osstest;
  use Osstest::TestSupport;
  use Osstest::BuildSupport;
+use Carp;
  tsreadconfig();
@@ -48,9 +49,24 @@ my $parms = $archparms->{ $r{arch} };
  sub enable_xen_config ();
+our $arch_envvars = '';
+
  sub cmd ($$) {
      my ($timeout, $script) = @_;
-    target_cmd_build($ho, $timeout, $builddir, $script);
+    target_cmd_build($ho, $timeout, $builddir, $arch_envvars.$script);
+}
+
+sub checkarch () {
+    return if $r{arch} eq $ho->{Arch};
+    my $cc = 'gcc';
+    if ($r{arch} eq 'armhf') {
+    target_install_packages($ho, "crossbuild-essential-$r{arch}");
+    $arch_envvars = <<END;
+    export ARCH=arm HOSTCC=$cc CROSS_COMPILE=arm-linux-gnueabihf-

NIT: HOSTCC is not necessary. It will be default to gcc if not passed.

Otherwise, the export looks write to me.

I forgot to mention that you can add my ack one in any case:

Acked-by: Julien Grall <julien.gr...@arm.com>

Cheers,

--
Julien Grall

_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xenproject.org
https://lists.xenproject.org/mailman/listinfo/xen-devel

Reply via email to