在 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

嗨David，尽管你标记了这个问题为已接受，但我对上面的看法是，我曾向Vojimir提出了一些建议，他尝试了，并且对他说它是有效的。我认为我们会很感兴趣Vojimir的更简洁的补丁。

Answer 8

由：vojko 发布的评论

嗨Mike，我附上新补丁，按照您的建议。

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 报告）