Page MenuHomeFreeBSD
Differential D18962

sysutils/py-consul -> sysutils/py-python-consul
ClosedPublic
Actions

Authored by jhixson on Jan 25 2019, 6:14 PM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Dec 6, 8:49 PM
Unknown Object (File)
Thu, Nov 28, 2:39 PM
Unknown Object (File)
Mon, Nov 25, 2:25 PM
Unknown Object (File)
Sun, Nov 24, 2:13 PM
Unknown Object (File)
Nov 8 2024, 2:25 PM
Unknown Object (File)
Oct 14 2024, 8:07 PM
Unknown Object (File)
Oct 4 2024, 10:04 PM
Unknown Object (File)
Oct 4 2024, 8:54 PM
View All Files
Subscribers
mat

Details

Reviewers
miwi
koobs
araujo
Commits
rP491686: sysutils/py-consul: rename to sysutils/py-python-consul
Summary

sysutils/py-consul -> sysutils/py-python-consul

Test Plan

Tested using poudriere for 11.2 and 12.0 with python

Diff Detail

Repository
rP FreeBSD ports repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

jhixson created this revision.Jan 25 2019, 6:14 PM
Herald added a subscriber: mat. · View Herald TranscriptJan 25 2019, 6:14 PM
Harbormaster completed remote builds in B22150: Diff 53209.Jan 25 2019, 6:14 PM
jhixson updated this revision to Diff 53211.Jan 25 2019, 6:17 PM
  • Add MOVED file
  • Remove py-python-nomad from diff
Harbormaster completed remote builds in B22152: Diff 53211.Jan 25 2019, 6:17 PM
koobs requested changes to this revision.Jan 26 2019, 2:54 AM
koobs added inline comments.
sysutils/Makefile
982 ↗(On Diff #53211)

This unrelated change is still included in the diff, needs to be removed

sysutils/py-python-consul/Makefile
26 ↗(On Diff #53211)

Not required after PORTNAME change (GH_PROJECT defaults to PORTNAME)

28 ↗(On Diff #53211)

Group USE_* together, under USES

This revision now requires changes to proceed.Jan 26 2019, 2:54 AM
jhixson updated this revision to Diff 53328.Jan 28 2019, 6:07 PM

Port fixes

  • Group USE_* together
  • Fix sysutils/Makefile with correct port entry
  • Fix MOVED with correct port entry
Harbormaster completed remote builds in B22223: Diff 53328.Jan 28 2019, 6:07 PM
jhixson updated this revision to Diff 53329.Jan 28 2019, 6:08 PM

Remove conman from Makefile

Harbormaster completed remote builds in B22224: Diff 53329.Jan 28 2019, 6:08 PM
jhixson marked 2 inline comments as done.Jan 29 2019, 3:20 AM
araujo added inline comments.Jan 29 2019, 3:38 AM
sysutils/py-python-consul/Makefile
6 ↗(On Diff #53329)

You should keep the PORTREVISION for safety even if you are renaming it.

jhixson updated this revision to Diff 53441.Jan 30 2019, 8:17 PM

sysutils/py-consul: rename to sysutils/py-python-consul

  • Rename to be consistent with github project
  • Add PORTREVISION back
  • Add MOVED entry
  • Edit sysutils/Makefile to reflect name change
Harbormaster completed remote builds in B22280: Diff 53441.Jan 30 2019, 8:17 PM
jhixson marked an inline comment as done.Jan 30 2019, 8:18 PM
araujo accepted this revision.Jan 31 2019, 2:17 AM
This revision was not accepted when it landed; it landed in state Needs Review.Jan 31 2019, 5:03 AM
Closed by commit rP491686: sysutils/py-consul: rename to sysutils/py-python-consul (authored by jhixson). · Explain Why
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
head/
1 line
sysutils/
2 lines
py-consul/
30 lines
3 lines
files/
23 lines
3 lines
py-python-consul/
30 lines
3 lines
files/
23 lines
3 lines

Diff 53461

View Options

head/MOVED

Loading...
View Options

head/sysutils/Makefile

Loading...
View Options

head/sysutils/py-consul/Makefile

Loading...
View Options

head/sysutils/py-consul/distinfo

Loading...
View Options

head/sysutils/py-consul/files/patch-setup.py

Loading...
View Options

head/sysutils/py-consul/pkg-descr

Loading...
View Options

head/sysutils/py-python-consul/Makefile

Loading...
View Options

head/sysutils/py-python-consul/distinfo

Loading...
View Options

head/sysutils/py-python-consul/files/patch-setup.py

Loading...
View Options

head/sysutils/py-python-consul/pkg-descr

Loading...