ipsilon

#56 pam: use a pam object method instead of pam module function
Closed by puiterwijk. Opened by cascardo.
cascardo/ipsilon pam  into  master

Download 56.patch
no initial comment

Looks good to me. Thanks!

Unfortunately though, this breaks compatibility with python-pam on Fedora/RHEL because that has a different API.
Could you possible rework it to check if pam.pam exists, and otherwise not do it?
If not, I can also change it to do that.

This appears to be a duplicate of ticket 200, which I'm about to submit a patch for. Is this PR associated with a ticket?

Per discussion in the Ipsilon conference call it was decided rather than checking the python pam version it would be better to check for the existence of the pam class in the pam module.

This has been fixed with #65 and #67.

Thank you for your patch!
Metadata
None
No Tags
Static archive. pagure.io was sunset in 2026; this is a read-only snapshot.
Red Hat Logo

Fedora is sponsored by Red Hat. Learn more about the relationship between Red Hat and Fedora »

© Red Hat, Inc. and others.