Spec URL: http://oget.fedorapeople.org/review/faust.spec SRPM URL: http://oget.fedorapeople.org/review/faust-0.9.9.4-1.b.fc10.src.rpm Description: Faust AUdio STreams is a functional programming language for real-time audio signal processing. Its programming model combines two approaches : functional programming and block diagram composition. You can think of FAUST as a structured block diagram language with a textual syntax. FAUST is intended for developers who need to develop efficient C/C++ audio plugins for existing systems or full standalone audio applications. Thanks to some specific compilation techniques and powerful optimizations, the C++ code generated by the Faust compiler is usually very fast. It can generally compete with (and sometimes outperform) hand-written C code. Programming with FAUST is somehow like working with electronic circuits and signals. A FAUST program is a list of definitions that defines a signal processor block-diagram : a piece of code that produces output signals according to its input signals (and maybe some user interface parameters) rpmlint gives two kinds of errors/warnings: devel-file-in-non-devel-package only-non-binary-in-usr-lib Both can be ignored since these files are really needed there during the runtime of faust compiler.
Created attachment 336130 [details] [Review] Attached is my initial review. Summary: - rpmlint: only-non-binary-in-usr-lib should be explained or fixed - should tools/%{name}2pd-* be %doc? - is the doc subpackage not requiring the base package intentional?
(In reply to comment #1) > Created an attachment (id=336130) [details] > [Review] > > Attached is my initial review. > Thank you for the initial review! > Summary: > - rpmlint: only-non-binary-in-usr-lib should be explained or fixed I fixed this. I put all those files in %{_datadir}/%{name} > - should tools/%{name}2pd-* be %doc? pd (Pure Data: http://puredata.info/) is a large programming environment that is unfortunately not packaged in Fedora yet. Thus installing these files in %{_bindir} will not mean much for the time being (We could even build these files from sources if we had pd in Fedora.). I am thinking of keeping these in %doc until the day we have pd packaged (which is no easy task). > - is the doc subpackage not requiring the base package intentional? Yes that was intentional. I checked the existing doc subpackages we have in Fedora. Some require the main package, some don't. On a second thought, I couldn't think of a case where someone would just want the doc subpackage for faust without needing the main package, so I added the dep. So here we are: Spec URL: http://oget.fedorapeople.org/review/faust.spec SRPM URL: http://oget.fedorapeople.org/review/faust-0.9.9.4-2.b.fc10.src.rpm Changelog: 0.9.9.4-2.b - Fix the year of the previous changelog entry - Install the nonbinary files in %%{_datadir}/%%{name}/ - Add Requires: %%{name}=%%{version}-%%{release} to the doc subpackage
This looks very good, thank you for packaging it for Fedora. APPROVED.
I thank you for the review! New Package CVS Request ======================= Package Name: faust Short Description: Compiled language for real-time audio signal processing Owners: oget Branches: F-10 InitialCC:
cvs done.
faust-0.9.9.4-2.b.fc10 has been submitted as an update for Fedora 10. http://admin.fedoraproject.org/updates/faust-0.9.9.4-2.b.fc10
faust-0.9.9.4-2.b.fc10 has been pushed to the Fedora 10 testing repository. If problems still persist, please make note of it in this bug report. If you want to test the update, you can install it with su -c 'yum --enablerepo=updates-testing update faust'. You can provide feedback for this update here: http://admin.fedoraproject.org/updates/F10/FEDORA-2009-2935
faust-0.9.9.4-2.b.fc10 has been pushed to the Fedora 10 stable repository. If problems still persist, please make note of it in this bug report.