RFR: JDK-8189682: JDK demos/tests should be in a separate bundle than hotspot ones

classic Classic list List threaded Threaded
3 messages Options
Reply | Threaded
Open this post in threaded view
|

RFR: JDK-8189682: JDK demos/tests should be in a separate bundle than hotspot ones

Erik Joelsson
This patch adds building of a new test bundle containing just the
remaining demos, which are used for client testing. It's basically
resurrecting the build logic for the old demo bundles, but adding tests
to the bundle name to make the intent clear. It's also a tar.gz on all
platforms.

Bug: https://bugs.openjdk.java.net/browse/JDK-8189682

Webrev: http://cr.openjdk.java.net/~erikj/8189682/webrev.01/

/Erik

Reply | Threaded
Open this post in threaded view
|

Re: RFR: JDK-8189682: JDK demos/tests should be in a separate bundle than hotspot ones

Phil Race
I've looked at the code and tested this too .. looks good.

Thanks,

-phil.

On 10/23/2017 07:39 AM, Erik Joelsson wrote:

> This patch adds building of a new test bundle containing just the
> remaining demos, which are used for client testing. It's basically
> resurrecting the build logic for the old demo bundles, but adding
> tests to the bundle name to make the intent clear. It's also a tar.gz
> on all platforms.
>
> Bug: https://bugs.openjdk.java.net/browse/JDK-8189682
>
> Webrev: http://cr.openjdk.java.net/~erikj/8189682/webrev.01/
>
> /Erik
>

Reply | Threaded
Open this post in threaded view
|

Re: RFR: JDK-8189682: JDK demos/tests should be in a separate bundle than hotspot ones

Magnus Ihse Bursie
In reply to this post by Erik Joelsson
On 2017-10-23 16:39, Erik Joelsson wrote:
> This patch adds building of a new test bundle containing just the
> remaining demos, which are used for client testing. It's basically
> resurrecting the build logic for the old demo bundles, but adding
> tests to the bundle name to make the intent clear. It's also a tar.gz
> on all platforms.
>
> Bug: https://bugs.openjdk.java.net/browse/JDK-8189682
>
> Webrev: http://cr.openjdk.java.net/~erikj/8189682/webrev.01/
Looks good to me.

/Magnus
>
> /Erik
>