Bug 835988

Summary:	dev-java/snakeyaml-1.30 fails tests
Product:	Gentoo Linux	Reporter:	Agostino Sarubbo <ago>
Component:	Current packages	Assignee:	Java team <java>
Status:	RESOLVED FIXED
Severity:	normal	CC:	flow, gentoo
Priority:	Normal	Keywords:	PullRequest
Version:	unspecified
Hardware:	All
OS:	Linux
See Also:	https://github.com/gentoo/gentoo/pull/24745 https://bugs.gentoo.org/show_bug.cgi?id=780663 https://github.com/gentoo/gentoo/pull/27027
Whiteboard:
Package list:		Runtime testing required:	---
Attachments:	build.log

Description Agostino Sarubbo gentoo-dev

2022-03-25 09:58:49 UTC

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

Issue: dev-java/snakeyaml-1.30 fails tests.
Discovered on: amd64 (internal ref: ci)

Comment 1 Agostino Sarubbo gentoo-dev

2022-03-25 09:58:51 UTC

CC'ing also the author of the commit (a77bfe0e1aea86bdb3c2b49eba2422a0f759f269)

Comment 2 Agostino Sarubbo gentoo-dev

2022-03-25 09:58:52 UTC

Created attachment 767774 [details]
build.log

build log and emerge --info

Comment 3 Agostino Sarubbo gentoo-dev

2022-03-25 09:58:53 UTC

Error(s) that match a know pattern:


FAILURES!!!

Comment 4 Florian Schmaus gentoo-dev

2022-03-25 12:30:10 UTC

Looking at https://bitbucket.org/snakeyaml/snakeyaml/src/49227c24d741dafe3b519a0d5f9d8413d803d9f3/src/test/java/org/yaml/snakeyaml/issues/issue377/BillionLaughsAttackTest.java#lines-72 this smells like a false positive: The test expects the exception message thrown on (stack?)overflow to contain the string 'heap'. But I guess this could vary between Java runtime systems, and hence the test is not robust.

Comment 5 Florian Schmaus gentoo-dev

2022-03-25 20:16:11 UTC

FWIW, this is what I proposed to upstream to make the test more robust: https://bitbucket.org/snakeyaml/snakeyaml/pull-requests/7

Comment 6 Larry the Git Cow gentoo-dev

2022-03-26 12:21:39 UTC

The bug has been referenced in the following commit(s):

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

commit dab318c383567c82677d3edeb6efd34f07d60cc2
Author:     Volkmar W. Pogatzki <gentoo@pogatzki.net>
AuthorDate: 2022-03-25 14:25:39 +0000
Commit:     Florian Schmaus <flow@gentoo.org>
CommitDate: 2022-03-26 12:20:53 +0000

    dev-java/snakeyaml: Add snakeyaml-1.30-fix-test-check.patch
    
    Bug: https://bugs.gentoo.org/835988
    Package-Manager: Portage-3.0.30, Repoman-3.0.3
    Signed-off-by: Volkmar W. Pogatzki <gentoo@pogatzki.net>
    Closes: https://github.com/gentoo/gentoo/pull/24745
    Signed-off-by: Florian Schmaus <flow@gentoo.org>

 .../files/snakeyaml-1.30-fix-test-check.patch          | 18 ++++++++++++++++++
 dev-java/snakeyaml/snakeyaml-1.30.ebuild               |  7 +++++++
 2 files changed, 25 insertions(+)

Comment 7 Larry the Git Cow gentoo-dev

2022-08-28 05:17:18 UTC

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

https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=184fd3afda3a9fe998ade5bea6ba18f75f2a7343

commit 184fd3afda3a9fe998ade5bea6ba18f75f2a7343
Author:     Volkmar W. Pogatzki <gentoo@pogatzki.net>
AuthorDate: 2022-08-27 08:59:53 +0000
Commit:     Miroslav Šulc <fordfrog@gentoo.org>
CommitDate: 2022-08-28 05:17:14 +0000

    dev-java/snakeyaml: drop 1.30
    
    Closes: https://bugs.gentoo.org/835988
    Signed-off-by: Volkmar W. Pogatzki <gentoo@pogatzki.net>
    Signed-off-by: Miroslav Šulc <fordfrog@gentoo.org>

 dev-java/snakeyaml/snakeyaml-1.30.ebuild | 95 --------------------------------
 1 file changed, 95 deletions(-)