RFR(M): 8175300 - Enable artifact resolution for jtreg tests

[Magnus Ihse Bursie]

Looks good to me now.

/Magnus

On 2017-03-09 17:42, Christian Tornqvist wrote:
> Updated webrev based on feedback from Magnus Ihse Bursie:
> http://cr.openjdk.java.net/~ctornqvi/webrev/8175300/webrev.01/
>
> -----Original Message-----
> From: hotspot-dev [mailto:hotspot-dev-bounces at openjdk.java.net] On Behalf Of
> Christian Tornqvist
> Sent: Tuesday, March 7, 2017 3:03 PM
> To: hotspot-dev at openjdk.java.net; 'build-dev' <build-dev at openjdk.java.net>
> Subject: RFR(M): 8175300 - Enable artifact resolution for jtreg tests
>
> Hi everyone,
>
>   
>
> Please review this change that adds the ability to have external
> dependencies in jtreg tests. As a proof of concept, I've added Scimark to
> the Hotspot repository. The concept is this, annotate the test with one or
> more @Artifact annotation describing the external dependencies, these can
> then either be resolved manually (using -Djdk.test.lib.artifacts.<name>) or
> automatically using a custom ArtifactManager.
>
>   
>
> The make changes were contributed by Erik Joelsson.
>
> These changes have been verified by running the Scimark test locally (using
> make) and in JPRT.
>
>   
>
> Bugs:
>
> https://bugs.openjdk.java.net/browse/JDK-8175300
>
> Webrev:
>
> http://cr.openjdk.java.net/~ctornqvi/webrev/8175300/webrev.00/
>
>   
>
> Thanks,
>
> Christian
>
>   
>
>   
>
>   
>
>