Re: FAILS: test/sun/management/jdp/JdpTest.sh: test: argument expected

2013-02-25 Thread Kelly O'Hair
Bingo. ;^) -kto On Feb 12, 2013, at 2:34 PM, alejandro murillo wrote: Dmitry, you have to use -testset core to run this test via JPRT. It's not run on regular jobs. Alejandro On 2/12/2013 3:18 PM, Dmitry Samersoff wrote: Kelly, Do you have an idea why I didn't see this failure

Re: FAILS: test/sun/management/jdp/JdpTest.sh: test: argument expected

2013-02-25 Thread Kelly O'Hair
+if [ ! -f ${_testclasses} ] needs to be +if [ ! -d ${_testclasses} ] -kto On Feb 12, 2013, at 9:38 AM, Chris Hegarty wrote: Dmitry, This test is now failing on several platforms, on jdk8 and 7u-dev --- result: Passed. Compilation successful #section:shell

Re: FAILS: test/sun/management/jdp/JdpTest.sh: test: argument expected

2013-02-13 Thread Chris Hegarty
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

Re: FAILS: test/sun/management/jdp/JdpTest.sh: test: argument expected

2013-02-13 Thread Dmitry Samersoff
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

RR(S): 8008095.JDP-TEST3 (was Re: FAILS: test/sun/management/jdp/JdpTest.sh: test: argument expected)

2013-02-13 Thread Dmitry Samersoff
Chris, Please take a look at the final changes: http://cr.openjdk.java.net/~dsamersoff/8008095.JDP-TEST3/webrev.01/jdk/ I still not able to get valuable results from jprt but tried couple of different solaris machines and hope this time the problem with this test is gone. I'm ready to push it

Re: RR(S): 8008095.JDP-TEST3 (was Re: FAILS: test/sun/management/jdp/JdpTest.sh: test: argument expected)

2013-02-13 Thread Chris Hegarty
Looks fine to me. -Chris. On 13/02/2013 14:40, Dmitry Samersoff wrote: Chris, Please take a look at the final changes: http://cr.openjdk.java.net/~dsamersoff/8008095.JDP-TEST3/webrev.01/jdk/ I still not able to get valuable results from jprt but tried couple of different solaris machines

FAILS: test/sun/management/jdp/JdpTest.sh: test: argument expected

2013-02-12 Thread Chris Hegarty
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

Re: FAILS: test/sun/management/jdp/JdpTest.sh: test: argument expected

2013-02-12 Thread Dmitry Samersoff
Kelly, Good catch but this part of test is not executed under JTReg, and this mistake cause extra mkdir call but not a test error. -Dmitry On 2013-02-12 21:52, Kelly O'Hair wrote: +if [ ! -f ${_testclasses} ] needs to be +if [ ! -d ${_testclasses} ] -kto On Feb 12, 2013, at

Re: FAILS: test/sun/management/jdp/JdpTest.sh: test: argument expected

2013-02-12 Thread Dmitry Samersoff
Kelly, Do you have an idea why I didn't see this failure under jprt? http://sthjprt.se.oracle.com/archives/2013/02/2013-02-08-192233.dsamerso.tl/ -Dmitry On 2013-02-12 21:52, Kelly O'Hair wrote: +if [ ! -f ${_testclasses} ] needs to be +if [ ! -d ${_testclasses} ] -kto On

Re: FAILS: test/sun/management/jdp/JdpTest.sh: test: argument expected

2013-02-12 Thread Dmitry Samersoff
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

Re: FAILS: test/sun/management/jdp/JdpTest.sh: test: argument expected

2013-02-12 Thread alejandro murillo
Dmitry, you have to use -testset core to run this test via JPRT. It's not run on regular jobs. Alejandro On 2/12/2013 3:18 PM, Dmitry Samersoff wrote: Kelly, Do you have an idea why I didn't see this failure under jprt?