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

Dmitry Samersoff Dmitry.Samersoff at oracle.com
Fri Feb 24 11:46:27 PST 2012


Staffan,

Looks good for me!

-Dmitry


On 2012-02-24 20:12, Staffan Larsen wrote:
> Please review the following changes:
> 
> webrev: http://cr.openjdk.java.net/~sla/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


-- 
Dmitry Samersoff
Java Hotspot development team, SPB04
* There will come soft rains ...


More information about the serviceability-dev mailing list