Chris, Will push it in few hours.
-Dmitry On 2013-02-13 14:53, Chris Hegarty wrote: > Dmitry, > > I have a trivial patch that resolves this issue. I would like to push it > to jdk8/tl today before we freeze for M7, and have this failure escape > into master. > > The change is to simply use conditions that are also supported by bourne > shell . > > diff -r 7dcb74c3ffba test/sun/management/jdp/JdpTest.sh > --- a/test/sun/management/jdp/JdpTest.sh Tue Feb 12 09:25:43 2013 > -0800 > +++ b/test/sun/management/jdp/JdpTest.sh Wed Feb 13 10:48:28 2013 > +0000 > @@ -51,7 +51,7 @@ _do_compile(){ > # sun.* packages is not included to symbol file lib/ct.sym so we have > # to ignore it > > - if [ ! -f ${_testclasses} ] > + if [ ! -d ${_testclasses} ] > then > mkdir -p ${_testclasses} > fi > @@ -319,7 +319,7 @@ rm -f ${_logname} > rm -f ${_logname} > rm -f ${_policyname} > > -if [ -e ${_testsrc}/policy.tpl ] > +if [ -f ${_testsrc}/policy.tpl ] > then > > cat ${_testsrc}/policy.tpl | \ > > -Chris. > > On 12/02/2013 22:19, Dmitry Samersoff wrote: >> Chris, >> >> I'm not able to reproduce it locally. Do you have a link to jprt job you >> see this failure? >> >> -Dmitry >> >> On 2013-02-12 21:38, Chris Hegarty wrote: >>> Dmitry, >>> >>> This test is now failing on several platforms, on jdk8 and 7u-dev >>> >>> ------- >>> >>> result: Passed. Compilation successful >>> >>> #section:shell >>> ----------messages:(3/154)---------- >>> command: shell JdpTest.sh [--jtreg, --no-compile] >>> reason: User specified action: run shell JdpTest.sh --jtreg --no-compile >>> elapsed time (seconds): 0.045 >>> ----------System.out:(0/0)---------- >>> ----------System.err:(1/110)---------- >>> /export2/Users/chris/repos/jdk8/tl/master_top/jdk/test/sun/management/jdp/JdpTest.sh: >>> >>> test: argument expected >>> result: Failed. Execution failed: exit code 1 >>> >>> >>> test result: Failed. Execution failed: exit code 1 >>> >>> -Chris. >>> >>> >>> On 02/12/2013 12:04 PM, dmitry.samers...@oracle.com wrote: >>>> Changeset: f7fb173ac833 >>>> Author: dsamersoff >>>> Date: 2013-02-12 16:02 +0400 >>>> URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/f7fb173ac833 >>>> >>>> 8007786: JDK-8002048 testcase doesn't work on Solaris >>>> Summary: test built in into Solaris shell doesn't have -e operator >>>> Reviewed-by: sla, sspitsyn >>>> >>>> ! test/sun/management/jdp/JdpTest.sh >>>> >> >> -- Dmitry Samersoff Oracle Java development team, Saint Petersburg, Russia * Give Rabbit time, and he'll always get the answer