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 >
