Page MenuHomeFreeBSD

Update sysutils/py-supervisor to 3.3.4. Improve port

Authored by on Aug 17 2018, 5:54 PM.



My efforts to cleanup some legacy and improve port:

  1. get rid of @[un]exec in pkg-plist
    • @exec mkdir -p %%PIDDIR%% goes to PLIST_DIRS
    • @unexec echo goes away, let pkg handle PIDDIR removal
    • @sample is the only line left, it goes to PLIST_FILES
    • This leaves us with empty pkg-plist, remove it.
  2. The port has PORTDOCS which makes DOCS option a MUST, adding it
  3. remove patch Failing test is already disabled upstream.
  4. regenerate to make it PORTish
  5. - fix failing test, regenerate (makepatch created it under slightly different name, mind it during commit)

TODO: the port still lacks license. supervisor itself uses REPOZE licence, e.g. ports with the same licence are py-repoze.who/Makefile, py-repoze.xmliter/Makefile
but supervisor has a bundle of licenses in it's LICENSES.txt, I just don't know how to put it/them into port properly. I'd appreciate any hints.

Test Plan

poudriere testport: OK with only supported flavor py27
make test: Ran 1130 tests in 2.540s, OK

Diff Detail

rP FreeBSD ports repository
Automatic diff as part of commit; lint not applicable.
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

17–20 ↗(On Diff #46853)

Wrong place in the Makefile. See Chapter 15. Order of Variables in Port Makefiles.

This revision is now accepted and ready to land.Aug 20 2018, 1:21 AM
31 ↗(On Diff #46889)

Remove, default.

This revision now requires review to proceed.Aug 20 2018, 2:21 PM
This revision was not accepted when it landed; it landed in state Needs Review.Sep 22 2018, 8:27 AM
Closed by commit rP480364: - Update to 3.3.4 (authored by wen). · Explain Why
This revision was automatically updated to reflect the committed changes.