在 Windows 上 repl.cljc 中构建 fileUrl 导致 java.net.UnknownHostException

Question

我认为以下行可能是在 Windows 上导致问题的原因
https://github.com/clojure/clojurescript/blob/master/src/main/clojure/cljs/repl.cljc#L713

例如，文件参数为 "src\duct3\client.cljs"

(str "file://" (.getAbsolutePath file))

在 Windows 上评估为
"file://C:\Projects\Playground\duct3\src\duct3\client.cljs"

这不是合法的文件 URL （https://en.wikipedia.org/wiki/File_URI_scheme#Windows）

最终结果为：java.net.UnknownHostException（Java 将该 URL 作为 FTP 地址处理）。

Answer 1

评论由：vojko 提供

到提到行的永久链接
https://github.com/clojure/clojurescript/blob/98656d305e6447c62e36849ee615532d53211fdd/src/main/clojure/cljs/repl.cljc#L736

Answer 2

评论由：vojko 提供

为 Windows 平台添加不同的前缀。

Answer 3

评论由：vojko 提供

附加补丁。

Answer 4

评论由：admin 提供

请勿将问题标记为已解决 - 只有像dnolen或mfikes这样的项目管理员才能这样做。

Answer 5

评论由：mfikes发表

我想知道{{file://}}字符串结构是否必要。

如果不是，那么一个更简单的补丁将涉及替换以下表达式

(ana/analyze-file (str "file://" (.getAbsolutePath file)) opts)

替换为

(ana/analyze-file file opts)

Answer 6

评论由：vojko 提供

如您所述，我已经尝试了WO "file://"，它工作得完美。
谢谢！

Answer 7

评论由：mfikes发表

嗨大卫，尽管你已经将这个问题标记为已接受，但我的看法是，我可能已经向Vojimir提出了一个更简洁的方法，他尝试了，而且它对他有效。我觉得我们会感兴趣看看Vojimir的简洁补丁。

Answer 8

评论由：vojko 提供

嗨迈克，我已经附上了你建议的新补丁。

Answer 9

评论由：mfikes发表

CLJS-2155-2.patch 通过CI(/)

Answer 10

评论由：mfikes发表

CLJS-2155-2.patch 已添加到补丁待办事项(i)

Answer 11

参考： https://clojure.atlassian.net/browse/CLJS-2155（由vojko报告）