Arch Linux

Please read this before reporting a bug:
https://wiki.archlinux.org/title/Bug_reporting_guidelines

Do NOT report bugs when a package is just outdated, or it is in the AUR. Use the 'flag out of date' link on the package page, or the Mailing List.

REPEAT: Do NOT report bugs for outdated packages!
Tasklist

FS#13488 - [coreutils-7.1] Outdated patches (enable-pam isn't recognized)

Attached to Project: Arch Linux
Opened by Gerardo Exequiel Pozzi (djgera) - Wednesday, 25 February 2009, 18:02 GMT
Last edited by Andreas Radke (AndyRTR) - Thursday, 26 February 2009, 04:46 GMT
Task Type Bug Report
Category Packages: Testing
Status Closed
Assigned To No-one
Architecture All
Severity Medium
Priority Normal
Reported Version
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Votes 0
Private No

Details

Description:
Seems that the patches apply, but with warnings about offsets, and when configure runs this warning appears

configure: WARNING: unrecognized options: --enable-pam

Steps to reproduce:
cp -a /var/abs/testing/coreutils coreutils
cd coreutils
# need to edit PKGBUILD because missing install file?
makepkg
This task depends upon

Closed by  Andreas Radke (AndyRTR)
Thursday, 26 February 2009, 04:46 GMT
Reason for closing:  Fixed
Additional comments about closing:  fixed in trunk.
Comment by Gerardo Exequiel Pozzi (djgera) - Wednesday, 25 February 2009, 18:03 GMT
This isn't related to the "su" problem  FS#13444  (missing setuid su)
Comment by Gerardo Exequiel Pozzi (djgera) - Wednesday, 25 February 2009, 18:30 GMT
Oooh, missing autoconf command before ./configure in PKGBUILD ;)
Comment by Gerardo Exequiel Pozzi (djgera) - Wednesday, 25 February 2009, 19:08 GMT
BTW: I rewrited the patches with the correct offsets, to avoid offsets warnings, with these simple steps. (But maybe a better idea is to search for new patches if exists)

tar xf coreutils-7.1.tar.gz
cp -a coreutils-7.1 coreutils-7.1.orig
for x in coreutils*.patch; do
NOMBRE=$(basename $x .patch)
patch --no-backup-if-mismatch -p1 -i ../$x -d coreutils-7.1
diff -Naru coreutils-7.1.orig coreutils-7.1 > ${NOMBRE}.FIXED.patch
patch --no-backup-if-mismatch -p1 -R -i ../$x -d coreutils-7.1
done
Comment by Andreas Radke (AndyRTR) - Wednesday, 25 February 2009, 19:44 GMT
* --enable pam is still included in configure(.ac) - check with --help, the output that it is no more supported comes from new autotools
* autoconf isn't needed, it's already called by autoreconf a few lines above
* all patches are at latest "upstream" state from Fedora and Gentoo. I don't care if there are some offsets. Feel free to contact the Gentoo/Fedora maintainers.

So this in all isn't a bug.
Comment by Gerardo Exequiel Pozzi (djgera) - Wednesday, 25 February 2009, 20:13 GMT
Oops, Ok warning appears about this, but "su" is compiled with "PAM". (and have same md5sum) :)

Question: Running autoreconf (like now) but after apliying all patches, not is a better idea? This resolves the "redundant" warning.

Thanks.
Comment by Andreas Radke (AndyRTR) - Wednesday, 25 February 2009, 20:22 GMT
you're right, autoreconf should be run after patching. will change that in the next revision.

what's wrong with su compiled with pam? what would be the difference if we don't do that? (same md5sum - hum?)
Comment by Gerardo Exequiel Pozzi (djgera) - Thursday, 26 February 2009, 00:41 GMT
No, there is no diferences. :) Only the warning.
It happened that I saw that warning several times, when I tried compiling, and then report the problem for sure.

Loading...