Re: [PATCH] u-boot: update to 2015.01

[Denys Dmytriyenko <denis@denix.org>]

On Thu, Jan 22, 2015 at 04:23:16PM -0500, Denys Dmytriyenko wrote:
> On Thu, Jan 22, 2015 at 07:50:28AM -0200, Raphael Philipe wrote:
> > Hey Denys, I tested u-boot-fw-utils-cross_2014.07.bb in Ubuntu 12.04
> > and got the same error .
> > 
> > Can you confirm that this recipe works for you?
> 
> Ok, trying to bisect the breakage of the current 2014.07 version of 
> fw-utils-cross now:
> 
> Poky snapshot from mid-September, when 2014.07 update patch was submitted and 
> when it was validated by the autobuilder - still works fine.
> 
> Since it was a feature freeze time before the release, the patch was not 
> accepted and was queued for later.
> 
> Poky snapshot from mid-November, when the patch was finally applied - the 
> build breaks. Again, it may not be showing the issue on the autobuilder due to 
> an earlier successful sstate.
> 
> So, somewhere in between a change was introduced that breaks fw-utils-cross. 
> BTW, neither 2014.07 update nor 2015.01 change anything to the way 
> fw-utils-cross gets built.

Heh, and the commit that breaks (on purpose!) building 2014.07 version of 
u-boot-fw-utils-cross is the one from Richard (which does make sense):

gcc: poison default sysroot path
Various pieces of the code assume that the --sysroot option gets passed
into the compiler tools. By having a "sane" default, we don't always
spot when this occurs and this can later show up as breakage in sstate,
or in usage of the external toolchain.

We've long since talked about poisoning the default such that it will
break unless the correct option is specified. This patch does just that.

If this patch causes something to fail to build, it most likely means
the various compiler flags and commands are not correctly being passed
through to the underlying piece of software and that there is a real
problem that needs fixing, its not the fault of this patch.

http://git.yoctoproject.org/cgit/cgit.cgi/poky/commit/?id=678e8798ebe0f4fd1bd347db136f1499b8fe00c9

-- 
Denys