Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 825078 - sys-libs/pam_wrapper: test and runtime errors against sys-libs/pam[-split-usr]
Summary: sys-libs/pam_wrapper: test and runtime errors against sys-libs/pam[-split-usr]
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Gentoo's SAMBA Team
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2021-11-20 00:34 UTC by Marek Szuba
Modified: 2024-02-06 22:41 UTC (History)
3 users (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 Marek Szuba archtester gentoo-dev 2021-11-20 00:34:25 UTC
Both pam_wrapper ebuilds currently present in the tree implicitly assume native-ABI libpam.so to be installed in ${EPREFIX}/$(get_libdir). If, however, sys-libs/pam has been emerged with USE=-split-usr (which is currently made possible by sys-libs/pam-1.5.2-r1, owing to this ebuild inheriting usr-ldscript.eclass) - all PAM libraries are located in ${EPREFIX}/usr/$(get_libdir), pam_wrapper produces the following error:

pwrap_init: Failed to read libpam.so link

both in its own test suite and when in use elsewhere (e.g. in sys-auth/fprintd tests).
Comment 1 Larry the Git Cow gentoo-dev 2021-11-21 22:35:38 UTC
The bug has been closed via the following commit(s):

https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=834518e670c13f7ccfcf8bff134074f36f404930

commit 834518e670c13f7ccfcf8bff134074f36f404930
Author:     Mike Gilbert <floppym@gentoo.org>
AuthorDate: 2021-11-21 22:32:49 +0000
Commit:     Mike Gilbert <floppym@gentoo.org>
CommitDate: 2021-11-21 22:35:36 +0000

    sys-libs/pam_wrapper: let cmake find libpam
    
    pam_wrapper will still need to be rebuilt when libpam changes location,
    but at least this lets it work after a rebuild.
    
    Closes: https://bugs.gentoo.org/825078
    Signed-off-by: Mike Gilbert <floppym@gentoo.org>

 .../{pam_wrapper-1.1.3-r2.ebuild => pam_wrapper-1.1.3-r3.ebuild}  | 8 --------
 1 file changed, 8 deletions(-)
Comment 2 Larry the Git Cow gentoo-dev 2022-03-09 23:53:29 UTC
The bug has been closed via the following commit(s):

https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=6b84b0b5a44afd525b2018db5a0813141554381a

commit 6b84b0b5a44afd525b2018db5a0813141554381a
Author:     Sam James <sam@gentoo.org>
AuthorDate: 2022-03-09 23:52:45 +0000
Commit:     Sam James <sam@gentoo.org>
CommitDate: 2022-03-09 23:52:45 +0000

    sys-libs/pam_wrapper: depend on newer PAM (non split-usr)
    
    We can be sure of the location of PAM if we depend on a new enough version.
    
    Closes: https://bugs.gentoo.org/825078
    Signed-off-by: Sam James <sam@gentoo.org>

 .../{pam_wrapper-1.1.4.ebuild => pam_wrapper-1.1.4-r1.ebuild}        | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)
Comment 3 Larry the Git Cow gentoo-dev 2024-01-05 04:10:40 UTC
The bug has been referenced in the following commit(s):

https://gitweb.gentoo.org/data/gentoo-news.git/commit/?id=114a15884faf88f202073de48812613b264f49e0

commit 114a15884faf88f202073de48812613b264f49e0
Author:     Eli Schwartz <eschwartz93@gmail.com>
AuthorDate: 2024-01-02 04:04:32 +0000
Commit:     Sam James <sam@gentoo.org>
CommitDate: 2024-01-05 04:10:37 +0000

    2024-01-05-usr-initramfs: add news item
    
    Revival of commit a79dd69b0cca439bc0c483c9193c79e0554819d0.
    
    Bug: https://bugs.gentoo.org/868306#c10
    Bug: https://bugs.gentoo.org/902829
    Bug: https://bugs.gentoo.org/915379
    Bug: https://bugs.gentoo.org/825078
    Signed-off-by: Eli Schwartz <eschwartz93@gmail.com>
    Signed-off-by: Sam James <sam@gentoo.org>

 .../2024-01-05-usr-initramfs.en.txt                | 46 ++++++++++++++++++++++
 1 file changed, 46 insertions(+)
Comment 4 Larry the Git Cow gentoo-dev 2024-02-06 22:41:09 UTC
The bug has been referenced in the following commit(s):

https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=be1b61e0867fc6528c1bdf47556c0da674e4349c

commit be1b61e0867fc6528c1bdf47556c0da674e4349c
Author:     Eli Schwartz <eschwartz93@gmail.com>
AuthorDate: 2024-02-06 05:27:28 +0000
Commit:     Sam James <sam@gentoo.org>
CommitDate: 2024-02-06 22:40:59 +0000

    sys-libs/pam_wrapper: revbump to force rebuilding against updated pam
    
    We got rid of the terrible usr-ldscript hack. It is now possible to
    build this sanely. Force it to rebuild against the updated pam (for the
    sake of split-usr profiles).
    
    Bug: https://bugs.gentoo.org/825078
    Closes: https://bugs.gentoo.org/825758
    Signed-off-by: Eli Schwartz <eschwartz93@gmail.com>
    Signed-off-by: Sam James <sam@gentoo.org>

 sys-libs/pam_wrapper/pam_wrapper-1.1.4-r3.ebuild | 88 ++++++++++++++++++++++++
 1 file changed, 88 insertions(+)

https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=72d0c560a13563ebd6e7b010cc5ab169fb2efc8b

commit 72d0c560a13563ebd6e7b010cc5ab169fb2efc8b
Author:     Eli Schwartz <eschwartz93@gmail.com>
AuthorDate: 2024-02-06 04:50:41 +0000
Commit:     Sam James <sam@gentoo.org>
CommitDate: 2024-02-06 22:40:58 +0000

    sys-libs/pam: remove usr-ldscript support
    
    Per news item 2024-01-05-usr-initramfs, we no longer support this use
    case. It is fragile and hacky and leads to bizarre forms of load errors.
    
    The functionality is, despite being called "split-usr", not really about
    split-usr at all.
    
    [sam: add bug #868306 ref.]
    
    Bug: https://bugs.gentoo.org/825078
    Bug: https://bugs.gentoo.org/825758
    Bug: https://bugs.gentoo.org/868306
    Signed-off-by: Eli Schwartz <eschwartz93@gmail.com>
    Signed-off-by: Sam James <sam@gentoo.org>

 sys-libs/pam/pam-1.5.3-r1.ebuild | 153 +++++++++++++++++++++++++++++++++++++++
 1 file changed, 153 insertions(+)