Code review request: 8015274, 8015276 and 8016158: policytool manual test problems

Sean Mullan sean.mullan at oracle.com
Tue Jun 11 13:16:53 UTC 2013


Why did you need to check for a $HOME/.java.policy file in i18n.sh? Can 
you explain that?

Otherwise, changes look good to me.

--Sean

On 06/08/2013 09:32 PM, Weijun Wang wrote:
> Updated at
>
>     http://cr.openjdk.java.net/~weijun/8015274/webrev.01/
>
> A new bug 8016158 is added.
>
> The changes include:
>
> 1. Add CYGWIN* as a case for uname in all tests
> 2. Add description about temp directory in all tests
> 3. i18.sh now checks for ~/.java.policy and creates a keystore
> 4. Many changes in i18n.html, keystore URLs, some warning now does not
> show.
>
> I personally run i18n.sh on Mac and Windows. Every step looks fine now.
>
> Thanks
> Max
>
>
> On 5/31/13 9:41 AM, Weijun Wang wrote:
>> Please review the fix at
>>
>>     http://cr.openjdk.java.net/~weijun/8015274/webrev.00/
>>
>> It's for two bugs:
>>
>>     8015274: TEST_BUG: Step2: After selecting 'View Warning Log', it is
>> empty instead of FileNotFound.
>>     8015276: TEST_BUG: The 'ptool.test' can't be saved in the 'tmp'
>> folder.
>>
>> Thanks
>> Max




More information about the security-dev mailing list