This is the mail archive of the cygwin-patches@cygwin.com mailing list for the Cygwin project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: A minor patch to Makefile.in


Christopher Faylor wrote:

On Wed, Jun 26, 2002 at 01:21:22PM -0400, Nicholas Wourms wrote:

Ok,

A correction:
"Due to the dependancy of gettext(libintl) and gettext, this pach..."

This should read:
"Due to the dependancy of gettext(libintl) on libiconv, this patch..."

Also, Netscape is stripping tabs, so I have attached the Changelog this time with the hope it won't be stripped.

Sorry, but this patch isn't acceptable.  If we are going to accommodate
libiconv then there needs to be a configure test for it.

cgf

Chris,

A better solution is to include "naked-intl" in the list of cvs modules for the "winsup" ampersand module, then when utils/Makefile.in evaluates "libintl:=${shell $(CC) -B$(bupdir2)/intl/ --print-file-name=libintl.a}", it will statically link dumper.exe to the libintl which *doesn't* depend on libiconv. Otherwise, it will evaluate to be /usr/lib/libintl.a, which is dependant on libiconv if you are using the gettext > 10.40. I think this would be a better solution because dumper.exe doesn't use any of the features provided by libiconv, so why use what isn't needed? Furthermore, FWICT, src/intl doesn't install anything durning "make install", so nothing should get clobbered. I'm going to checkout "naked-intl" to test this theory and re-run the build. I'll report my findings shortly...

Cheers,
Nicholas



Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]