Summary: | dev-perl/Crypt-Cracklib-1.700.0-r1 : /.../features.h:381:4: error: #warning _FORTIFY_SOURCE requires compiling with optimization (-O) [-Werror=cpp] | ||
---|---|---|---|
Product: | Gentoo Linux | Reporter: | Toralf Förster <toralf> |
Component: | Current packages | Assignee: | Gentoo Perl team <perl> |
Status: | RESOLVED FIXED | ||
Severity: | normal | CC: | kentnl, vapier |
Priority: | Normal | ||
Version: | unspecified | ||
Hardware: | All | ||
OS: | Linux | ||
URL: | https://github.com/dsully/perl-crypt-cracklib/issues/5 | ||
Whiteboard: | |||
Package list: | Runtime testing required: | --- | |
Attachments: |
emerge-info.txt
dev-perl:Crypt-Cracklib-1.700.0-r1:20180612-061414.log emerge-history.txt environment etc.portage.tbz2 temp.tbz2 |
Description
Toralf Förster
![]() Created attachment 535714 [details]
emerge-info.txt
Created attachment 535716 [details]
dev-perl:Crypt-Cracklib-1.700.0-r1:20180612-061414.log
Created attachment 535718 [details]
emerge-history.txt
Created attachment 535720 [details]
environment
Created attachment 535722 [details]
etc.portage.tbz2
Created attachment 535724 [details]
temp.tbz2
@toolchain, care to comment on this? features.h is from glibc Though it does look like replications of this are found in the wild: http://www.cpantesters.org/cpan/report/95b9fbb4-eb04-11e6-92d0-e49cd7b5bb8e It *could* be related to https://metacpan.org/source/DANIEL/Crypt-Cracklib-1.7/Makefile.PL#L18 But its not obvious to me why there's this sudden need to compile things against features.h differently. _FORTIFY_SOURCE requires optimisations to be enabled. At least -O1. (In reply to Sergei Trofimovich from comment #8) > _FORTIFY_SOURCE requires optimisations to be enabled. At least -O1. Also note: features.h only issues a warning but build system sets -Werror and -Werror=cpp fails your build. this is entirely a bug in Crypt-Cracklib: (1) it's ignoring CFLAGS (2) it's using -Werror which it should not once both of those get fixed, the fortify issue should be resolved. (In reply to SpanKY from comment #10) > this is entirely a bug in Crypt-Cracklib: > (1) it's ignoring CFLAGS > (2) it's using -Werror which it should not > > once both of those get fixed, the fortify issue should be resolved. Sadly, ignoring CFLAGS is a widespread hard-to-fix problem with all of upstreams tooling, that will require lots of upstream abuse to get working right. But the remaining question is why this wasn't an error until now. i'm not sure the history really matters. we know the build files are bad. if it's one or two bad lines in the Makefile.PL, just sed it out in the ebuild and be done. if it wasn't fortify tickling the bad build, it'd still be a QA issue. https://bugs.gentoo.org/621036 likely fixed it on gcc side. The bug has been referenced in the following commit(s): https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=974c3c3a68dc7390ef544b261b83d8d1ba797f67 commit 974c3c3a68dc7390ef544b261b83d8d1ba797f67 Author: Andreas K. Hüttel <dilfridge@gentoo.org> AuthorDate: 2019-10-16 04:00:25 +0000 Commit: Andreas K. Hüttel <dilfridge@gentoo.org> CommitDate: 2019-10-16 04:00:25 +0000 dev-perl/Crypt-Cracklib: don't "enhance" CFLAGS in Makefile.PL, bug 658000 Bug: https://bugs.gentoo.org/658000 Package-Manager: Portage-2.3.76, Repoman-2.3.17 Signed-off-by: Andreas K. Hüttel <dilfridge@gentoo.org> .../Crypt-Cracklib-1.700.0-r2.ebuild | 26 ++++++++++++++++++++++ .../files/Crypt-Cracklib-1.700.0-CFLAGS.patch | 11 +++++++++ 2 files changed, 37 insertions(+) I've made further fixes for this in -r3 (1d79ac57f4449ed34d97ddea06fedcfd479f5a5c), as -r2 was still ignoring CFLAGS in ENV. This is remedied, and I can now confirm that: - Even with CFLAGS="-O0", compile succeeds. - CFLAGS="-O0 -frecord-gcc-switches" does what its supposed to Please test. dev-perl/Crypt-Cracklib-1.700.0-r3 is fine hera at various images The bug has been closed via the following commit(s): https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=846e88466158456f93887b34bdf2d0ecc3c56b6a commit 846e88466158456f93887b34bdf2d0ecc3c56b6a Author: Andreas K. Hüttel <dilfridge@gentoo.org> AuthorDate: 2021-10-09 20:00:16 +0000 Commit: Andreas K. Hüttel <dilfridge@gentoo.org> CommitDate: 2021-10-10 14:01:55 +0000 dev-perl/Crypt-Cracklib: Remove old Closes: https://bugs.gentoo.org/658000 Package-Manager: Portage-3.0.28, Repoman-3.0.3 Signed-off-by: Andreas K. Hüttel <dilfridge@gentoo.org> .../Crypt-Cracklib-1.700.0-r2.ebuild | 27 ---------------------- 1 file changed, 27 deletions(-) |