[Bug 821233] Review Request: vdr-live - An interactive web interface for VDR

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



https://bugzilla.redhat.com/show_bug.cgi?id=821233

--- Comment #23 from Matthias Runge <mrunge@xxxxxxxxxx> ---
(In reply to comment #22)

> > javascript dir comes apparently from a different project, license is ASL 2.0
> > vdr-live is licensed under GPLv2+, which is incompatible with ASL 2.0
> > according to
> > https://fedoraproject.org/wiki/Licensing:Main?rd=Licensing#Good_Licenses
> > 
^^^^^^^
Incompatible licenses is a blocker! That is much harder than mootools-bundling.


> > buildutils pot2i18n.pl is copied from somewhere else, Readme is referenced
> > but missing.
> > 
You should ask upstream regarding this. pot2i18n.pl is just required for
building, so this is not the huge problem here. It's license won't even affect
the license of the package at all, as long it's a free license.

> > live/js/mootools is copied from somewhere else, my educated guess, it comes
> > from http://mootools.net/
> > License MIT (which is compatible with GPLv2+
> > 
> > 
> > You should unbundle that stuff, at least that javascript piece. In this way,
> > IMHO, it can't be packaged in one package.
> 
> hmm, why does zoneminder-1.25.0-9.fc18 comes with bundled mootools ?
> 

I can't say, why zoneminder....

I agree, that especially JavaScript libraries (such as jQuery) are a problem in
general. At least, your package license must also include  MIT such as
License: GPLv2+ and MIT

It seems, it's an unspoken law, bundling JavaScript libs might be ok, as long
as licenses fit together. You should try to talk to upstream and ask them to
unbundle that. In usual cases, upstream is more cooperative, when submitting
patches together with the request ;-)

When packaging in a good style, you will also list (in the spec), which parts
are GPL and which is MIT licensed.

> rpm -ql zoneminder |grep tools
> /usr/share/zoneminder/www/js/mootools.ext.js
> /usr/share/zoneminder/www/tools
> /usr/share/zoneminder/www/tools/mootools
> /usr/share/zoneminder/www/tools/mootools/mootools-core-1.3.2-nc.js
> /usr/share/zoneminder/www/tools/mootools/mootools-core-1.3.2-yc.js
> /usr/share/zoneminder/www/tools/mootools/mootools-core.js
> /usr/share/zoneminder/www/tools/mootools/mootools-more-1.3.2.1-nc.js
> /usr/share/zoneminder/www/tools/mootools/mootools-more-1.3.2.1-yc.js
> /usr/share/zoneminder/www/tools/mootools/mootools-more.js
> 
> i need

Current upstream version of mootools is 1.4.5.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
_______________________________________________
package-review mailing list
package-review@xxxxxxxxxxxxxxxxxxxxxxx
https://admin.fedoraproject.org/mailman/listinfo/package-review



[Index of Archives]     [Fedora Legacy]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]     [Fedora Tools]