Bug 814944

Summary:	dev-libs/jansson-2.14 fails tests
Product:	Gentoo Linux	Reporter:	Agostino Sarubbo <ago>
Component:	Current packages	Assignee:	David Zero <zero-one>
Status:	RESOLVED FIXED
Severity:	normal	CC:	eike, kevinlyles, proxy-maint, sam
Priority:	Normal
Version:	unspecified
Hardware:	All
OS:	Linux
Whiteboard:
Package list:		Runtime testing required:	---
Attachments:	build.log 1-test-suite.log

Description Agostino Sarubbo gentoo-dev

2021-09-26 07:25:52 UTC

https://blogs.gentoo.org/ago/2020/07/04/gentoo-tinderbox/

Issue: dev-libs/jansson-2.14 fails tests.
Discovered on: amd64 (internal ref: ci)

NOTE:
This machine uses GCC-11: https://gcc.gnu.org/gcc-11/porting_to.html
If you think this is a GCC-11 related issue, please block bug 732706.

Comment 1 Agostino Sarubbo gentoo-dev

2021-09-26 07:25:54 UTC

Created attachment 741330 [details]
build.log

build log and emerge --info

Comment 2 Agostino Sarubbo gentoo-dev

2021-09-26 07:25:55 UTC

Created attachment 741333 [details]
1-test-suite.log

1-test-suite.log

Comment 3 David Zero 2021-09-26 08:17:13 UTC

The "check-exports" test is failing when run by portage, but passing when I run "make check" from the shell. For some reason, the sed script in this line isn't being applied:

> grep ' [DT] ' $test_log/symbols | cut -d' ' -f3 | grep -v '^_' | sed 's/@@libjansson.*//' | sort >$test_log/output

Comment 4 Rolf Eike Beer archtester

2021-10-13 15:59:21 UTC

> > grep ' [DT] ' $test_log/symbols | cut -d' ' -f3 | grep -v '^_' | sed 's/@@libjansson.*//' | sort >$test_log/output

I can't see that line anywhere. It would likely work if you backport https://github.com/akheron/jansson/commit/0677666f65b988b2dd44d02966a08fea490d5883 into the ebuild…

Comment 5 Larry the Git Cow gentoo-dev

2021-11-12 22:24:00 UTC

The bug has been closed via the following commit(s):

https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=2df0372e65704f40fa9529f36d468cec59dd8ca0

commit 2df0372e65704f40fa9529f36d468cec59dd8ca0
Author:     Rolf Eike Beer <eike@sf-mail.de>
AuthorDate: 2021-11-12 21:59:32 +0000
Commit:     Sam James <sam@gentoo.org>
CommitDate: 2021-11-12 22:22:56 +0000

    dev-libs/jansson: properly handle symbol names in tests
    
    Closes: https://bugs.gentoo.org/814944
    Signed-off-by: Rolf Eike Beer <eike@sf-mail.de>
    Signed-off-by: Sam James <sam@gentoo.org>

 .../jansson/files/jansson-2.14-test-symbols.patch  | 22 ++++++++++++++++++++++
 dev-libs/jansson/jansson-2.14.ebuild               |  2 ++
 2 files changed, 24 insertions(+)

Additionally, it has been referenced in the following commit(s):

https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=5cf776af229d8f55d1daf955e14d2280a1c61429

commit 5cf776af229d8f55d1daf955e14d2280a1c61429
Author:     Sam James <sam@gentoo.org>
AuthorDate: 2021-11-12 22:23:45 +0000
Commit:     Sam James <sam@gentoo.org>
CommitDate: 2021-11-12 22:23:45 +0000

    dev-libs/jansson: add links to patch
    
    Bug: https://bugs.gentoo.org/814944
    Signed-off-by: Sam James <sam@gentoo.org>

 dev-libs/jansson/files/jansson-2.14-test-symbols.patch | 3 +++
 1 file changed, 3 insertions(+)