This is the mail archive of the cygwin-apps mailing list for the Cygwin project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [ITA] w32api-3.0b_svn5368-1


On 14 August 2012 08:34, Corinna Vinschen wrote:
> On Aug 14 09:29, Corinna Vinschen wrote:
>> On Aug 14 08:25, Andy Koppe wrote:
>> > On 14 August 2012 08:18, Corinna Vinschen wrote:
>> > > On Aug 13 21:51, Yaakov (Cygwin/X) wrote:
>> > >> On 2012-08-12 01:49, JonY wrote:
>> > >> >New w32api preliminary upload, now with mingw-w64 parts.
>> > >>
>> > >> Nack.  Both mintty and xorg-server FTBFS with this w32api.
>> > >
>> > > Er... what?
>> >
>> > I had to look it up: Fails To Build From Source.
>> >
>> > /usr/lib/gcc/i686-pc-cygwin/4.5.3/../../../../include/w32api/windef.h:23:27:
>> > error: expected â=â, â,â, â;â, âasmâ or â__attribute__â before âULONGâ
>> >
>> > windef.h:23: typedef unsigned __LONG32 ULONG;
>> >
>> > It doesn't like the __LONG32.
>>
>> Ok, that's fixable.  Do you include windef before windows.h?  If so,
>> does it work if you switch the includes?

I haven't included windows.h, but just the headers that are actually
needed. (Yeah, I know MSDN doesn't like that, but then why do they
bother documenting which header each function is in.)

> Alternatively, can you apply this patch to windef.h and try again?
>
> Index: windef.h
> ===================================================================
> --- windef.h    (revision 5370)
> +++ windef.h    (working copy)
> @@ -14,6 +14,8 @@
>  extern "C" {
>  #endif
>
> +#include <_mingw.h>
> +
>  #ifndef WINVER
>  #define WINVER 0x0502
>  #endif

Yep, mintty builds fine with that, and appears to work. For some
reason it's 9K bigger than with the current w32api though.

Andy


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]