Re: [PATCH] Update FAQ listing required packages for building Cygwin
Hi Jon, On Jun 28 15:21, Corinna Vinschen wrote: > On Jun 28 13:39, Jon Turney wrote: > > docbook2X is now required for building documentation > > libiconv differences between x86_64 and x86 no longer exist > > Please apply. Sorry, but that was not quite correct. Apparently I only skimmed the log text, not the actual patch. Doh. When building 32 bit Cygwin, you need the 64 bit Mingw compiler for cyglsa64.dll. The reason is that 32 bit cyglsa.dll won't work on 64 bit systems, even if Cygwin itself is running under WOW64. Since we don't know if the user runs Cygwin on 32 bit or under WOW64, we have to create *both* cyglsa DLLs for 32 bit Cygwin, while it's obviously sufficient to build only the 64 bit version for 64 bit Cygwin. Care to fix this part of your patch? Thanks, Corinna -- Corinna Vinschen Please, send mails regarding Cygwin to Cygwin Maintainer cygwin AT cygwin DOT com Red Hat signature.asc Description: PGP signature
Re: [PATCH] Update FAQ listing required packages for building Cygwin
On 29/06/2016 09:04, Corinna Vinschen wrote: Hi Jon, On Jun 28 15:21, Corinna Vinschen wrote: On Jun 28 13:39, Jon Turney wrote: docbook2X is now required for building documentation libiconv differences between x86_64 and x86 no longer exist Please apply. Sorry, but that was not quite correct. Apparently I only skimmed the log text, not the actual patch. Doh. When building 32 bit Cygwin, you need the 64 bit Mingw compiler for cyglsa64.dll. The reason is that 32 bit cyglsa.dll won't work on 64 bit systems, even if Cygwin itself is running under WOW64. Since we don't know if the user runs Cygwin on 32 bit or under WOW64, we have to create *both* cyglsa DLLs for 32 bit Cygwin, while it's obviously sufficient to build only the 64 bit version for 64 bit Cygwin. Yes, I somehow though it said mingw64-i686-gcc-core rather than mingw64-x86_64-gcc-core. How about the attached? From e27cfa4631e9ff7ead7b417e87d160cc20c9d03e Mon Sep 17 00:00:00 2001 From: Jon Turney Date: Wed, 29 Jun 2016 15:08:19 +0100 Subject: [PATCH] Restore mingw64-x86_64-gcc-core in FAQ listing required packages Restore mingw64-x86_64-gcc-core to requirements for 32-bit builds in FAQ listing required packages, and give reason. Signed-off-by: Jon Turney --- winsup/doc/faq-programming.xml | 1 + 1 file changed, 1 insertion(+) diff --git a/winsup/doc/faq-programming.xml b/winsup/doc/faq-programming.xml index 1f595f8..c64ab4a 100644 --- a/winsup/doc/faq-programming.xml +++ b/winsup/doc/faq-programming.xml @@ -683,6 +683,7 @@ installed; you at least need gcc-g++, make perl, cocom, gettext-devel, libiconv-devel and zlib-devel. Building for 32-bit Cygwin also requires +mingw64-x86_64-gcc-core (for building the cyglsa64 DLL for WoW64), mingw64-i686-gcc-g++ and mingw64-i686-zlib. Building for 64-bit Cygwin also requires mingw64-x86_64-gcc-g++ and -- 2.8.3
Re: [PATCH] Update FAQ listing required packages for building Cygwin
On Jun 29 15:18, Jon Turney wrote: > On 29/06/2016 09:04, Corinna Vinschen wrote: > > Hi Jon, > > > > On Jun 28 15:21, Corinna Vinschen wrote: > > > On Jun 28 13:39, Jon Turney wrote: > > > > docbook2X is now required for building documentation > > > > libiconv differences between x86_64 and x86 no longer exist > > > > > > Please apply. > > > > Sorry, but that was not quite correct. Apparently I only skimmed the > > log text, not the actual patch. Doh. > > > > When building 32 bit Cygwin, you need the 64 bit Mingw compiler for > > cyglsa64.dll. The reason is that 32 bit cyglsa.dll won't work on 64 bit > > systems, even if Cygwin itself is running under WOW64. Since we don't > > know if the user runs Cygwin on 32 bit or under WOW64, we have to create > > *both* cyglsa DLLs for 32 bit Cygwin, while it's obviously sufficient to > > build only the 64 bit version for 64 bit Cygwin. > > Yes, I somehow though it said mingw64-i686-gcc-core rather than > mingw64-x86_64-gcc-core. > > How about the attached? Should be fine, given that cyglsa is plain C. Please go ahead. Thanks, Corinna -- Corinna Vinschen Please, send mails regarding Cygwin to Cygwin Maintainer cygwin AT cygwin DOT com Red Hat signature.asc Description: PGP signature
