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 209894
Summary: | Review Request: python-eyed3 - Python module for processing ID3 tags | ||
---|---|---|---|
Product: | [Fedora] Fedora | Reporter: | Brian Pepple <bdpepple> |
Component: | Package Review | Assignee: | Patrice Dumas <pertusus> |
Status: | CLOSED NEXTRELEASE | QA Contact: | Fedora Package Reviews List <fedora-package-review> |
Severity: | medium | Docs Contact: | |
Priority: | medium | ||
Version: | rawhide | CC: | i |
Target Milestone: | --- | Flags: | gwync:
fedora-cvs+
|
Target Release: | --- | ||
Hardware: | All | ||
OS: | Linux | ||
Whiteboard: | |||
Fixed In Version: | Doc Type: | Bug Fix | |
Doc Text: | Story Points: | --- | |
Clone Of: | Environment: | ||
Last Closed: | 2006-10-10 21:57:28 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: | |||
Bug Depends On: | |||
Bug Blocks: | 163779, 794971 |
Description
Brian Pepple
2006-10-07 16:22:21 UTC
* as it is a noarch package it may be better to have BuildRequires: python instead of BuildRequires: python-devel * you could remove the -f option of rm such that it breaks if the file isn't there anymore * You could add README.html and THANKS to %doc * the build is done twice since make triggers the build and the install target depends on all. So I think the make in %build should be removed or commented out with a comment indicating that install triggers the build unconditionally * I think that it would be better to have a * for man files to catch man pages when no compressed or compressed using something different than gz, like %{_mandir}/man1/*.1* * Given that it only creates the directory in python_sitelib, you may optionally set %{python_sitelib}/eyeD3/ instead of %{python_sitelib}/* These issues are not blockers. Shouldn't the name be python-eyeD3? Otherwise * rpmlint is silent * follow packaging guidelines * spec legible * free software, licence GPL included * sane provides python-eyed3 = 0.6.10-1 * match upstream source 2f7ee6749b993faba3b5d10d9621d314 ./eyeD3-0.6.10.tar.gz * %files right * buildrequires/requires right The only remaining issue is the name. (In reply to comment #2) > Shouldn't the name be python-eyeD3? It's up to the maintainers discretion. I based the name on how other distros package it also. http://fedoraproject.org/wiki/Packaging/NamingGuidelines#head-96b3088669f40824665abf97ff34841f9b65172d Ok, APPROVED. But please take into consideration my comments, even though they aren't blockers. It seems to be built in devel and added to owners.list, please don't forget to close the review. I'm aware, but I'm waiting for the FC5 branch to be created before closing this. Package Change Request ====================== Package Name: python-eyed3 New Branches: el6 Owners: cicku Git done (by process-git-requests). python-eyed3-0.7.4-1.el6 has been submitted as an update for Fedora EPEL 6. https://admin.fedoraproject.org/updates/python-eyed3-0.7.4-1.el6 python-eyed3-0.7.4-1.el6 has been pushed to the Fedora EPEL 6 stable repository. Package Change Request ====================== Package Name: python-eyed3 New Branches: epel7 Owners: cicku Git done (by process-git-requests). |