Page MenuHomeFreeBSD
Differential D13189

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

Authored by yuri on Nov 21 2017, 11:59 PM.
Tags
None
Referenced Files
Unknown Object (File)
Oct 20 2024, 7:38 PM
Unknown Object (File)
Oct 14 2024, 2:32 AM
Unknown Object (File)
Oct 7 2024, 11:40 AM
Unknown Object (File)
Sep 4 2024, 9:22 AM
Unknown Object (File)
Aug 17 2024, 2:49 AM
Unknown Object (File)
May 14 2024, 12:16 PM
Unknown Object (File)
May 3 2024, 3:58 AM
Unknown Object (File)
May 2 2024, 9:40 AM
View All 46 Files
Subscribers
mat

Details

Reviewers
tcberner
adamw
Commits
rP454893: audio/py-pyliblo: Update: Removed the unnecessary dependency on pyrex; Made it…
Summary

Verified that it builds and works.

Diff Detail

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

Event Timeline

yuri created this revision.Nov 21 2017, 11:59 PM
Herald added a subscriber: mat. · View Herald TranscriptNov 21 2017, 11:59 PM
Harbormaster completed remote builds in B12931: Diff 35572.Nov 21 2017, 11:59 PM
tcberner added inline comments.Nov 22 2017, 5:48 AM
audio/py-pyliblo/Makefile
17

^ 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 updated this revision to Diff 35582.Nov 22 2017, 5:50 AM

.

yuri marked an inline comment as done.Nov 22 2017, 5:51 AM
yuri added inline comments.
audio/py-pyliblo/Makefile
17

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
mat added inline comments.Nov 24 2017, 6:05 PM
audio/py-pyliblo/Makefile
17

There is one, it comes with the flavors patch.

tcberner added a comment.Nov 24 2017, 6:06 PM

So let's wait for that then? :D

yuri added a comment.Nov 24 2017, 7:13 PM
In D13189#275515, @tcberner wrote:

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
tcberner accepted this revision.Nov 25 2017, 11:01 AM

we can :)

This revision is now accepted and ready to land.Nov 25 2017, 11:01 AM
Closed by commit rP454893: audio/py-pyliblo: Update: Removed the unnecessary dependency on pyrex; Made it… (authored by yuri). · Explain WhyNov 25 2017, 8:30 PM
This revision was automatically updated to reflect the committed changes.
mat added inline comments.Nov 27 2017, 12:09 PM
head/audio/py-pyliblo/Makefile
22 ↗(On Diff #35774)

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.

Revision Contents
Changeset List

PathSize
audio/
py-pyliblo/
10 lines

Diff 35582

View Options

audio/py-pyliblo/Makefile

Loading...