Page MenuHomeFreeBSD

new port: security/py-xkcdpass
ClosedPublic

Authored by jacob_jwfh.ca on Aug 12 2019, 6:47 PM.
Tags
None
Referenced Files
Unknown Object (File)
Dec 20 2023, 1:56 AM
Unknown Object (File)
Nov 27 2023, 3:38 AM
Unknown Object (File)
Nov 11 2023, 2:35 AM
Unknown Object (File)
Nov 9 2023, 6:51 PM
Unknown Object (File)
Oct 28 2023, 8:34 AM
Unknown Object (File)
Oct 14 2023, 2:28 AM
Unknown Object (File)
Oct 14 2023, 2:25 AM
Unknown Object (File)
Oct 14 2023, 2:23 AM
Subscribers

Details

Summary

Add port for XKCDPass, a password generator modelled after XKCD #936

WWW: https://github.com/jwfh/xkcdpass

Diff Detail

Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

Modified security/Makefile to include the new port before py-xmlsec.

Previously the new port was not added in alphabetical order.

0mp requested changes to this revision.Aug 13 2019, 9:03 AM
0mp added a subscriber: 0mp.

Thanks for submission. There are a couple of issues with the current patch, however.

security/py-xkcdpass/Makefile
11

It would be nice to set LICENSE_FILE

17

This is the default.

18

We could use ${DISTVERSION} here.

20

Is this necessary?

security/py-xkcdpass/pkg-descr
4

Missing WWW.

security/py-xkcdpass/pkg-plist
22

This is invalid. Use make makeplist to generate it properly.

This revision now requires changes to proceed.Aug 13 2019, 9:03 AM
security/py-xkcdpass/Makefile
13

Also, upstream README suggests that Python 2.7 and 3.4<= are supported.

https://github.com/redacted/XKCD-password-generator

Fixed requested changes from 0mp.

Also, since creating the revision a new release (1.17.3.1 -> 1.17.4) is available
so this has also been reflected in the diff.

Note: Setting USES=python:2.7+ will allow building with Python 2.7, 3.5, 3.6, 3.7,
and 3.8, though Python 3.4 is also supported by upstream.

tobik added inline comments.
security/py-xkcdpass/Makefile
15

python:2.7+ is unnecessary. Just python means the same thing.

Remove tar. It is useless for this port since USE_GITHUB does not use EXTRACT_SUFX.

16

It seems wrong to have a pkg-plist and USE_PYTHON=autoplist. You probably have duplicate plist entries and make check-plist will complain about with a weird error.

19

Remove and set DISTVERSIONPREFIX=${PORTNAME}- instead.

security/py-xkcdpass/pkg-descr
2

I think this description could be improved a lot by explaining XKCD 936 a little.

Updated the port to include the changes requested by tobik.

I also added the DOCS option to allow installing the package's
README file to ${DOCSDIR} as this provides a little more usage
information to the user.

Created a new diff from master to show all changes for the new port rather than just changes made since the last diff was uploaded to Phabricator.

I am new to Phabricator so please advise if this was necessary. I didn't realize the previous diff did not show all of the changes when I attached it to the revision.

This revision was not accepted when it landed; it landed in state Needs Review.Jun 28 2020, 11:39 AM
Closed by commit rP540701: New port: security/py-xkcdpass (authored by 0mp). · Explain Why
This revision was automatically updated to reflect the committed changes.

Committed with very minor changes. Nice work! Thanks for your patience and patches! Looking forward to future contributions! ;)