Bug 597888
Summary: | Review Request: spek - free acoustic spectrum analyzer | ||
---|---|---|---|
Product: | [Fedora] Fedora | Reporter: | cdonatsch |
Component: | Package Review | Assignee: | Nobody's working on this, feel free to take it <nobody> |
Status: | CLOSED CANTFIX | QA Contact: | Fedora Extras Quality Assurance <extras-qa> |
Severity: | medium | Docs Contact: | |
Priority: | low | ||
Version: | rawhide | CC: | alexander, bugs.michael, cse.cem+redhatbugz, fabian.deutsch, fedora-package-review, notting, opensource, thibault.north |
Target Milestone: | --- | ||
Target Release: | --- | ||
Hardware: | All | ||
OS: | Linux | ||
Whiteboard: | |||
Fixed In Version: | Doc Type: | Bug Fix | |
Doc Text: | Story Points: | --- | |
Clone Of: | Environment: | ||
Last Closed: | 2010-07-26 06:25:16 UTC | Type: | --- |
Regression: | --- | Mount Type: | --- |
Documentation: | --- | CRM: | |
Verified Versions: | Category: | --- | |
oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |
Cloudforms Team: | --- | Target Upstream Version: | |
Embargoed: |
Description
cdonatsch
2010-05-30 22:00:54 UTC
Hi, As I've been told, this package is pretty useless without dependencies on gstreamer-plugin-base and good packages. So here are the new versions. Spec URL: http://dl.dropbox.com/u/6598054/spek.spec SRPM URL: http://dl.dropbox.com/u/6598054/spek-0.4-2.fc13.src.rpm ScratchBuild: http://koji.fedoraproject.org/koji/taskinfo?taskID=2222419 Looks sane on a quick look and builds fine on koji. In the meantime Spek 0.5 has been released: http://spek.googlecode.com/files/spek-0.5.tar.bz2 Christophe, did you perform some unofficial reviews? > Requires: gtk2 >= 2.14 > Requires: gstreamer >= 0.10.17 > Requires: gstreamer-plugins-base >= 0.10.17 > Requires: gstreamer-plugins-good >= 0.10.17 Be aware of the following guideline: https://fedoraproject.org/wiki/Packaging:Guidelines#Explicit_Requires Dependency on gtk2 and gstreamer library SONAMEs is automatic already. The explicit versions are much lower than what is available with Fedora. And gstreamer-plugins-good requires gstreamer-plugins-base. So, you should reduce these explicit Requires to just gstreamer-plugins-good with a comment that this app strictly needs this collection of plugins. > %install > rm -rf %{buildroot} > %clean > rm -rf %{buildroot} Since you also omitted the BuildRoot definition, you could drop further stuff: https://fedoraproject.org/wiki/Packaging:Guidelines#BuildRoot_tag Just FYI, Spek 0.6 no longer depends on GStreamer, it now uses FFmpeg libraries: http://gitorious.org/spek/spek/blobs/master/NEWS#line49 Which means it's a candidate for RPMFusion, not Fedora. Might as well close this review... I'd like to see this land in RPMFusion, though. > I'd like to see this land in RPMFusion, though. See https://bugzilla.rpmfusion.org/show_bug.cgi?id=1718 |