Re: [PATCH] Win32: make buildable with MinGW-w64 gcc

2015-10-19 Thread Kouhei Sutou
Hi, In <20151019205435.gv48...@mdounin.ru> "Re: [PATCH] Win32: make buildable with MinGW-w64 gcc" on Mon, 19 Oct 2015 23:54:35 +0300, Maxim Dounin wrote: >> The attached patch fixes the following warning: >> >> x86_64-w64-mingw32-gcc -c -pipe -O -W -Wall -Wpointer-arith >> -Wno-unuse

[PATCH] Win32: make buildable with MinGW-w64 gcc

2015-10-17 Thread Kouhei Sutou
^ cc1: all warnings being treated as errors Thanks, -- kou # HG changeset patch # User Kouhei Sutou # Date 1445085662 -32400 # Sat Oct 17 21:41:02 2015 +0900 # Node ID a1e6710e934117a4a7dc7b1720f441207550d665 # Parent 2f34ea503ac4e015cc08f6efbb279b360eda609c Win32: make buildable

[PATCH] Add Win32 related files into tar.gz

2014-10-28 Thread Kouhei Sutou
Hi, The current nginx can be built with MinGW-w64. Could you add Win32 related files into tar.gz? If tar.gz includes Win32 related file, I can build nginx for Windows from tar.gz. Thanks, -- kou # HG changeset patch # User Kouhei Sutou # Date 1414505796 -32400 # Tue Oct 28 23:16:36 2014

Re: [PATCH] Support cross compiling with MinGW-w64 on Debian GNU/Linux

2014-10-28 Thread Kouhei Sutou
Hi, In <20141028133802.gl45...@mdounin.ru> "Re: [PATCH] Support cross compiling with MinGW-w64 on Debian GNU/Linux" on Tue, 28 Oct 2014 16:38:02 +0300, Maxim Dounin wrote: >> > - the NGX_GNUC_64 macro was removed, __MINGW64_VERSION_MAJOR >> > tested directly instead; >> > >> > - the #in

Re: [PATCH] Support cross compiling with MinGW-w64 on Debian GNU/Linux

2014-10-28 Thread Kouhei Sutou
Hi, In <20141027174711.gd45...@mdounin.ru> "Re: [PATCH] Support cross compiling with MinGW-w64 on Debian GNU/Linux" on Mon, 27 Oct 2014 20:47:11 +0300, Maxim Dounin wrote: > - the NGX_GNUC_64 macro was removed, __MINGW64_VERSION_MAJOR > tested directly instead; > > - the #include <_ming

Re: [PATCH] Support cross compiling with MinGW-w64 on Debian GNU/Linux

2014-10-27 Thread Kouhei Sutou
4 00:42:50 +0900 (JST), Kouhei Sutou wrote: > Hi, > > In <20141017144512.gf35...@mdounin.ru> > "Re: [PATCH] Support cross compiling with MinGW-w64 on Debian GNU/Linux" on > Fri, 17 Oct 2014 18:45:12 +0400, > Maxim Dounin wrote: > >>> I add on

Re: [PATCH] Support cross compiling with MinGW-w64 on Debian GNU/Linux

2014-10-17 Thread Kouhei Sutou
%20Check%20For%20Mingw-w64/ Thanks for the information. I don't know about it. (I just checked only pre-defined macros by "x86_64-w64-mingw32-gcc -dM -xc -E /dev/null"...) I confirmed the the approach can work with both 32bit version and 64bit version MinGW-w64 GCCs. Could you

Re: [PATCH] Support cross compiling with MinGW-w64 on Debian GNU/Linux

2014-10-15 Thread Kouhei Sutou
rted Could you review it? Thanks, -- kou # HG changeset patch # User Kouhei Sutou # Date 1413375301 -32400 # Wed Oct 15 21:15:01 2014 +0900 # Node ID 92d92c63ae66b2cb082794838d315306de541bf8 # Parent d91b98232e4c1b6a9a37c957d713269089fcce60 Win32: made build-able with 64bit version

Re: [PATCH] Support cross compiling with MinGW-w64 on Debian GNU/Linux

2014-10-14 Thread Kouhei Sutou
t; Style nit: this needs more empty lines before/after preprocessor > directives. It means the following? #ifdef XXX #define A #define B #else #define C #define D #endif Or the following? #ifdef XXX #define A #define B #else #define C #define D #endif Other code

Re: [PATCH] Support cross compiling with MinGW-w64 on Debian GNU/Linux

2014-10-14 Thread Kouhei Sutou
return 1; } if (ngx_get_options(argc, argv) != NGX_OK) { return 1; I run objs/nginx on Wine and got the following output: 8:8:8 So I added the following change to the attached patch: +#if defined(__GNUC__) && defined(_WIN64) +#define NGX_PTR_SIZE8 +#de

Re: [PATCH] Support cross compiling with MinGW-w64 on Debian GNU/Linux

2014-10-11 Thread Kouhei Sutou
Hi, In <9179053.BLszIlEqpy@vbart-laptop> "Re: [PATCH] Support cross compiling with MinGW-w64 on Debian GNU/Linux" on Sat, 11 Oct 2014 13:17:36 +0400, "Valentin V. Bartenev" wrote: > On Saturday 11 October 2014 10:41:23 Kouhei Sutou wrote: >> Hi, &

Re: [PATCH] Support cross compiling with MinGW-w64 on Debian GNU/Linux

2014-10-11 Thread Kouhei Sutou
, this breaks at least compilation with MSVC. You're absolutely right. This type name change is needless. (off64_t was introduced in my first try but it was removed in my second try. So I should remove the change before sending my patch.) The attached patch just adds "#ifdef __GNUC__&q

[PATCH] Win32: suppressed sign-compare warnings

2014-10-11 Thread Kouhei Sutou
nginx.org/pipermail/nginx-devel/2014-October/006028.html It was suggested at http://mailman.nginx.org/pipermail/nginx-devel/2014-October/006054.html Thanks, -- kou # HG changeset patch # User Kouhei Sutou # Date 1413029369 -32400 # Sat Oct 11 21:09:29 2014 +0900 # Node ID 7e6e4627e9ca0

[PATCH] Fix wrong type cast in ngx_iocp_module.c

2014-10-11 Thread Kouhei Sutou
t http://mailman.nginx.org/pipermail/nginx-devel/2014-October/006054.html Thanks, -- kou # HG changeset patch # User Kouhei Sutou # Date 1413028595 -32400 # Sat Oct 11 20:56:35 2014 +0900 # Node ID baaa9448adcc11db6f9e1e99c8a8570674e1baea # Parent 21043ce2a0058154687a779b4c9b0f241fa7e88a

Re: [PATCH] Support cross compiling with MinGW-w64 on Debian GNU/Linux

2014-10-10 Thread Kouhei Sutou
Hi, Could anyone review the patch? In <20141004.195156.801414609668336877@cozmixng.org> "[PATCH] Support cross compiling with MinGW-w64 on Debian GNU/Linux" on Sat, 04 Oct 2014 19:51:56 +0900 (JST), Kouhei Sutou wrote: > Hi, > > I'm trying cross compili

[PATCH] Support cross compiling with MinGW-w64 on Debian GNU/Linux

2014-10-04 Thread Kouhei Sutou
Hi, I'm trying cross compiling nginx for Windows on Debian GNU/Linux. The current nginx doesn't support it. I created a patch to do it. With the attached patch, I can cross compile nginx for Windows on Debian GNU/Linux. Please review the attached patch. The current status (without patch)