Spec URL: http://rdieter.fedorapeople.org/rpms/pam-kwallet/pam-kwallet.spec SRPM URL: http://rdieter.fedorapeople.org/rpms/pam-kwallet/pam-kwallet-0-0.1.20140409git7604d424.fc20.src.rpm Description: PAM module for kwallet Fedora Account System Username: rdieter This is the KDE/kwallet equivalent to gnome-keyring-pam
Spec URL: http://rdieter.fedorapeople.org/rpms/pam-kwallet/pam-kwallet.spec SRPM URL: http://rdieter.fedorapeople.org/rpms/pam-kwallet/pam-kwallet-0-0.2.20140428gitaf786456.fc20.src.rpm %changelog * Mon Apr 28 2014 Rex Dieter <rdieter> 0-0.2.20140428gitaf786456 - 20140428 git snapshot
1) License should be LGPLv2+ 2) rpmlint says: pam-kwallet.i686: E: missing-call-to-setgroups /lib/security/pam_kwallet.so I was Googling it and it seems that pam_kwallet should call setgroup before using setuid/setgid. I was also talking with Alex Fieastas and he is rather busy, so I'll try to provide a patch for that.
Spec URL: http://rdieter.fedorapeople.org/rpms/pam-kwallet/pam-kwallet.spec SRPM URL: http://rdieter.fedorapeople.org/rpms/pam-kwallet/pam-kwallet-0-0.3.20140428gitaf786456.fc20.src.rpm %changelog * Wed Apr 30 2014 Rex Dieter <rdieter> 0-0.3.20140428gitaf786456 - License: LGPLv2+ (imo, we can fix the setgroups thing post-review)
That's true, at least it's not something what could be fixed on our side. Except the setgroups thing the package looks good.
New Package SCM Request ======================= Package Name: pam-kwallet Short Description: PAM module for KWallet Owners: rdieter Branches: f20 InitialCC:
Git done (by process-git-requests).
imported, thanks. comps updated, pam configs updates submitted for kde-settings(kdm), lightdm, sddm so far. will hit others (like gdm) soon.
pam-kwallet-0-0.3.20140428gitaf786456.fc20 has been submitted as an update for Fedora 20. https://admin.fedoraproject.org/updates/pam-kwallet-0-0.3.20140428gitaf786456.fc20
pam-kwallet-0-0.3.20140428gitaf786456.fc20 has been pushed to the Fedora 20 stable repository.
Would pam_kwallet be a better package name?
Maybe a little, but I don't think that's significant enough to warrant renaming things now.