[Gc] Extend win32 static root handling to Cygwin targets.
Hans.Boehm at hp.com
Mon Mar 22 21:59:33 PST 2010
Dave's latest patch looks fine to me. I didn't personally test it,
but Dave apparently did, so I would go ahead and check it in. It
seems unlikely that it would break other platforms.
On Tue, 23 Mar 2010, Ivan Maidanski wrote:
> Sat, 20 Mar 2010 13:12:19 +0000 ?????? ?? Dave Korn <dave.korn.cygwin at googlemail.com>:
>>> Sat, 20 Mar 2010 00:45:29 +0000 ?????? ?? Dave Korn <dave.korn.cygwin at googlemail.com>:
>>>> On 19/03/2010 23:31, Boehm, Hans wrote:
>>>>> Dave -
>>>>> Many thanks for the patch. I suspect that this has been causing a variety
>>>>> of problems for a while.
>>> I've checked in the test files.
>> And I've got the solution to the problem that libtool thinks it doesn't make
>> sense to build a library shared if you aren't going to install it (thanks Ralf
>> W!), you just have to spoof it out by adding a dummy -rpath argument. The
>> attached patch enables the tests without causing anything to be installed.
> "Boehm, Hans" <hans.boehm at hp.com> wrote (earlier):
>> It looks like the current patch installs the test library? That's not good. We should probably hold off on the tests.am changes until we figure out how to solve this problem.
> Hans -
> should I check in the proposed patch?
>> * Makefile.am (check_LTLIBRARIES): Initialise to empty.
>> * tests/tests.am (TESTS): Add staticrootstest.
>> (check_PROGRAMS): Likewise.
>> (staticrootstest_SOURCES): Define.
>> (staticrootstest_LDADD): Likewise.
>> (check_LTLIBRARIES): Add libstaticrootslib.la.
>> (libstaticrootslib_la_SOURCES): Define.
>> (libstaticrootslib_la_LIBADD): Likewise.
>> (libstaticrootslib_la_LDFLAGS): Likewise.
>> (libstaticrootslib_la_DEPENDENCIES): Likewise.
>> * .cvsignore: Ignore libstaticrootslib.la.
>> As always, needs a full "autoreconf -vif" after applying. With my various
>> installed versions of everything, I get a lot of re-libtoolize churn, so I've
>> left off the various "Regenerate" lines from the changelog because whoever
>> applies the patch will likely get a different set of files.
>> ATTACHMENT: text/x-c (add-tests.diff)
> Gc mailing list
> Gc at linux.hpl.hp.com
More information about the Gc