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.ri...@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.he...@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 >
