[Bug 654037] gst-ffmpeg enables GPL parts in ffmpeg/libav

[GStreamer (bugzilla.gnome.org)]

https://bugzilla.gnome.org/show_bug.cgi?id=654037
  GStreamer | gst-ffmpeg | git

Edward Hervey <bilboed> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
 Attachment #191353|none                        |needs-work
             status|                            |

--- Comment #2 from Edward Hervey <bilboed at gmail.com> 2011-07-11 08:42:36 UTC ---
Review of attachment 191353:
 --> (https://bugzilla.gnome.org/review?bug=654037&attachment=191353)

In addition to the comment about LGPL, this patch should also check dynamically
how libav (whether included or system-wide) was built in order to modify the
license field of the plugin details.

It's a bit late to include this one in the release

::: configure.ac
@@ +207,3 @@
+AC_ARG_ENABLE(lgpl,
+              [AC_HELP_STRING([--enable-lgpl], [build a LGPL licensed
gst-ffmpeg])])
+AM_CONDITIONAL(LGPL, test "x$enable_lgpl" = "xyes")

LGPL is too generic of a name. Something like GST_FFMPEG_ENABLE_LGPL would be
better.

-- 
Configure bugmail: https://bugzilla.gnome.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug.
You are the assignee for the bug.