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 733426 - gl/sdl example fails to build (missing include directory?)
gl/sdl example fails to build (missing include directory?)
Status: RESOLVED FIXED
Product: GStreamer
Classification: Platform
Component: gst-plugins-bad
1.x
Other Windows
: Normal normal
: 1.4.1
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2014-07-19 22:02 UTC by LRN
Modified: 2014-07-22 12:33 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Add missing include directory (1.24 KB, patch)
2014-07-21 09:47 UTC, LRN
needs-work Details | Review
Reorder CFLAGS to include in-source dirs first (1.17 KB, patch)
2014-07-21 11:08 UTC, LRN
committed Details | Review

Description LRN 2014-07-19 22:02:08 UTC
While building 1.4.0:
Making all in sdl
make[5]: Entering directory `/f/gstreamer-buildslave/slave/gst-bad-bin/build/tests/examples/gl/sdl'
gcc -std=gnu99 -DHAVE_CONFIG_H -I. -I../../../..   -D_WIN32_WINNT=0x500 -pthread -mms-bitfields -If:/gstreamer-buildslave/mingw/include/gstreamer-1.0 -If:/gstreamer-buildslave/mingw/include/glib-2.0 -If:/gstreamer-buildslave/mingw/lib/glib-2.0/include -DGST_USE_UNSTABLE_API  -DG_THREADS_MANDATORY -DG_DISABLE_CAST_CHECKS -DG_DISABLE_ASSERT -Wall -Wdeclaration-after-statement -Wvla -Wpointer-arith -Wmissing-declarations -Wmissing-prototypes -Wredundant-decls -Wwrite-strings -Wformat-security -Wold-style-definition -Winit-self -Wmissing-include-dirs -Waddress -Wno-multichar -Wnested-externs   -g     -D_GNU_SOURCE=1 -Dmain=SDL_main -If:/gstreamer-buildslave/mingw/include/SDL  -I../../../../gst-libs -g -O2 -MT sdlshare-sdlshare.o -MD -MP -MF .deps/sdlshare-sdlshare.Tpo -c -o sdlshare-sdlshare.o `test -f 'sdlshare.c' || echo './'`sdlshare.c
In file included from f:/gstreamer-buildslave/mingw/include/gstreamer-1.0/gst/gl/gstglmixerpad.h:25:0,
                 from f:/gstreamer-buildslave/mingw/include/gstreamer-1.0/gst/gl/gstglmixer.h:27,
                 from f:/gstreamer-buildslave/mingw/include/gstreamer-1.0/gst/gl/gl.h:41,
                 from sdlshare.c:40:
f:/gstreamer-buildslave/mingw/include/gstreamer-1.0/gst/video/gstvideoaggregator.h:28:35: fatal error: gstvideoaggregatorpad.h: No such file or directory
 #include "gstvideoaggregatorpad.h"
                                   ^
compilation terminated.
make[5]: *** [sdlshare-sdlshare.o] Error 1
make[5]: Leaving directory `/f/gstreamer-buildslave/slave/gst-bad-bin/build/tests/examples/gl/sdl'
make[4]: Leaving directory `/f/gstreamer-buildslave/slave/gst-bad-bin/build/tests/examples/gl'
make[3]: Leaving directory `/f/gstreamer-buildslave/slave/gst-bad-bin/build/tests/examples'
make[2]: Leaving directory `/f/gstreamer-buildslave/slave/gst-bad-bin/build/tests'
make[1]: Leaving directory `/f/gstreamer-buildslave/slave/gst-bad-bin/build'

Obviously, gstvideoaggregatorpad.h is in $(top_srcdir)/gst-libs/gst/video
Comment 1 LRN 2014-07-21 09:47:47 UTC
Created attachment 281295 [details] [review]
Add missing include directory
Comment 2 Sebastian Dröge (slomo) 2014-07-21 10:08:04 UTC
It looks like it is using the libgstgl headers from an installed version of GStreamer, not the currently build one (and not 1.4.0 but 1.3.90).
Comment 3 LRN 2014-07-21 10:33:49 UTC
Yes, it appears to be so.
-bad has been broken for me for quite some time, thus all that time -bad headers and libraries in /mingw were not getting updated (because buildbot doesn't install something that it can't compile completely). That might explain the discrepancy between sdl example expectations and reality.

It's mildly disturbing that makefiles put system include dir (/mingw/include/gstreamer-1.0) before in-source include dirs (i.e. $(top_srcdir)/...), etc). Do you want me to file that as a separate bug? Do you need a patch for this?
Comment 4 Sebastian Dröge (slomo) 2014-07-21 10:40:39 UTC
That would be the right fix for this here, yes. No need for another bug :)
Comment 5 LRN 2014-07-21 11:08:57 UTC
Created attachment 281302 [details] [review]
Reorder CFLAGS to include in-source dirs first
Comment 6 Sebastian Dröge (slomo) 2014-07-21 11:11:33 UTC
commit 27df659fd805592783d13a7c664339eb577ac666
Author: Руслан Ижбулатов <lrn1986@gmail.com>
Date:   Mon Jul 21 11:07:28 2014 +0000

    gl: Reorder CFLAGS to include in-source dirs first in examples
    
    https://bugzilla.gnome.org/show_bug.cgi?id=733426