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 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
>> ----------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
