On 8/7/2012 10:29 AM, Daniel Colascione wrote: > On 8/7/12 10:22 AM, Eli Zaretskii wrote: >>> Date: Tue, 07 Aug 2012 01:19:27 -0700 >>> From: Daniel Colascione >>> >>> This set of patches allows a Cygwin Emacs to use the w32 window-system >>> instead of X11. (The ability to use X11 with Cygwin is retained.) >>> This patchset also fixes a few bugs in the Windows code and makes the >>> window-system configuration mechanism more generic, eliminating >>> inclusion of window-system specific headers in most of the code. >> >> Thanks. Just for my information: did you try building the modified >> sources with MinGW? > > Not yet, but definitely before I check in. Doing so looks like a pain > because I have a Cygwin build environment, not a MinGW one, and Emacs > won't build as NT-native under cross-compilation from Cygwin. (But > it'd be nice to make that work.) > Is building with cl.exe (Visual Studio) supposed to work? I remember some discussion on the mailing list, but I don't know whether compilation was made to actually work. When I try building with CL, I get a bunch of errors, many apparently unrelated to my changes. For example, emacs.c c:\users\dancol\software\emacs\cyg.w32test\src\lisp.h(336) : error C2065: 'PTRDI FF_MIN' : undeclared identifier c:\users\dancol\software\emacs\cyg.w32test\src\lisp.h(336) : error C2099: initia lizer is not a constant c:\users\dancol\software\emacs\cyg.w32test\src\lisp.h(2375) : error C2065: 'PTRD IFF_MIN' : undeclared identifier