Summary: | [NEW PORT] audio/stk: Synthesis ToolKit in C++ | ||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|
Product: | Ports & Packages | Reporter: | Yuri Victorovich <yuri> | ||||||||
Component: | Individual Port(s) | Assignee: | Richard Gallamore <ultima> | ||||||||
Status: | Closed FIXED | ||||||||||
Severity: | Affects Only Me | CC: | ultima | ||||||||
Priority: | --- | ||||||||||
Version: | Latest | ||||||||||
Hardware: | Any | ||||||||||
OS: | Any | ||||||||||
Bug Depends on: | |||||||||||
Bug Blocks: | 221400 | ||||||||||
Attachments: |
|
QAs: * pourdriere 11.1 amd64 * portlint Created attachment 185265 [details]
shar archive
Created attachment 185501 [details]
shar
Improved patch and submitted it as an upstream PR.
Looks good, couple things * CONFIGURE_ARGS append flag is not needed. * One of the patch files has dos endlines, this is not acceptable and can't be added to the ports tree as is. The USES dos2unix[1] must be used in this case. Already fixed these entries if you would like to see the changes they are here[2]. [1] https://www.freebsd.org/doc/en/books/porters-handbook/slow-patch.html (near the bottom) [2] https://reviews.freebsd.org/D12062 Thanks! A commit references this bug: Author: ultima Date: Thu Aug 17 19:43:00 UTC 2017 New revision: 448181 URL: https://svnweb.freebsd.org/changeset/ports/448181 Log: The Synthesis ToolKit in C++ (STK) is a set of open source audio signal processing and algorithmic synthesis classes written in the C++ programming language. WWW: https://ccrma.stanford.edu/software/stk PR: 221387 Submitted by: Yuri Victorovich (maintainer) Reviewed by: matthew (mentor) Approved by: matthew (mentor) Differential Revision: https://reviews.freebsd.org/D12062 Changes: head/audio/Makefile head/audio/stk/ head/audio/stk/Makefile head/audio/stk/distinfo head/audio/stk/files/ head/audio/stk/files/patch-configure.ac head/audio/stk/files/patch-src_RtAudio.cpp head/audio/stk/pkg-descr head/audio/stk/pkg-plist More one thing that Matthew mentioned about the patch-configure.ac. It would be good to get these sort of changes upstreamed -- well, not exactly these changes, but something that will still work on other OSes but that avoids trying to configure linux specific sound systems on non-linux. Thanks for your contributions, port has been committed, thanks! I suggested the changes to upstream yesterday. (In reply to Yuri Victorovich from comment #8) Awesome. |
Created attachment 185228 [details] shar