| Message ID | 1390096790-1864-1-git-send-email-danomimanchego123@gmail.com |
|---|---|
| State | Superseded |
| Headers | show |
Dear Danomi Manchego, Danomi Manchego wrote: > Add option to build a LGPL licensed gst-ffmpeg. > > Signed-off-by: Danomi Manchego <danomimanchego123@gmail.com> > > --- > > v1 -> v2: > - Move GST_FFMPEG_CONF_OPT adjustment (introduced by this patch) > higher in the file, so that GST_FFMPEG_CONF_EXTRA_OPT lines > don't get split. > --- > package/gstreamer/gst-ffmpeg/Config.in | 13 ++++++++++++- > package/gstreamer/gst-ffmpeg/gst-ffmpeg.mk | 11 ++++++++++- > 2 files changed, 22 insertions(+), 2 deletions(-) > > diff --git a/package/gstreamer/gst-ffmpeg/Config.in b/package/gstreamer/gst-ffmpeg/Config.in > index d879f5e..220f4d7 100644 > --- a/package/gstreamer/gst-ffmpeg/Config.in > +++ b/package/gstreamer/gst-ffmpeg/Config.in > @@ -1,4 +1,4 @@ > -config BR2_PACKAGE_GST_FFMPEG > +menuconfig BR2_PACKAGE_GST_FFMPEG > bool "gst-ffmpeg" > select BR2_PACKAGE_GST_PLUGINS_BASE > depends on BR2_LARGEFILE > @@ -8,5 +8,16 @@ config BR2_PACKAGE_GST_FFMPEG > > http://gstreamer.freedesktop.org/ Not related to your patch, but how about providing a more specific URL such as http://gstreamer.freedesktop.org/modules/gst-ffmpeg.html? It would be a separate patch of course. > > +if BR2_PACKAGE_GST_FFMPEG > + > +config BR2_PACKAGE_GST_FFMPEG_LGPL_BUILD > + bool "Build a LGPL licensed gst-ffmpeg" > + help > + This removes --enable-gpl and --enable-postproc from the > + ffmpeg configure line, and disables building the postproc > + gstreamer plugin. I personally prefer positive options. If you enable this options, it will remove stuff. I suggest you mimic the BR2_PACKAGE_FFMPEG_GPL option, from the ffmpeg package. Also unrelated to this patch, but is this package actually using ffmpeg or libav? All kconfig stuff suggest ffmpeg, but the package tarball actually contains a copy of the library in a directory named gst-libs/ext/libav, and the licensing info you add below mention libav. It's worth adding a separate patch to clarify to the user that: - this package uses an internal copy of the library, and - which source library it actually is. > + > +endif > + > comment "gst-ffmpeg needs a toolchain w/ largefile, IPv6" > depends on !(BR2_LARGEFILE && BR2_INET_IPV6) > diff --git a/package/gstreamer/gst-ffmpeg/gst-ffmpeg.mk b/package/gstreamer/gst-ffmpeg/gst-ffmpeg.mk > index 6534f93..ed2b519 100644 > --- a/package/gstreamer/gst-ffmpeg/gst-ffmpeg.mk > +++ b/package/gstreamer/gst-ffmpeg/gst-ffmpeg.mk > @@ -10,6 +10,15 @@ GST_FFMPEG_SITE = http://gstreamer.freedesktop.org/src/gst-ffmpeg > GST_FFMPEG_INSTALL_STAGING = YES > GST_FFMPEG_DEPENDENCIES = host-pkgconf gstreamer gst-plugins-base > > +ifeq ($(BR2_PACKAGE_GST_FFMPEG_LGPL_BUILD),y) > +GST_FFMPEG_CONF_OPT += --enable-lgpl Your patch does not apply anymore to current master because _CONF_OPT have been renamed to _CONF_OPTS. Care to you rebase and resend? Otherwise looks good. I did a quick build test and it worked.
Luca, On Sun, Oct 12, 2014 at 5:06 AM, Luca Ceresoli <luca@lucaceresoli.net> wrote: > Dear Danomi Manchego, > > Danomi Manchego wrote: >> >> Add option to build a LGPL licensed gst-ffmpeg. >> >> Signed-off-by: Danomi Manchego <danomimanchego123@gmail.com> >> >> --- >> >> v1 -> v2: >> - Move GST_FFMPEG_CONF_OPT adjustment (introduced by this patch) >> higher in the file, so that GST_FFMPEG_CONF_EXTRA_OPT lines >> don't get split. >> --- >> package/gstreamer/gst-ffmpeg/Config.in | 13 ++++++++++++- >> package/gstreamer/gst-ffmpeg/gst-ffmpeg.mk | 11 ++++++++++- >> 2 files changed, 22 insertions(+), 2 deletions(-) >> >> diff --git a/package/gstreamer/gst-ffmpeg/Config.in >> b/package/gstreamer/gst-ffmpeg/Config.in >> index d879f5e..220f4d7 100644 >> --- a/package/gstreamer/gst-ffmpeg/Config.in >> +++ b/package/gstreamer/gst-ffmpeg/Config.in >> @@ -1,4 +1,4 @@ >> -config BR2_PACKAGE_GST_FFMPEG >> +menuconfig BR2_PACKAGE_GST_FFMPEG >> bool "gst-ffmpeg" >> select BR2_PACKAGE_GST_PLUGINS_BASE >> depends on BR2_LARGEFILE >> @@ -8,5 +8,16 @@ config BR2_PACKAGE_GST_FFMPEG >> >> http://gstreamer.freedesktop.org/ > > > Not related to your patch, but how about providing a more specific URL > such as http://gstreamer.freedesktop.org/modules/gst-ffmpeg.html? > > It would be a separate patch of course. Okay, on v3, I'll add a patch updating the Config.in. >> +if BR2_PACKAGE_GST_FFMPEG >> + >> +config BR2_PACKAGE_GST_FFMPEG_LGPL_BUILD >> + bool "Build a LGPL licensed gst-ffmpeg" >> + help >> + This removes --enable-gpl and --enable-postproc from the >> + ffmpeg configure line, and disables building the postproc >> + gstreamer plugin. > > > I personally prefer positive options. If you enable this options, it > will remove stuff. I suggest you mimic the BR2_PACKAGE_FFMPEG_GPL > option, from the ffmpeg package. Well, the option is --enable-lgpl, not --enable-gpl, and I was following the upstream description at http://cgit.freedesktop.org/gstreamer/gst-ffmpeg/commit/?id=2d767fe58f6cc95a66f58f3de5a2fece7f3521ee . But okay, I'll switch this to a BR2_PACKAGE_GST_FFMPEG_GPL that resembles the ffmpeg option. > Also unrelated to this patch, but is this package actually using > ffmpeg or libav? All kconfig stuff suggest ffmpeg, but the package > tarball actually contains a copy of the library in a directory named > gst-libs/ext/libav, and the licensing info you add below mention libav. Yes, gst-ffmpeg switched to libav back in 2011. > It's worth adding a separate patch to clarify to the user that: > - this package uses an internal copy of the library, and > - which source library it actually is. Okay, I'll add some detail in that Config.in patch. >> + >> +endif >> + >> comment "gst-ffmpeg needs a toolchain w/ largefile, IPv6" >> depends on !(BR2_LARGEFILE && BR2_INET_IPV6) >> diff --git a/package/gstreamer/gst-ffmpeg/gst-ffmpeg.mk >> b/package/gstreamer/gst-ffmpeg/gst-ffmpeg.mk >> index 6534f93..ed2b519 100644 >> --- a/package/gstreamer/gst-ffmpeg/gst-ffmpeg.mk >> +++ b/package/gstreamer/gst-ffmpeg/gst-ffmpeg.mk >> @@ -10,6 +10,15 @@ GST_FFMPEG_SITE = >> http://gstreamer.freedesktop.org/src/gst-ffmpeg >> GST_FFMPEG_INSTALL_STAGING = YES >> GST_FFMPEG_DEPENDENCIES = host-pkgconf gstreamer gst-plugins-base >> >> +ifeq ($(BR2_PACKAGE_GST_FFMPEG_LGPL_BUILD),y) >> +GST_FFMPEG_CONF_OPT += --enable-lgpl > > > Your patch does not apply anymore to current master because _CONF_OPT > have been renamed to _CONF_OPTS. Care to you rebase and resend? Okay, will be part of the change to resemble the ffmpeg GPL option. Danomi - > Otherwise looks good. I did a quick build test and it worked. > > -- > Luca
diff --git a/package/gstreamer/gst-ffmpeg/Config.in b/package/gstreamer/gst-ffmpeg/Config.in index d879f5e..220f4d7 100644 --- a/package/gstreamer/gst-ffmpeg/Config.in +++ b/package/gstreamer/gst-ffmpeg/Config.in @@ -1,4 +1,4 @@ -config BR2_PACKAGE_GST_FFMPEG +menuconfig BR2_PACKAGE_GST_FFMPEG bool "gst-ffmpeg" select BR2_PACKAGE_GST_PLUGINS_BASE depends on BR2_LARGEFILE @@ -8,5 +8,16 @@ config BR2_PACKAGE_GST_FFMPEG http://gstreamer.freedesktop.org/ +if BR2_PACKAGE_GST_FFMPEG + +config BR2_PACKAGE_GST_FFMPEG_LGPL_BUILD + bool "Build a LGPL licensed gst-ffmpeg" + help + This removes --enable-gpl and --enable-postproc from the + ffmpeg configure line, and disables building the postproc + gstreamer plugin. + +endif + comment "gst-ffmpeg needs a toolchain w/ largefile, IPv6" depends on !(BR2_LARGEFILE && BR2_INET_IPV6) diff --git a/package/gstreamer/gst-ffmpeg/gst-ffmpeg.mk b/package/gstreamer/gst-ffmpeg/gst-ffmpeg.mk index 6534f93..ed2b519 100644 --- a/package/gstreamer/gst-ffmpeg/gst-ffmpeg.mk +++ b/package/gstreamer/gst-ffmpeg/gst-ffmpeg.mk @@ -10,6 +10,15 @@ GST_FFMPEG_SITE = http://gstreamer.freedesktop.org/src/gst-ffmpeg GST_FFMPEG_INSTALL_STAGING = YES GST_FFMPEG_DEPENDENCIES = host-pkgconf gstreamer gst-plugins-base +ifeq ($(BR2_PACKAGE_GST_FFMPEG_LGPL_BUILD),y) +GST_FFMPEG_CONF_OPT += --enable-lgpl +GST_FFMPEG_LICENSE = LGPL2+ (gst-ffmpeg), LGPLv2.1+/LGPL3+ (libav) +GST_FFMPEG_LICENSE_FILES = COPYING.LIB gst-libs/ext/libav/COPYING.LGPLv2.1 gst-libs/ext/libav/COPYING.LGPLv3 +else +GST_FFMPEG_LICENSE = GPLv2+ (gst-ffmpeg), GPLv2+/GPL3+ (libav) +GST_FFMPEG_LICENSE_FILES = COPYING gst-libs/ext/libav/COPYING.GPLv2 gst-libs/ext/libav/COPYING.GPLv3 +endif + GST_FFMPEG_CONF_EXTRA_OPT = \ --cross-prefix=$(TARGET_CROSS) \ --target-os=linux @@ -76,6 +85,6 @@ ifeq ($(BR2_PREFER_STATIC_LIB),) GST_FFMPEG_CONF_EXTRA_OPT += --enable-pic endif -GST_FFMPEG_CONF_OPT = --with-ffmpeg-extra-configure="$(GST_FFMPEG_CONF_EXTRA_OPT)" +GST_FFMPEG_CONF_OPT += --with-ffmpeg-extra-configure="$(GST_FFMPEG_CONF_EXTRA_OPT)" $(eval $(autotools-package))
Add option to build a LGPL licensed gst-ffmpeg. Signed-off-by: Danomi Manchego <danomimanchego123@gmail.com> --- v1 -> v2: - Move GST_FFMPEG_CONF_OPT adjustment (introduced by this patch) higher in the file, so that GST_FFMPEG_CONF_EXTRA_OPT lines don't get split. --- package/gstreamer/gst-ffmpeg/Config.in | 13 ++++++++++++- package/gstreamer/gst-ffmpeg/gst-ffmpeg.mk | 11 ++++++++++- 2 files changed, 22 insertions(+), 2 deletions(-)