FS#34368 - [cinnamon-screensaver] Needs dependency 'gnome-keyring'

Attached to Project: Community Packages
Opened by jstjohn (jstjohn) - Monday, 18 March 2013, 23:29 GMT
Last edited by Alexandre Filgueira (faidoc) - Tuesday, 19 March 2013, 22:57 GMT
Task Type Bug Report
Category Packages
Status Closed
Assigned To Alexandre Filgueira (faidoc)
Architecture All
Severity Low
Priority Normal
Reported Version
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Votes 1
Private No

Details

Description: gnome-keyring should be a dependency, or at least an optional dependency, of cinnamon-screensaver.

Additional info:
* package version(s): cinnamon-screensaver-1.7.1-4

`journalctl -b` shows these errors when 'gnome-keyring' is not installed, and the screen gets locked:
<hostname> cinnamon-screensaver-dialog[25522]: PAM unable to dlopen(/usr/lib/security/pam_gnome_keyring.so): /usr/lib/security/pam_gnome_keyring.so
<hostname> cinnamon-screensaver-dialog[25522]: PAM adding faulty module: /usr/lib/security/pam_gnome_keyring.so


Steps to reproduce:
1.) Remove or leave 'gnome-keyring' uninstalled
2.) Lock the screen and then unlock it
3.) Run `journalctl -b` and scroll to the end (notice the errors here)
4.) Install 'gnome-keyring'
5.) Lock the screen and then unlock it
6.) Run `journalctl -b` and scroll to the end (notice that the errors have gone away)
This task depends upon

Closed by  Alexandre Filgueira (faidoc)
Tuesday, 19 March 2013, 22:57 GMT
Reason for closing:  Fixed
Additional comments about closing:  Fixed in cinnamon-screensaver 1.7.1-5
Comment by Balló György (City-busz) - Tuesday, 19 March 2013, 07:56 GMT
No, it's not a dependency. We have to add a minus sign before the pam_gnome_keyring.so line in file /etc/pam.d/cinnamon-screensaver, which marks that it's optional.
Comment by Alexandre Filgueira (faidoc) - Tuesday, 19 March 2013, 22:57 GMT
Yep, making pam_gnome_keyring.so optional fixes the problem

Loading...