[Bug 1174] Review request: get_iplayer - Lists, Records and Streams BBC iPlayer TV and Radio programmes

[RPM Fusion Bugzilla]

http://bugzilla.rpmfusion.org/show_bug.cgi?id=1174


Rex Dieter <rdieter at math.unl.edu> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|rdieter at math.unl.edu        |
             Blocks|3                           |4
             Status|NEW                         |ASSIGNED




--- Comment #1 from Rex Dieter <rdieter at math.unl.edu>  2010-04-28 14:44:15 ---
Fairly simple package, though a hefty perl script at that.

naming: ok

scriplets: ok (none)

MUST: %files: as discussed on irc, %{_sysconfdir}/get_iplayer/ needs to be
owned

SHOULD: add to %files:  %doc README

licensing: ok

runtime: ok, I only verified basic functionality, fetching listings.

SHOULD: consider using 'install -p' to preserve timestamps

I think noarch should be ok here, provided all the Requires: deps are available
on all archs.


I trust you can address the relatively minor MUST item and look at the
SHOULD's, prior to building.


APPROVED.


-- 
Configure bugmail: http://bugzilla.rpmfusion.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.