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
    from: :boot-cljs
        clojure.lang.ExceptionInfo: 解析后重复的模块路径:/home/corin/Projects/Demos/boot-parsets/node_modules/d3/d3.js
    line: 33


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

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

13 答案

0
by

评论者:mfikes

你好,Corin,

  1. 你是否已经签署了CA?(你的名字未出现在https://clojure.org/community/contributors上)
  2. 你能提供不使用下游工具的最小重现示例吗?(bug上报详情位于https://script.clojure.org/community/reporting-issues
0

评论区:phiware

  1. 我已经签署了CA(我是在提交这个错误之后做的)。
  2. 没问题,我今天稍后会处理。把项目链接到GitHub可以吗?还是我应该上传一个tarball文件?
0

评论者:mfikes

你好Corin,链接到GitHub是不行的,任何重现案例也不应使用任何下游工具(Leiningen,Boot等)——这意味着一个重现案例理想情况下只依赖于shipped {{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
by

评论者:mfikes

你好,Corin,

是的,内联代码块非常好。任何只依赖于发货 {{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
by
_评论者:phiware_

步骤二:重现问题的步骤。

考虑以下三个源文件

{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('<script>goog.require("distinct_inputs.core");</script>');


{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


实际输出。

{{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.invoke(api.clj:204)
在cljs.build.api$build.invoke(api.clj:189)
在cljs.build.api$build.invoke(api.clj:204)
在cljs.build.api$build.invokeStatic(api.clj:192)
在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 more

无任何预期文件生成。

异常产生原因。

{{ModuleLoader#resolvePaths}}发射的异常是由于相同的输入文件(即{{node_modules/d3/d3.js}})被多次指定给{{Compiler#initModules}}。{{Compiler#getAllInputsFromModules}}中有以下备注:
{code:title=src/com/google/javascript/jscomp/Compiler.java}
        // 备注(nicksantos): 如果一个输入在多个模块中,
        // 它将在输入列表中出现两次,然后我们将


        // 在接下来的步骤中获得错误。

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

简而言之,D3的依赖项是由ClojureScript源文件和JavaScript模块源文件拉取的。

在此场景中,尽管在{{cljs.closure/node-inputs}}中使用了{{distinct}},但{{:foreign-libs}}向量中仍包含重复条目。一种可能的解决方案是移除{{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]>
日期:Tue, 21 Nov 2017 01:31:53 +1100
主题:[补丁] 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 @@
        输出目录 (util/output-directory opts)
        opts (update opts :foreign-libs
                 fn [库s]
-                 (into (if (= 目标 :nodejs)
-                         []
-                         (index-node-modules 节点所需的)))
-                   (into 扩展库s
-                              (节点输入s (filter (fn [{:keys [模块类型]}]
-                                              有一种模块类型
-                                                  不等于 :amd)
-                         扩展库s))))))
+                 (distinct
+                   (into (if (= 目标 :nodejs)
+                         []
+                         (index-node-modules 节点所需的)))
+                         (into 扩展库s
+                              (节点输入s (filter (fn [{:keys [模块类型]}]
+                                              有一种模块类型
+                                                  不等于 :amd)
+                         扩展库s)))))))
         opts (if (some
                      (fn [ijs]
                      (let [dest (io/file 输出目录 (rel-output-path (assoc ijs :foreign true) opts))]
--
2.13.0



更通用的解决方案是 {{cljs.closure/process-js-modules}} 必须确保输入文件集(即 {{js-modules}})是唯一的。这个补丁会更简单(即不会弄乱我不理解的部分)并且更接近 Google Closure Compiler 的调用。

{code:title=解决方案 B}
From 6bf11a24b93642e118e6d29c5af8a137fa01ea94 Mon Sep 17 00:00:00 2001
作者:Corin Lawson <[email protected]>
Date: Sun, 19 Nov 2017 20:25:31 +1100
Subject: [PATCH] CLJS-2402: 确保输入源文件是唯一的。

---
 src/main/clojure/cljs/closure.clj | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

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 @@
   (let [;; 来自 :foreign-libs(编译器选项)和 :ups-foreign-libs(deps.cljs)的模块
         ;; 它们被一起处理,这样两个来源的文件可以相互依赖。
         ;; 例如,:foreign-libs中的CommonJS模块可以依赖:ups-foreign-libs中的CommonJS模块。
-        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

评论区:phiware

附上建议的方案B

0

评论区:phiware

Hi Mike,

我希望现在的结果让您(以及BDFL)满意;我为两个建议的解决方案都运行了{{lein test}},我没有收到任何失败。然而,确实收到了一些错误,但这些错误在断言中并未出现。我假设原因是我的设置中某些特别(或缺乏)之处。如果还需要我做其他事情,请让我知道。

祝好,
Corin.

0

评论者:mfikes

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

0

评论者:mfikes

CLJS-2402.patch已添加到补丁帐篷(i)

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