2024 Clojure 状态调查! 中分享你的想法。

欢迎!请参阅 关于 页面以了解更多有关此工作方式的信息。

0
ClojureScript
影响:1.9.1033+

我在这里有一个最小化的复 repro 仓库:[https://github.com/au-phiware/cljs-2818](https://github.com/au-phiware/cljs-2818)

h2. 问题描述

在 CLJS-2389 中更新 Closure 编译器之前,使用 {{export { x as y } from './x'}} 或 {{export { default as y } from './x'}} 语法的 ES6 模块可以正确编译。而其他这种语法的表单,如 {{export { default } from './x'}}, 则不能。

从 1.9.1033 开始,编译器不再发出 {{goog.require}} 语句,也不在 {{cljs_deps.js}} 发出完整的 {{goog.addDependency}} 语句集。

h2. 重复问题的步骤

考虑以下源文件

{code:title=src/foo/core.cljs|language=clojure}
(ns foo.core (:require [hello :refer [helloGreet]]))                                 

(def ^:export sayHello
    (helloGreet "World"))

(sayHello)


{code:title=es6/hello.js|language=javascript}
export {
    default as helloGreet
} from "./greet";


{code:title=es6/greet.js|language=javascript}
export default function greet(m) {
    document.write("\nHello, " + m);
};


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

(cljs.build.api/build
  "src"
  {:main 'foo.core
   :output-to "target/main.js"
   :output-dir "target/main.out"
   :asset-path "main.out"
   :foreign-libs [{:file "es6/hello.js"
                   :provides ['hello]
                   :module-type :es6}]
   :verbose true
   :npm-deps {"@cljs-oss/module-deps" "*"}
   :install-deps true})


执行 {{cljs}}


java -cp cljs.jar:src clojure.main build.clj


h2. 预期结果

{{cljs}} 应该干净退出并写入以下文件(大约)。

{code:title=target/main.out/cljs_deps.js|language=javascript}
goog.addDependency("base.js", ['goog'], []);
goog.addDependency("../cljs/core.js", ['cljs.core'], ['goog.string', 'goog.Uri', 'goog.object', 'goog.math.Integer', 'goog.string.StringBuffer', 'goog.array', 'goog.math.Long']);
goog.addDependency("../process/env.js", ['process.env'], ['cljs.core']);
goog.addDependency("../es6/greet.js", ['module$usr$src$es6$greet'], []);
goog.addDependency("../es6/hello.js", ['module$usr$src$es6$hello'], ['module$usr$src$es6$greet']);
goog.addDependency("../foo/core.js", ['foo.core'], ['cljs.core', 'module$usr$src$es6$hello']);


{code:title=target/main.out/es6/hello.js|language=javascript}
goog.provide("module$usr$src$es6$hello");
goog.require("module$usr$src$es6$greet");
module$usr$src$es6$hello.helloGreet=module$usr$src$es6$greet["default"]


h2. 实际结果

{{cljs_deps.js}} 中缺少 {{es6/greet.js}} 依赖

{code:title=target/main.out/cljs_deps.js|language=javascript}
goog.addDependency("base.js", ['goog'], []);
goog.addDependency("../cljs/core.js", ['cljs.core'], ['goog.string', 'goog.Uri', 'goog.object', 'goog.math.Integer', 'goog.string.StringBuffer', 'goog.array', 'goog.math.Long']);
goog.addDependency("../process/env.js", ['process.env'], ['cljs.core']);
goog.addDependency("../es6/hello.js", ['module$usr$src$es6$hello'], []);
goog.addDependency("../foo/core.js", ['foo.core'], ['cljs.core', 'module$usr$src$es6$hello']);


{{es6/hello.js}} 文件中缺少 {{goog.requires}} 语句

{code:title=target/main.out/es6/hello.js|language=javascript}
goog.provide("module$usr$src$es6$hello");
var module$usr$src$es6$hello={get helloGreet(){return module$usr$src$es6$greet["default"]}}


此外,浏览器控制台显示


>>> module$usr$src$es6$hello.helloGreet
hello.js:2 未捕获的引用错误: module$usr$src$es6$greet未定义
    在 Object.get helloGreet [as helloGreet] (hello.js:2)
    在 <匿名>:1:65


h2. 尝试的解决方案

明确定义了{{:requires}}选项到{{es6/hello.js}} {{:foreign-libs}}条目,但并未解决这个问题(也无法通过使用整个{{es6}}目录的条目或使用npm模块,例如d3-scale)。添加{{[greet]}}到{{foo.core}}的requires也同样无效。

6 个回答

0
by

评论由: phiware

这好像是在Google Closure Compiler上游的一个问题...

0
by

评论由: phiware

在gcc中使用相同的es6代码编译可以正常进行,使用{{java -jar $JAR -O WHITESPACE_ONLY --js_output_file=/dev/stdout -W VERBOSE --module_resolution NODE greet.js hello.js}}。但当cljs调用gcc转换es6代码时,生成的源代码缺少{{goog.require}}语句...这可能是不是因为module_resolution的问题?

0
by

评论由: phiware

我注意到gcc使用{{-O BUNDLE --dependency_mode STRICT}}选项无法发出greet依赖...这时我有点困惑...

0
by

评论由: phiware

如果我对gcc应用此补丁,那么它将能够检测到hello.js的正确requires。

`
diff --git a/src/com/google/javascript/jscomp/CompilerInput.java b/src/com/google/javascript/jscomp/CompilerInput.java
index 62609562b..4e4d3e3dc 100644
--- a/src/com/google/javascript/jscomp/CompilerInput.java
+++ b/src/com/google/javascript/jscomp/CompilerInput.java
@@ -285,7 +285,7 @@ public class CompilerInput extends DependencyInfo.Base implements SourceAst {

 // If the code is a JsAst, then it was originally JS code, and is compatible with the
 // regex-based parsing of JsFileParser.
  • if (ast instanceof JsAst && JsFileParser.isSupported()) {
    • if (false && ast instanceof JsAst && JsFileParser.isSupported()) {
      // Look at the source code.
      // Note: it's OK to use getName() instead of
      // getPathRelativeToClosureBase() here because we're not using
      `

但现在cljs认为greet应该命名为{{"module$usr$src$greet"}}而不是{{"module$usr$src$es6$greet"}},这看起来也是gcc的一个bug。

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