Bug 477958 - Review Request: id3mtag - Command line mass ID3 tagging utility for audio files
Review Request: id3mtag - Command line mass ID3 tagging utility for audio files
Status: CLOSED DUPLICATE of bug 594064
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: FE-DEADREVIEW
  Show dependency treegraph
 
Reported: 2008-12-26 08:32 EST by Avi Alkalay
Modified: 2010-05-20 09:17 EDT (History)
7 users (show)

See Also:
Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
: 594064 (view as bug list)
Environment:
Last Closed: 2009-05-21 12:40:18 EDT
Type: ---
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---


Attachments (Terms of Use)

  None (edit)
Description Avi Alkalay 2008-12-26 08:32:14 EST
Spec URL: http://avi.alkalay.net/software/id3mtag/
SRPM URL: http://avi.alkalay.net/software/id3mtag/
Description: ID3 mass tagger is a tool for manipulating id3 and id3v2 tags in multiple files. It can generate tag fields from the filename and other variables, and/or rename files, using an intuitive syntax.
Comment 1 Fabian Affolter 2008-12-27 06:57:56 EST
The rpmlint output...

[fab@laptop024 i386]$ rpmlint id3mtag*
id3mtag.i386: W: invalid-license Two-clause BSD
id3mtag-debuginfo.i386: E: empty-debuginfo-package
id3mtag-debuginfo.i386: W: invalid-license Two-clause BSD
2 packages and 0 specfiles checked; 1 errors, 2 warnings.

[fab@laptop024 SRPMS]$ rpmlint id3mtag-0.78-1.fc9.src.rpm
id3mtag.src: W: invalid-license Two-clause BSD
1 packages and 0 specfiles checked; 0 errors, 1 warnings.

For more details check https://fedoraproject.org/wiki/Licensing

Some comments on your spec file

- CHANGES, COPYING, and so on should be placed in the %files section 
  %doc CHANGES COPYING
  %doc %{_docdir}/%{name} looks a bit unusual

- Don't mix $RPM_BUILD_ROOT and %{buildroot}
  https://fedoraproject.org/wiki/Packaging/Guidelines#Using_.25.7Bbuildroot.7D_and_.25.7Boptflags.7D_vs_.24RPM_BUILD_ROOT_and_.24RPM_OPT_FLAGS

- Is 'Prefix: %{_usr}' really needed?
  https://fedoraproject.org/wiki/Packaging/Guidelines#Relocatable_packages

- Remove the 'fc9' in the changelog entry '...com> - 0.78-1'
  https://fedoraproject.org/wiki/Packaging/Guidelines#Changelogs

- %{?_smp_mflags} and %{optflags} are not honored in the %build section
Comment 2 Avi Alkalay 2009-01-02 00:03:39 EST
All fixed.

Please recheck updates at http://avi.alkalay.net/software/id3mtag/
Comment 3 Fabian Affolter 2009-01-09 05:12:05 EST
Two other small things

- BuildRoot tag doesn't match the guidelines
  https://fedoraproject.org/wiki/Packaging/Guidelines#BuildRoot_tag

- the manpage should not have %doc

I will try to make an informal review today.  I'm not a sponsor but I can help you to make some progress with this package till a sponsor will take care of it.
Comment 4 Fabian Affolter 2009-03-09 11:12:34 EDT
ping?
Comment 5 Fabian Affolter 2009-04-20 03:23:22 EDT
Are you still interested in maintaining this package?
Comment 6 Mamoru TASAKA 2009-05-10 05:09:58 EDT
I will close this bug as NOTABUG if no response is received
from the reporter within ONE WEEK.
Comment 7 Mamoru TASAKA 2009-05-21 12:40:18 EDT
Once closing.

If someone wants to import this package into Fedora, please
file a new review request and mark this bug as a duplicate
of the new one.

Thank you!
Comment 8 Avi Alkalay 2010-05-20 09:17:25 EDT

*** This bug has been marked as a duplicate of bug 594064 ***

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