[95509] trunk/dports/multimedia/mlt/Portfile

[Ryan Schmidt]

On Jul 14, 2012, at 12:13, cal at macports.org wrote:

> Revision: 95509
>          https://trac.macports.org/changeset/95509
> Author:   cal at macports.org
> Date:     2012-07-14 10:13:33 -0700 (Sat, 14 Jul 2012)
> Log Message:
> -----------
> mlt: Fix license, fix lint warning, closes #35194, maintainer
> 
> Modified Paths:
> --------------
>    trunk/dports/multimedia/mlt/Portfile
> 
> Modified: trunk/dports/multimedia/mlt/Portfile
> ===================================================================
> --- trunk/dports/multimedia/mlt/Portfile	2012-07-14 14:30:06 UTC (rev 95508)
> +++ trunk/dports/multimedia/mlt/Portfile	2012-07-14 17:13:33 UTC (rev 95509)
> @@ -9,7 +9,7 @@
> version             0.8.0
> categories          multimedia
> maintainers         dennedy.org:dan
> -license             LGPL-2.1+
> +license             GPL-2+

Where did you see GPL? It seemed LGPL to me. The COPYING file says LGPL 2.1. The README says the framework is LGPL, and that "melted" is GPL, but "port contents mlt" doesn't show anything called "melted" being installed.


> -variant no_x11 {
> +variant no_x11 description exclude the gtk2 module to prevent pulling in X11 dependencies {

Ideally, variant descriptions should be enclosed in curly brackets or double quotes.