[PATCH] Add libz to dumper.exe link [was Re: Re: speclib vs. -lc trouble.]

Dave Korn dave.korn.cygwin@googlemail.com
Mon Apr 13 19:20:00 GMT 2009


Christopher Faylor wrote:
> On Mon, Apr 13, 2009 at 06:34:04PM +0100, Dave Korn wrote:
>> Christopher Faylor wrote:
>>> 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.
>> I thought that might happen in a combined tree build with /src and /gcc
>> together?
> 
> That wasn't the intent of the current checks in the makefile.  They were
> just to detect newer versions of libbfd.a or libintl.a.  Even libintl.a
> isn't really necessary IMO.  It's not likely that is going to be under
> active development and different from what should be installed in /lib.

  Ah, OK, so just like this then:

winsup/utils/ChangeLog

	* Makefile.in (dumper.exe):  Add -lz to ALL_LDFLAGS.

  Tested by making clean then all in an existing
$objdir/i686-pc-cygwin/winsup/utils build directory.  Ok?

    cheers,
      DaveK
-------------- next part --------------
A non-text attachment was scrubbed...
Name: dumper-lz-take2.patch
Type: text/x-patch
Size: 741 bytes
Desc: not available
URL: <http://cygwin.com/pipermail/cygwin-patches/attachments/20090413/36dc2e2b/attachment.bin>


More information about the Cygwin-patches mailing list