diff mbox

[autobuild.buildroot.net] Build results for 2017-03-15

Message ID CAMo8BfJLJ6VQhdfJUhNoMTmWom3MrEyygVQy8-d+xJqhPM=Wfg@mail.gmail.com
State Not Applicable
Headers show

Commit Message

Max Filippov March 16, 2017, 6:20 p.m. UTC 
On Thu, Mar 16, 2017 at 12:48 AM, Baruch Siach <baruch@tkos.co.il> wrote:
> Hi Thomas, all,
>
> On Thu, Mar 16, 2017 at 08:29:14AM +0100, Thomas Petazzoni wrote:
>>       xtensa |            uboot-tools-2017.03 | NOK | http://autobuild.buildroot.net/results/439d6dacfaa7f9244bd15ca04990c12a25ff4777
>
>   HOSTLD  tools/mkenvimage
>   HOSTLD  tools/fdtgrep
> cc1: error: unrecognized command line option "-mlongcalls"
> cc1: error: unrecognized command line option "-mauto-litpools"
> error: command '/usr/bin/gcc' failed with exit status 1
>
> Host toolchain used instead of target. Can't reproduce it here.

It does not reproduce unless host package swig is installed. Then it looks
like it's supposed to build for the host, but somehow python uses the
value of CFLAGS environment variable. Given that host cflags are passed
to the python script through the command line I'd propose the following fix:
diff mbox

Patch

diff --git a/tools/Makefile b/tools/Makefile
index 1c840d7..c224e8a 100644
--- a/tools/Makefile
+++ b/tools/Makefile
@@ -120,7 +120,7 @@  _libfdt.so-sharedobjs += $(LIBFDT_OBJS)
 libfdt:

 tools/_libfdt.so: $(patsubst %.o,%.c,$(LIBFDT_OBJS)) tools/libfdt_wrap.c
-       LDFLAGS="$(HOSTLDFLAGS)" python $(srctree)/lib/libfdt/setup.py \
+       LDFLAGS="$(HOSTLDFLAGS)" CFLAGS="" python
$(srctree)/lib/libfdt/setup.py \
                "$(_hostc_flags)" $^
        mv _libfdt.so $@