Page MenuHomeFreeBSD

sysutils/py-consul: fix PORTNAME
AbandonedPublic

Authored by jhixson on Jan 18 2019, 3:37 AM.
Tags
None
Referenced Files
Unknown Object (File)
Jan 17 2024, 11:33 AM
Unknown Object (File)
Jan 2 2024, 11:28 AM
Unknown Object (File)
Dec 20 2023, 7:13 AM
Unknown Object (File)
Nov 10 2023, 1:21 PM
Unknown Object (File)
Nov 5 2023, 1:30 PM
Unknown Object (File)
Oct 9 2023, 12:09 PM
Unknown Object (File)
Oct 4 2023, 12:27 PM
Unknown Object (File)
Sep 30 2023, 12:26 PM
Subscribers

Details

Reviewers
araujo
koobs
Summary

sysutils/py-consul: fix PORTNAME

Test Plan

Tested with poudriere for 11.2 and 12.0

Diff Detail

Repository
rP FreeBSD ports repository
Lint
No Lint Coverage
Unit
No Test Coverage
Build Status
Buildable 22121
Build 21343: arc lint + arc unit

Event Timeline

Because, if you want to change, well, it seems that it should be python-consul.

Fix name of port to match github project

I am trying to remove PORTREVISION, but arc is failing with a curl error (92). I will update this when this gets resolved.

Add Makefile for review & remove port revision since name has changed

PORTREVISION cannot go backwards.

koobs requested changes to this revision.Jan 25 2019, 2:20 AM

@jhixson Could you please add the full port/diff context to the review please. I'm under the impression arc diff create created full file context unless --less-context was used, so not sure why/how there's no full context here. Maybe its a default config file or something.

Also, this needs a MOVED entry

sysutils/py-python-consul/Makefile
6

Add python to CATEGORIES as a secondary virtual category, for python packages

12

Add LICENSE_FILE when one is provided in the distribution file. This packages sdist does: LICENSE`

This revision now requires changes to proceed.Jan 25 2019, 2:20 AM

@mat , I am confused. Since this is technically a "new port", why would the PORTREVISION need to remain? Is that standard?

@mat , I am confused. Since this is technically a "new port", why would the PORTREVISION need to remain? Is that standard?

This is not a new port, it is a move, versions cannot go backwards.