Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 667546 - sys-libs/pam-1.3.1: Doesn't honor $prefix
Summary: sys-libs/pam-1.3.1: Doesn't honor $prefix
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Gentoo Linux bug wranglers
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2018-10-02 13:54 UTC by Bernd Feige
Modified: 2018-10-02 18:37 UTC (History)
1 user (show)

See Also:
Package list:
Runtime testing required: ---


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Bernd Feige 2018-10-02 13:54:04 UTC
The subject says it all: In the ebuild, --libdir=/usr/$(get_libdir) is hardcoded, while it is '$(prefix)'/$(get_libdir) in 1.3.0-r2.
This leads to plugins being installed in /usr/lib64/security instead of /lib64/security with version 1.3.0-r2, and system breakage.

It may be that there is an error in the previous version ($prefix isn't actually set in multilib_src_configure), but the consequence was correct since all other ebuilds including systemd install to /lib64/security.

Thanks and best regards,
Bernd
Comment 1 Jonas Stein gentoo-dev 2018-10-02 15:55:57 UTC
please write the full package name in the summary. There are many pam packages.
Comment 2 Torbjörn Lönnemark 2018-10-02 18:13:22 UTC
The full package name is sys-libs/pam-1.3.1.

The bad ebuild was introduced by this commit:

commit 42bd1f101e777d779dd4b740a0a7f2bbe41a1432
Author: Mikle Kolyada <zlogene@gentoo.org>
Date:   Tue Oct 2 01:20:22 2018 +0300

    sys-libs/pam: Version bump (v1.3.1)


and later fixed by this commit:

commit 5922f2cc41fd354eea0a8b632fef55fbb48db228
Author: Mikle Kolyada <zlogene@gentoo.org>
Date:   Tue Oct 2 10:46:52 2018 +0300

    sys-libs/pam: add missing enable-securedir option


However, the revision number was not bumped by the fix, so anyone who fetched the portage tree between those commits and built pam from it will be stuck with the broken version (until they manually rebuild it).
Comment 3 Bernd Feige 2018-10-02 18:35:42 UTC
(In reply to Torbjörn Lönnemark from comment #2)
> The full package name is sys-libs/pam-1.3.1.
> 
> The bad ebuild was introduced by this commit:
> 
> commit 42bd1f101e777d779dd4b740a0a7f2bbe41a1432
> Author: Mikle Kolyada <zlogene@gentoo.org>
> Date:   Tue Oct 2 01:20:22 2018 +0300
> 
>     sys-libs/pam: Version bump (v1.3.1)
> 
> 
> and later fixed by this commit:
> 
> commit 5922f2cc41fd354eea0a8b632fef55fbb48db228
> Author: Mikle Kolyada <zlogene@gentoo.org>
> Date:   Tue Oct 2 10:46:52 2018 +0300
> 
>     sys-libs/pam: add missing enable-securedir option
> 
> 
> However, the revision number was not bumped by the fix, so anyone who
> fetched the portage tree between those commits and built pam from it will be
> stuck with the broken version (until they manually rebuild it).

... or those who used the daily snapshot / patches like I did ...
Thanks for the explanation, I see it is now fixed and will be fixed in tomorrow's snapshot...