Bug 794990

Summary:	Review Request: get-flash-videos - CLI tool to download flash video from websites
Product:	[Fedora] Fedora	Reporter:	Alec Leamas <leamas.alec>
Component:	Package Review	Assignee:	Petr Šabata <psabata>
Status:	CLOSED CANTFIX	QA Contact:	Fedora Extras Quality Assurance <extras-qa>
Severity:	medium	Docs Contact:
Priority:	unspecified
Version:	rawhide	CC:	dtimms, notting, package-review, psabata
Target Milestone:	---	Flags:	psabata: fedora-review?
Target Release:	---
Hardware:	All
OS:	Linux
Whiteboard:
Fixed In Version:		Doc Type:	Bug Fix
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2012-02-28 13:54:48 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:	794985, 794988
Bug Blocks:

Description Alec Leamas 2012-02-18 15:17:58 UTC

Spec : http://dl.dropbox.com/u/17870887/perl-Crypt-Blowfish_PP.spec
SRPM: http://dl.dropbox.com/u/17870887/perl-Crypt-Blowfish_PP-1.12-1.fc15.src.rpm

Download videos from various Flash-based video hosting sites, without
having to use the Flash player. Handy for saving videos for watching
offline, and means you don't have to keep upgrading Flash for sites that
insist on a newer version of the player.

It's similar to youtube-dl, but covers a larger set of websites, including some where I live :)

This has been on RPMFusion's wishlist for some time, but I see no reason why this shouldn't go to Fedora so I submit it here.

rpmlint is quiet besides weird spelling-errors warnings (also on installed package) and a warning about a properly commented source URL.

koji: http://koji.fedoraproject.org/koji/taskinfo?taskID=3801176  (f17)

Comment 1 Alec Leamas 2012-02-18 15:20:04 UTC

I don't have a package, and thus needs a sponsor

Comment 2 David Timms 2012-02-18 23:50:31 UTC

(In reply to comment #0)
> Spec : http://dl.dropbox.com/u/17870887/perl-Crypt-Blowfish_PP.spec
> SRPM:
> http://dl.dropbox.com/u/17870887/perl-Crypt-Blowfish_PP-1.12-1.fc15.src.rpm

Alec: something seems wrong with your URLs !

Comment 3 Alec Leamas 2012-02-19 09:16:33 UTC

Odd. Works for me. Transient?

Comment 4 David Timms 2012-02-19 10:03:28 UTC

We usually name the spec according to the package name, not:
ie get-flash-videos rather than perl-Crypt-Blowfish_PP.spec

Comment 5 Alec Leamas 2012-02-19 10:37:16 UTC

Woa... A certain class of errors are simply embarrasing. Sorry for missing twice. New links:
spec: http://dl.dropbox.com/u/17870887/get-flash-videos.spec
srpm: http://dl.dropbox.com/u/17870887/get-flash-videos-1.24-2.20120205git8abc6c6.fc15.src.rpm

Comment 6 Alec Leamas 2012-02-21 18:47:08 UTC

Updated after review of perl-Data-AMF. New links:
spec: http://dl.dropbox.com/u/17870887/get-flash-videos-2/get_flash_videos.spec
srpm: http://dl.dropbox.com/u/17870887/get-flash-videos-2/get-flash-videos-1.24-3.20120205git8abc6c6.fc15.src.rpm

Comment 7 Alec Leamas 2012-02-24 09:46:37 UTC

Updated after test-run with  fedora-review. New links:
http://dl.dropbox.com/u/17870887/get-flash-videos-4/get-flash-videos.spec
http://dl.dropbox.com/u/17870887/get-flash-videos-4/get-flash-videos-1.24-3.20120224git8abc6c6.fc15.src.rpm

Comment 8 Petr Šabata 2012-02-28 13:54:48 UTC

I've just noticed (halfway through the review) this package relies on rtmpdump which is not (and won't be) in Fedora.

You'd have to strip all the modules that use it which would cripple the package.  I'm closing this as CANTFIX.

Comment 9 Alec Leamas 2012-02-28 16:58:39 UTC

It's probably best to close it here, agreed. Sorry for taking your time w this.
--a