Page MenuHomeFreeBSD

new port: net/py-nmsg
ClosedPublic

Authored by truckman on Apr 1 2015, 7:48 PM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Oct 7, 5:30 AM
Unknown Object (File)
Sun, Oct 5, 3:40 AM
Unknown Object (File)
Sat, Oct 4, 7:08 AM
Unknown Object (File)
Fri, Oct 3, 1:19 AM
Unknown Object (File)
Thu, Oct 2, 5:09 AM
Unknown Object (File)
Wed, Oct 1, 4:11 PM
Unknown Object (File)
Mon, Sep 29, 10:38 PM
Unknown Object (File)
Fri, Sep 26, 8:22 AM
Subscribers
None

Details

Reviewers
mat
Summary

This is pynmsg, a Python extension module implemented in Cython for
the nmsg C library.

Test Plan

Fetch distfiles from both MASTER_SITES

Build on FreeBSD 8.4, 9.3, 10.1, and 11.0-CURRENT, i386 and amd64.

Diff Detail

Repository
rP FreeBSD ports repository
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

truckman retitled this revision from to new port: net/py-nmsg.
truckman updated this object.
truckman edited the test plan for this revision. (Show Details)
truckman added a reviewer: mat.
mat requested changes to this revision.Apr 1 2015, 9:11 PM
mat edited edge metadata.
mat added inline comments.
net/py-nmsg/Makefile
3

Same here, should be pynmsg, and in net/py-pynmsg.

6–7

If you use both a lot, it could be interesting to end up creating a MASTER_SITE_FARSIGHT in Mk/bsd.sites.mk with a bit of magic in the "macro magic" at the end to put PORTNAME in the right place, and maybe a shortcut like FS.

25

Same question here.

This revision now requires changes to proceed.Apr 1 2015, 9:11 PM
net/py-nmsg/Makefile
6–7

Yeah, there are about a dozen existing or potential ports that could use MASTER_SITE_FARSIGHT.

net/py-nmsg/Makefile
6–7

I don't think I can combine the two since the subdir substitution differs between the two sites.

truckman edited edge metadata.

Change PORTNAME to pynmsg

Use MASTER_SITE_FARSIGHT

Nuke unnecessary ${REINPLACE_CMD}

net/py-pynmsg/Makefile
6–7 ↗(On Diff #4589)

Likewise, would fit on one line.

truckman edited edge metadata.

Collapse MASTER_SITES to a single line.

Add LICENSE FILE.

mat edited edge metadata.
This revision is now accepted and ready to land.Apr 2 2015, 11:29 PM