After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 706055 - gstvaapi jpeg decoder check failed in autogen.sh
gstvaapi jpeg decoder check failed in autogen.sh
Status: RESOLVED FIXED
Product: gstreamer-vaapi
Classification: Other
Component: general
git master
Other Linux
: Normal major
: ---
Assigned To: gstreamer-vaapi maintainer(s)
gstreamer-vaapi maintainer(s)
Depends on:
Blocks:
 
 
Reported: 2013-08-15 10:21 UTC by Wind Yuan
Modified: 2013-08-30 01:43 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
0001-jpeg-fix-compile-miss-on-va_dec_jpeg.h.patch (1.66 KB, patch)
2013-08-15 10:23 UTC, Wind Yuan
none Details | Review
0002-jpeg-fix-compile-flags-in-jpeg-check.patch (869 bytes, patch)
2013-08-22 05:31 UTC, Wind Yuan
none Details | Review

Description Wind Yuan 2013-08-15 10:21:32 UTC
with some version of libva, JPEG parameters moved from va/va.h to va/va_dec_jpeg.h; configure.ac only include <va/va.h> to check libva JPEG supports. some version failed with
checking for JPEG decoding API... no
but it has va/va_dec_jpeg.h header file.
Comment 1 Wind Yuan 2013-08-15 10:23:52 UTC
Created attachment 251712 [details] [review]
0001-jpeg-fix-compile-miss-on-va_dec_jpeg.h.patch

this patch include <va/va_dec_jpeg.h> check, please help review
Comment 2 Wind Yuan 2013-08-22 05:31:11 UTC
Created attachment 252695 [details] [review]
0002-jpeg-fix-compile-flags-in-jpeg-check.patch

0002 is based on 0001.
After some tests, 0001 is not enough for all platforms, 0002 is a fix based on 0001 to make most platform work on jpeg build
Comment 3 Gwenole Beauchesne 2013-08-26 09:27:28 UTC
Hi,

(In reply to comment #0)
> with some version of libva, JPEG parameters moved from va/va.h to
> va/va_dec_jpeg.h; configure.ac only include <va/va.h> to check libva JPEG
> supports. some version failed with
> checking for JPEG decoding API... no
> but it has va/va_dec_jpeg.h header file.

Please define which version of libva. Testing for <va/va.h> only, and this is what is done already, should be enough for all released versions of libva, down to including 1.0.16.
Comment 4 Wind Yuan 2013-08-27 02:35:03 UTC
0.34.0
Comment 5 Gwenole Beauchesne 2013-08-27 04:25:36 UTC
Hi,

(In reply to comment #4)
> 0.34.0

That's the API version, please define the libva version. All released versions of libva include <va/va_dec_jpeg.h> from <va/va.h>. So, where does yours come from?
Comment 6 Wind Yuan 2013-08-27 04:47:52 UTC
#define PACKAGE_VERSION "1.2.2.pre1"
try libva *staging* branch. gst-vaapi should work on both master and staging.
Comment 7 Gwenole Beauchesne 2013-08-27 06:02:22 UTC
(In reply to comment #6)
> #define PACKAGE_VERSION "1.2.2.pre1"
> try libva *staging* branch. gst-vaapi should work on both master and staging.

Libva staging branch is a development branch and should not be used as is. So, this is a libva issue. I will push the fix there.
Comment 8 Wind Yuan 2013-08-27 06:18:54 UTC
new features and fast bug fix are in staging branch. since gst-vaapi works closely with libva, it need to support staging. if you push fix in libva staging, that's ok. all I want is making gst-vaapi work well under QA tests.
Comment 9 Gwenole Beauchesne 2013-08-29 17:46:37 UTC
Fixed in libva git staging branch. Also squashed and included your patches to gstreamer-vaapi, just in case someone already shipped with invalid VA-API headers.
Comment 10 Gwenole Beauchesne 2013-08-29 17:46:50 UTC
commit 621d48a5b1d1b51256ef8a6e76b8ee625fd4f7c3
Author: Wind Yuan <feng.yuan@intel.com>
Date:   Thu Aug 15 17:59:37 2013 +0800

    configure: fix detection of VA/JPEG decoding API.
    
    Fix detection of VA/JPEG decoding API with non-standard libva packages.
    More precisely, some packages were shipping with a <va/va.h> header that
    did not include <va/va_dec_jpeg.h>.
    
    https://bugzilla.gnome.org/show_bug.cgi?id=706055
Comment 11 Wind Yuan 2013-08-30 01:43:30 UTC
Good to know:)