RFR: JDK-8180651: Make target to run tests on exploded image

[Jonathan Gibbons]

"exploded" as an adjective applies well enough to "image" but not to the 
imperative "run-test".

At the risk of a long name, can you move exploded to the end, with 
something like
     run-test-exploded-image

-- Jon

On 05/19/2017 08:36 AM, Robbin Ehn wrote:
> Hi,
>
> On 05/19/2017 12:55 PM, Erik Joelsson wrote:
>> Sure we can pause this. Robbin has a local patch to play with for now.
>
> Yes, no rush for me.
>
>>
>> If you want another name, then we should also rename exploded-image. 
>> As I understand it, the name exploded is well established. It refers 
>> to the java class files/resources not being in jars/jimage format, 
>> but exploded as separate files in a directory structure.
>
> I would prefer to keep the name exploded for these reasons.
>
> /Robbin
>
>>
>> /Erik
>>
>>
>> On 2017-05-19 11:45, Magnus Ihse Bursie wrote:
>>> I like the idea, the changes in itself look good, but I really 
>>> *really* do not like the name "exploded". It was not good before as 
>>> in "exploded-image", but this is even worse. :-(
>>>
>>> Can we pause this one for just a while and try really hard to come 
>>> up with a better name? If we fail to do that in a couple of days, 
>>> I'll admit defeat and accept the patch as it is. OK?
>>>
>>> What about "local" image? "simple"? "quick"? "bare"?
>>>
>>> /Magnus
>>>
>>> On 2017-05-19 11:26, Erik Joelsson wrote:
>>>> In most cases, when running regression tests, you want to run them 
>>>> on the jdk image because that's the correct image and some tests 
>>>> will not work on other variants. However, many tests can be run 
>>>> successfully on the exploded image and since building the full jdk 
>>>> image takes quite a bit of extra time, especially when running a 
>>>> slowdebug build, enabling this as an option can significantly 
>>>> improve developer efficiency.
>>>>
>>>> I therefor propose the new target "exploded-run-test". It will work 
>>>> just like run-test but it depends on and runs on the exploded image.
>>>>
>>>> Bug: https://bugs.openjdk.java.net/browse/JDK-8180651
>>>>
>>>> Webrev: http://cr.openjdk.java.net/~erikj/8180651/webrev.01/
>>>>
>>>> /Erik
>>>>
>>>
>>