RFR: JDK-8067151 - [TESTBUG] com/sun/corba/5036554/TestCorbaBug.sh

[Mark Sheppard]

Hi Chris,
    thanks for the reply. The bug refers to TestCorbaBug.sh, but I added 
the same change to 7056731.sh as a "pre-emptive measure".
If the first is failing then the second will also (at some point). I 
looked through the various shell script tests, while some use TESTJAVA, 
the majority are using COMPILEJAVA
as the env variable with the path to the jdk tools. So while making the 
first change, I thought it convenient to make the second change as well.

I can remove it in this case, if it is thought to be unnecessary at this 
juncture.

regards
Mark

On 17/12/2014 09:01, Chris Hegarty wrote:
> Mark,
>
> The changes in *test/com/sun/corba/5036554/TestCorbaBug.sh*look fine 
> to me. I don’t understand the reason for the ones in 
> *test/com/sun/corba/cachedSocket/7056731.sh.*
>
> -Chris.
>
> On 17 Dec 2014, at 00:48, Mark Sheppard <mark.sheppard at oracle.com 
> <mailto:mark.sheppard at oracle.com>> wrote:
>
>> Hi
>>   please oblige and review the follow changes
>> http://cr.openjdk.java.net/~msheppar/8067151/webrev/ 
>> <http://cr.openjdk.java.net/%7Emsheppar/8067151/webrev/>
>>
>> to address the issue
>> https://bugs.openjdk.java.net/browse/JDK-8067151
>>
>> which sees a test failure for JRE only in an embedded environment.
>>
>> the test scripts now use the env variable COMPILEJAVA rather than 
>> TESTJAVA
>>
>> regards
>> Mark
>