在 repl.cljc 中在 Windows 上构建 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

我已经试验过“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 已添加到Patch Tender（i）

Answer 11

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