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 690163 - Add a pre-configured gio/gnetworking.h for Visual C++ builds
Add a pre-configured gio/gnetworking.h for Visual C++ builds
Status: RESOLVED FIXED
Product: glib
Classification: Platform
Component: gio
unspecified
Other Windows
: Normal normal
: ---
Assigned To: gtkdev
gtkdev
Depends on:
Blocks:
 
 
Reported: 2012-12-13 15:50 UTC by Fan, Chun-wei
Modified: 2012-12-14 16:31 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Add a preconfigured gnetworking.h(.win32) for Visual C++ builds (4.83 KB, patch)
2012-12-13 15:50 UTC, Fan, Chun-wei
reviewed Details | Review

Description Fan, Chun-wei 2012-12-13 15:50:50 UTC
Created attachment 231484 [details] [review]
Add a preconfigured gnetworking.h(.win32) for Visual C++ builds

Hi,

As gio/gnetworking.h(.in) was added as a result of bug 623187, we would need a pre-configured gio/gnetworking.h(.win32) for Visual C++ builds, as those builds do not make use of autotools during the build process.  This patch adds a gio/gnetworking.h.win32 to address that need, and updates the relevant build-related files.

With blessings, thank you!
Comment 1 Dan Winship 2012-12-14 12:49:38 UTC
Comment on attachment 231484 [details] [review]
Add a preconfigured gnetworking.h(.win32) for Visual C++ builds

>+#include <wspiapi.h>

The reason gnetworking.h is generated rather than static is because we need to check at compile time whether or not wspiapi.h is available. (And likewise another #include on the UNIX side.)

Or maybe that only applies to mingw, and not Visual C? If Visual C will always have wspiapi.h, then go ahead and commit.
Comment 2 Fan, Chun-wei 2012-12-14 16:31:19 UTC
Hi Dan,

I understood your point, hence the .win32 extension for the pre-configured gnetworking.h.win32.

Yes, AFAICT wspiapi.h is shipped with the Windows SDK, which is needed (included) with a Visual C++/Studio installation.

Pushed as 4b859e4e and 0b2919a9 (two patches) as git am did not like a patch that deals with files having different line endings somehow. :|

With blessings, thank you!