RFR: JDK-8210958 Rename "make run-test" to "make test"

[Magnus Ihse Bursie]

On 2018-10-23 10:45, Magnus Ihse Bursie wrote:

> On 2018-10-19 18:11, Erik Joelsson wrote:
>
>> Looks good, and thanks for fixing test-make. I found the same issue 
>> today.
>>
>> Could you also change the make target we call for 
>> run-test/run-test-prebuilt profiles in jib-profiles.js?
>
> There's no target for run-test in jib-profiles.js; only for 
> run-test-prebuilt (as far as I can understand, at least). I've updated 
> this, and I also renamed all references to "run-test" to just "test". 
> Maybe this was going too far. Let me know if you want me to revert 
> that part.
>
> Updated webrev (only jib-profiles.js has changed):
>
> http://cr.openjdk.java.net/~ihse/JDK-8210958-rename-run-test-to-test/webrev.03 
>
By way of private discussion, I learned that this had a bit too much 
repercussions. I've reverted the change to jib-profiles.js, and only 
changed the name of the target.

New webrev:
http://cr.openjdk.java.net/~ihse/JDK-8210958-rename-run-test-to-test/webrev.04

(only change is in jib-profiles.js)

/Magnus

>
> /Magnus
>>
>> /Erik
>>
>>
>> On 2018-10-19 05:32, Magnus Ihse Bursie wrote:
>>> Now that JDK-8212028 is pushed, it's time to revisit this.
>>>
>>> In the meantime, the compile-command test had been added, which 
>>> caused me to realize some better handling of the make tests were 
>>> needed. These fixes are also included. Now "make test-make" will run 
>>> all build system tests with proper dependencies.
>>>
>>> I also fixed a test in TestMakeBase.gmk which became broken with 
>>> JDK-8212028.
>>>
>>> Updated webrev:
>>> http://cr.openjdk.java.net/~ihse/JDK-8210958-rename-run-test-to-test/webrev.02 
>>>
>>>
>>> /Magnus
>>>
>>>
>>> On 2018-09-20 12:47, Magnus Ihse Bursie wrote:
>>>> The time has come to start phasing out the old test running 
>>>> framework ("cd test && make"). This patch will change the behavior 
>>>> of "make test" to use the new run-test framework, instead of the old.
>>>>
>>>> The old framework is still available as of now by using "cd test && 
>>>> make".
>>>>
>>>> The "run-test" target (and variants like exploded-run-test or 
>>>> run-test-tier1) are kept as aliases.
>>>>
>>>> Using "cd test && make" will result in a warning being printed that 
>>>> it is recommended to stop using this way of running tests.
>>>>
>>>> Bug: https://bugs.openjdk.java.net/browse/JDK-8210958
>>>> WebRev: 
>>>> http://cr.openjdk.java.net/~ihse/JDK-8210958-rename-run-test-to-test/webrev.01
>>>>
>>>> /Magnus
>>>>
>>>
>>
>