poudriere log: http://p.defau.lt/?bDOrY0fxKi4zCdQKF_j70A
% portlint -AC
looks fine.
Details
Details
Diff Detail
Diff Detail
- Repository
- rP FreeBSD ports repository
- Lint
Lint Not Applicable - Unit
Tests Not Applicable
Event Timeline
Comment Actions
I am accepting this review, since all of my comments do not 'necessarily' require changes, and are feedback for you to make judgement/decisions on if you choose. They are not blockers to approval/commit.
You clearly know exactly what you're doing, great job :)
| net/py-python-bitcoinrpc/Makefile | ||
|---|---|---|
| 5 ↗ | (On Diff #7714) | Upstream needs to update their PyPi version :) Also their __version__ string: https://github.com/jgarzik/python-bitcoinrpc/blob/master/setup.py#L6 Note: This comment is not a commit blocker. |
| 7 ↗ | (On Diff #7714) | Maybe finance as secondary (leaving python as tertiary virtual)? What secondary categories are other bitcoin ports in? |
| 13 ↗ | (On Diff #7714) | Upstream doesn't have a LICENSE_FILE, so ask them to add one :) I added comment here: https://github.com/jgarzik/python-bitcoinrpc/issues/42#issuecomment-128411210 Not a commit blocker. |
| 22 ↗ | (On Diff #7714) | Rationale? v0.3 doesnt appear to have the bitcoinrpc directory in the tarball: https://github.com/jgarzik/python-bitcoinrpc/tree/v0.3/ I'd either:
I know last revision I said that backporting changes is often the best thing to do. In this case its a substantial changeset, so to save you time, use the hash you want, but just be 'smart' about correctly 'naming' the version so users aren't confused :) And ask the author for a new tag/release :) |
Comment Actions
Thanks @koobs
poudriere log: http://p.defau.lt/?lBL9SeHafTM1E1dlsfatEg
% portlint -AC
looks fine.
| net/py-python-bitcoinrpc/Makefile | ||
|---|---|---|
| 5 ↗ | (On Diff #7744) | I would use 0.3.0.<date> to mean, v.0.3.0 + commits up to <datestamp> This way when the next version is released (0.3.1, 0.4.0, etc), the port wont require a PORTEPOCH bump. |
| net/py-python-bitcoinrpc/Makefile | ||
|---|---|---|
| 6 ↗ | (On Diff #7744) | Sure. submitted it to upstream. https://github.com/ubldr/python-bitcoinrpc/commit/2cf384a6782a419b903225c7390ee6d90c09c555 |
| 8 ↗ | (On Diff #7744) | Thanks @koobs |
| 21 ↗ | (On Diff #7744) | please let me know if it's okay to use the last commit date from upstream as the PORTVERSION BTW. Is there any best practices for creating a patch with an empty__init__.py? |
| net/py-python-bitcoinrpc/Makefile | ||
|---|---|---|
| 5 ↗ | (On Diff #7744) | thank you for your quick reply, I will upload another patch. |
| net/py-python-bitcoinrpc/Makefile | ||
|---|---|---|
| 6 ↗ | (On Diff #7744) | oops, the URL should be https://github.com/jgarzik/python-bitcoinrpc/pull/53 |
| net/py-python-bitcoinrpc/Makefile | ||
|---|---|---|
| 21 ↗ | (On Diff #7744) | I can't think of a better way than touch(8) if our patch can't do it |
Comment Actions![20150807.143047-⚙ D3317 [NEW PORT] net_py-python-bitcoinrpc.png (262×593 px, 5 KB)](https://reviews.freebsd.org/file/data/m7kgx2umldvqbr3pupyx/PHID-FILE-itbu25kgywwyq4fhmnve/preview-20150807.143047-%E2%9A%99_D3317_%5BNEW_PORT%5D_net_py-python-bitcoinrpc.png)
Please also include a 'commit summary' each time you arc diff --update so change summary is visible per commit, making it easier to understand change context
![20150807.143047-⚙ D3317 [NEW PORT] net_py-python-bitcoinrpc.png (262×593 px, 5 KB)](https://reviews.freebsd.org/file/data/5ddwxzrbmsgec56fljbc/PHID-FILE-srx7kyixm2biygqnnhep/20150807.143047-%E2%9A%99_D3317_%5BNEW_PORT%5D_net_py-python-bitcoinrpc.png)