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)
Wed, Jan 22, 11:00 AM
Unknown Object (File)
Nov 26 2024, 7:24 AM
Unknown Object (File)
Oct 4 2024, 3:01 PM
Unknown Object (File)
Oct 1 2024, 4:05 PM
Unknown Object (File)
Oct 1 2024, 12:12 PM
Unknown Object (File)
Sep 24 2024, 7:14 AM
Unknown Object (File)
Sep 22 2024, 8:04 AM
Unknown Object (File)
Sep 1 2024, 2:52 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.