Summary: | sys-auth/pambase: move pam_env.so last on the stack, move pam_systemd.so back to system-login (from system-auth) | ||
---|---|---|---|
Product: | Gentoo Linux | Reporter: | Samuli Suominen (RETIRED) <ssuominen> |
Component: | Current packages | Assignee: | Mikle Kolyada (RETIRED) <zlogene> |
Status: | RESOLVED FIXED | ||
Severity: | enhancement | CC: | alexander, gentoo, jasmin+gentoo, kensington, pam-bugs+disabled, sam |
Priority: | Normal | ||
Version: | unspecified | ||
Hardware: | All | ||
OS: | Linux | ||
See Also: | https://bugs.gentoo.org/show_bug.cgi?id=780441 | ||
Whiteboard: | |||
Package list: | Runtime testing required: | --- | |
Bug Depends on: | |||
Bug Blocks: | 485470 | ||
Attachments: |
0001-According-to-pam_env-manpage-it-should-be-last-on-th.patch
Improved patch, solves some bugs |
Description
Samuli Suominen (RETIRED)
![]() I believe the fix for bug 485470 was incorrect and this was the correct fix, and after merging this, we should again move '-session optional pam_systemd.so' second-last line of system-login (before pam_env) And if you could possibly also comment on this change for pam_mail.so (adding quiet parameter): https://projects.archlinux.org/svntogit/packages.git/commit/trunk?h=packages/pambase&id=6bcaa2f5f6831ab91bc80f87f036886760e9c973 Discussion about it: https://bugs.archlinux.org/task/31999 That's it, if we could get all of this done, there is no reason why ArchLinux couldn't merge our latest pambase again Both get boths improvements, possibly This is very experimental, so don't rush things and push anything into git yet, I'd like to test this for a while first too and get peoples comments here Created attachment 372644 [details, diff] Improved patch, solves some bugs - Adds 'quiet' to pam_mail.so as per bug #473678 - Moves pam_systemd.so back to system-login from system-auth -> systemd-user pam file in sys-apps/systemd should be fixed 's/system-auth/system-login/ bug #485470) - Moves pam_env.so last in the stack like in pam_env.so manpage says as it can have unexpected results on other modules. Makes use of XDG_RUNTIME_DIR within pam_env.conf easier. - Puts our pambase in line with ArchLinux's, so they can again merge our latest pambase. Sharing this looks like a good idea. They are using our fourth latest copy currently (there are some packaging differences, but the base is same) Just saying, even if this wasn't true, all of the changes make sense imho Any news here? Comment on attachment 372644 [details, diff]
Improved patch, solves some bugs
please don't squash unrelated things into one big fat commit. each independent change should be an independent commit/patch.
i've moved the pam_mail.so/quiet topic back to bug 473678 *** Bug 592616 has been marked as a duplicate of this bug. *** Should be fixed in the latest pambase release. |