You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@sling.apache.org by "Craig L. Ching (JIRA)" <ji...@apache.org> on 2008/07/30 17:29:31 UTC
[jira] Commented: (SLING-582) scripting/jst JsCodeGeneratorTest
broken under Cygwin on Windows
[ https://issues.apache.org/jira/browse/SLING-582?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12618385#action_12618385 ]
Craig L. Ching commented on SLING-582:
--------------------------------------
Thank you very much, I've built on Cygwin and this is fixed. The issue appears closed to me, so not taking any further action. Thanks again!
> scripting/jst JsCodeGeneratorTest broken under Cygwin on Windows
> ----------------------------------------------------------------
>
> Key: SLING-582
> URL: https://issues.apache.org/jira/browse/SLING-582
> Project: Sling
> Issue Type: Bug
> Components: Testing
> Environment: Maven version: 2.0.9
> Java version: 1.6.0_03
> OS name: "windows vista" version: "6.0" arch: "x86" Family: "windows"
> Reporter: Craig L. Ching
> Priority: Minor
> Attachments: SLING-582.patch
>
>
> This is probably not that important to too many people, but for me it is. I use cygwin on Windows for development and CRLF issues are always a problem here. So this is a fix to the JsCodeGeneratorTest that allows it to work on Cygwin.
> Basically, normal Windows uses CRLF for line-endings, but Cygwin uses just CR if installed the way Cygwin recommends. There's a subtle problem where the test generates code that has CRLF in it, but since the test's expected input doesn't use CRLF, the test fails. There is already a helper that is supposed to turn line-endings into '.', but, unfortunately it's broken as well. Currently it simply turns '\n' into '.', but in the case of Windows this ends up leaving the line-feeds in, which doesn't cause any problems, but the intention of the code isn't met. So, not only is this a better implementation that fixes a subtle issue, but it also makes it work on Cygwin.
> Patch will be attached in my next post.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.