* [Buildroot] [PATCH 1/3] package/libva: bump version to 1.8.0
@ 2017-04-02 8:13 Bernd Kuhls
2017-04-02 8:13 ` [Buildroot] [PATCH 2/3] package/libva-intel-driver: " Bernd Kuhls
` (2 more replies)
0 siblings, 3 replies; 7+ messages in thread
From: Bernd Kuhls @ 2017-04-02 8:13 UTC (permalink / raw)
To: buildroot
Removed patch because libva-utils are no longer part of the libva
package:
https://lists.01.org/pipermail/intel-vaapi-media/2017-March/000015.html
* Move all utilities and tests to libva-utils
Package libva-utils will be added later.
Signed-off-by: Bernd Kuhls <bernd.kuhls@t-online.de>
---
package/libva/0001-no_test_decode.patch | 17 -----------------
package/libva/libva.hash | 4 +++-
package/libva/libva.mk | 12 +++---------
3 files changed, 6 insertions(+), 27 deletions(-)
delete mode 100644 package/libva/0001-no_test_decode.patch
diff --git a/package/libva/0001-no_test_decode.patch b/package/libva/0001-no_test_decode.patch
deleted file mode 100644
index 0f0d78667..000000000
--- a/package/libva/0001-no_test_decode.patch
+++ /dev/null
@@ -1,17 +0,0 @@
-mpeg2vldemo is the only file requiring cxx support. It is not needed for
-buildroot, so disable its compilation to avoid depending on cxx.
-
-Signed-off-by: Bernd Kuhls <bernd.kuhls@t-online.de>
-
-diff -uNr libva-1.3.0.org/test/Makefile.am libva-1.3.0/test/Makefile.am
---- libva-1.3.0.org/test/Makefile.am 2014-03-24 05:49:46.000000000 +0100
-+++ libva-1.3.0/test/Makefile.am 2014-04-26 18:12:25.644390292 +0200
-@@ -20,7 +20,7 @@
- # TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION WITH THE
- # SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE.
-
--SUBDIRS = common decode encode vainfo
-+SUBDIRS = common encode vainfo
-
- if USE_X11
- SUBDIRS += basic putsurface
diff --git a/package/libva/libva.hash b/package/libva/libva.hash
index 1f248f8b6..086a010b9 100644
--- a/package/libva/libva.hash
+++ b/package/libva/libva.hash
@@ -1,2 +1,4 @@
+# From https://lists.01.org/pipermail/intel-vaapi-media/2017-March/000015.html
+sha1 d9468956c301784b6b12c39cafd30ac52e24290b libva-1.8.0.tar.bz2
# Locally computed
-sha256 f0f816a725007fbf3c7ed816ff71ba420696bec63ff49045e9b23d9b34ac73c8 libva-1.7.3.tar.gz
+sha256 eb92f3dcbe3981df3575348377263b31361113c77b2c3745f23174d1f562d658 libva-1.8.0.tar.bz2
diff --git a/package/libva/libva.mk b/package/libva/libva.mk
index bce69072e..f16be900b 100644
--- a/package/libva/libva.mk
+++ b/package/libva/libva.mk
@@ -4,11 +4,11 @@
#
################################################################################
-LIBVA_VERSION = 1.7.3
-LIBVA_SITE = $(call github,01org,libva,libva-$(LIBVA_VERSION))
+LIBVA_VERSION = 1.8.0
+LIBVA_SOURCE = libva-$(LIBVA_VERSION).tar.bz2
+LIBVA_SITE = https://github.com/01org/libva/releases/download/$(LIBVA_VERSION)
LIBVA_LICENSE = MIT
LIBVA_LICENSE_FILES = COPYING
-LIBVA_AUTORECONF = YES
LIBVA_INSTALL_STAGING = YES
LIBVA_DEPENDENCIES = host-pkgconf libdrm
@@ -46,10 +46,4 @@ else
LIBVA_CONF_OPTS += --disable-egl
endif
-# Autoreconf requires an m4 directory to exist
-define LIBVA_PATCH_M4
- mkdir -p $(@D)/m4
-endef
-LIBVA_POST_PATCH_HOOKS += LIBVA_PATCH_M4
-
$(eval $(autotools-package))
--
2.11.0
^ permalink raw reply related [flat|nested] 7+ messages in thread* [Buildroot] [PATCH 2/3] package/libva-intel-driver: bump version to 1.8.0
2017-04-02 8:13 [Buildroot] [PATCH 1/3] package/libva: bump version to 1.8.0 Bernd Kuhls
@ 2017-04-02 8:13 ` Bernd Kuhls
2017-04-02 21:45 ` Thomas Petazzoni
2017-04-02 8:13 ` [Buildroot] [PATCH 3/3] package/libva-utils: new package Bernd Kuhls
2017-04-02 21:45 ` [Buildroot] [PATCH 1/3] package/libva: bump version to 1.8.0 Thomas Petazzoni
2 siblings, 1 reply; 7+ messages in thread
From: Bernd Kuhls @ 2017-04-02 8:13 UTC (permalink / raw)
To: buildroot
Autoreconf is not necessary anymore after a fix was commit upstream:
https://github.com/01org/intel-vaapi-driver/commit/fe6280841dfaaee94e8664517420f351cd334c86
Signed-off-by: Bernd Kuhls <bernd.kuhls@t-online.de>
---
package/libva-intel-driver/libva-intel-driver.hash | 4 +++-
package/libva-intel-driver/libva-intel-driver.mk | 7 +++----
2 files changed, 6 insertions(+), 5 deletions(-)
diff --git a/package/libva-intel-driver/libva-intel-driver.hash b/package/libva-intel-driver/libva-intel-driver.hash
index a9688763a..ea885bc52 100644
--- a/package/libva-intel-driver/libva-intel-driver.hash
+++ b/package/libva-intel-driver/libva-intel-driver.hash
@@ -1,2 +1,4 @@
+# From https://lists.01.org/pipermail/intel-vaapi-media/2017-March/000016.html
+sha1 9982122c8ef36aad01aa09c083974532afa8f8ca intel-vaapi-driver-1.8.0.tar.bz2
# Locally computed
-sha256 c1f02cbdf0398f096ddc91096fac6ee62267a54955f6d235d73a8ff2ea888118 libva-intel-driver-1.7.3.tar.gz
+sha256 4fcbb282957cf4f33b6c43a0f476d8df21f7721e586a8546c71242551986b3a6 intel-vaapi-driver-1.8.0.tar.bz2
diff --git a/package/libva-intel-driver/libva-intel-driver.mk b/package/libva-intel-driver/libva-intel-driver.mk
index e2f855bb3..ef14ca1e2 100644
--- a/package/libva-intel-driver/libva-intel-driver.mk
+++ b/package/libva-intel-driver/libva-intel-driver.mk
@@ -4,13 +4,12 @@
#
################################################################################
-LIBVA_INTEL_DRIVER_VERSION = 1.7.3
+LIBVA_INTEL_DRIVER_VERSION = 1.8.0
+LIBVA_INTEL_DRIVER_SOURCE = intel-vaapi-driver-$(LIBVA_INTEL_DRIVER_VERSION).tar.bz2
LIBVA_INTEL_DRIVER_SITE = \
- $(call github,01org,intel-vaapi-driver,$(LIBVA_INTEL_DRIVER_VERSION))
+ https://github.com/01org/intel-vaapi-driver/releases/download/$(LIBVA_INTEL_DRIVER_VERSION)
LIBVA_INTEL_DRIVER_LICENSE = MIT
LIBVA_INTEL_DRIVER_LICENSE_FILES = COPYING
-# needed to work around https://bugs.freedesktop.org/show_bug.cgi?id=79478
-LIBVA_INTEL_DRIVER_AUTORECONF = YES
LIBVA_INTEL_DRIVER_DEPENDENCIES = host-pkgconf libdrm libva
ifeq ($(BR2_PACKAGE_XORG7),y)
--
2.11.0
^ permalink raw reply related [flat|nested] 7+ messages in thread
* [Buildroot] [PATCH 2/3] package/libva-intel-driver: bump version to 1.8.0
2017-04-02 8:13 ` [Buildroot] [PATCH 2/3] package/libva-intel-driver: " Bernd Kuhls
@ 2017-04-02 21:45 ` Thomas Petazzoni
0 siblings, 0 replies; 7+ messages in thread
From: Thomas Petazzoni @ 2017-04-02 21:45 UTC (permalink / raw)
To: buildroot
Hello,
On Sun, 2 Apr 2017 10:13:26 +0200, Bernd Kuhls wrote:
> Autoreconf is not necessary anymore after a fix was commit upstream:
> https://github.com/01org/intel-vaapi-driver/commit/fe6280841dfaaee94e8664517420f351cd334c86
>
> Signed-off-by: Bernd Kuhls <bernd.kuhls@t-online.de>
> ---
> package/libva-intel-driver/libva-intel-driver.hash | 4 +++-
> package/libva-intel-driver/libva-intel-driver.mk | 7 +++----
> 2 files changed, 6 insertions(+), 5 deletions(-)
Applied to master, thanks.
Thomas
--
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com
^ permalink raw reply [flat|nested] 7+ messages in thread
* [Buildroot] [PATCH 3/3] package/libva-utils: new package
2017-04-02 8:13 [Buildroot] [PATCH 1/3] package/libva: bump version to 1.8.0 Bernd Kuhls
2017-04-02 8:13 ` [Buildroot] [PATCH 2/3] package/libva-intel-driver: " Bernd Kuhls
@ 2017-04-02 8:13 ` Bernd Kuhls
2017-04-02 21:46 ` Thomas Petazzoni
2017-04-02 21:45 ` [Buildroot] [PATCH 1/3] package/libva: bump version to 1.8.0 Thomas Petazzoni
2 siblings, 1 reply; 7+ messages in thread
From: Bernd Kuhls @ 2017-04-02 8:13 UTC (permalink / raw)
To: buildroot
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 <bernd.kuhls@t-online.de>
---
package/Config.in | 1 +
package/libva-utils/Config.in | 15 +++++++++++++++
package/libva-utils/libva-utils.hash | 4 ++++
package/libva-utils/libva-utils.mk | 14 ++++++++++++++
4 files changed, 34 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
diff --git a/package/Config.in b/package/Config.in
index d6d134d0c..364766e37 100644
--- a/package/Config.in
+++ b/package/Config.in
@@ -1049,6 +1049,7 @@ menu "Graphics"
source "package/libsvgtiny/Config.in"
source "package/libva/Config.in"
source "package/libva-intel-driver/Config.in"
+ source "package/libva-utils/Config.in"
source "package/libvdpau/Config.in"
source "package/libvips/Config.in"
source "package/menu-cache/Config.in"
diff --git a/package/libva-utils/Config.in b/package/libva-utils/Config.in
new file mode 100644
index 000000000..89615cad3
--- /dev/null
+++ b/package/libva-utils/Config.in
@@ -0,0 +1,15 @@
+config BR2_PACKAGE_LIBVA_UTILS
+ bool "libva-utils"
+ select BR2_PACKAGE_LIBVA
+ depends on BR2_INSTALL_LIBSTDCPP
+ depends on BR2_TOOLCHAIN_HAS_THREADS # libva
+ depends on !BR2_STATIC_LIBS # 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_TOOLCHAIN_HAS_THREADS || BR2_STATIC_LIBS
diff --git a/package/libva-utils/libva-utils.hash b/package/libva-utils/libva-utils.hash
new file mode 100644
index 000000000..5a51e86c8
--- /dev/null
+++ b/package/libva-utils/libva-utils.hash
@@ -0,0 +1,4 @@
+# From https://lists.01.org/pipermail/intel-vaapi-media/2017-March/000017.html
+sha1 1cdd17ee0cd5ccf0da2ed85512d87bf1931b2bc1 libva-utils-1.8.0.tar.bz2
+# Locally computed
+sha256 ca4574111fd5658d07dcf883fa1a9914b02f77011f51afcc880af1c302afdbcc libva-utils-1.8.0.tar.bz2
diff --git a/package/libva-utils/libva-utils.mk b/package/libva-utils/libva-utils.mk
new file mode 100644
index 000000000..e8ed31b2b
--- /dev/null
+++ b/package/libva-utils/libva-utils.mk
@@ -0,0 +1,14 @@
+################################################################################
+#
+# libva-utils
+#
+################################################################################
+
+LIBVA_UTILS_VERSION = 1.8.0
+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))
--
2.11.0
^ permalink raw reply related [flat|nested] 7+ messages in thread
* [Buildroot] [PATCH 3/3] package/libva-utils: new package
2017-04-02 8:13 ` [Buildroot] [PATCH 3/3] package/libva-utils: new package Bernd Kuhls
@ 2017-04-02 21:46 ` Thomas Petazzoni
2017-04-03 13:33 ` Arnout Vandecappelle
0 siblings, 1 reply; 7+ messages in thread
From: Thomas Petazzoni @ 2017-04-02 21:46 UTC (permalink / raw)
To: buildroot
Hello,
On Sun, 2 Apr 2017 10:13:27 +0200, Bernd Kuhls wrote:
> 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 <bernd.kuhls@t-online.de>
> ---
> package/Config.in | 1 +
> package/libva-utils/Config.in | 15 +++++++++++++++
> package/libva-utils/libva-utils.hash | 4 ++++
> package/libva-utils/libva-utils.mk | 14 ++++++++++++++
Entry to DEVELOPERS file is missing.
> diff --git a/package/Config.in b/package/Config.in
> index d6d134d0c..364766e37 100644
> --- a/package/Config.in
> +++ b/package/Config.in
> @@ -1049,6 +1049,7 @@ menu "Graphics"
> source "package/libsvgtiny/Config.in"
> source "package/libva/Config.in"
> source "package/libva-intel-driver/Config.in"
> + source "package/libva-utils/Config.in"
Does it really makes sense to have this package in Libraries->Graphics
when it is in fact not a library at all?
> diff --git a/package/libva-utils/Config.in b/package/libva-utils/Config.in
> new file mode 100644
> index 000000000..89615cad3
> --- /dev/null
> +++ b/package/libva-utils/Config.in
> @@ -0,0 +1,15 @@
> +config BR2_PACKAGE_LIBVA_UTILS
> + bool "libva-utils"
> + select BR2_PACKAGE_LIBVA
> + depends on BR2_INSTALL_LIBSTDCPP
> + depends on BR2_TOOLCHAIN_HAS_THREADS # libva
> + depends on !BR2_STATIC_LIBS # libva
Perhaps this can be summarized as:
depends on BR2_PACKAGE_LIBVA
indeed, someone wanting the libva tests will surely realize that he
needs to enable libva in the first place, no?
Thanks!
Thomas
--
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com
^ permalink raw reply [flat|nested] 7+ messages in thread
* [Buildroot] [PATCH 3/3] package/libva-utils: new package
2017-04-02 21:46 ` Thomas Petazzoni
@ 2017-04-03 13:33 ` Arnout Vandecappelle
0 siblings, 0 replies; 7+ messages in thread
From: Arnout Vandecappelle @ 2017-04-03 13:33 UTC (permalink / raw)
To: buildroot
On 02-04-17 23:46, Thomas Petazzoni wrote:
> Hello,
>
> On Sun, 2 Apr 2017 10:13:27 +0200, Bernd Kuhls wrote:
>> 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 <bernd.kuhls@t-online.de>
>> ---
>> package/Config.in | 1 +
>> package/libva-utils/Config.in | 15 +++++++++++++++
>> package/libva-utils/libva-utils.hash | 4 ++++
>> package/libva-utils/libva-utils.mk | 14 ++++++++++++++
>
> Entry to DEVELOPERS file is missing.
>
>> diff --git a/package/Config.in b/package/Config.in
>> index d6d134d0c..364766e37 100644
>> --- a/package/Config.in
>> +++ b/package/Config.in
>> @@ -1049,6 +1049,7 @@ menu "Graphics"
>> source "package/libsvgtiny/Config.in"
>> source "package/libva/Config.in"
>> source "package/libva-intel-driver/Config.in"
>> + source "package/libva-utils/Config.in"
>
> Does it really makes sense to have this package in Libraries->Graphics
> when it is in fact not a library at all?
>
>> diff --git a/package/libva-utils/Config.in b/package/libva-utils/Config.in
>> new file mode 100644
>> index 000000000..89615cad3
>> --- /dev/null
>> +++ b/package/libva-utils/Config.in
>> @@ -0,0 +1,15 @@
>> +config BR2_PACKAGE_LIBVA_UTILS
>> + bool "libva-utils"
>> + select BR2_PACKAGE_LIBVA
>> + depends on BR2_INSTALL_LIBSTDCPP
>> + depends on BR2_TOOLCHAIN_HAS_THREADS # libva
>> + depends on !BR2_STATIC_LIBS # libva
>
> Perhaps this can be summarized as:
>
> depends on BR2_PACKAGE_LIBVA
>
> indeed, someone wanting the libva tests will surely realize that he
> needs to enable libva in the first place, no?
IMO, if you move the package somewhere else in the menu structure, and then
replace the select with a depends, you're really making life difficult for the
user because they have to go to two places to select this package.
For me, it does make sense to keep the _UTILS in the library menu. I think the
typical use case would be that you select libva because you know you'll need it,
then you see the menu entry for the _UTILS and realize that a test program would
be convenient.
By the way, Bernd, the help text says "Libva-utils is a collection of tests",
so would libva-tests (or perhaps libva-demos" be a more appropriate name?
Regards,
Arnout
>
> Thanks!
>
> Thomas
>
--
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: 7493 020B C7E3 8618 8DEC 222C 82EB F404 F9AC 0DDF
^ permalink raw reply [flat|nested] 7+ messages in thread
* [Buildroot] [PATCH 1/3] package/libva: bump version to 1.8.0
2017-04-02 8:13 [Buildroot] [PATCH 1/3] package/libva: bump version to 1.8.0 Bernd Kuhls
2017-04-02 8:13 ` [Buildroot] [PATCH 2/3] package/libva-intel-driver: " Bernd Kuhls
2017-04-02 8:13 ` [Buildroot] [PATCH 3/3] package/libva-utils: new package Bernd Kuhls
@ 2017-04-02 21:45 ` Thomas Petazzoni
2 siblings, 0 replies; 7+ messages in thread
From: Thomas Petazzoni @ 2017-04-02 21:45 UTC (permalink / raw)
To: buildroot
Hello,
On Sun, 2 Apr 2017 10:13:25 +0200, Bernd Kuhls wrote:
> Removed patch because libva-utils are no longer part of the libva
> package:
> https://lists.01.org/pipermail/intel-vaapi-media/2017-March/000015.html
>
> * Move all utilities and tests to libva-utils
>
> Package libva-utils will be added later.
>
> Signed-off-by: Bernd Kuhls <bernd.kuhls@t-online.de>
> ---
> package/libva/0001-no_test_decode.patch | 17 -----------------
> package/libva/libva.hash | 4 +++-
> package/libva/libva.mk | 12 +++---------
> 3 files changed, 6 insertions(+), 27 deletions(-)
> delete mode 100644 package/libva/0001-no_test_decode.patch
Applied to master, thanks.
Thomas
--
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com
^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2017-04-03 13:33 UTC | newest]
Thread overview: 7+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2017-04-02 8:13 [Buildroot] [PATCH 1/3] package/libva: bump version to 1.8.0 Bernd Kuhls
2017-04-02 8:13 ` [Buildroot] [PATCH 2/3] package/libva-intel-driver: " Bernd Kuhls
2017-04-02 21:45 ` Thomas Petazzoni
2017-04-02 8:13 ` [Buildroot] [PATCH 3/3] package/libva-utils: new package Bernd Kuhls
2017-04-02 21:46 ` Thomas Petazzoni
2017-04-03 13:33 ` Arnout Vandecappelle
2017-04-02 21:45 ` [Buildroot] [PATCH 1/3] package/libva: bump version to 1.8.0 Thomas Petazzoni
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox