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 701316 - rtspconnection: using g_pollable_stream_read and write breaks builds on Ubuntu and Debian stable
rtspconnection: using g_pollable_stream_read and write breaks builds on Ubunt...
Status: RESOLVED FIXED
Product: GStreamer
Classification: Platform
Component: gst-plugins-base
git master
Other Linux
: Normal normal
: 1.1.1
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2013-05-30 22:12 UTC by Brendan Long
Modified: 2013-05-31 12:12 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Replace g_pollable_stream_{read,write} with the functions that utility function calls (1.92 KB, patch)
2013-05-30 22:12 UTC, Brendan Long
none Details | Review

Description Brendan Long 2013-05-30 22:12:36 UTC
Created attachment 245682 [details] [review]
Replace g_pollable_stream_{read,write} with the functions that utility function calls

When I build gst-plugins-base today, I got:

gstrtspconnection.c: In function 'write_bytes':
gstrtspconnection.c:866:5: error: implicit declaration of function 'g_pollable_stream_write' [-Werror=implicit-function-declaration]
gstrtspconnection.c:866:5: error: nested extern declaration of 'g_pollable_stream_write' [-Werror=nested-externs]
gstrtspconnection.c: In function 'fill_raw_bytes':
gstrtspconnection.c:921:5: error: implicit declaration of function 'g_pollable_stream_read' [-Werror=implicit-function-declaration]
gstrtspconnection.c:921:5: error: nested extern declaration of 'g_pollable_stream_read' [-Werror=nested-externs]
cc1: all warnings being treated as errors

The problem is because those two functions were added in GLib 2.34[1], which isn't available on Ubuntu 12.04 or Debian Wheezy.

This patch just replaces those functions with the functions they call[2].

[1] https://developer.gnome.org/gio/2.36/gio-gpollableutils.html#g-pollable-stream-write
[2] https://git.gnome.org/browse/glib/tree/gio/gpollableutils.c
Comment 1 Wim Taymans 2013-05-31 12:12:54 UTC
commit 63961242dfe4a7624280a2e971b92245d9f421c6
Author: Brendan Long <b.long@cablelabs.com>
Date:   Thu May 30 16:00:35 2013 -0600

    rtspconnection: remove functions added in GLib 2.34
    
    g_pollable_stream_read and g_pollable_stream_write were added in GLib 2.34,
    but Ubuntu 12.04 and Debian Wheezy still use GLib 2.32.
    
    Fixes: https://bugzilla.gnome.org/show_bug.cgi?id=701316