Page MenuHomeFreeBSD

audio/py-pyliblo: Update: Removed the unnecessary dependency on pyrex; Made it python3-compatible; Added USES=localbase:ldflags.
ClosedPublic

Authored by yuri on Nov 21 2017, 11:59 PM.

Diff Detail

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

Event Timeline

audio/py-pyliblo/Makefile
17 ↗(On Diff #35572)

^ Apart from looking a bit on the overdesigned side, also you should not need to add a path for BUILD/RUN_DEPENDS in $PATH.

yuri marked an inline comment as done.Nov 22 2017, 5:51 AM
yuri added inline comments.
audio/py-pyliblo/Makefile
17 ↗(On Diff #35572)

This should be made into USE_PYTHON=cython.
Maybe I will submit such patch later.

yuri marked 2 inline comments as done.Nov 22 2017, 5:51 AM
audio/py-pyliblo/Makefile
17 ↗(On Diff #35572)

There is one, it comes with the flavors patch.

So let's wait for that then? :D

So let's wait for that then? :D

Can we not wait, and instead add a note to update this when available?

yuri marked an inline comment as done.Nov 24 2017, 7:13 PM
This revision is now accepted and ready to land.Nov 25 2017, 11:01 AM
This revision was automatically updated to reflect the committed changes.
head/audio/py-pyliblo/Makefile
22

I don't understand the comment here.

USE_PYTHON=autoplist looks at what files distutils has installed and uses that to generate the plist.

In this port, the man pages are installed manually in post-install, so there is no way that distutils knows about them, so it seems normal that they are not present in the plist generated by autoplist.