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 733052 - Include windsock header for Windows targets
Include windsock header for Windows targets
Status: RESOLVED FIXED
Product: GStreamer
Classification: Platform
Component: gst-plugins-bad
1.x
Other Windows
: Normal normal
: 1.4.0
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2014-07-11 11:10 UTC by Alexey Pavlov
Modified: 2014-07-11 19:23 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
issue fixing (1.50 KB, patch)
2014-07-11 11:11 UTC, Alexey Pavlov
committed Details | Review

Description Alexey Pavlov 2014-07-11 11:10:20 UTC
Windows need include <winsock.h> instead netinet/*.h and arpa/*.h headers.
Comment 1 Alexey Pavlov 2014-07-11 11:11:42 UTC
Created attachment 280495 [details] [review]
issue fixing
Comment 2 Sebastian Dröge (slomo) 2014-07-11 11:26:50 UTC
commit 198d8dd5f12036b84c09d69f6d9209ec4c7e9e5d
Author: Sebastian Dröge <sebastian@centricular.com>
Date:   Fri Jul 11 13:25:37 2014 +0200

    curl: Use winsock2 instead and the GLib macro for checking if we're on Windows

commit 0ea13d7a65e5fcbe1ce452f0b3843895652061e1
Author: Sebastian Dröge <sebastian@centricular.com>
Date:   Fri Jul 11 13:25:18 2014 +0200

    motioncells: Use no network API if not required

commit 365d0cfb320369144ba6a632c1a27920e850fb7f
Author: Alexey Pavlov <alexpux@gmail.com>
Date:   Fri Jul 11 13:21:21 2014 +0200

    ext: Include winsock.h on Windows when required
    
    https://bugzilla.gnome.org/show_bug.cgi?id=733052
Comment 3 Alexey Pavlov 2014-07-11 13:53:51 UTC
Need add
#include <gst/gst.h>

before using G_OS_WIN32 macros also.