Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!

Bug 834068

Summary: dev-cpp/gtest-1.11.0 fails tests on MUSL
Product: Gentoo Linux Reporter: Agostino Sarubbo <ago>
Component: Current packagesAssignee: Peter Levine <plevine457>
Status: RESOLVED FIXED    
Severity: normal CC: plevine457, proxy-maint
Priority: Normal Keywords: PullRequest
Version: unspecified   
Hardware: All   
OS: Linux   
See Also: https://github.com/gentoo/gentoo/pull/25073
Whiteboard:
Package list:
Runtime testing required: ---
Attachments: build.log
1-LastTest.log.xz
gtest-1.10.0_p20200702-increase-clone-stack-size.patch

Description Agostino Sarubbo gentoo-dev 2022-02-26 09:20:06 UTC
https://blogs.gentoo.org/ago/2020/07/04/gentoo-tinderbox/

Issue: dev-cpp/gtest-1.11.0 fails tests.
Discovered on: amd64 (internal ref: tinderbox_musl)

NOTE:
This machine uses MUSL libc
Comment 1 Agostino Sarubbo gentoo-dev 2022-02-26 09:20:09 UTC
Created attachment 765845 [details]
build.log

build log and emerge --info
Comment 2 Agostino Sarubbo gentoo-dev 2022-02-26 09:20:10 UTC
Created attachment 765846 [details]
1-LastTest.log.xz

1-LastTest.log (compressed because it exceeds attachment limit, use 'xzless' to read it)
Comment 3 Peter Levine 2022-03-16 06:35:29 UTC
Created attachment 767130 [details, diff]
gtest-1.10.0_p20200702-increase-clone-stack-size.patch

Could you try replacing "gtest-1.10.0_p20200702-increase-clone-stack-size.patch" in "/var/db/repos/gentoo/dev-cpp/gtest/files/" directory with this one and see if you can reproduce the issue?
Comment 4 Larry the Git Cow gentoo-dev 2022-04-18 20:28:36 UTC
The bug has been closed via the following commit(s):

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

commit 482e7cfce9dff16b7457c507accbc7e7dcf8ed6c
Author:     Peter Levine <plevine457@gmail.com>
AuthorDate: 2022-04-18 06:17:10 +0000
Commit:     Sam James <sam@gentoo.org>
CommitDate: 2022-04-18 20:28:28 +0000

    dev-cpp/gtest: fix testcase failure with musl
    
    Empirically, "getpagesize() * 12" instead of "getpagesize() * 10"
    appears to be enough to memory to stave off a stack overflow when built
    against musl libc.
    
    Closes: https://bugs.gentoo.org/834068
    Closes: https://github.com/gentoo/gentoo/pull/25073
    Package-Manager: Portage-3.0.30, Repoman-3.0.3
    Signed-off-by: Peter Levine <plevine457@gmail.com>
    Signed-off-by: Sam James <sam@gentoo.org>

 .../gtest/files/gtest-1.10.0_p20200702-increase-clone-stack-size.patch | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)
Comment 5 Peter Levine 2022-04-19 00:15:33 UTC
WFM.  Reopen if the issue persists with a rebuilt gtest.