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

欢迎!请查看 关于 页面了解关于这个工作的更多信息。

0
ClojureScript
影响:1.9.1033+

我这里有一个最小的重现仓库: https://github.com/au-phiware/cljs-2818

## 问题描述

在 CLJS-2389 更新关闭器之前,使用 {{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}} 语句集。

## 重复问题步骤

考虑以下源文件

{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


## 预期结果

{{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"]

## 实际结果

{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']);
在 {{cljs_deps.js}} 中缺少 {{es6/greet.js}} 依赖
"../es6/hello.js", ['module$usr$src$es6$hello'], ['module$usr$src$es6$greet']);


并且,{{es6/hello.js}} 文件缺少了 {{goog.requires}} 声明

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


此外,浏览器控制台显示


>>> module$usr$src$es6$hello.helloGreet
hello.js:2 Uncaught ReferenceError: module$usr$src$es6$greet is not defined
    在 Object.get helloGreet [as helloGreet] (hello.js:2)
    在 <anonymous>: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

使用相同版本的 es6 代码,通过 gcc 编译效果良好,使用 {{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

我注意到使用 {{-O BUNDLE --dependency_mode STRICT}} 选项的 gcc 无法生成 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
      `

但现在 ClojureScript 认为 greet 应该命名为 {{"module$usr$src$greet"}}, 而不是 {{"module$usr$src$es6$greet"}},但这似乎也是一个 GCC 的 bug。

0
0
...