Re: [OE-core] [PATCH v2] go: ensure use of BUILD_CC when building bootstrap tools

2017-11-22 Thread Burton, Ross
Yes, just marked it. Thanks for the reminder. Ross On 22 November 2017 at 20:58, Matt Madison wrote: > On Fri, Nov 17, 2017 at 8:02 AM, Burton, Ross > wrote: > > I had a hunch that was the problem this morning after looking again at > the > > patch set. Thanks for confirming. > > Ross, > > C

Re: [OE-core] [PATCH v2] go: ensure use of BUILD_CC when building bootstrap tools

2017-11-22 Thread Matt Madison
On Fri, Nov 17, 2017 at 8:02 AM, Burton, Ross wrote: > I had a hunch that was the problem this morning after looking again at the > patch set. Thanks for confirming. Ross, Can the BUILD_CC patch go back into your queue again? Thanks, -Matt > > Ross > > On 17 November 2017 at 15:49, Matt Madis

Re: [OE-core] [PATCH v2] go: ensure use of BUILD_CC when building bootstrap tools

2017-11-17 Thread Khem Raj
On Fri, Nov 17, 2017 at 7:49 AM, Matt Madison wrote: > On Thu, Nov 16, 2017 at 3:29 PM, Matt Madison wrote: >> On Thu, Nov 16, 2017 at 12:29 PM, Burton, Ross wrote: >>> Looks like this is failing on some of our autobuilders, such as >>> https://autobuilder.yocto.io/builders/nightly-musl-x86-64/b

Re: [OE-core] [PATCH v2] go: ensure use of BUILD_CC when building bootstrap tools

2017-11-17 Thread Burton, Ross
I had a hunch that was the problem this morning after looking again at the patch set. Thanks for confirming. Ross On 17 November 2017 at 15:49, Matt Madison wrote: > On Thu, Nov 16, 2017 at 3:29 PM, Matt Madison > wrote: > > On Thu, Nov 16, 2017 at 12:29 PM, Burton, Ross > wrote: > >> Looks

Re: [OE-core] [PATCH v2] go: ensure use of BUILD_CC when building bootstrap tools

2017-11-17 Thread Matt Madison
On Thu, Nov 16, 2017 at 3:29 PM, Matt Madison wrote: > On Thu, Nov 16, 2017 at 12:29 PM, Burton, Ross wrote: >> Looks like this is failing on some of our autobuilders, such as >> https://autobuilder.yocto.io/builders/nightly-musl-x86-64/builds/58/steps/BuildImages/logs/stdio > > I'm looking into

Re: [OE-core] [PATCH v2] go: ensure use of BUILD_CC when building bootstrap tools

2017-11-16 Thread Matt Madison
On Thu, Nov 16, 2017 at 12:29 PM, Burton, Ross wrote: > Looks like this is failing on some of our autobuilders, such as > https://autobuilder.yocto.io/builders/nightly-musl-x86-64/builds/58/steps/BuildImages/logs/stdio I'm looking into it. Thanks, -Matt > > | cmd/go/internal/vet > | # cmd/api >

Re: [OE-core] [PATCH v2] go: ensure use of BUILD_CC when building bootstrap tools

2017-11-16 Thread Burton, Ross
Looks like this is failing on some of our autobuilders, such as https://autobuilder.yocto.io/builders/nightly-musl-x86-64/builds/58/steps/BuildImages/logs/stdio | cmd/go/internal/vet | # cmd/api | /home/pokybuild/yocto-autobuilder/yocto-worker/nightly-musl-x86-64/build/build/tmp/work/core2-64-poky

Re: [OE-core] [PATCH v2] go: ensure use of BUILD_CC when building bootstrap tools

2017-11-14 Thread Matt Madison
Yes, drat it, v3 should fix that. Thanks, -Matt On Tue, Nov 14, 2017 at 9:20 AM, Otavio Salvador wrote: > Hello Matt, > > A minor thing: > > Upstream-Status: Pending > > On Tue, Nov 14, 2017 at 3:18 PM, Matt Madison wrote: >> For cross-canadian builds, we were accidentally using >> the crosssdk

Re: [OE-core] [PATCH v2] go: ensure use of BUILD_CC when building bootstrap tools

2017-11-14 Thread Otavio Salvador
Hello Matt, A minor thing: Upstream-Status: Pending On Tue, Nov 14, 2017 at 3:18 PM, Matt Madison wrote: > For cross-canadian builds, we were accidentally using > the crosssdk C compiler when building the Go compiler > bootstrap. Add a patch to the make script to let us > use BUILD_CC, and pre

[OE-core] [PATCH v2] go: ensure use of BUILD_CC when building bootstrap tools

2017-11-14 Thread Matt Madison
For cross-canadian builds, we were accidentally using the crosssdk C compiler when building the Go compiler bootstrap. Add a patch to the make script to let us use BUILD_CC, and prepend do_compile to set it in the local environment to ensure that the trailing blank gets stripped, since that confus