Re: [oe] [meta-oe][PATCH] protobuf-c: fix race condition

2019-10-23 Thread Martin Jansa
On Wed, Oct 23, 2019 at 08:50:10AM -0700, akuster808 wrote: > > > On 10/22/19 2:37 PM, Martin Jansa wrote: > > On Fri, Oct 18, 2019 at 09:06:54PM +0200, Martin Jansa wrote: > >> One more data point is that it might be fixed completely with 1.3.2 version > >> used in zeus/master and failing less

Re: [oe] [meta-oe][PATCH] protobuf-c: fix race condition

2019-10-23 Thread akuster808
On 10/22/19 2:37 PM, Martin Jansa wrote: > On Fri, Oct 18, 2019 at 09:06:54PM +0200, Martin Jansa wrote: >> One more data point is that it might be fixed completely with 1.3.2 version >> used in zeus/master and failing less often then without this patch in 1.3.1 >> used in thud/warrior. >> >> I

Re: [oe] [meta-oe][PATCH] protobuf-c: fix race condition

2019-10-22 Thread Martin Jansa
On Fri, Oct 18, 2019 at 09:06:54PM +0200, Martin Jansa wrote: > One more data point is that it might be fixed completely with 1.3.2 version > used in zeus/master and failing less often then without this patch in 1.3.1 > used in thud/warrior. > > I was rebuilding it 300 times in thud/warrior/zeus

Re: [oe] [meta-oe][PATCH] protobuf-c: fix race condition

2019-10-18 Thread Martin Jansa
One more data point is that it might be fixed completely with 1.3.2 version used in zeus/master and failing less often then without this patch in 1.3.1 used in thud/warrior. I was rebuilding it 300 times in thud/warrior/zeus and got 5-6 failures in thud/warrior but none in zeus. I've started

Re: [oe] [meta-oe][PATCH] protobuf-c: fix race condition

2019-10-18 Thread Khem Raj
Yes I have seen same build failures on master even after this fix on machine with more cpus so the fix while seems to address the problem is not fixing is completely when I get a chance I will look into the logic myself too but I think we have to delve into it a bit More On Fri, Oct 18, 2019 at

Re: [oe] [meta-oe][PATCH] protobuf-c: fix race condition

2019-10-17 Thread Sinan Kaya
On 10/17/2019 3:11 PM, Martin Jansa wrote: > Hmm I spoke too soon, at least rebuilding protobuf-c-native with this > change backported to thud still failed once with: Maybe, there are multiple issues as you mentioned. I'm monitoring our build pipeline for failures too. I have always seen this:

Re: [oe] [meta-oe][PATCH] protobuf-c: fix race condition

2019-10-17 Thread Martin Jansa
Hmm I spoke too soon, at least rebuilding protobuf-c-native with this change backported to thud still failed once with: ../git/t/generated-code2/cxx-generate-packed-data.cc:28:17: error: ‘foo’ is not a namespace-name using namespace foo; ^~~

Re: [oe] [meta-oe][PATCH] protobuf-c: fix race condition

2019-10-17 Thread Martin Jansa
On Sun, Sep 29, 2019 at 05:29:56PM +0800, Chen Qi wrote: > Signed-off-by: Chen Qi > --- > .../0001-avoid-race-condition.patch | 36 +++ > .../protobuf/protobuf-c_1.3.2.bb | 1 + > 2 files changed, 37 insertions(+) > create mode 100644 >

[oe] [meta-oe][PATCH] protobuf-c: fix race condition

2019-09-29 Thread Chen Qi
Signed-off-by: Chen Qi --- .../0001-avoid-race-condition.patch | 36 +++ .../protobuf/protobuf-c_1.3.2.bb | 1 + 2 files changed, 37 insertions(+) create mode 100644 meta-oe/recipes-devtools/protobuf/protobuf-c/0001-avoid-race-condition.patch diff --git