This is the mail archive of the
cygwin-patches
mailing list for the Cygwin project.
Re: [PATCH] Add libz to dumper.exe link [was Re: Re: speclib vs. -lc trouble.]
- From: Christopher Faylor <cgf-use-the-mailinglist-please at cygwin dot com>
- To: cygwin-patches at cygwin dot com
- Date: Mon, 13 Apr 2009 12:59:23 -0400
- Subject: Re: [PATCH] Add libz to dumper.exe link [was Re: Re: speclib vs. -lc trouble.]
- References: <49E3641E.6040407@gmail.com>
- Reply-to: cygwin-patches at cygwin dot com
On Mon, Apr 13, 2009 at 05:11:10PM +0100, Dave Korn wrote:
> Ok?
>
>winsup/utils/ChangeLog
>
> * Makefile.in (libz): New makefile variable.
> (build_dumper): Test it was correctly set.
> (dumper.exe): Use it.
>
>Index: winsup/utils/Makefile.in
>===================================================================
>RCS file: /cvs/src/src/winsup/utils/Makefile.in,v
>retrieving revision 1.82
>diff -p -u -r1.82 Makefile.in
>--- winsup/utils/Makefile.in 18 Mar 2009 04:19:05 -0000 1.82
>+++ winsup/utils/Makefile.in 13 Apr 2009 15:07:21 -0000
>@@ -81,14 +81,16 @@ ldh.exe: MINGW_LDFLAGS := -nostdlib -lke
> # Check for dumper's requirements and enable it if found.
> LIBICONV := @libiconv@
> libbfd := ${shell $(CC) -B$(bupdir2)/bfd/ --print-file-name=libbfd.a}
>+# Recent libbfd requires libz as it handles compressed files.
>+libz := ${shell $(CC) -B$(bupdir2)/zlib/ --print-file-name=libz.a}
> libintl := ${shell $(CC) -B$(bupdir2)/intl/ --print-file-name=libintl.a}
>-build_dumper := ${shell test -r $(libbfd) -a -r $(libintl) -a -n "$(LIBICONV)" && echo 1}
>+build_dumper := ${shell test -r $(libbfd) -a -r $(libz) -a -r $(libintl) -a -n "$(LIBICONV)" && echo 1}
> ifdef build_dumper
> CYGWIN_BINS += dumper.exe
> dumper.o module_info.o parse_pe.o: CXXFLAGS += -I$(bupdir2)/bfd -I$(updir1)/include
> dumper.o parse_pe.o: dumper.h
> dumper.exe: module_info.o parse_pe.o
>-dumper.exe: ALL_LDFLAGS += ${libbfd} ${libintl} -L$(bupdir1)/libiberty $(LIBICONV) -liberty
>+dumper.exe: ALL_LDFLAGS += ${libbfd} ${libintl} ${libz} -L$(bupdir1)/libiberty $(LIBICONV) -liberty
> else
> all: warn_dumper
> endif
I think you can get by with just adding -lz to the ALL_LDFLAGS line and
removing the other stuff. The tests for libintl and libbfd are supposed
to just detect if the appropriate directories are available. There
isn't likely going to be a libz two levels above cygwin's source
directory so I don't see any reason to specfically check for it.
cgf