Bug 643391 - Review Request: pytrailer - Python module to simplify access to movies on apple.com/trailers
Summary: Review Request: pytrailer - Python module to simplify access to movies on app...
Keywords:
Status: CLOSED RAWHIDE
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Petr Šabata
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks: 643392
TreeView+ depends on / blocked
 
Reported: 2010-10-15 13:33 UTC by Stanislav Ochotnicky
Modified: 2010-10-21 13:58 UTC (History)
3 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2010-10-21 13:58:26 UTC
Type: ---
Embargoed:
psabata: fedora-review+
kevin: fedora-cvs+


Attachments (Terms of Use)

Description Stanislav Ochotnicky 2010-10-15 13:33:12 UTC
Spec URL: http://sochotni.fedorapeople.org/packages/pytrailer.spec
SRPM URL: http://sochotni.fedorapeople.org/packages/pytrailer-0.3.1-1.fc13.src.rpm

Description: 

Pytrailer is very simple python module using JSON and few simple
regular expressions to simplify access to trailer information on
www.apple.com/trailers.

Comment 1 Petr Šabata 2010-10-15 14:10:45 UTC
rpmlint results seem okay (although the output is different for RPMs and SPEC; might be an rpmlint issue):
$ rpmlint pytrailer-0.3.1-1.fc13.noarch.rpm 
pytrailer.noarch: W: spelling-error %description -l en_US www -> WWW, waw, wow
1 packages and 0 specfiles checked; 0 errors, 1 warnings.
$ rpmlint pytrailer-0.3.1-1.fc13.src.rpm
pytrailer.src: W: spelling-error %description -l en_US www -> WWW, waw, wow
1 packages and 0 specfiles checked; 0 errors, 1 warnings.
$ rpmlint pytrailer.spec 
0 packages and 1 specfiles checked; 0 errors, 0 warnings.

The package name is okay.
The SPEC file name matches the package name and its %name.
The package uses an approved license (LGPLv2) and seems legible.
Provided SRPM sources match the upstream sources.
Package builds fine (noarch).
BuildRequires looks legit.
The %files section lists files correctly.
The package meets the packaging guidelines.

Issues:
Don't mix RPM macros and variables. Change $RPM_BUILD_ROOT to %buildroot.
Upstream version license is LGPLv3 but the SPEC files states it's LGPLv2.

Comment 2 Stanislav Ochotnicky 2010-10-15 14:40:32 UTC
* Fri Oct 15 2010 Stanislav Ochotnicky <sochotnicky> - 0.3.1-2
- Fix License tag to match license of the package
- Change RPM_BUILD_ROOT to buildroot macro

Spec URL: http://sochotni.fedorapeople.org/packages/pytrailer.spec
SRPM URL: http://sochotni.fedorapeople.org/packages/pytrailer-0.3.1-2.fc13.src.rpm

Comment 3 Petr Šabata 2010-10-18 07:50:27 UTC
It looks okay now.

Comment 4 Stanislav Ochotnicky 2010-10-18 08:06:39 UTC
New Package CVS Request
=======================
Package Name: pytrailer
Short Description: Python module to simplify access to movie trailers on apple.com
Owners: sochotni
Branches: f12 f13 f14
InitialCC:

Comment 5 Kevin Fenzi 2010-10-21 13:30:22 UTC
Git done (by process-git-requests).

Comment 6 Stanislav Ochotnicky 2010-10-21 13:58:26 UTC
Thanks for review & scm. Package built, closing.

http://koji.fedoraproject.org/koji/taskinfo?taskID=2546648


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