[NEW PORT] security/py-python-pam: Python PAM module
ClosedPublic
Actions

Authored by vishwin on Jan 26 2019, 7:18 AM.

Details

Reviewers

koobs

Group Reviewers

Python

Commits

rP491641: security/py-python-pam: create port

Summary

[NEW PORT] security/py-python-pam: Python PAM module

Python PAM module

WWW: https://github.com/FirefighterBlu3/python-pam

PR: 235212
Submitted by: Charlie Li <ml+freebsd vishwin info>
Reviewed_by: koobs
Approved by: koobs
Differential_Revision: D18979

Test Plan

portlint: OK
testport: OK
ran python3.7 -m pam using own credentials successfully

Diff Detail

Repository

rP FreeBSD ports repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

vishwin created this revision.Jan 26 2019, 7:18 AM

Harbormaster completed remote builds in B22173: Diff 53240.Jan 26 2019, 7:18 AM

vishwin set the repository for this revision to rP FreeBSD ports repository.Jan 26 2019, 7:18 AM

Please confirm this also passes portlint

security/py-python-pam/pkg-descr
2 ↗	(On Diff #53240)	Insufficient for pkg-descr contents, and its really unfortunate upstream doesn't provide anything except CLI examples in their README. I don't know what to suggest, but if we can't find something upstream or make something up, we'll have to accept it as is.
3 ↗	(On Diff #53240)	Match metadata from setup.py where possible, as it is canonical. In this case `setup.py:url` is `https://github.com/FirefighterBlu3/python-pam`

portlint was run as part of testport; both pass.

Harbormaster completed remote builds in B22176: Diff 53244.Jan 26 2019, 11:28 AM

vishwin edited the test plan for this revision. (Show Details)Jan 26 2019, 11:29 AM

vishwin marked an inline comment as done.Jan 26 2019, 11:32 AM

vishwin added inline comments.

security/py-python-pam/pkg-descr
2 ↗	(On Diff #53240)	I wish there was something better to come up with, but the actual code is such a thin wrapper around the native PAM library that any more description would be redundant at best. Even upstream's "supporting py3 (and py2)" doesn't serve much of a purpose.