sys-auth/pambase needs a new USE flag "elogind". Additionally it is needed that pkg_postinst() counts elogind as a session tracker to warn users if more than one session tracker was enabled. For elogind to be supported, pam_elogind.so must be patched into system-auth.in as an additional optional session module.
Created attachment 453022 [details] pambase-20150213-r2.ebuild Updated ebuild that accepts USE="elogind" and patches in elogind support. Patches will follow.
Created attachment 453024 [details, diff] Patch to enable elogind support in the Makefile This patch add support for HAVE_ELOGIND in the main Makefile.
Created attachment 453026 [details, diff] Patch to enable pam_elogind.so in system-auth This patch adds pam_elogind.so as an additional optional session module in system-auth.
@Michael : I have not yet found any problems with adding elogind to pambase. I guess you can add it to the KDE Overlay. @KDE
(In reply to Sven Eden from comment #4) > @Michael : I have not yet found any problems with adding elogind to pambase. > I guess you can add it to the KDE Overlay. dammit, wrong key. sorry. @PAM Gentoo Team: Are you okay with the patches, or have I missed something?
The REQUIRED_USE you've set makes the pkg_postinst partially irrelevant - $stcnt will never be > 2. Is there a technical reason for the REQUIRED_USE? Otherwise just remove it. I don't know why it is handled like that in the first place, otherwise we could remove the entire pkg_postinst with REQUIRED_USE="?? ( consolekit elogind systemd )", but there may be some reason for allowing more than 1 session tracker at the same time (if only to avoid support hours because of REQUIRED_USE...).
Also, repoman's got something to say: > KEYWORDS.invalid [fatal] 2 > sys-auth/pambase/pambase-20150213-r1.ebuild: ~-sparc-fbsd > sys-auth/pambase/pambase-20150213-r1.ebuild: ~-x86-fbsd
Created attachment 474506 [details, diff] pambase-20150213-r1.ebuild.diff - bump to EAPI 6 - cleanup obsolete blockers and version reqs - Sven's elogind changes with minor improvements - pkg_postinst -> pkg_setup (warn users beforehand)
*sigh* I forgot to answer to your comments in january due to a rather ... erm ... stormy private life at that time. I am very sorry! I really like the new diff, it looks very good. pambase was one of the first packages I patched, and the patch wasn't exactly beautiful. Yours is much better! However, it makes no sense to install more than one session trackers at the same time. And installing elogind where systemd runs the show gains nothing but a big headache. :-) That's the reason why I put in the REQUIRED_USE lines everywhere. Well, this, and because of the tiny fact that I didn't know of the ?? operator. So thank you very much for reworking the ebuild!
Thanks, fixed in git. https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=fb917030ea1e32b9e920cfa4b8d1003a5c90f183
