Page MenuHomeFreeBSD

devel/buildbot*: Rename all buildbot ports
ClosedPublic

Authored by rodrigc on Sep 26 2017, 5:28 AM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, Dec 21, 5:41 AM
Unknown Object (File)
Wed, Dec 18, 6:58 AM
Unknown Object (File)
Mon, Dec 16, 8:29 AM
Unknown Object (File)
Sat, Dec 14, 1:12 PM
Unknown Object (File)
Wed, Dec 4, 9:54 AM
Unknown Object (File)
Wed, Dec 4, 8:58 AM
Unknown Object (File)
Wed, Dec 4, 1:04 AM
Unknown Object (File)
Tue, Dec 3, 11:00 AM
Subscribers

Details

Summary
devel/buildbot*: Rename all buildbot ports

  devel/buildbot -> devel/py-buildbot
  devel/buildbot-worker -> devel/py-buildbot-worker
  devel/buildbot-console-view -> devel/py-buildbot-console-view
  devel/buildbot-grid-view -> devel/py-buildbot-grid-view
  devel/buildbot-pkg -> devel/py-buildbot-pkg
  devel/buildbot-waterfall-view -> devel/py-buildbot-waterfall-view
  devel/buildbot-www -> devel/py-buildbot-www

  - Update MAINTAINER
  - Allow builds with Python 3.x

Reviewed_by: koobs
Approved by: koobs (maintainer), portmgr (maintainer timeout, grembo, 3+ weeks)
Differential_Revision: D12499
Test Plan
  • portlint: OK (all ports)
  • testport: OK (poudriere: 10.3-RELEASE-amd64, Python 2.7)
  • maketest: OK (4 tests failing, 5060 passing for buildbot, 0 tests failing, 229 passing for buildbot-worker)

Diff Detail

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

Event Timeline

koobs requested changes to this revision.EditedSep 26 2017, 6:23 AM

All ports:

  • Add PKGNAMEPREFIX=${PYTHON_PKGNAMEPREFIX}
  • Add LICENCE_FILE=WRKSRC/path/to/file if/where one exists in WRKSRC (DISTFILES)
  • Add and verify USE_PYTHON=concurrent
  • Update USES=python to allow 2/3 (removed in previous review)
  • Update MAINTAINER=koobs@FreeBSD.org (or take maintainership). All buildbot ports should be maintained by the same person. Some are already different
  • Update COMMENT to match that in setup.py:description examplt buildbot-pkg COMMENT=Buildbot packaging tools

General:

  • Add MOVED entries
devel/py-buildbot-www/Makefile
14

Add LICENCE_FILE=WRKSRC/path/to/file if/where one exists in WRKSRC (DISTFILES)

devel/py-buildbot/Makefile
10–11

Update MAINTAINER=koobs@FreeBSD.org (all ports, or take maintainership)

11–12

No longer called master, update (bit)

34

Revert to allow 2/3

36

Add and verify concurrent support

This revision now requires changes to proceed.Sep 26 2017, 6:23 AM
rodrigc marked 3 inline comments as done.

Update patch

rodrigc added inline comments.
devel/py-buildbot-www/Makefile
14

There isn't one.

devel/py-buildbot/Makefile
36

How do I verify concurrent support?

devel/py-buildbot/Makefile
36

All files are uniquely named (are version suffixed), between Python 2.x and 3.x built versions, in path locations are not python version specific, so that concurrently installed port versions have no conflicting files. USE_PYTHON=concurrent is not perfect (eg: does not currently suffix man files)

In particular:

  • script dirs (eg: localbase/bin)
  • share/doc
  • man pages

concurrent does some/most things automatically, but check final results

devel/py-buildbot/Makefile
17–32

It would be great if it's sorted. Thanks!

38

It should be OK to use = instead of +=.

We are not ready to make these ports as concurrent yet.
There are still some conflicts.

@koobs I removed concurrent from these Makefiles. We aren't ready to do that yet, since there are conflicts.
The port works on Python 3 though, so removing 2.7 as a restriction is OK.

koobs requested changes to this revision.Nov 3 2017, 12:33 AM
koobs retitled this revision from devel/buildbot -> devel/py-buildbot: Rename to devel/buildbot*: Rename all buildbot ports.
koobs edited the summary of this revision. (Show Details)
koobs edited the summary of this revision. (Show Details)
koobs added 1 blocking reviewer(s): grembo.
koobs added inline comments.
devel/py-buildbot/Makefile
35

Minor nit, independent of what version our mock port is, but this is probably >=

This revision now requires changes to proceed.Nov 3 2017, 12:35 AM
This revision was not accepted when it landed; it landed in state Needs Review.Dec 15 2017, 12:06 AM
This revision was automatically updated to reflect the committed changes.