[Buildroot] [PATCH v1 1/1] vidstab: new package
Thomas Petazzoni
thomas.petazzoni at free-electrons.com
Wed Jul 15 09:53:07 UTC 2015
Dear Steve Kenton,
On Mon, 27 Apr 2015 22:42:00 -0500, Steve Kenton wrote:
> Vid.stab is a video image stabilization library
> that can be used with ffmpeg --enable-libvidstab
> and is acknowledged by the ffmpeg developers to
> be superior to the built-in deshake filter. It
> works best in a 2-pass configuration but 1-pass
> is also available when needed/acceptable.
No need to indent the commit log.
> package/Config.in | 1 +
> package/vidstab/Config.in | 20 ++++++++++++++++++++
> package/vidstab/vidstab.mk | 14 ++++++++++++++
> 3 files changed, 35 insertions(+)
> create mode 100644 package/vidstab/Config.in
> create mode 100644 package/vidstab/vidstab.mk
>
> diff --git a/package/Config.in b/package/Config.in
> index 4e20e4d..99935d8 100644
> --- a/package/Config.in
> +++ b/package/Config.in
> @@ -802,6 +802,7 @@ menu "Graphics"
> source "package/poppler/Config.in"
> source "package/powervr/Config.in"
> source "package/tiff/Config.in"
> + source "package/vidstab/Config.in"
> source "package/wayland/Config.in"
> source "package/webkit/Config.in"
> source "package/webp/Config.in"
> diff --git a/package/vidstab/Config.in b/package/vidstab/Config.in
> new file mode 100644
> index 0000000..d900d17
> --- /dev/null
> +++ b/package/vidstab/Config.in
> @@ -0,0 +1,20 @@
> +config BR2_PACKAGE_VIDSTAB
> + bool "vidstab"
> + depends on BR2_INSTALL_LIBSTDCPP
> + help
> + Vid.stab is a video image stabilization library
> + that can be used with ffmpeg --enable-libvidstab
probably no need to mention --enable-libvidstab here explicitly.
> + and is acknowledged by the ffmpeg developers to
> + be superior to the built-in deshake filter. It
> + works best in a 2-pass configuration but 1-pass
> + is also available when needed/acceptable.
> +
> + Buildroot does not support packages with a '.'
> + in their name, which explains why it is named
> + vidstab and not vid.stab as might be expected.
> +
> + https://github.com/georgmartius/vid.stab
> +
> +comment "vidstab needs a toolschain w/ C++"
> + depends on !BR2_INSTALL_LIBSTDCPP
> +
Unneeded empty new line.
> diff --git a/package/vidstab/vidstab.mk b/package/vidstab/vidstab.mk
> new file mode 100644
> index 0000000..53fb420
> --- /dev/null
> +++ b/package/vidstab/vidstab.mk
> @@ -0,0 +1,14 @@
> +################################################################################
> +#
> +# vidstab
> +#
> +################################################################################
> +
> +VIDSTAB_VERSION = 3b35b4ddeb595511593a862849e750c459b99cb8
> +VIDSTAB_SITE = $(call github,georgmartius,vid.stab,$(VIDSTAB_VERSION))
> +VIDSTAB_LICENSE = GPL
License is GPLv2+
> +VIDSTAB_LICENSE_FILES = LICENSE
> +VIDSTAB_INSTALL_STAGING = YES
> +
> +$(eval $(cmake-package))
> +
Unneeded new line.
However, unfortunately, it doesn't build here: it uses x86 specific gcc
flags even on ARM:
>>> vidstab 3b35b4ddeb595511593a862849e750c459b99cb8 Building
PATH="/home/thomas/projets/buildroot/output/host/bin:/home/thomas/projets/buildroot/output/host/sbin:/home/thomas/projets/buildroot/output/host/usr/bin:/home/thomas/projets/buildroot/output/host/usr/sbin:/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin:/usr/games:/usr/local/games:/home/thomas/sys/bin" /usr/bin/make -j5 -C /home/thomas/projets/buildroot/output/build/vidstab-3b35b4ddeb595511593a862849e750c459b99cb8/
Scanning dependencies of target vidstab
[ 16%] [ 16%] [ 25%] Building C object CMakeFiles/vidstab.dir/src/transformtype.c.o
Building C object CMakeFiles/vidstab.dir/src/libvidstab.c.o
[ 33%] [ 41%] Building C object CMakeFiles/vidstab.dir/src/frameinfo.c.o
Building C object CMakeFiles/vidstab.dir/src/transformfixedpoint.c.o
Building C object CMakeFiles/vidstab.dir/src/transform.c.o
arm-linux-gcc: error: unrecognized command line option ‘-msse2’
arm-linux-gcc: error: unrecognized command line option ‘-msse2’
CMakeFiles/vidstab.dir/build.make:100: recipe for target 'CMakeFiles/vidstab.dir/src/libvidstab.c.o' failed
make[4]: *** [CMakeFiles/vidstab.dir/src/libvidstab.c.o] Error 1
Could you fix those issues and send an updated version of the patch?
Also, make sure that it builds fine with a static only toolchain like
http://autobuild.buildroot.org/toolchains/configs/br-arm-full-static.config
and if it doesn't, make the package depends on !BR2_STATIC_LIBS (see
other packages).
I'll mark your patch as Changes Requested in our patch tracking system.
Thanks!
Thomas
--
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com
More information about the buildroot
mailing list