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 561990 - qtdemux build fails on mingw due to missing definition of BIG_ENDIAN
qtdemux build fails on mingw due to missing definition of BIG_ENDIAN
Status: RESOLVED FIXED
Product: GStreamer
Classification: Platform
Component: gst-plugins-good
git master
Other All
: Normal normal
: 0.10.14
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2008-11-23 10:02 UTC by Tal Shalif
Modified: 2009-01-23 08:41 UTC
See Also:
GNOME target: ---
GNOME version: ---

Attachments
patch to convert {BIG,LITTLE}_ENDIAN to G_{BIG,LITTLE}_ENDIAN (2.11 KB, patch)
2008-11-23 10:04 UTC, Tal Shalif 		committed Details | Review

Description Tal Shalif 2008-11-23 10:02:03 UTC 
Please describe the problem:
My build of the qtdemux plugin fails due to missing definition of {BIG,LITTLE}_ENDIAN. I am using the mingw cross compiler. I have locally fixed that problem by replacing  BIG_ENDIAN -> G_BIG_ENDIAN, LITTLE_ENDIAN -> G_LITTLE_ENDIAN in qtdemux.c (see attached patch)

Steps to reproduce:


Actual results:


Expected results:


Does this happen every time?


Other information:
Comment 1 Tal Shalif 2008-11-23 10:04:22 UTC 
Created attachment 123258 [details] [review]
patch to convert {BIG,LITTLE}_ENDIAN to G_{BIG,LITTLE}_ENDIAN
Comment 2 Sebastian Dröge (slomo) 2008-11-23 11:14:43 UTC 
2008-11-23  Sebastian Dröge  <sebastian.droege@collabora.co.uk>

        Patch by: Tal Shalif <tshalif at nargila dot org>

        * gst/qtdemux/qtdemux.c: (qtdemux_audio_caps):
        Use G_{BIG,LITTLE}_ENDIAN instead of the non-GLib variants as
        the latter don't exist on some systems (mingw). Fixes bug #561992.