RFR:

[Roger Riggs]

$.02, for testoutput/hotspot

so that if other repos do something similar they are all under 
testoutput/...

Roger


On 1/13/16 10:11 AM, Erik Helin wrote:
> On 2016-01-13, Staffan Larsen wrote:
>> Looks good!
> Thanks!
>
> On 2016-01-13, Staffan Larsen wrote:
>> (Although <top-level>/build/<conf>/testoutput/hotspot/ could be even better?)
> Doesn't really matter to me, I'm fine with either hotspot/testoutput or
> testoutput/hotspot. Anyone else have a preference?
>
> Thanks,
> Erik
>
>> Thanks,
>> /Staffan
>>
>>
>>> On 13 jan. 2016, at 15:01, Erik Helin <erik.helin at oracle.com> wrote:
>>>
>>> Hi all,
>>>
>>> this patch changes the output directory for hotspot's jtreg tests when
>>> run via the top-level Makefile targets such as
>>> `make test-hotspot-jtreg`.
>>>
>>> The current directory is
>>> <top-level>/build/<conf>/hotspot/linux-x64/testoutput
>>> (on an x86-64 machine running Linux). There is no need to place the
>>> "testoutput" directory in a directory which name is based on OS and arch,
>>> that is already done by the current configuration. Therefore, we can
>>> instead place the "testoutput" directory in
>>> <top-level>/build/<conf>/hotspot/, which is a more reasonable location
>>> for the output from hotspot's tests.
>>>
>>> Enhancement:
>>> https://bugs.openjdk.java.net/browse/JDK-8146985
>>>
>>> Webrev:
>>> http://cr.openjdk.java.net/~ehelin/8146985/00/webrev
>>>
>>> Testing:
>>> - Running `make test-hotspot-jtreg` locally
>>> - JPRT (the patch does not affect JPRT since JPRT does not use
>>>   ALT_OUTPUTDIR)
>>>
>>> Thanks,
>>> Erik