Per instructions from redinger and jgehtland (patch addressing one issue
attached).
Three separate issues so far re: path support on windows
* compiler path regex is not portable, cljs/compiler.clj:1096 (see attached
patch, thanks amalloy for the assist)
* generated JS has path problems on windows
goog.addDependency(*
"..\..\..\..\..\..\..\/C:/tmp/clojurescript/samples/twitterbuzz/out/cljs/core.js"
*, ['cljs.core'], ['goog.string', 'goog.string.StringBuffer', 'goog.object',
'goog.array']);
* helper launch scripts on windows (under cygwin), more a feature than bug
--
You received this message because you are subscribed to the Google
Groups "Clojure" group.
To post to this group, send email to clojure@googlegroups.com
Note that posts from new members are moderated - please be patient with your
first post.
To unsubscribe from this group, send email to
clojure+unsubscr...@googlegroups.com
For more options, visit this group at
http://groups.google.com/group/clojure?hl=en
>From 6b7b35504af661e16fc24487d309719cda974d31 Mon Sep 17 00:00:00 2001
From: unknown <pba...@.corp.shutterfly.com>
Date: Wed, 20 Jul 2011 18:56:19 -0700
Subject: [PATCH] Made path pattern portable.
---
src/clj/cljs/compiler.clj | 2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
mode change 100644 => 100755 src/clj/cljs/compiler.clj
diff --git a/src/clj/cljs/compiler.clj b/src/clj/cljs/compiler.clj
old mode 100644
new mode 100755
index b1f2213..b7090b0
--- a/src/clj/cljs/compiler.clj
+++ b/src/clj/cljs/compiler.clj
@@ -1093,7 +1093,7 @@ goog.require = function(rule){Packages.clojure.lang.RT[\"var\"](\"cljs.compiler\
(defn path-seq
[file-str]
- (string/split file-str (re-pattern java.io.File/separator)))
+ (string/split file-str (re-pattern (java.util.regex.Pattern/quote (java.io.File/separator)))))
(defn to-path
[parts]
--
1.7.5.1
