Re: GOLD: PATCH: Support parallel build

["H.J. Lu" <hjl dot tools at gmail dot com>]

On Fri, Dec 4, 2009 at 9:49 PM, Ian Lance Taylor <iant@google.com> wrote:
> "H.J. Lu" <hongjiu.lu@intel.com> writes:
>
>> If we don't change LDADD, we need to add $(DEPENDENCIES) to
>> xxx_DEPENDENCIES. ?Otherwise, parallel build will fail since
>> $(LDADD) will be used for link. ?OK to install?
>>
>>
>> H.J.
>> ---
>> 2009-12-03 ?H.J. Lu ?<hongjiu.lu@intel.com>
>>
>> ? ? ? * Makefile.am (constructor_test_DEPENDENCIES): Add $(DEPENDENCIES).
>> ? ? ? (two_file_test_DEPENDENCIES): Likewise.
>> ? ? ? (common_test_1_DEPENDENCIES): Likewise.
>> ? ? ? (exception_test_DEPENDENCIES) Likewise.
>> ? ? ? (weak_test_DEPENDENCIES): Likewise.
>> ? ? ? (many_sections_test_DEPENDENCIES): Likewise.
>> ? ? ? (initpri1_DEPENDENCIES): Likewise.
>> ? ? ? (script_test_1_DEPENDENCIES): Likewise.
>> ? ? ? (script_test_2_DEPENDENCIES): Likewise.
>> ? ? ? (justsyms_DEPENDENCIES): Likewise.
>> ? ? ? (binary_test_DEPENDENCIES): Likewise.
>> ? ? ? (large_DEPENDENCIES): Likewise.
>> ? ? ? * Makefile.in: Regenerated.
>
> This is not the right fix. ?These tests should not be linked against
> the libraries listed in LDADD. ?It may be that we should avoid using
> LDADD at all.
>

Here is a patch to define empty xxx_LDADD.  OK to install?


-- 
H.J.

Attachment: gold-foo-2.patch
Description: Binary data