Perl 5.36 doesn't like "-I" or forcing cflag Duplicated fixes from Irssi project, mask cflag and use "-isystem" instead of "-II" Reproducible: Always Steps to Reproduce: 1. Upgrade to Perl 5.36 2. Install perl-core/Encode-3.120.0 3. It fails Actual Results: In file included from /usr/lib64/perl5/5.36/x86_64-linux-thread-multi/CORE/perl.h:7244, from Encode.xs:8: /usr/lib64/perl5/5.36/x86_64-linux-thread-multi/CORE/inline.h: In function ‘Perl_cop_file_avn’: /usr/lib64/perl5/5.36/x86_64-linux-thread-multi/CORE/inline.h:3489:5: error: ISO C90 forbids mixed declarations and code [-Werror=declaration-after-statement] Expected Results: Encode installs successfully Works with Perl stable at the moment but this will need patching/modding when Perl 5.36 goes stable - possibly pushing to their actual project as a fix :) Patches included
Created attachment 843281 [details, diff] Patch to fix -I and cflags
Created attachment 843283 [details] Ebuild updated to include patch
You shouldn't actually need perl-core/Encode, I think, as the virtual is satisfied by Perl 5.36 which contains a newer version. Is it in your world file or something? perl-core/* is only ever for satisfying a virtual/perl-* to provide a newer version of a module that is bundled with dev-lang/perl. Anyway, please include the full build.log and emerge --info.
We discussed this on IRC and turned out the package.accept_keywords list was wrong. I'll still just throw in the patch at https://github.com/dankogai/p5-encode/commit/7c9c5be4e658a5b37632b46925a2735123f65c6e as it's trivial.
The bug has been closed via the following commit(s): https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=038c222a22b535f2b118c02091c7b53e2d9cb7a6 commit 038c222a22b535f2b118c02091c7b53e2d9cb7a6 Author: Sam James <sam@gentoo.org> AuthorDate: 2022-12-18 01:03:14 +0000 Commit: Sam James <sam@gentoo.org> CommitDate: 2022-12-18 01:03:34 +0000 perl-core/Encode: backport -Werror patch It's not strictly needed (as this version isn't pulled in w/ Perl 5.36) but the patch exists and it's cheap to pull in, so if it avoids someone wasting some time looking into it, why not? Closes: https://bugs.gentoo.org/886507 Signed-off-by: Sam James <sam@gentoo.org> perl-core/Encode/Encode-3.120.0.ebuild | 5 +++- perl-core/Encode/files/Encode-3.120.0-Werror.patch | 29 ++++++++++++++++++++++ 2 files changed, 33 insertions(+), 1 deletion(-)
