Note: This is a public test instance of Red Hat Bugzilla. The data contained within is a snapshot of the live data so any changes you make will not be reflected in the production Bugzilla. Email is disabled so feel free to test any aspect of the site that you want. File any problems you find or give feedback at bugzilla.redhat.com.
Bug 191592 - Review Request: vorbisgain - Adds tags to Ogg Vorbis files to adjust the volume
Summary: Review Request: vorbisgain - Adds tags to Ogg Vorbis files to adjust the volume
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Brian Pepple
QA Contact: Fedora Package Reviews List
URL:
Whiteboard:
Depends On:
Blocks: FE-ACCEPT
TreeView+ depends on / blocked
 
Reported: 2006-05-13 15:48 UTC by Noa Resare
Modified: 2007-11-30 22:11 UTC (History)
0 users

Fixed In Version: 0.34-2.fc5
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2006-05-28 13:15:38 UTC
Type: ---
Embargoed:

Attachments (Terms of Use)

Description Noa Resare 2006-05-13 15:48:47 UTC 
Spec URL: http://resare.com/noa/fedora-extras/vorbisgain.spec
SRPM URL: http://resare.com/noa/fedora-extras/vorbisgain-0.34-1.fc5.src.rpm
Description: 
VorbisGain is a utility that uses a psychoacoustic method to correct the
volume of an Ogg Vorbis file to a predefined standardized loudness.

It needs player support to work. Non-supporting players will play back
the files without problems, but you'll miss out on the benefits.
Nowadays most good players such as ogg123, xmms and mplayer are already
compatible.

I'd like to have a sponsor also :)
Comment 1 Brian Pepple 2006-05-13 17:46:33 UTC 
MD5Sums:
ee62352b74c610bb3eeddda038819fc8  vorbisgain-0.34.zip

Good:
* Source URL is canonical
* Upstream source tarball verified
* Group Tag is from the official list
* Buildroot has all required elements
* All paths begin with macros
* All directories are owned by this or other packages
* All necessary BuildRequires listed.
* All desired features are enabled
* Make succeeds even when %{_smp_mflags} is defined
* Files have appropriate permissions and owners
* rpmlint produces no errors
* Package installs and uninstalls cleanly on FC5
* Software runs fine

Minor:
* License is LGPL, not GPL.
* Inconsistant use of '$RPM_BUILD_ROOT' & '%buildroot'.  Use only one or the other.

These two items can be corrected after you import them into FE CVS

+1 APPROVE

I'll also be your sponser.  If you haven't created an account yet, you will need
to do so.

http://fedoraproject.org/wiki/Extras/Contributors#head-a89c07b5b8abe7748b6b39f0f89768d595234907
Comment 2 Noa Resare 2006-05-14 06:26:17 UTC 
Thanks! My account name is 'noa'
Comment 3 Brian Pepple 2006-05-14 13:55:00 UTC 
I've gone ahead and approved you.  Once this goes thru the system, you should be
able to go to the next step.

http://fedoraproject.org/wiki/Extras/Contributors#head-0956b12959af46cfe0aa12d09ed15e573bfd9ef4
Comment 4 Brian Pepple 2006-05-26 19:29:50 UTC 
Has this been build for FC5 & devel yet?  If so, this can be closed.
Comment 5 Noa Resare 2006-05-28 13:15:38 UTC 
Yup. And it seems to work also :)
Comment 6 Brian Pepple 2006-07-26 16:52:01 UTC 
This package still needs to be built for the devel branch.  Refer to
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=200268 and
http://fedoraproject.org/wiki/Extras/PackageStatus#head-e3ffc07096e28c4a16bc8a8a0c088d4916fe0d21.
Comment 7 Noa Resare 2006-08-08 14:46:51 UTC 
I'm just back from a few weeks of vacation, and I'm sorry about my slow response
to this issue.

I'm a bit uncertain if the tag generated by 'make tag' is correct
(vorbisgain-0_34-1_fc6). However the package is built by the plague build system
(as job id 13875) and is in status needsign at the moment.
Note You need to log in before you can comment on or make changes to this bug.