Skip to contentSkip to conversation

package/libva-utils: new package

PATCH New Bernd Kuhls 2 participants 2 messages Jul 3, 2017

lore
Share & tools
lore

Bernd Kuhls

author
#0 lore raw

PATCH v4 1/1 package/libva-utils: new package

The utils provided by this package were formerly packaged with libva.

Libva contained a patch not to compile mpeg2vldemo to reduce dependencies by avoiding C++, this patch is not moved to this package.

Signed-off-by Bernd Kuhls <[email protected]>
Reviewed-by Romain Naour <[email protected]>

---
v4: - bumped version to 1.8.3
v3: - bumped version to 1.8.1
- used "select BR2_PACKAGE_LIBVA" instead of "depends on" (Arnout) Imho it makes sense to have only one Kconfig option for libva- utils. @Arnout:
I kept the package in "Graphic libraries and applications" because it is not a library and also kept the package name so it stays in sync with the upstream name.
- added comment about c++ dependency (Thomas)
v2: addressed review by Thomas:
- added entry in DEVELOPERS
- moved package to Graphic libraries and applications
- package depends on libva

Signed-off-by Bernd Kuhls <[email protected]>
---
 DEVELOPERS                           |  1 +
 package/Config.in                    |  1 +
 package/libva-utils/Config.in        | 15 +++++++++++++++
 package/libva-utils/libva-utils.hash |  4 ++++
 package/libva-utils/libva-utils.mk   | 14 ++++++++++++++
 5 files changed, 35 insertions(+)
 create mode 100644 package/libva-utils/Config.in
 create mode 100644 package/libva-utils/libva-utils.hash
 create mode 100644 package/libva-utils/libva-utils.mk
DEVELOPERS +1 -0
index d121bd816..55d2c7a1d 100644
@@ -249,6 +249,7 @@ F:	package/libsquish/
 F:	package/liburiparser/
 F:	package/libva/
 F:	package/libva-intel-driver/
+F:	package/libva-utils/
 F:	package/libvorbis/
 F:	package/libvpx/
 F:	package/libyuv/
package/Config.in +1 -0
index 355aa5201..9b5e94576 100644
@@ -247,6 +247,7 @@ comment "Graphic applications"
 	source "package/gnuplot/Config.in"
 	source "package/jhead/Config.in"
 	source "package/kmscube/Config.in"
+	source "package/libva-utils/Config.in"
 	source "package/mesa3d-demos/Config.in"
 	source "package/pngquant/Config.in"
 	source "package/qt5cinex/Config.in"
package/libva-utils/Config.in new file +15 -0
new file mode 100644
index 000000000..ad299234c
@@ -0,0 +1,15 @@
+config BR2_PACKAGE_LIBVA_UTILS
+	bool "libva-utils"
+	depends on BR2_INSTALL_LIBSTDCPP
+	depends on !BR2_STATIC_LIBS # libva
+	depends on BR2_TOOLCHAIN_HAS_THREADS # libva
+	select BR2_PACKAGE_LIBVA
+	help
+	  Libva-utils is a collection of tests for VA-API (Video
+	  Acceleration API)
+
+	  https://01.org/linuxmedia/vaapi
+
+comment "libva-utils needs a toolchain w/ C++, threads, dynamic library"
+	depends on !BR2_INSTALL_LIBSTDCPP || \
+		BR2_STATIC_LIBS || !BR2_TOOLCHAIN_HAS_THREADS
package/libva-utils/libva-utils.hash new file +4 -0
new file mode 100644
index 000000000..737b246a0
@@ -0,0 +1,4 @@
+# From https://github.com/01org/libva-utils/releases
+sha1 8684aa2b28fc08bf62b1fa5acf95d23fab5faf9a libva-utils-1.8.3.tar.bz2
+# Locally computed
+sha256 c59de4fb6f1021c435b3f49e2410760692324ee5bb464c716d674fcb626a7e03  libva-utils-1.8.3.tar.bz2
package/libva-utils/libva-utils.mk new file +14 -0
new file mode 100644
index 000000000..80ab529fb
@@ -0,0 +1,14 @@
+################################################################################
+#
+# libva-utils
+#
+################################################################################
+
+LIBVA_UTILS_VERSION = 1.8.3
+LIBVA_UTILS_SOURCE = libva-utils-$(LIBVA_UTILS_VERSION).tar.bz2
+LIBVA_UTILS_SITE = https://github.com/01org/libva-utils/releases/download/$(LIBVA_UTILS_VERSION)
+LIBVA_UTILS_LICENSE = MIT
+LIBVA_UTILS_LICENSE_FILES = COPYING
+LIBVA_UTILS_DEPENDENCIES = host-pkgconf libva
+
+$(eval $(autotools-package))
Signature · 1 line
2.11.0
Applied
#1 lore raw

PATCH v4 1/1 package/libva-utils: new package

Hello,

Quoted 7 lines · Bernd Kuhls

The utils provided by this package were formerly packaged with libva.

Libva contained a patch not to compile mpeg2vldemo to reduce dependencies by avoiding C++, this patch is not moved to this package.

Signed-off-by Bernd Kuhls <[email protected]>
Reviewed-by Romain Naour <[email protected]>
Quoted 14 lines

---
v4: - bumped version to 1.8.3
v3: - bumped version to 1.8.1
- used "select BR2_PACKAGE_LIBVA" instead of "depends on" (Arnout) Imho it makes sense to have only one Kconfig option for libva- utils. @Arnout:
I kept the package in "Graphic libraries and applications" because it is not a library and also kept the package name so it stays in sync with the upstream name.
- added comment about c++ dependency (Thomas)
v2: addressed review by Thomas:
- added entry in DEVELOPERS
- moved package to Graphic libraries and applications
- package depends on libva

Applied to master, thanks.

Thomas

Signature · 3 lines
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com