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
