Spec URL: http://www.davidcornette.com/EQ10Q.spec SRPM URL: http://www.davidcornette.com/EQ10Q-1.0-1.fc11.src.rpm Description: This is an LV2 parametric equalizer plugin. Filters implemented include peaking, HPF, LPF, Shelving and Notch filters. It has a gtk-based gui which displays the equalization curve. This package depends on lv2-c++-tools, which is currently under review (Bug 537363).
(In reply to comment #0) I neglected to mention that this is my first package, and I need a sponsor.
*** Bug 581103 has been marked as a duplicate of this bug. ***
Hi David, Since I am not a sponsor, I can't review the package. I just have a request: Can we rename this package to lv2-eq10q-plugins (or to lv2-EQ10Q-plugins), for consistency with our other lv2 plugin packages? A note on the specfile: The package needs to have a "Requires: lv2core" for directory ownership.
I have made the requested changes. Since the name has changed, the urls of the files have changed. Spec URL: http://www.davidcornette.com/lv2-EQ10Q-plugins.spec SRPM URL: http://www.davidcornette.com/lv2-EQ10Q-plugins-1.0-1.fc11.src.rpm
I noticed a missing BuildRequires. I needed to add lv2core-devel. I have re-uploaded the spec and SRPM. Spec URL: http://www.davidcornette.com/lv2-EQ10Q-plugins.spec SRPM URL: http://www.davidcornette.com/lv2-EQ10Q-plugins-1.0-2.fc11.src.rpm
Based on some comments I received on #fedora-devel, I have made 3 changes. * I changed the Source0: listing, I replaced the literal 1.0 version in the tarball name with the %{version} macro. * One of the classes, vuwidget, is under GPLv3+, while the rest is under GPLv2+. The vuwidget class is only linked into one of the two libraries, parameq_gui.so, while the other library, paramEQ.so, contains only GPLv2+ code. Therefore I have set the license field to "GPLv2+ and GPLv3+". * I have patched the Makefile so that the optimization flags were not hard coded, and could be overridden with the %{optflags} macro. Spec File: http://www.davidcornette.com/lv2-EQ10Q-plugins.spec SRPM URL: http://www.davidcornette.com/lv2-EQ10Q-plugins-1.0-3.fc11.src.rpm
I was doing some testing, and I have made a few more changes. The parameq.ttl file was missing the definition of foaf, which caused lv2rack to hang trying to read it. Also, the paths of the image files were hard coded to be under /usr/local, so I patched the code and makefiles to have it look under /usr/share/lv2-EQ10Q-plugins/ for them. Spec File: http://www.davidcornette.com/lv2-EQ10Q-plugins.spec SRPM URL: http://www.davidcornette.com/lv2-EQ10Q-plugins-1.0-4.fc11.src.rpm
I will finish this review once we have lv2-c++-tools in the repo.
Since I originally submitted this package, lv2-c++-tools switched from packaging libraries as shared to static. This package builds and works fine with the updated version of that package without modification. If anyone built this package with the older version of lv2-c++-tools, I expect they would expect that you would need to rebuild this package.
I had not changed the BuildRequires: line to lv2-c++-tools-static, so I made a new version of the spec file, which also fixes a couple of warnings in the -debuginfo file about spurious exec permissions on two of the source files. Spec File: http://www.davidcornette.com/lv2-EQ10Q-plugins.spec SRPM URL: http://www.davidcornette.com/lv2-EQ10Q-plugins-1.0-5.fc11.src.rpm
I made a full review. A few minor things could be added/corrected. Otherwise we are good to go. ! rpmlint complains about spelling errors. lv2-EQ10Q-plugins.x86_64: W: spelling-error Summary(en_US) diferent -> different, difference, deferential lv2-EQ10Q-plugins.x86_64: W: spelling-error %description -l en_US plugin -> plug in, plug-in, plugging lv2-EQ10Q-plugins.x86_64: W: spelling-error %description -l en_US diferent -> different, difference, deferential lv2-EQ10Q-plugins.x86_64: W: spelling-error %description -l en_US biquadratic -> bi quadratic, bi-quadratic, quadratic lv2-EQ10Q-plugins-debuginfo.x86_64: W: spelling-error Summary(en_US) lv -> lb, l, v lv2-EQ10Q-plugins-debuginfo.x86_64: W: spelling-error %description -l en_US lv -> lb, l, v "plug-in" and "different" can be corrected. The rest can be ignored. ? A little explanation on patches would be nice. The patches look quite clean and portable. Did you contact upstream about these? ! Upstream could be reminded about missing license text files. * koji rawhide build is fine: http://koji.fedoraproject.org/koji/taskinfo?taskID=2218755 - I couldn't do a functionality test yet with the static linkage. I will do that whenever people are not sleeping :)
I tried the plugin with lv2rack host and got the following error: Error: file can't be openError: file can't be open Nothing is functional except In and Out gain that are on the left hand side of the window. Same result with the elven host (from lv2-ll-plugins). However the plugin works fine under ardour. Can you reproduce these?
I can confirm. I believe the errors are harmless. From what I can gather, when the gui is opened, it creates the directory ~/.RafolsEQ/ and it tries to open the file ~/.RafolsEQ/eq_presets.prs to load its presets. The error message occurs because the file doesn't exist (you haven't saved any presets). Once you have saved a preset, this error message goes away. However, any time you open the gui after the first, the error message "mkdir: cannot create directory `/home/david/.RafolsEQ': File exists" is printed. So these error messages are sloppy, but harmless. They don't really indicate errors. I can also confirm the problem with using the plugin under lv2rack. When I tried to track it down, It seemed that when a change was made to one of the filter type drop downs, it reports the change to the host, which then informs the plugin of the change, but the host gives the plugin that the old value instead of the new value, so you can't change the filter type to anything except "off". I don't know why this doesn't happen with ardour. I am not sure if this is a bug with the plugin, or with the host. Since presumably other plugins work correctly, probably it is a bug with the plugin. I suspect that the plugin was only ever tested with ardour by the developer. Without one of the patches that I added, attempting to use lv2rack after installing EQ10Q would actually cause lv2rack to hang (which is certainly a bug in lv2rack, but it is avoidable by correcting EQ10Q). I have submitted all of my patches to upstream's patch tracker on Sourceforge. I will also add a bug report for the lv2rack usage issue. Someone from Debian already filed a bug report about the license file. I'll send an email about all of the issues and patches, too. I'll update the spec with spelling corrections and info on the patches, too.
ping? Is everything alright?
I have updated the spec file to fix the spelling errors and explain the patches. I also sent an email to upstream about the patches and the missing license file. Spec File: http://www.davidcornette.com/lv2-EQ10Q-plugins.spec SRPM URL: http://www.davidcornette.com/lv2-EQ10Q-plugins-1.0-6.fc13.src.rpm
Thanks. Good to go ---------------------------------------------------- This package (lv2-EQ10Q-plugins) is APPROVED by oget ----------------------------------------------------
New Package CVS Request ======================= Package Name: lv2-EQ10Q-plugins Short Description: LV2 Plugin: Parametric equalizer with 12 different filter types Owners: davidcornette Branches: F-12 F-13 InitialCC:
CVS done (by process-cvs-requests.py).
lv2-EQ10Q-plugins-1.0-6.fc13 has been submitted as an update for Fedora 13. http://admin.fedoraproject.org/updates/lv2-EQ10Q-plugins-1.0-6.fc13
lv2-EQ10Q-plugins-1.0-6.fc12 has been submitted as an update for Fedora 12. http://admin.fedoraproject.org/updates/lv2-EQ10Q-plugins-1.0-6.fc12
lv2-EQ10Q-plugins-1.0-6.fc13 has been pushed to the Fedora 13 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 lv2-EQ10Q-plugins'. You can provide feedback for this update here: http://admin.fedoraproject.org/updates/lv2-EQ10Q-plugins-1.0-6.fc13
lv2-EQ10Q-plugins-1.0-6.fc12 has been pushed to the Fedora 12 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 lv2-EQ10Q-plugins'. You can provide feedback for this update here: http://admin.fedoraproject.org/updates/lv2-EQ10Q-plugins-1.0-6.fc12
lv2-EQ10Q-plugins-1.0-7.fc14 has been submitted as an update for Fedora 14. http://admin.fedoraproject.org/updates/lv2-EQ10Q-plugins-1.0-7.fc14
lv2-EQ10Q-plugins-1.0-7.fc14 has been pushed to the Fedora 14 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 lv2-EQ10Q-plugins'. You can provide feedback for this update here: http://admin.fedoraproject.org/updates/lv2-EQ10Q-plugins-1.0-7.fc14
lv2-EQ10Q-plugins-1.0-6.fc13 has been pushed to the Fedora 13 stable repository. If problems still persist, please make note of it in this bug report.
lv2-EQ10Q-plugins-1.0-6.fc12 has been pushed to the Fedora 12 stable repository. If problems still persist, please make note of it in this bug report.
lv2-EQ10Q-plugins-1.0-7.fc14 has been pushed to the Fedora 14 stable repository. If problems still persist, please make note of it in this bug report.