Conversation
|
Hi @classabbyamp, I saw this PR was closed with the If there is a technical issue with the patch, I am happy to fix it — just let me know what needs changing. I tested this on my daily driver (Void x86_64-glibc, runit, Cinnamon) and it resolves real If the concern is about AI-assisted contributions: I use AI as a drafting and research tool, but every line is reviewed, built in clean chroot, and tested on real hardware by me. The bug, the root cause, the fix, and the testing are all real. I stand behind the contribution. That said, if Void has a policy against this kind of contribution I will respect it — I would just appreciate knowing clearly so I can adjust. Thanks for your time. |
|
the fix you generated was incorrect, I implemented the correct fix in 05fdd1e. Additionally, Void does have a policy against AI-generated contributions |
Problem
The LightDM PAM files (lightdm, lightdm-greeter, lightdm-autologin) contain duplicate pam_elogind.so entries. The template post_install replaces pam_systemd.so with pam_elogind.so across all PAM files, which can create duplicate entries when source files already contain pam_elogind.so.
This causes errors during session transitions:
Fix
Add a targeted deduplication step in post_install using awk, applied only to pam_elogind.so lines in the three LightDM PAM files.
Testing