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 788064 - zynqultrascaleplus: Requires third party OMX header not included and not checked
zynqultrascaleplus: Requires third party OMX header not included and not checked
Status: RESOLVED FIXED
Product: GStreamer
Classification: Platform
Component: gst-omx
1.13.x
Other Linux
: Normal normal
: 1.13.1
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2017-09-22 20:36 UTC by Nicolas Dufresne (ndufresne)
Modified: 2017-10-19 14:25 UTC
See Also:
GNOME target: ---
GNOME version: ---

Attachments
check if Allegro headers are present when building zynqultrascaleplus (1.86 KB, patch)
2017-10-17 10:19 UTC, Guillaume Desmottes 		committed Details | Review

Description Nicolas Dufresne (ndufresne) 2017-09-22 20:36:27 UTC 
The zynqultrascaleplus rely on Allegro HEADERS and extension that are neither present (like we do for RPi externsion) or checked (at least not by meson). This should be fixed.
Comment 1 Guillaume Desmottes 2017-10-16 08:36:38 UTC 
You mean that we should be able to build for the zynqultrascaleplus but using standard OMX headers by disabling the Allegro specific code?
Comment 2 Nicolas Dufresne (ndufresne) 2017-10-16 13:17:50 UTC 
No, you only want to build this target in a functional way. What's wrong here is that you rely on system headers, but do not check if they are present in configure and meson. Check their presence, and print a useful message on where to find them and how to configure them.
Comment 3 Guillaume Desmottes 2017-10-17 10:19:08 UTC 
Created attachment 361731 [details] [review]
check if Allegro headers are present when building zynqultrascaleplus

The Zynq UltraScale+ uses a custom version of OMX implementing several
3rd party extensions. Make sure those are present when building this
target.
Comment 4 Sebastian Dröge (slomo) 2017-10-19 14:23:57 UTC 
Attachment 361731 [details] pushed as a0e7fc7 - check if Allegro headers are present when building zynqultrascaleplus