Re: Fwd: Re: [patch]: Decouple cygwin building from in-tree mingw/w32api building

2012-10-24 Thread Corinna Vinschen
On Oct 23 17:48, Christopher Faylor wrote: > On Tue, Oct 23, 2012 at 09:50:10PM +0100, Keith Marshall wrote: > >And just like Earnie's, the response he requested from me also bounced. > >Forwarded copy below: > > > Original Message > >Subject: Re: [patch]: Decouple cygwin building

[patch cygwin]: Replace inline-assembler in string.h by C implementation

2012-10-24 Thread Kai Tietz
Hello, this patch replaces the inline-assember used in string.h by C implementation. There are three reasons why I want to suggest this. First, the C-code might be optimized further by fixed (constant) arguments. Secondly, it is architecture independent and so we just need to maintain on code-pa

Re: [patch]: Decouple cygwin building from in-tree mingw/w32api building

2012-10-24 Thread Yaakov (Cygwin/X)
On Mon, 2012-10-22 at 14:23 +0200, Corinna Vinschen wrote: > If the original patch with the aforementioned changes is ok with > everybody, I'd apply it asap and remove lsaauth/cyglsa64.dll, > lsaauth/make-64bit-version-with-mingw-w64.sh, and utils/mingw. Revised patches for winsup/cygwin and winsu

Re: [patch]: Decouple cygwin building from in-tree mingw/w32api building

2012-10-24 Thread Corinna Vinschen
On Oct 24 04:30, Yaakov (Cygwin/X) wrote: > On Mon, 2012-10-22 at 14:23 +0200, Corinna Vinschen wrote: > > If the original patch with the aforementioned changes is ok with > > everybody, I'd apply it asap and remove lsaauth/cyglsa64.dll, > > lsaauth/make-64bit-version-with-mingw-w64.sh, and utils/m

Re: [patch]: Decouple cygwin building from in-tree mingw/w32api building

2012-10-24 Thread Corinna Vinschen
On Oct 24 11:50, Corinna Vinschen wrote: > On Oct 24 04:30, Yaakov (Cygwin/X) wrote: > > On Mon, 2012-10-22 at 14:23 +0200, Corinna Vinschen wrote: > > > If the original patch with the aforementioned changes is ok with > > > everybody, I'd apply it asap and remove lsaauth/cyglsa64.dll, > > > lsaaut

Re: [patch]: Decouple cygwin building from in-tree mingw/w32api building

2012-10-24 Thread Yaakov (Cygwin/X)
On Wed, 2012-10-24 at 12:01 +0200, Corinna Vinschen wrote: > > Checking in toplevel patches requires global checkin rights. I can > > apply the toplevel patch when you applied the rest. Other than that, > > toplevel patches also have to be kept aligned with the gcc repo. I'll > > make sure to in

[Fwd: Re: Fwd: Re: [patch]: Decouple cygwin building from in-tree mingw/w32api building]

2012-10-24 Thread Corinna Vinschen
FYI. - Forwarded message from Earnie Boyd <...> - > Date: Wed, 24 Oct 2012 07:15:52 -0400 > From: Earnie Boyd <...> > To: Keith Marshall <...>, Christopher Faylor > <...>, cygwin-patches@cygwin.com, Corinna Vinschen > <...> > Subject: Re: Fwd: Re: [patch]: Decouple cygwin buil

Re: Fwd: Re: [patch]: Decouple cygwin building from in-tree mingw/w32api building

2012-10-24 Thread Corinna Vinschen
On Oct 24 07:15, Earnie Boyd wrote: > On Wed, Oct 24, 2012 at 3:49 AM, Corinna Vinschen wrote: > > > > Just to be sure: Does that mean we can simply remove the mingw and > > w32api dirs in the sourceware repo any time? > > Yes. Ok, thank you! Corinna -- Corinna Vinschen Plea

Re: [patch]: Decouple cygwin building from in-tree mingw/w32api building

2012-10-24 Thread Corinna Vinschen
On Oct 24 05:26, Yaakov (Cygwin/X) wrote: > On Wed, 2012-10-24 at 12:01 +0200, Corinna Vinschen wrote: > > > Checking in toplevel patches requires global checkin rights. I can > > > apply the toplevel patch when you applied the rest. Other than that, > > > toplevel patches also have to be kept al

[toplevel patch] Simplify FLAGS_FOR_TARGET for Cygwin

2012-10-24 Thread Corinna Vinschen
Hi guys, I just applied the below patch to the sourceware src repo. The reason for the patch is that Cygwin won't be using the in-tree mingw and w32api any longer, but instead it requires an external installation of a Mingw64 based w32api, and a Mingw64 build environment to build the native Windo

Re: [patch cygwin]: Replace inline-assembler in string.h by C implementation

2012-10-24 Thread Ryan Johnson
On 24/10/2012 5:16 AM, Kai Tietz wrote: Hello, this patch replaces the inline-assember used in string.h by C implementation. There are three reasons why I want to suggest this. First, the C-code might be optimized further by fixed (constant) arguments. Secondly, it is architecture independent

Re: [patch cygwin]: Replace inline-assembler in string.h by C implementation

2012-10-24 Thread Christopher Faylor
On Wed, Oct 24, 2012 at 11:07:47AM -0400, Ryan Johnson wrote: >On 24/10/2012 5:16 AM, Kai Tietz wrote: >> Hello, >> >> this patch replaces the inline-assember used in string.h by C implementation. >> There are three reasons why I want to suggest this. First, the C-code might >> be optimized furthe

Re: [patch cygwin]: Replace inline-assembler in string.h by C implementation

2012-10-24 Thread Corinna Vinschen
On Oct 24 11:42, Christopher Faylor wrote: > On Wed, Oct 24, 2012 at 11:07:47AM -0400, Ryan Johnson wrote: > >On 24/10/2012 5:16 AM, Kai Tietz wrote: > >> Hello, > >> > >> this patch replaces the inline-assember used in string.h by C > >> implementation. > >> There are three reasons why I want to

Re: [patch cygwin]: Replace inline-assembler in string.h by C implementation

2012-10-24 Thread Kai Tietz
2012/10/24 Christopher Faylor wrote: > On Wed, Oct 24, 2012 at 11:07:47AM -0400, Ryan Johnson wrote: >>On 24/10/2012 5:16 AM, Kai Tietz wrote: >>> Hello, >>> >>> this patch replaces the inline-assember used in string.h by C >>> implementation. >>> There are three reasons why I want to suggest this