HI Heinrich, On Thu, 11 Feb 2021 at 10:34, Simon Glass <[email protected]> wrote: > > On Wed, 10 Feb 2021 at 10:59, Heinrich Schuchardt <[email protected]> wrote: > > > > UEFI test files like helloworld.efi require an architecture specific > > PE-COFF header. > > > > Currently this does not work for cross compiling. If $CROSS_COMPILE is set, > > use the first part of the architecture triplet from the variable to > > choose the PE-COFF header. > > > > Now we can cross-compile the sandbox, e.g. > > > > make sandbox_defconfig NO_SDL=1 > > CROSS_COMPILE=/opt/bin/aarch64-linux-gnu- NO_SDL=1 MK_ARCH=aarch64 make > > > > Signed-off-by: Heinrich Schuchardt <[email protected]> > > --- > > v2: > > use $CROSS_COMPILE instead of an extra environment variable > > --- > > Makefile | 10 +++++++--- > > 1 file changed, 7 insertions(+), 3 deletions(-) >
With this patch if I try to build sandbox with CROSS_COMPILE=/bin/ it fails. In fact 'buildman -A sandbox' will return '/bin/' so I'd like your thoughts on what to do about this. Should we modify your patch or perhaps make buildman return something else with -A sandbox? Regards, Simon > Reviewed-by: Simon Glass <[email protected]>
