This service will be undergoing maintenance at 00:00 UTC, 2016-08-01. It is expected to last about 1 hours
Bug 597888 - Review Request: spek - free acoustic spectrum analyzer
Review Request: spek - free acoustic spectrum analyzer
Status: CLOSED CANTFIX
Product: Fedora
Classification: Fedora
Component: Package Review (Show other bugs)
rawhide
All Linux
low Severity medium
: ---
: ---
Assigned To: Nobody's working on this, feel free to take it
Fedora Extras Quality Assurance
:
Depends On:
Blocks:
  Show dependency treegraph
 
Reported: 2010-05-30 18:00 EDT by cdonatsch
Modified: 2013-10-19 10:42 EDT (History)
8 users (show)

See Also:
Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
Environment:
Last Closed: 2010-07-26 02:25:16 EDT
Type: ---
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:


Attachments (Terms of Use)

  None (edit)
Description cdonatsch 2010-05-30 18:00:54 EDT
Hi,
This is my first package, so I need a sponsor and a review.
Thanks

Spec URL: http://dl.dropbox.com/u/6598054/spek.spec
SRPM URL: http://dl.dropbox.com/u/6598054/spek-0.4-1.fc13.src.rpm

Description: 
Spek helps to analyze your audio files by showing their spectrogram.
Spectrograms are used to analyze the quality of audio files, you can easily
detect lossy re-encodes, web-rips and other badness by just looking at the
spectrogram. Spek uses GStreamer to read audio files which means it supports
most file formats you can think of.

ScratchBuild : http://koji.fedoraproject.org/koji/taskinfo?taskID=2218397
Comment 1 cdonatsch 2010-06-01 09:44:55 EDT
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
Comment 2 Fabian Deutsch 2010-06-11 06:08:30 EDT
Looks sane on a quick look and builds fine on koji.
Comment 3 Alexander Kojevnikov 2010-06-26 21:45:34 EDT
In the meantime Spek 0.5 has been released:
http://spek.googlecode.com/files/spek-0.5.tar.bz2
Comment 4 Till Maas 2010-06-30 09:58:09 EDT
Christophe, did you perform some unofficial reviews?
Comment 5 Michael Schwendt 2010-07-18 04:44:00 EDT
> 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
Comment 6 Alexander Kojevnikov 2010-07-18 05:17:13 EDT
Just FYI, Spek 0.6 no longer depends on GStreamer, it now uses FFmpeg libraries:

http://gitorious.org/spek/spek/blobs/master/NEWS#line49
Comment 7 Conrad Meyer 2010-07-26 02:25:16 EDT
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.
Comment 8 Thibault North 2011-04-27 11:09:07 EDT
> I'd like to see this land in RPMFusion, though.

See https://bugzilla.rpmfusion.org/show_bug.cgi?id=1718

Note You need to log in before you can comment on or make changes to this bug.