https://blogs.gentoo.org/ago/2020/07/04/gentoo-tinderbox/ Issue: dev-cpp/valijson-0.7 adds -Werror flag on its own (LIBARCHIVE-SYSTEM). Discovered on: amd64 (internal ref: libarchive_tinderbox) NOTE: (LIBARCHIVE-SYSTEM) in the summary means that the bug was found on a machine that runs app-alternatives/tar and app-alternatives/cpio with [-gnu,+libarchive] but this bug MAY or MAY NOT BE related to the tar/cpio implementation. -Werror is not recommended for releases and should always be disabled when encountered in build-logs, because there are numerous cases where this breaks without purpose. See also: https://devmanual.gentoo.org/ebuild-writing/common-mistakes/index.html.
Created attachment 848977 [details] build.log build log and emerge --info
Thanks for the heads-up. We already remove -Werror, but apparently some parts of the vendored test suite add it back. Since the test support will go away with the next update to 1.0 (as upstream has changed its packaging) I'm closing this as WONTFIX. It's really not worth the trouble for a header-only library.
(In reply to Holger Hoffstätte from comment #2) > Since the test support will go away with the next update to 1.0 (as upstream has changed its packaging) I'm > closing this as WONTFIX. Let's keep open to avoid duplicates
The bug has been referenced in the following commit(s): https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=44f039de30123a9472590cc6d2c15aa520a1d82b commit 44f039de30123a9472590cc6d2c15aa520a1d82b Author: Holger Hoffstätte <holger@applied-asynchrony.com> AuthorDate: 2023-01-23 11:00:05 +0000 Commit: Viorel Munteanu <ceamac@gentoo.org> CommitDate: 2023-02-08 12:19:40 +0000 dev-cpp/valijson: bump to 1.0 This also removes the test support since upstream has removed the vendored test dependencies from the tarball in favor of git submodules. Bug: https://bugs.gentoo.org/891683 Signed-off-by: Holger Hoffstätte <holger@applied-asynchrony.com> Closes: https://github.com/gentoo/gentoo/pull/29227 Signed-off-by: Viorel Munteanu <ceamac@gentoo.org> dev-cpp/valijson/Manifest | 1 + dev-cpp/valijson/files/1.0-fix-macro-typos.patch | 77 ++++++++++++++++++++++++ dev-cpp/valijson/valijson-1.0.ebuild | 24 ++++++++ 3 files changed, 102 insertions(+)
The bug has been closed via the following commit(s): https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=8907498d925ed490841e9208e84c01994316c3af commit 8907498d925ed490841e9208e84c01994316c3af Author: Holger Hoffstätte <holger@applied-asynchrony.com> AuthorDate: 2023-05-15 09:44:49 +0000 Commit: Joonas Niilola <juippis@gentoo.org> CommitDate: 2023-05-27 07:49:18 +0000 dev-cpp/valijson: clean up old Can be cleaned up since 1.0 is stable. Closes: https://bugs.gentoo.org/891683 Signed-off-by: Holger Hoffstätte <holger@applied-asynchrony.com> Closes: https://github.com/gentoo/gentoo/pull/31040 Signed-off-by: Joonas Niilola <juippis@gentoo.org> dev-cpp/valijson/Manifest | 1 - dev-cpp/valijson/valijson-0.7.ebuild | 44 ------------------------------------ 2 files changed, 45 deletions(-)