Page MenuHomeFreeBSD

devel/buildbot*: Rename all buildbot ports

Authored by rodrigc on Sep 26 2017, 5:28 AM.
Referenced Files
Unknown Object (File)
Fri, Jun 14, 2:52 PM
Unknown Object (File)
May 15 2024, 1:03 AM
Unknown Object (File)
May 11 2024, 6:03 AM
Unknown Object (File)
May 10 2024, 7:58 PM
Unknown Object (File)
May 7 2024, 2:43 AM
Unknown Object (File)
May 6 2024, 6:59 AM
Unknown Object (File)
May 5 2024, 9:38 PM
Unknown Object (File)
May 4 2024, 3:17 PM


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

  - 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

rP FreeBSD ports repository
Lint Not Applicable
Tests Not Applicable

Event Timeline

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

All ports:

  • 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 (or take maintainership). All buildbot ports should be maintained by the same person. Some are already different
  • Update COMMENT to match that in examplt buildbot-pkg COMMENT=Buildbot packaging tools


  • Add MOVED entries
13 ↗(On Diff #33429)

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

10 ↗(On Diff #33429)

Update (all ports, or take maintainership)

11 ↗(On Diff #33429)

No longer called master, update (bit)

33 ↗(On Diff #33429)

Revert to allow 2/3

35 ↗(On Diff #33429)

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.
13 ↗(On Diff #33429)

There isn't one.

35 ↗(On Diff #33429)

How do I verify concurrent support?

35 ↗(On Diff #33429)

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

17–32 ↗(On Diff #33498)

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

38 ↗(On Diff #33498)

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.
32 ↗(On Diff #33679)

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.