After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 484629 - [pam] Doesn't link against libpam
[pam] Doesn't link against libpam
Status: RESOLVED FIXED
Product: gnome-keyring
Classification: Core
Component: general
git master
Other Linux
: Normal minor
: ---
Assigned To: GNOME keyring maintainer(s)
GNOME keyring maintainer(s)
Depends on:
Blocks:
 
 
Reported: 2007-10-08 08:04 UTC by Sebastian Dröge (slomo)
Modified: 2007-10-09 15:39 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
g-k-pam.diff (590 bytes, patch)
2007-10-08 08:05 UTC, Sebastian Dröge (slomo)
committed Details | Review
g-k-pam-configure.diff (603 bytes, patch)
2007-10-08 08:23 UTC, Sebastian Dröge (slomo)
committed Details | Review

Description Sebastian Dröge (slomo) 2007-10-08 08:04:51 UTC
Hi,
currently the pam module doesn't link against libpam. This isn't a problem normally, as everybody using that module must link against libpam anyway but not exactly beautiful either:

ldd -r /lib/security/pam_gnome_keyring.so 
undefined symbol: pam_getenv    (/lib/security/pam_gnome_keyring.so)
undefined symbol: pam_set_data  (/lib/security/pam_gnome_keyring.so)
undefined symbol: pam_putenv    (/lib/security/pam_gnome_keyring.so)
undefined symbol: pam_get_item  (/lib/security/pam_gnome_keyring.so)
undefined symbol: pam_strerror  (/lib/security/pam_gnome_keyring.so)
undefined symbol: pam_set_item  (/lib/security/pam_gnome_keyring.so)
undefined symbol: pam_get_user  (/lib/security/pam_gnome_keyring.so)
undefined symbol: pam_get_data  (/lib/security/pam_gnome_keyring.so)
        linux-gate.so.1 =>  (0xffffe000)
        libc.so.6 => /lib/tls/i686/cmov/libc.so.6 (0xb7e53000)
        /lib/ld-linux.so.2 (0x80000000)


Ok to commit the trivial attached patch?
Comment 1 Sebastian Dröge (slomo) 2007-10-08 08:05:17 UTC
Created attachment 96863 [details] [review]
g-k-pam.diff
Comment 2 Sebastian Dröge (slomo) 2007-10-08 08:23:17 UTC
Created attachment 96864 [details] [review]
g-k-pam-configure.diff

Might also make sense to add this to configure.ac then
Comment 3 Stef Walter 2007-10-09 15:39:21 UTC
Thanks committed:

2007-10-09  Stef Walter  <stef@memberwebs.com>

	* configure.in:
	* pam/Makefile.am: Link pam module with libpam.
	Patch by Sebastian Dröge. Fixes bug #484629