Review Request: 8001460: build-infra: Linker warnings on macosx

[Erik Joelsson]

Yes, you did.

This patch was sent by me over two weeks ago, but the mail server didn't 
agree with me and just swallowed it. I noticed today, after I sent the 
new patch out for review, that the bug it tried to fix, which was 
introduced by this patch, wasn't there yet. Yesterday I finally managed 
to send this one out after a couple of new failed attempts.

So yes, part of this 8001460 will get overwritten by 8003528, but they 
both also address other issues.

/Erik

On 2012-11-27 04:31, David Holmes wrote:
> Did I just see this same issue addressed with a slightly different 
> patch ???
>
> David
>
> On 27/11/2012 1:45 AM, Erik Joelsson wrote:
>> (resend attempt 4)
>>
>> On macosx there were a bunch of warnings from the linker when building
>> jdk that were easily fixed.
>>
>> Examples:
>> ld: warning: ignoring file
>> /HUDSON/workspace/2-build-macosx-x86_64/jdk8-newmake/1571/build/macosx-x86_64/jdk/objs/libfdlibm/e_acos.o 
>>
>>
>> ld: warning: directory not found for option
>> '-L/localhome/erik/build-infra-jdk8/build/macosx-x86_64-normal-server-release/jdk/lib/client' 
>>
>>
>>
>> http://cr.openjdk.java.net/~erikj/8001460/webrev.root.01/
>> http://cr.openjdk.java.net/~erikj/8001460/webrev.jdk.01/
>>
>> /Erik