Buildroot Archive on lore.kernel.org
 help / color / mirror / Atom feed
From: Arnout Vandecappelle <arnout@mind.be>
To: buildroot@busybox.net
Subject: [Buildroot] [PATCH v10 06/13] gstreamer1: Add gstreamer version 1.1.1 package
Date: Wed, 12 Jun 2013 07:51:34 +0200	[thread overview]
Message-ID: <51B80C66.3070602@mind.be> (raw)
In-Reply-To: <1370647138-26623-7-git-send-email-spenser@gillilanding.com>

  Hi Spenser,

On 08/06/13 01:18, Spenser Gilliland wrote:
> This patch adds the gstreamer version 1.x series to buildroot.

  Missing SoB

> ---
>   package/multimedia/Config.in                |    1 +
>   package/multimedia/gstreamer1/Config.in     |   41 +++++++++++++++++++++++++++
>   package/multimedia/gstreamer1/gstreamer1.mk |   38 +++++++++++++++++++++++++
>   3 files changed, 80 insertions(+), 0 deletions(-)
>   create mode 100644 package/multimedia/gstreamer1/Config.in
>   create mode 100644 package/multimedia/gstreamer1/gstreamer1.mk
>
> diff --git a/package/multimedia/Config.in b/package/multimedia/Config.in
> index dd74863..0351fd0 100644
> --- a/package/multimedia/Config.in
> +++ b/package/multimedia/Config.in
> @@ -6,6 +6,7 @@ source "package/multimedia/faad2/Config.in"
>   source "package/multimedia/flac/Config.in"
>   source "package/multimedia/ffmpeg/Config.in"
>   source "package/multimedia/gstreamer/Config.in"
> +source "package/multimedia/gstreamer1/Config.in"

  I think this should come below the gst-0.10 packages (so below 
gst-plugins-ugly) so they're more logically collected.


>   source "package/multimedia/gst-ffmpeg/Config.in"
>   source "package/multimedia/gst-dsp/Config.in"
>   source "package/multimedia/gst-fsl-plugins/Config.in"
> diff --git a/package/multimedia/gstreamer1/Config.in b/package/multimedia/gstreamer1/Config.in
> new file mode 100644
> index 0000000..96b32e9
> --- /dev/null
> +++ b/package/multimedia/gstreamer1/Config.in
> @@ -0,0 +1,41 @@
> +config BR2_PACKAGE_GSTREAMER1
> +	bool "gstreamer1"
> +	depends on BR2_USE_WCHAR # glib2
> +	depends on BR2_TOOLCHAIN_HAS_THREADS
> +	select BR2_PACKAGE_LIBGLIB2
> +	help
> +	  GStreamer is an open source multimedia framework. This 1.x version of
> +	  Gstreamer is incompatible with Gstreamer 0.1x.

  Well, gstreamer 0.11 was actually an experimental pre-release branch 
for 1.x (although there are still lots of incompatible changes between 
0.11 and 1.0). I think it's more clear and accurate if you say it's 
incompatible with Gstreamer 0.10.


> +
> +	  http://gstreamer.freedesktop.org/
> +
> +if BR2_PACKAGE_GSTREAMER1
> +
> +config BR2_PACKAGE_GSTREAMER1_GST_DEBUG
> +	bool "enable gst-debug trace support"
> +	default y
> +	help
> +	  Enable support for the gst-debug tracing functionality in gstreamer.
> +	  This has limited CPU overhead, but does increase the rootfs size
> +	  somewhat.

  I believe the disable-parse and disable-trace options have at least as 
much impact on CPU and size overhead as the gst-debug, so it would make 
sense to export them as well. Although actually I expect the difference 
to be vanishingly small for any of them.

> +
> +config BR2_PACKAGE_GSTREAMER1_PLUGIN_REGISTRY
> +	bool "enable plugin registry"
> +	default y
> +	help
> +	  Enable support for the GStreamer plugin registry. This may increase
> +	  the launch-time for a GStreamer application.
> +
> +config BR2_PACKAGE_GSTREAMER1_INSTALL_TOOLS
> +	bool "install gst-launch gst-inspect"

  "install gst-launch & gst-inspect"

> +	default y
> +	help
> +	  Install the gst-launch and gst-inspect tools. This will take up
> +	  additional space on the target
> +
> +endif
> +
> +comment "gstreamer1 requires a toolchain with WCHAR and threads support"
> +	depends on !BR2_USE_WCHAR || !BR2_TOOLCHAIN_HAS_THREADS
> +
> +
> diff --git a/package/multimedia/gstreamer1/gstreamer1.mk b/package/multimedia/gstreamer1/gstreamer1.mk
> new file mode 100644
> index 0000000..90cf0f6
> --- /dev/null
> +++ b/package/multimedia/gstreamer1/gstreamer1.mk
> @@ -0,0 +1,38 @@
> +#############################################################
> +#
> +# gstreamer1
> +#
> +#############################################################
> +
> +GSTREAMER1_VERSION = 1.1.1
> +GSTREAMER1_SOURCE = gstreamer-$(GSTREAMER1_VERSION).tar.xz
> +GSTREAMER1_SITE = http://gstreamer.freedesktop.org/src/gstreamer
> +GSTREAMER1_INSTALL_STAGING = YES
> +GSTREAMER1_LICENSE_FILES = COPYING
> +GSTREAMER1_LICENSE = GPLv2 LGPLv2+ LGPLv2.1+

  I couldn't find any GPLv2 license headers except some tests and an m4 
header (which we normally don't consider to be part of the code requiring 
a license).


  Regards,
  Arnout

> +
> +# Checking if unaligned memory access works correctly cannot be done when cross
> +# compiling. For the following architectures there is no information available
> +# in the configure script.
> +ifeq ($(BR2_avr32),y)
> +GSTREAMER1_CONF_ENV = as_cv_unaligned_access=no
> +endif
> +ifeq ($(BR2_aarch64),y)
> +GSTREAMER1_CONF_ENV = as_cv_unaligned_access=yes
> +endif
> +
> +GSTREAMER1_CONF_OPT = \
> +	--disable-examples \
> +	--disable-tests \
> +	--disable-failing-tests \
> +	--disable-debug \
> +	--disable-valgrind \
> +	--disable-benchmarks \
> +	--disable-check \
> +	$(if $(BR2_PACKAGE_GSTREAMER1_GST_DEBUG),,--disable-gst-debug) \
> +	$(if $(BR2_PACKAGE_GSTREAMER1_PLUGIN_REGISTRY),,--disable-registry) \
> +	$(if $(BR2_PACKAGE_GSTREAMER1_INSTALL_TOOLS),,--disable-tools)
> +
> +GSTREAMER1_DEPENDENCIES = libglib2 host-pkgconf host-bison host-flex
> +
> +$(eval $(autotools-package))
>


-- 
Arnout Vandecappelle                          arnout at mind be
Senior Embedded Software Architect            +32-16-286500
Essensium/Mind                                http://www.mind.be
G.Geenslaan 9, 3001 Leuven, Belgium           BE 872 984 063 RPR Leuven
LinkedIn profile: http://www.linkedin.com/in/arnoutvandecappelle
GPG fingerprint:  7CB5 E4CC 6C2E EFD4 6E3D A754 F963 ECAB 2450 2F1F

  reply	other threads:[~2013-06-12  5:51 UTC|newest]

Thread overview: 23+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-06-07 23:18 [Buildroot] [PATCH v10 00/13] add Gstreamer 1.X for gst-omx support Spenser Gilliland
2013-06-07 23:18 ` [Buildroot] [PATCH v10 01/13] libglib2: Bump libglib2 to 2.36.1 Spenser Gilliland
2013-06-07 23:18 ` [Buildroot] [PATCH v10 02/13] gdk-pixbuf: bump to version 2.28.1 Spenser Gilliland
2013-06-07 23:18 ` [Buildroot] [PATCH v10 03/13] libgtk2: bump to version 2.24.18 and drop DirectFB Spenser Gilliland
2013-06-08 22:38   ` Peter Korsgaard
2013-06-07 23:18 ` [Buildroot] [PATCH v10 04/13] libsoup: bump to version 2.43.1 Spenser Gilliland
2013-06-08 22:38   ` Peter Korsgaard
2013-06-07 23:18 ` [Buildroot] [PATCH v10 05/13] webkit: bump to version 1.9.6 Spenser Gilliland
2013-06-08 22:38   ` Peter Korsgaard
2013-06-07 23:18 ` [Buildroot] [PATCH v10 06/13] gstreamer1: Add gstreamer version 1.1.1 package Spenser Gilliland
2013-06-12  5:51   ` Arnout Vandecappelle [this message]
2013-06-07 23:18 ` [Buildroot] [PATCH v10 07/13] gstreamer: update Config.in with gstreamer1 info Spenser Gilliland
2013-06-12  5:52   ` Arnout Vandecappelle
2013-06-07 23:18 ` [Buildroot] [PATCH v10 08/13] gst1-plugins-base: add gstreamer1 base plugins Spenser Gilliland
2013-06-12  6:21   ` Arnout Vandecappelle
2013-06-12 19:19     ` Spenser Gilliland
2013-06-13  5:57       ` Arnout Vandecappelle
2013-06-13 15:10         ` Spenser Gilliland
2013-06-07 23:18 ` [Buildroot] [PATCH v10 09/13] gst1-plugins-good: add gstreamer1 good plugins Spenser Gilliland
2013-06-07 23:18 ` [Buildroot] [PATCH v10 10/13] gst1-plugins-bad: add gstreamer1 bad plugins Spenser Gilliland
2013-06-07 23:18 ` [Buildroot] [PATCH v10 11/13] gst1-plugins-ugly: add gstreamer1 ugly plugins Spenser Gilliland
2013-06-07 23:18 ` [Buildroot] [PATCH v10 12/13] gst1-plugins-good: add libvpx support Spenser Gilliland
2013-06-07 23:18 ` [Buildroot] [PATCH v10 13/13] gst-omx: add gst-omx package Spenser Gilliland

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=51B80C66.3070602@mind.be \
    --to=arnout@mind.be \
    --cc=buildroot@busybox.net \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox