DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT <http://nagoya.apache.org/bugzilla/show_bug.cgi?id=20992>. ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND INSERTED IN THE BUG DATABASE.
http://nagoya.apache.org/bugzilla/show_bug.cgi?id=20992 [PATCH] ant test-encoding fails on Windows Summary: [PATCH] ant test-encoding fails on Windows Product: Velocity Version: 1.0-Release Platform: PC OS/Version: Other Status: NEW Severity: Minor Priority: Other Component: Source AssignedTo: [EMAIL PROTECTED] ReportedBy: [EMAIL PROTECTED] Ant test fails on Windows (specifically ant test-encoding) due to a problem matching newlines between the result of parsing "encodingtest2.vm" and the expected result "encodingtest2.cmp". Apparently, WinCVS translates Unix newline (LF) to the Windows newline (CR LF) when checking out the files. But the file "encodingtest2.cmp" already has newlines in the windows format CR LF. When it's checked out to a Win machine, this turns into CR CR LF. The result file (generated by the Velocity test), generates CR LF for newline. According to the Java Spec, the unusual case of CR CR LF should be treated as a single newline [1], along with CR LF and LF. The Velocity test is smart enough to normalize newlines (to make CR LF the same as LF). But does not treat CR CR LF as a newline. Thus the compare file doesn't match the result file and the test fails. Got all that? One solution would be to use CygWin CVS, which preserves the Unix newline (LF), or check the box in WinCVS "Checkout files with the Unix LF". A better solution is to change the routine in Velocity test (normalizeNewlines) that normalizes the newlines, to treat the following as equivalent newlines LF (unix), CR LF (win), and CR CR LF (win mistranslated). The one-line patch is attached, WILL [1] http://java.sun.com/docs/books/jls/second_edition/html/lexical.doc.html#231571 --------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
