Bug 958094 - Review Request: movgrab - An online video downloader in console
Summary: Review Request: movgrab - An online video downloader in console
Keywords:
Status: CLOSED CANTFIX
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
unspecified
medium
Target Milestone: ---
Assignee: Nobody's working on this, feel free to take it
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2013-04-30 11:34 UTC by Christopher Meng
Modified: 2013-06-26 10:45 UTC (History)
4 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2013-06-26 10:45:22 UTC
Type: ---
Embargoed:


Attachments (Terms of Use)

Description Christopher Meng 2013-04-30 11:34:19 UTC
Spec URL: http://cicku.me/movgrab.spec
SRPM URL: http://cicku.me/movgrab-1.2.1-1.fc20.src.rpm

Description: movgrab is a downloader for all those pesky sites that insist you use a big 
fat browser that runs flash in order to see their content.

Features:
Command-line application.
No dependancies.
HTTP Proxy server support.
Write download to std-out to feed into another application.
Progress display while downloading.
Fork into background.

Fedora Account System Username: cicku

Comment 1 Christopher Meng 2013-04-30 11:39:28 UTC
Koji success:

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

Comment 2 Mario Blättermann 2013-06-22 11:16:52 UTC
$ licensecheck -r *
libUseful-2.0/sha1.c: GPL (v2 or later)
libUseful-2.0/sha1.h: GPL (v2 or later)
libUseful-2.0/sha2.c: BSD (2 clause)
main.c: GPL (v3 or later)

(files with unknown licenses filtered out)

The license tag has to be "GPLv3+ and BSD".


The tarball bundles a library named libuseful, which is not available from Fedora yet, but there's a location where it is developed separately:
https://github.com/breckinloggins/libuseful
Would be better to package this first and make sure that movgrab uses that instead of the bundled one.

Moreover, there's another bundled library in libuseful itself. The files sha.* come from sha2 which is already in Fedora. The feature "No dependencies" says it actually, all you need is bundled with the tarball, which breaks one of the most fundamental Fedora rules.

Comment 3 Mario Blättermann 2013-06-22 11:23:36 UTC
(In reply to Mario Blättermann from comment #2)
> $ licensecheck -r *
> libUseful-2.0/sha1.c: GPL (v2 or later)
> libUseful-2.0/sha1.h: GPL (v2 or later)
> libUseful-2.0/sha2.c: BSD (2 clause)
> main.c: GPL (v3 or later)
> 
> (files with unknown licenses filtered out)
> 
> The license tag has to be "GPLv3+ and BSD".

Forget about the BSD license. Tha appropriate files are part of the bundled library, so you get "GPLv3+" only.


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