Bug 1174

Summary:	Review request: get_iplayer - Lists, Records and Streams BBC iPlayer TV and Radio programmes
Product:	Package Reviews	Reporter:	dwmw2 <dwmw2>
Component:	Review Request	Assignee:	Rex Dieter <rdieter>
Status:	RESOLVED FIXED
Severity:	normal	CC:	chris, hobbes1069, rpmfusion-package-review, rpmfusion.org
Priority:	P5
Version:	Current
Hardware:	All
OS:	GNU/Linux
namespace:
Bug Depends on:
Bug Blocks:	4

Description dwmw2@infradead.org 2010-04-20 11:17:49 CEST

Spec: http://david.woodhou.se/get_iplayer.spec
SRPM: http://david.woodhou.se/get_iplayer-2.76-1.fc14.src.rpm

Description:
get_iplayer is a tool for listing, recording and streaming BBC iPlayer TV 
programmes, and other programmes via 3rd-party plugins.

Unsuitable for Fedora because it requires rtmpdump.

rpmlint whines about spelling errors but nothing relevant.

Comment 1 Rex Dieter 2010-04-28 14:44:15 CEST

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.

Comment 2 dwmw2@infradead.org 2010-04-28 14:49:57 CEST

Package CVS request
======================
Package Name: get_iplayer
Short Description: Lists, Records and Streams BBC iPlayer TV and Radio programmes
Owners: dwmw2
Branches: F-12 F-13
InitialCC:
----------------------
License tag: free

Comment 3 Chris Nolan 2010-04-29 01:46:17 CEST

Great to see this here, but didn't upstream drop support for this [1]? What is the status, did someone else take over upstream maintenance?

[1] http://linuxcentre.net/getiplayer

Comment 4 dwmw2@infradead.org 2010-04-29 01:57:45 CEST

We're getting it from the new upstream: http://github.com/jjl/get_iplayer

Comment 5 Richard 2011-10-06 20:50:59 CEST

Closing and Resolved:Fixed since the CVS request has been completed.

Comment 6 Peter Oliver 2014-01-10 14:39:46 CET

Package Change Request
======================
Package Name: get_iplayer
Owners: dwmw2,mavit

Applying for co-maintainership of this package.  See http://lists.infradead.org/pipermail/get_iplayer/2014-January/005330.html for background.