[Gc] Extend win32 static root handling to Cygwin targets.

Hans Boehm 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.

Thanks.

Hans



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?
>
>>
>> ChangeLog:
>>
>> * 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.
>>
>>     cheers,
>>       DaveK
>>
>>
>> ATTACHMENT: text/x-c (add-tests.diff)
>>
> _______________________________________________
> Gc mailing list
> Gc at linux.hpl.hp.com
> http://www.hpl.hp.com/hosted/linux/mail-archives/gc/
>


More information about the Gc mailing list