public inbox for gcc@gcc.gnu.org
 help / color / mirror / Atom feed
* Cygwin + zlib
@ 2017-02-24 17:55 NightStrike
  2017-02-25  0:10 ` Matthias Klose
  2017-03-11  6:30 ` Daniel Santos
  0 siblings, 2 replies; 3+ messages in thread
From: NightStrike @ 2017-02-24 17:55 UTC (permalink / raw)
  To: gcc

Currently, to build natively on cygwin, this patch is required to zlib:

https://github.com/Alexpux/MSYS2-packages/blob/master/zlib/1.2.11-cygwin-no-widechar.patch

Otherwise, this error occurs:

./../zlib/libz.a(libz_a-gzlib.o):gzlib.c:(.text+0x646): undefined
reference to `_wopen'
./../zlib/libz.a(libz_a-gzlib.o):gzlib.c:(.text+0x646): relocation
truncated to fit: R_X86_64_PC32 against undefined symbol `_wopen'


Given that cygwin is a supported system, and given that zlib is
in-tree (and was recently updated), does it make sense to include this
patch in the gcc in-tree version of zlib?

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: Cygwin + zlib
  2017-02-24 17:55 Cygwin + zlib NightStrike
@ 2017-02-25  0:10 ` Matthias Klose
  2017-03-11  6:30 ` Daniel Santos
  1 sibling, 0 replies; 3+ messages in thread
From: Matthias Klose @ 2017-02-25  0:10 UTC (permalink / raw)
  To: NightStrike, gcc

On 24.02.2017 18:55, NightStrike wrote:
> Currently, to build natively on cygwin, this patch is required to zlib:
> 
> https://github.com/Alexpux/MSYS2-packages/blob/master/zlib/1.2.11-cygwin-no-widechar.patch
> 
> Otherwise, this error occurs:
> 
> ./../zlib/libz.a(libz_a-gzlib.o):gzlib.c:(.text+0x646): undefined
> reference to `_wopen'
> ./../zlib/libz.a(libz_a-gzlib.o):gzlib.c:(.text+0x646): relocation
> truncated to fit: R_X86_64_PC32 against undefined symbol `_wopen'
> 
> 
> Given that cygwin is a supported system, and given that zlib is
> in-tree (and was recently updated), does it make sense to include this
> patch in the gcc in-tree version of zlib?

please could you forward this issue upstream as well?

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: Cygwin + zlib
  2017-02-24 17:55 Cygwin + zlib NightStrike
  2017-02-25  0:10 ` Matthias Klose
@ 2017-03-11  6:30 ` Daniel Santos
  1 sibling, 0 replies; 3+ messages in thread
From: Daniel Santos @ 2017-03-11  6:30 UTC (permalink / raw)
  To: NightStrike, gcc

On 02/24/2017 11:55 AM, NightStrike wrote:
> Currently, to build natively on cygwin, this patch is required to zlib:
>
> https://github.com/Alexpux/MSYS2-packages/blob/master/zlib/1.2.11-cygwin-no-widechar.patch

Thank you for this.  Unfortunately, I didn't see this message prior to 
filing my bug report, but here it is anyway: 
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=79771

Daniel

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2017-03-11  6:30 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-02-24 17:55 Cygwin + zlib NightStrike
2017-02-25  0:10 ` Matthias Klose
2017-03-11  6:30 ` Daniel Santos

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).