Pure python bindings for augeas WWW: https://github.com/hercules-team/python-augeas
Created attachment 147701 [details] textproc/py-augeas shar
Created attachment 147702 [details] Testport output from 10.0-amd64
Initial porting work made by Andrew Childs: https://github.com/thefloweringash/freebsd-ports-misc/tree/master/textproc/py-augeas
Created attachment 147727 [details] textproc/py-augeas shar Updated shar with me as maint instead of original author.
Thanks for your submission Johan! Review: * Use USE_GITHUB and GH_* variables (See: Mk/bsd.sites.mk) * Use v0.5.0 for GH_TAGNAME (not direct to commit hash) * PORTREVISION for a new port unnecessary * Add LICENSE_FILE if a standalone license file comes with the distribution * Tab align USE{S} section values * pkg-descr: WWW: URL location is http://augeas.net as per setup.py * Capitalize Augeas in COMMENT Please also: * Attach portlint -AC output (after addressing any outstanding issues)
Created attachment 147746 [details] cleaned up py-augeas.shar Thanks for the feedback! It has now been cleaned up. I had to override WRKSRC, since bsd.ports.mk would set it to ${WRKDIR}/${GH_ACCOUNT}-${GH_PROJECT}-${GH_COMMIT}. Is this correct? portlint -AC says "looks fine."
A commit references this bug: Author: koobs Date: Fri Jan 2 11:39:54 UTC 2015 New revision: 376043 URL: https://svnweb.freebsd.org/changeset/ports/376043 Log: [NEW] textproc/py-python-augeas: Pure python bindings for Augeas Pure python bindings for Augeas PR: 193960 Submitted by: johan at stromnet se Changes: head/textproc/Makefile head/textproc/py-python-augeas/ head/textproc/py-python-augeas/Makefile head/textproc/py-python-augeas/distinfo head/textproc/py-python-augeas/pkg-descr
Apologies for the delay Johan. I committed your patch with a few more changes: - Add GH_COMMIT, which points to the git hash, which answers your question about WRKSRC overrides in comment 6. The answer was actually in your question :) - Add regression-test: target to run the unit tests and QA Most importantly, I changed portname to match the upstream (pypi) name (python-augeas). This while not a strict policy, is something the Python team will likely be standardizing on going forwards. It helps users find the right packages they want, assists with google juice, and prevents conflicts. Thanks again for your submission! Looking forward to the next one.
Thanks for the fixes, I'll keep that in mind for my next submission! :)