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 653964 - Installing MinGW32 build misses gdk/win32 headers
Installing MinGW32 build misses gdk/win32 headers
Status: RESOLVED FIXED
Product: gtk+
Classification: Platform
Component: Backend: X11
3.1.x
Other Windows
: Normal normal
: ---
Assigned To: gtk-bugs
gtk-bugs
Depends on:
Blocks:
 
 
Reported: 2011-07-04 19:11 UTC by greg.hellings
Modified: 2011-08-16 06:28 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Patch against Makefile.am to install win32 headers (1.17 KB, patch)
2011-07-04 19:11 UTC, greg.hellings
needs-work Details | Review
Patch with modifications requested (1.05 KB, patch)
2011-08-14 21:51 UTC, greg.hellings
committed Details | Review

Description greg.hellings 2011-07-04 19:11:48 UTC
Created attachment 191245 [details] [review]
Patch against Makefile.am to install win32 headers

Cross compiling GTK+ 3 on FC15 for the mingw32 environment misses headers which ought to be installed into gtk-3.0/gdk/win32.  The header gtk-3.0/gdk/gdkwin32.h references these files, and it is impossible to build e.g. webkitgtk without them.  Other GDK backends install files in e.g. gtk-3.0/gdk/x11 but the win32 headers are not included.

I have submitted a patch to Fedora to fix this in their packaging. Attached is a patch to fix this here in the upstream location.  Alternatively I have pushed my fix to http://dl.thehellings.com/gtk3.git
Comment 1 Kalev Lember 2011-07-10 18:10:50 UTC
Review of attachment 191245 [details] [review]:

The patch currently installs ALL the header files from win32/ directory, but we should only be installing the public ones. A good list of public headers to start with would be the ones referenced from gdkwin32.h:

gdkwin32cursor.h
gdkwin32display.h
gdkwin32displaymanager.h
gdkwin32dnd.h
gdkwin32keys.h
gdkwin32screen.h
gdkwin32window.h

Also, while you are at it, could you fix your email address in the git commit log to be something saner than <greg@Greg-PC.Belkin>?
Comment 2 greg.hellings 2011-08-14 21:51:07 UTC
Created attachment 193830 [details] [review]
Patch with modifications requested

I have pushed the requested git changes to http://dl.thehellings.com/gtk3.git in the branch gdk-headers and also included them in this patch. They include the modifications that Kalev requested.
Comment 3 Kalev Lember 2011-08-15 09:47:54 UTC
Review of attachment 193830 [details] [review]:

Looks good to me.
Comment 4 Kalev Lember 2011-08-15 10:01:50 UTC
Review of attachment 193830 [details] [review]:

Pushed to master as bd26ce1d.
Comment 5 Fan, Chun-wei 2011-08-16 06:28:59 UTC
Hi,

Thanks for reporting this issue-the Visual Studio 2008/2010 property sheets has
also been updated to copy the Win32-specific backend headers for GDK, as they were also missed there (pushed to gtk-3-0 as 9afbfdc5f40; pushed to master as 
e2397bf0da2ec06).

God Bless!