Page MenuHomeFreeBSD
Differential D12904

New port: audio/polyphone: Graphical user interface for editing soundfont (sf2 and sfz) files
ClosedPublic
Actions

Authored by yuri on Nov 2 2017, 4:24 AM.
Tags
None
Referenced Files
F136493594: D12904.id35057.diff
Tue, Nov 18, 1:14 AM
F136351251: D12904.id34679.diff
Mon, Nov 17, 9:09 AM
Unknown Object (File)
Mon, Nov 17, 3:29 AM
Unknown Object (File)
Sun, Nov 16, 7:46 PM
Unknown Object (File)
Sat, Nov 15, 1:28 AM
Unknown Object (File)
Sun, Nov 2, 5:26 AM
Unknown Object (File)
Sat, Nov 1, 6:04 AM
Unknown Object (File)
Tue, Oct 28, 5:21 AM
View All Files
Subscribers
mat

Details

Reviewers
tcberner
adamw
Commits
rP453960: New port: audio/polyphone: Graphical user interface for editing soundfont (sf2…
Summary

https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=221400

Diff Detail

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

Event Timeline

yuri created this revision.Nov 2 2017, 4:24 AM
Herald added a subscriber: mat. · View Herald TranscriptNov 2 2017, 4:24 AM
tcberner added inline comments.Nov 2 2017, 6:04 AM
audio/polyphone/Makefile
27

^ I think you can drop the '"'s

36

If you switch to qmake:outsource you could probably drop this (would also mean using BUILD_WRKSRC below in the do-install).

yuri updated this revision to Diff 34657.Nov 2 2017, 6:21 AM
yuri marked 2 inline comments as done.
tcberner added inline comments.Nov 2 2017, 6:24 AM
audio/polyphone/Makefile
44

INSTALL_WRKSRC should be correct

yuri updated this revision to Diff 34659.Nov 2 2017, 6:27 AM
yuri marked an inline comment as done.
mat added inline comments.Nov 2 2017, 2:58 PM
audio/polyphone/Makefile
8
DISTVERSIONSUFFIX= -src
22

keep this sorted, you may discover something.

23

too early.

yuri marked 2 inline comments as done.Nov 2 2017, 3:34 PM
yuri updated this revision to Diff 34679.Nov 2 2017, 3:37 PM

Reviewers: tcberner, adamw

Harbormaster completed remote builds in B12399: Diff 34679.Nov 2 2017, 3:37 PM
yuri marked an inline comment as done.Nov 7 2017, 11:02 PM
tcberner added inline comments.Nov 10 2017, 8:38 PM
audio/polyphone/Makefile
15

you could sort these some more

23

you could move this above USE_QT5

yuri marked 2 inline comments as done.Nov 10 2017, 9:39 PM
yuri updated this revision to Diff 35057.Nov 10 2017, 9:43 PM

Requested changes.

tcberner added inline comments.Nov 10 2017, 10:00 PM
audio/Makefile
132–133

careful ^ you got yor diffs mixed

tcberner added inline comments.Nov 10 2017, 10:01 PM
audio/Makefile
132–133

sorry, the diff-of-the-diff was misleading

yuri marked 2 inline comments as done.Nov 10 2017, 10:08 PM
yuri added inline comments.
audio/Makefile
132–133

I always check in meld before committing.
For the patches here I update them manually.

tcberner accepted this revision.Nov 11 2017, 8:14 AM

Looks good now.

This revision is now accepted and ready to land.Nov 11 2017, 8:14 AM
Closed by commit rP453960: New port: audio/polyphone: Graphical user interface for editing soundfont (sf2… (authored by yuri). · Explain WhyNov 11 2017, 8:32 AM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Diff 34679

View Options

audio/Makefile

Loading...
View Options

audio/polyphone/

Loading...
View Options

audio/polyphone/Makefile

Loading...
View Options

audio/polyphone/distinfo

Loading...
View Options

audio/polyphone/files/

Loading...
View Options

audio/polyphone/files/patch-sfark_sfarkextractor1.cpp

Loading...
View Options

audio/polyphone/files/polyphone.desktop

Loading...
View Options

audio/polyphone/files/polyphone.xml

Loading...
View Options

audio/polyphone/pkg-descr

Loading...