Skip to content

Commit cdaeff2

Browse files
author
dnolen
committed
under Node.js we don't need require entries in the goog.addDependency calls in cljs_deps.js - runtime require is possible. It's surprising this hasn't come up as a problem before.
1 parent a64ab2c commit cdaeff2

1 file changed

Lines changed: 3 additions & 1 deletion

File tree

src/main/clojure/cljs/closure.clj

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1352,7 +1352,9 @@
13521352
"\", ["
13531353
(ns-list (deps/-provides input))
13541354
"], ["
1355-
(ns-list (deps/-requires input))
1355+
;; under Node.js runtime require is possible
1356+
(when-not (= :nodejs (:target opts))
1357+
(ns-list (deps/-requires input)))
13561358
"]);\n")))
13571359

13581360
(defn deps-file

0 commit comments

Comments
 (0)