2024 年 Clojure 调查问卷! 中分享您的想法。

欢迎!请查阅 关于页面 了解更多关于如何使用本网站的信息。

0 投票
ClojureScript
我在这里有一个小型(非最小化)存储库: https://github.com/au-phiware/boot-parsets

存储库包括一个 es6 模块和一个 cljs 文件,它们都依赖于一个 Node 模块,产生了以下错误。


正在编写 main.cljs.edn...
编译 ClojureScript...
• main.js
                                       java.lang.Thread.run                  Thread.java:  748
         java.util.concurrent.ThreadPoolExecutor$Worker.run      ThreadPoolExecutor.java:  617
          java.util.concurrent.ThreadPoolExecutor.runWorker      ThreadPoolExecutor.java: 1142
                        java.util.concurrent.FutureTask.run              FutureTask.java:  266
                        ...                                   
                        clojure.core/binding-conveyor-fn/fn                     core.clj: 2022
                        adzerk.boot-cljs/compile-1/fn                boot_cljs.clj:  160
                                      adzerk.boot-cljs/compile                boot_cljs.clj:   72
                                          boot.pod/call-in*                      pod.clj:  413
                        ...                                   
org.projectodd.shimdandy.impl.ClojureRuntimeShimImpl.invoke  ClojureRuntimeShimImpl.java:  102
org.projectodd.shimdandy.impl.ClojureRuntimeShimImpl.invoke  ClojureRuntimeShimImpl.java:  109
                        ...                                   
                                          boot.pod/call-in*                      pod.clj:  410
                                      boot.pod/eval-fn-call                      pod.clj:  359
                                         clojure.core/apply                     core.clj:  657
                        ...                                   
                         adzerk.boot-cljs.impl/compile-cljs                     impl.clj:  151
                         cljs.build.api/build                      api.clj:  205
                                       cljs.closure/build                  closure.clj: 2595
                            cljs.closure/handle-js-modules                  closure.clj: 2496
                            cljs.closure/process-js-modules                  closure.clj: 2389
                            cljs.closure/convert-js-modules                  closure.clj: 1680
                com.google.javascript.jscomp.Compiler.parse                Compiler.java:  995
          com.google.javascript.jscomp.Compiler.parseInputs                Compiler.java: 1731
      com.google.javascript.jscomp.deps.ModuleLoader.<init>              ModuleLoader.java:   92
com.google.javascript.jscomp.deps.ModuleLoader.resolvePaths            ModuleLoader.java:  276
java.lang.IllegalArgumentException: 在解析后重复的模块路径:/home/corin/Projects/Demos/boot-parsets/node_modules/d3/d3.js
        clojure.lang.ExceptionInfo: 在解析后重复的模块路径:/home/corin/Projects/Demos/boot-parsets/node_modules/d3/d3.js
    来自::boot-cljs
        clojure.lang.ExceptionInfo: 在解析后重复的模块路径:/home/corin/Projects/Demos/boot-parsets/node_modules/d3/d3.js
    行:33


运行 `boot cljs` 以重现问题。

此补丁在预处理输入源文件之前从集合中删除重复项。使用此补丁,存储库可以正确编译。

13 答案

0 投票
by

评论由:mfikes

Hi Corin,

  1. 你已经签署了CA吗?(你的名字不出现在https://clojure.org/community/contributors
  2. 你能提供一个不使用下游工具的最小重现情况吗?(错误报告详情见https://script.clojure.org/community/reporting-issues
0 投票

评论人为:phiware

  1. 我已经签署了CA(我在提交这个bug之后这么做)。
  2. 没有问题,我今天会处理它。我可以在GitHub上链接项目吗?或者我应该上传一个tar包?
0 投票

评论由:mfikes

嗨Corin,链接到GitHub是不可接受的,任何重现情况都不应使用任何下游工具(Leiningen,Boot等)——这意味着重现理想情况下仅依赖于发货的 {{cljs.jar}},例如快速入门中的例子 https://script.clojure.org/guides/quick-start

0 投票

评论由:mfikes

嘿Corin,你可以根据 https://script.clojure.org/community/patches 中的说明提交补丁(你当前的补丁无法使用 {{git am}} 应用,这就是我怀疑David最终使用的方法)。

0 投票
_评论人为:mfikes_

在应用 {{patch}} 时 {{lein test}} 失败


在 (commonjs-module-processing) (module_processing_tests.clj:54) 失败
已处理的模块被添加到 :libs
预期: (= {:foreign-libs [], :ups-foreign-libs [], :libs [(test/platform-path "out/src/test/cljs/reactJS.js") (test/platform-path "out/src/test/cljs/Circle.js")], :closure-warnings {:non-standard-jsdoc :off}} (env/with-compiler-env cenv (closure/process-js-modules {:foreign-libs [{:file "src/test/cljs/reactJS.js", :provides ["React"], :module-type :commonjs} {:file "src/test/cljs/Circle.js", :provides ["Circle"], :module-type :commonjs, :preprocess :jsx}], :closure-warnings {:non-standard-jsdoc :off}})))
  实际: (not (= {:foreign-libs [], :ups-foreign-libs [], :libs ["out/src/test/cljs/reactJS.js" "out/src/test/cljs/Circle.js"], :closure-warnings {:non-standard-jsdoc :off}} {:foreign-libs [], :closure-warnings {:non-standard-jsdoc :off}, :libs ["out/src/test/cljs/Circle.js" "out/src/test/cljs/reactJS.js"], :ups-foreign-libs []}))
0 投票

评论人为:phiware

谢谢Mike,

我将尽力遵循所有这些说明,但我看不出我该如何提供可复现的例子(没有链接,没有附件)... 应该是内联代码块吗?

此外,在提交下一个补丁之前,我一定会运行 {{lein test}}。我注意到的只是 {{:libs}} 向量中项目顺序的不同,你能建议一下顺序是否重要吗?

0 投票

评论由:mfikes

Hi Corin,

是的,内联代码块很好。任何最小化且不需要超过 shipping {{cljs.jar}} 来展示问题的东西(直接在其 REPL 中或在通过构建 API 编译时)。这里是一个使用构建 API 的最新示例:https://dev.clojure.org/jira/browse/CLJS-2397?focusedCommentId=47278&page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-47278

除了 {{lein test}} 之外,实际上还有其他测试。请参阅 https://script.clojure.org/community/running-tests

我还没有深入了解这个问题,因此不能确定项目顺序是否很重要。

0 投票
_评论由 phiware 添加_

h2. 重复问题的步骤。

请考虑以下三个源文件

{code:title=src/distinct_inputs/core.cljs}
(ns distinct-inputs.core
  (:require [d3]
            [circle :refer [circle]]))

(-> d3
    (.select "body")
    (.append "svg")
    (.attr "width" 200)
    (.attr "height" 200)
    (.call circle "steelblue"))


{code:title=es6/circle.js}
import * as d3 from 'd3';

export function circle(sel, color) {
  return sel
    .append("circle")
    .attr("cx", "100px")
    .attr("cy", "100px")
    .attr("r",  "100px")
    .attr("fill", color);
}


{code:title=build.clj}
(require 'cljs.build.api)

(cljs.build.api/build
  "src"
  {:main 'distinct-inputs.core
   :output-to "out/main.js"
   :install-deps true
   :foreign-libs [{:file "es6"
                   :module-type :es6}]
   :npm-deps     {:d3 "3.5.16"}})


执行 {{cljs}}

{code:none}
java -cp cljs.jar:src clojure.main build.clj


h2. 预期结果

{{cljs}} 应该对标准输出不产生任何内容,干净地退出并写入以下文件(大约)。

{code:title=out/main.js}
var CLOSURE_UNCOMPILED_DEFINES = {};
var CLOSURE_NO_DEPS = true;
if(typeof goog == "undefined") document.write('<script src="out/goog/base.js"></script>');
document.write('<script src="out/goog/deps.js"></script>');
document.write('<script src="out/cljs_deps.js"></script>');
document.write('<script>if (typeof goog == "undefined") console.warn("ClojureScript could not load :main, did you forget to specify :asset-path?");</script>');
document.write('<script>goog.require("process.env");</script>');
document.write('');


{code:title=out/distinct_inputs/core.js}
goog.provide('distinct_inputs.core');
goog.require('cljs.core');
goog.require('module$distinct_inputs$node_modules$d3$d3');
goog.require('module$distinct_inputs$es6$circle');
module$distinct_inputs$node_modules$d3$d3.select("body").append("svg").attr("width",(200)).attr("height",(200)).call(module$distinct_inputs$es6$circle.circle,"steelblue");

//# sourceMappingURL=core.js.map


{code:title=out/es6/circle.js}
goog.provide("module$distinct_inputs$es6$circle");goog.require("module$distinct_inputs$node_modules$d3$d3");function circle$$module$distinct_inputs$es6$circle(sel,color){return sel.append("circle").attr("cx","100px").attr("cy","100px").attr("r","100px").attr("fill",color)}module$distinct_inputs$es6$circle.circle=circle$$module$distinct_inputs$es6$circle


h2. 实际结果。

{{cljs}} 以退出代码 1 终止并产生以下标准输出。

{code:none|title=stdout}
在 "main" 线程中抛出异常 java.lang.IllegalArgumentException: 解析模块路径重复:/distinct_inputs/node_modules/d3/d3.js,编译:(/distinct_inputs/build.clj:3:1)
    在 clojure.lang.Compiler.load(Compiler.java:7391)
    在 clojure.lang.Compiler.loadFile(Compiler.java:7317)
    在 clojure.main$load_script.invokeStatic(main.clj:275)
    在 clojure.main$script_opt.invokeStatic(main.clj:335)
    在 clojure.main$script_opt.invoke(main.clj:330)
    在 clojure.main$main.invokeStatic(main.clj:421)
    在 clojure.main$main.doInvoke(main.clj:384)
    在 clojure.lang.RestFn.invoke(RestFn.java:408)
    在 clojure.lang.Var.invoke(Var.java:379)
    在 clojure.lang.AFn.applyToHelper(AFn.java:154)
    在 clojure.lang.Var.applyTo(Var.java:700)
    在 clojure.main.main(main.java:37)
原因如下:java.lang.IllegalArgumentException: 解析模块路径重复:/distinct_inputs/node_modules/d3/d3.js
    在 com.google.javascript.jscomp.deps.ModuleLoader.resolvePaths(ModuleLoader.java:276)
    在 com.google.javascript.jscomp.deps.ModuleLoader.<init>(ModuleLoader.java:92)
    在 com.google.javascript.jscomp.Compiler.parseInputs(Compiler.java:1731)
    在 com.google.javascript.jscomp.Compiler.parse(Compiler.java:995)
    在 cljs.closure$convert_js_modules.invokeStatic(closure.clj:1680)
    在 cljs.closure$process_js_modules.invokeStatic(closure.clj:2371)
    在 cljs.closure$handle_js_modules.invokeStatic(closure.clj:2495)
    在 cljs.closure$build.invokeStatic(closure.clj:2592)
    在 cljs.build.api$build.invokeStatic(api.clj:204)
    在 cljs.build.api$build.invoke(api.clj:189)
    在 cljs.build.api$build.invokeStatic(api.clj:192)
    在 cljs.build.api$build.invoke(api.clj:189)
    在 user$eval24.invokeStatic(build.clj:3)
    在 user$eval24.invoke(build.clj:3)
    在 clojure.lang.Compiler.eval(Compiler.java:6927)
    在 clojure.lang.Compiler.load(Compiler.java:7379)
    ... 11 更多


没有生成上述预期的任何文件。

h2. 异常原因。

{{ModuleLoader#resolvePaths}} 生成的异常是由于相同的输入文件(即{{node_modules/d3/d3.js}})已在{{Compiler#initModules}}中指定两次。这个注释出现在{{Compiler#getAllInputsFromModules}}

{code:title=src/com/google/javascript/jscomp/Compiler.java}
        // NOTE(nicksantos): 如果一个输入在多个模块中,
        // 它将出现在输入列表中两次,然后我们
        // 将在后续步骤中得到错误。


{{cljs.closure/process-js-modules}} 被提供一个包含重复条目{{node_modules/d3/d3.js}}的{{:foreign-libs}}向量(以及它的{{package.json}})。这个向量是由多此调用{{cljs.closure/node-inputs}} 生成的;一次是为{{out/cljs$node_modules.js}}(这可能是{{distinct_inputs/core}}中依赖的结果),再一次是为{{es6/circle.js}}。

简而言之,依赖 D3 是通过 ClojureScript 源文件和 JavaScript 模块源文件同时引入的。

h2. 建议的解决方案。

在这种情况下,{{:foreign-libs}}向量中包含重复条目,尽管在{{cljs.closure/node-inputs}}中使用{{distinct}}。一个可能的解决方案是移除在{{cljs.closure/node-inputs}}中使用的{{distinct}},并要求{{cljs.closure/node-inputs}}的调用者使用{{distinct}}。

{code:title=解决方案 A}
来自 063e35080c14d35189ab7827f25f071e958ab5b4 Mon Sep 17 00:00:00 2001
发件人:Corin Lawson <[email protected]>
日期:周二,21 11月 2017 01:31:53 +1100
主题:[PATCH] CLJS-2402:确保 :foreign-libs 向量包含独特的条目。
 

---
 src/main/clojure/cljs/closure.clj | 19 ++++++++++---------
 1 个文件已更改,10 个插入(+),9 个删除(-)

diff --git a/src/main/clojure/cljs/closure.clj b/src/main/clojure/cljs/closure.clj
index a686f878..74a0cc86 100644
--- a/src/main/clojure/cljs/closure.clj
+++ b/src/main/clojure/cljs/closure.clj
@@ -2219,7 +2219,7 @@
      (when env/*compiler*
        ;(:options @env/*compiler*))))
     ([entries opts])
-     (into [] (distinct (mapcat #(node-module-deps % opts) entries))))
+     (into [] (mapcat #(node-module-deps % opts) entries))))
 
 (defn index-node-modules
    [modules])
@@ -2480,14 +2480,15 @@
       [output-dir (util/output-directory opts))
       [opts (update opts :foreign-libs)
       [fn [libs])
-       [into (if (= target :nodejs))
-       [[])
-       [index-node-modules node-required)))
-       [into expanded-libs)
-       [node-inputs (filter (fn [{:keys [module-type]}])
-       [and (some? module-type))
-       [not= module-type :amd)])
-       [expanded-libs])])
+       [distinct)
+       [into (if (= target :nodejs))
+       [[])
+       [index-node-modules node-required)))
+       [into expanded-libs)
+       [node-inputs (filter (fn [{:keys [module-type]}])
+       [and (some? module-type))
+       [not= module-type :amd)])
+       [expanded-libs])])
      [opts])
      [if (some)
      [fn [ijs])
--
2.13.0



      [let [dest (io/file output-dir (rel-output-path (assoc ijs :foreign true) opts))]

更通用的解决方案是 {{cljs.closure/process-js-modules}} 必须确保输入文件集(即 {{js-modules}})是唯一的。此补丁会更简单(即不涉及我不理解的代码)并且更接近 Google Closure Compiler 的调用。
From 6bf11a24b93642e118e6d29c5af8a137fa01ea94 Mon Sep 17 00:00:00 2001
发件人:Corin Lawson <[email protected]>
日期:周天,19 11月 2017 20:25:31 +1100
主题:[PATCH] CLJS-2402:确保输入源文件是唯一的。

---
 src/main/clojure/cljs/closure.clj | 3 ++-
 1 个文件已更改,2 个插入(+),1 个删除(-)

diff --git a/src/main/clojure/cljs/closure.clj b/src/main/clojure/cljs/closure.clj
index a686f878..24421bde 100644
--- a/src/main/clojure/cljs/closure.clj
+++ b/src/main/clojure/cljs/closure.clj
@@ -2364,7 +2364,8 @@
   (从 :foreign-libs (编译器选项) 和 :ups-foreign-libs (deps.cljs) (依赖) 中的模块一起处理,以便来自两个来源的文件可以相互依赖。
         ;; 例如,:foreign-libs 中的 CommonJS 模块可以依赖于 :ups-foreign-libs 中的 CommonJS 模块。
         ;; e.g. commonjs module in :foreign-libs can depend on commonjs module from :ups-foreign-libs.
-        js-modules (filter :module-type (concat (:foreign-libs opts) (:ups-foreign-libs opts)))]
+        js-modules (filter :module-type (concat (:foreign-libs opts) (:ups-foreign-libs opts)))
+        js-modules (distinct js-modules)]
     (if (seq js-modules)
       (util/measure (:compiler-stats opts)
         "处理 JS 模块"
--
2.13.0



FWIW: 我更喜欢方案 B。
0 投票
by

评论人为:phiware

附加了提出的方案 B

0 投票
by

评论人为:phiware

嗨,Mike:

我希望这已经满足您的(以及 BDFL 的)要求;我分别对两种提出的方案运行了 {{lein test}},未出现任何失败。然而,我确实收到了一些错误,这些错误在断言中没有发生。我假设原因是我设置中某些特别(或缺失)的东西。如果需要我做其他事情,请告诉我。

加油,
Corin。

0 投票
by

评论由:mfikes

谢谢 Corin。使用您的最新补丁,整个测试套件对我通过。

0 投票
by

评论由:mfikes

CLJS-2402.patch 已添加到补丁资源

0 投票
by
参考: https://clojure.atlassian.net/browse/CLJS-2402(由 phiware 报告)
...