RFR(S): 7079093 & 7073626: cygwin problems in tests

serguei.spitsyn at oracle.com serguei.spitsyn at oracle.com
Fri Feb 24 10:04:31 PST 2012


Looks good.
A copyright comment year must be 2012.

Thanks,
Serguei


On 2/24/12 8:12 AM, Staffan Larsen wrote:
> Please review the following changes:
>
> webrev: http://cr.openjdk.java.net/~sla/7073626/webrev.00/ 
> <http://cr.openjdk.java.net/%7Esla/7073626/webrev.00/>
>
> This is a combo review for the following two bugs (although I will 
> push as two changes):
>
> http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=7079093
> http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=7073626
>
>
> The first problem is to recognize cygwin as a valid platform and to 
> set up the SystemRoot environment variable (which can be called 
> SYSTEMROOT) in some cases.
>
> The second problem is that some versions of cygwin (or at least some 
> installations of cygwin) will add extra newlines when cat:ing files, 
> so I remove those.
>
> The fixes passes JRPT runs.
>
> Thanks,
> /Staffan

-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://mail.openjdk.java.net/pipermail/serviceability-dev/attachments/20120224/b738e440/attachment.html 


More information about the serviceability-dev mailing list