RFR:

[Magnus Ihse Bursie]

I agree. It seems better to collect all test output in a central place. 

/Magnus

> 13 jan. 2016 kl. 16:15 skrev Roger Riggs <roger.riggs at oracle.com>:
> 
> $.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
>