RFR(XS) : 8187020 : AOT tests should not fail if devkit dependency isn't resolved

[Vladimir Kozlov]

On 9/1/17 11:07 AM, Igor Ignatyev wrote:
> H Vladimir,
> 
> I was planning to add execution of ld -v by a separate patch, but if you want I can merge it into this one.

Yes, please.

Thanks,
Vladimir

> 
> -- Igor
> 
>> On Aug 31, 2017, at 2:12 PM, Vladimir Kozlov <vladimir.kozlov at oracle.com> wrote:
>>
>> On MacOS we also have problem when 'ld' is present on path but Xcode is missing:
>>
>> % /usr/bin/ld -v
>> xcode-select: error: no developer tools were found at '/Applications/Xcode.app', and no install could be requested (perhaps no UI is present), please install manually from 'developer.apple.com'.
>>
>> AOT tests will fail in such case too. May be AotCompiler test code also have to execute ld -v (or ld -V on Solaris) to check that we can use linker similar to JAOTC tool:
>>
>> http://hg.openjdk.java.net/jdk10/hs/hotspot/file/a8ec32aa385e/src/jdk.aot/share/classes/jdk.tools.jaotc/src/jdk/tools/jaotc/Linker.java#l104
>>
>> Vladimir
>>
>> On 8/31/17 11:02 AM, Igor Ignatyev wrote:
>>> http://cr.openjdk.java.net/~iignatyev//8187020/webrev.00/index.html
>>>> 11 lines changed: 5 ins; 0 del; 6 mod;
>>> Hi all,
>>> could you please review this tiny fix or AOT tests? Prior this fix, the tests would fail w/ FileNotFoundException when devkit dependency can't be resolved even if jaotc is able to find a linker, e.g. in one of VS*COMNTOOLS. after this fix, if tests aren't able to download devkit, they will allow jaotc to try to find a linker. This patch also remove search for 'ld' in PATH on windows b/c jaotc doesn't use it on windows.
>>> webrev: http://cr.openjdk.java.net/~iignatyev//8187020/webrev.00/index.html
>>> jbs: https://bugs.openjdk.java.net/browse/JDK-8187020
>>> testing: compiler/aot tests
>>> Thanks,
>>> -- Igor
>