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 723176 - omx: Examples need to be ported to 1.0
omx: Examples need to be ported to 1.0
Status: RESOLVED FIXED
Product: GStreamer
Classification: Platform
Component: gst-omx
git master
Other Linux
: Normal normal
: 1.2.0
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2014-01-28 17:06 UTC by Marco Barisione
Modified: 2014-07-23 08:24 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
configure: don't generate Makefiles for the examples directory (1021 bytes, patch)
2014-01-28 17:11 UTC, Marco Barisione
none Details | Review
configure: don't generate Makefiles for the examples directory (1.00 KB, patch)
2014-01-28 17:17 UTC, Marco Barisione
none Details | Review

Description Marco Barisione 2014-01-28 17:06:21 UTC
Commit c10031f371e5e26075b7ae79e9984c36d68c4231 introduced the examples directory to gst-omx, but the examples are not ported to 1.x so they are disabled. They are disabled like this:
# if BUILD_EXAMPLES
# SUBDIRS += examples
# endif

But then in the configure file there's:
AC_CONFIG_FILE(
...
examples/Makefile
examples/egl/Makefile
)

So make dist generates a tarball without the examples directory, but then the directory is needed when compiling the tarball.
Comment 1 Marco Barisione 2014-01-28 17:11:17 UTC
Created attachment 267419 [details] [review]
configure: don't generate Makefiles for the examples directory

The examples directory is ignored in Makefile.am and so it's not
included in tarballs created with make dist. This made configure fail as
it tried to generate make files in that non-existing directory.

See c10031f371e5e26075b7ae79e9984c36d68c4231 for more details on the
introduction of the examples directory.
Comment 2 Marco Barisione 2014-01-28 17:17:24 UTC
Created attachment 267420 [details] [review]
configure: don't generate Makefiles for the examples directory

The examples directory is ignored in Makefile.am and so it's not
included in tarballs created with make dist. This made configure fail as
it tried to generate make files in that non-existing directory.

See c10031f371e5e26075b7ae79e9984c36d68c4231 for more details on the
introduction of the examples directory.
Comment 3 Julien Isorce 2014-01-29 12:24:18 UTC
ok we thought that the example was not supposed to work. So I'm going to check that and fix if needed.
Also I'll ensure it still work with https://bugzilla.gnome.org/show_bug.cgi?id=722686 changes.
Comment 4 Josep Torra Valles 2014-03-11 09:19:55 UTC
The examples in master does work in 1.x.

The SUBDIRS commented block just was missed to be enabled when it was ported.

I've also pushed in master the changes to avoid clashing names with the recent move of EGLImagePol to gst egl library.

Then I think that this bug could be clossed.
Comment 5 Marco Barisione 2014-03-11 16:13:43 UTC
Yep, I can confirm that compiling from the tarball generated with make dist now works. Thanks!