[PATCH] libpcre: fix occasionally do_package_write_rpm error

[PATCH] libpcre: fix occasionally do_package_write_rpm error

[Changqing Li]

From: Changqing Li <changqing.li@windriver.com>

Error info:
ERROR: libpcre-8.44-r0 do_package_write_rpm: Failure expanding variable
SUMMARY_libpcrecpp, expression was ${SUMMARY} - C++ wrapper functions
which triggered exception RecursionError: maximum recursion depth
exceeded while calling a Python object

this error is hard to reproduce, only met one time. Seems expand SUMMARY
with reference to SUMMARY cause a dead loop.

Signed-off-by: Changqing Li <changqing.li@windriver.com>
---
 meta/recipes-support/libpcre/libpcre_8.44.bb | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/meta/recipes-support/libpcre/libpcre_8.44.bb b/meta/recipes-support/libpcre/libpcre_8.44.bb
index e5471e81da..102b92b4dd 100644
--- a/meta/recipes-support/libpcre/libpcre_8.44.bb
+++ b/meta/recipes-support/libpcre/libpcre_8.44.bb
@@ -40,8 +40,8 @@ EXTRA_OECONF = "--enable-utf"
 
 PACKAGES =+ "libpcrecpp libpcreposix pcregrep pcregrep-doc pcretest pcretest-doc"
 
-SUMMARY_libpcrecpp = "${SUMMARY} - C++ wrapper functions"
-SUMMARY_libpcreposix = "${SUMMARY} - C wrapper functions based on the POSIX regex API"
+SUMMARY_libpcrecpp = "Perl Compatible Regular Expressions - C++ wrapper functions"
+SUMMARY_libpcreposix = "Perl Compatible Regular Expressions - C wrapper functions based on the POSIX regex API"
 SUMMARY_pcregrep = "grep utility that uses perl 5 compatible regexes"
 SUMMARY_pcregrep-doc = "grep utility that uses perl 5 compatible regexes - docs"
 SUMMARY_pcretest = "program for testing Perl-comatible regular expressions"
-- 
2.17.1

Re: [OE-core] [PATCH] libpcre: fix occasionally do_package_write_rpm error

[Richard Purdie]

On Fri, 2020-08-28 at 14:14 +0800, Changqing Li wrote:
> From: Changqing Li <changqing.li@windriver.com>
> 
> Error info:
> ERROR: libpcre-8.44-r0 do_package_write_rpm: Failure expanding
> variable
> SUMMARY_libpcrecpp, expression was ${SUMMARY} - C++ wrapper functions
> which triggered exception RecursionError: maximum recursion depth
> exceeded while calling a Python object
> 
> this error is hard to reproduce, only met one time. Seems expand
> SUMMARY with reference to SUMMARY cause a dead loop.
> 
> Signed-off-by: Changqing Li <changqing.li@windriver.com>

What is worrying me is why we can't reproduce this. It should fail or
not fail, why do we only sometimes see this?

I'd like to understand the cause as it sounds like a determinism
problem somewhere.

Is there a sequence we can reproduce it with?

Cheers,

Richard

Re: [OE-core] [PATCH] libpcre: fix occasionally do_package_write_rpm error

[Changqing Li]

[-- Attachment #1: Type: text/plain, Size: 1093 bytes --]


On 9/1/20 7:08 PM, Richard Purdie wrote:
> On Fri, 2020-08-28 at 14:14 +0800, Changqing Li wrote:
>> From: Changqing Li <changqing.li@windriver.com>
>>
>> Error info:
>> ERROR: libpcre-8.44-r0 do_package_write_rpm: Failure expanding
>> variable
>> SUMMARY_libpcrecpp, expression was ${SUMMARY} - C++ wrapper functions
>> which triggered exception RecursionError: maximum recursion depth
>> exceeded while calling a Python object
>>
>> this error is hard to reproduce, only met one time. Seems expand
>> SUMMARY with reference to SUMMARY cause a dead loop.
>>
>> Signed-off-by: Changqing Li <changqing.li@windriver.com>
> What is worrying me is why we can't reproduce this. It should fail or
> not fail, why do we only sometimes see this?
>
> I'd like to understand the cause as it sounds like a determinism
> problem somewhere.
>
> Is there a sequence we can reproduce it with?

Unfortunately no, we also hard to reproduce this, only met one time on 
08 Jul when build a wrlinux image.

And don't see it for about 2 months.

>
> Cheers,
>
> Richard
>
>
> 

[-- Attachment #2: Type: text/html, Size: 3006 bytes --]

Re: [OE-core] [PATCH] libpcre: fix occasionally do_package_write_rpm error

[Richard Purdie]

On Wed, 2020-09-02 at 10:36 +0800, Changqing Li wrote:
> 
> On 9/1/20 7:08 PM, Richard Purdie wrote:
> > On Fri, 2020-08-28 at 14:14 +0800, Changqing Li wrote:
> > > From: Changqing Li <changqing.li@windriver.com>
> > > 
> > > Error info:
> > > ERROR: libpcre-8.44-r0 do_package_write_rpm: Failure expanding
> > > variable
> > > SUMMARY_libpcrecpp, expression was ${SUMMARY} - C++ wrapper
> > > functions
> > > which triggered exception RecursionError: maximum recursion depth
> > > exceeded while calling a Python object
> > > 
> > > this error is hard to reproduce, only met one time. Seems expand
> > > SUMMARY with reference to SUMMARY cause a dead loop.
> > > 
> > > Signed-off-by: Changqing Li <changqing.li@windriver.com>
> > 
> > What is worrying me is why we can't reproduce this. It should fail
> > or
> > not fail, why do we only sometimes see this?
> > 
> > I'd like to understand the cause as it sounds like a determinism
> > problem somewhere.
> > 
> > Is there a sequence we can reproduce it with?
> 
> Unfortunately no, we also hard to reproduce this, only met one time
> on 08 Jul when build a wrlinux image.
> 
> And don't see it for about 2 months.

I'd like to have a better understanding of the issue before we merge
potential fixes as there are probably other recipes which also
reference SUMMARY like this and I don't want to patch them all without
understanding the problem.

Cheers,

Richard