Skip to content

Commit 343573f

Browse files
committed
CLJS-1184: log module building activity under verbose
1 parent 2b7017f commit 343573f

File tree

1 file changed

+13
-2
lines changed

1 file changed

+13
-2
lines changed

src/clj/cljs/closure.clj

Lines changed: 13 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -631,7 +631,7 @@ should contain the source for the given namespace name."
631631
(fn [modules module-name]
632632
(if-not (= module-name :cljs-base)
633633
(update-in modules [module-name :depends-on]
634-
(fnil conj #{}) :cljs-base)
634+
(fnil identity #{:cljs-base}))
635635
modules))
636636
(update-in modules [:cljs-base :output-to]
637637
(fnil io/file
@@ -701,6 +701,8 @@ should contain the source for the given namespace name."
701701
(fn [[sources ret] [name {:keys [entries output-to depends-on] :as module-desc}]]
702702
(assert (or (= name :cljs-base) (not (empty? entries)))
703703
(str "Module " name " does not define any :entries"))
704+
(when (and (:verbose opts) (not= name :cljs-base))
705+
(util/debug-prn "Building module" name))
704706
(let [js-module (JSModule. (clojure.core/name name))
705707
[sources' module-sources]
706708
;; compute inputs for a closure module
@@ -723,6 +725,8 @@ should contain the source for the given namespace name."
723725
foreign-deps (atom [])]
724726
;; add inputs to module
725727
(doseq [ijs module-sources]
728+
(when (:verbose opts)
729+
(util/debug-prn " adding entry" (:provides ijs)))
726730
(if-not (deps/-foreign? ijs)
727731
(.add js-module
728732
^SourceFile (js-source-file (javascript-name ijs) ijs))
@@ -731,7 +735,10 @@ should contain the source for the given namespace name."
731735
;; since modules are already in dependency order
732736
(doseq [dep depends-on]
733737
(if-let [parent-module (get-in (into {} ret) [dep :closure-module])]
734-
(.addDependency js-module ^JSModule parent-module)
738+
(do
739+
(when (:verbose opts)
740+
(util/debug-prn " module" name "depends on" dep))
741+
(.addDependency js-module ^JSModule parent-module))
735742
(throw (IllegalArgumentException.
736743
(str "Parent module " dep " does not exist")))))
737744
[sources'
@@ -742,8 +749,12 @@ should contain the source for the given namespace name."
742749
[sources []] (sort-modules (add-cljs-base-module (:modules opts) opts)))
743750
cljs-base-closure-module (get-in (into {} modules) [:cljs-base :closure-module])
744751
foreign-deps (atom [])]
752+
(when (:verbose opts)
753+
(util/debug-prn "Building module" :cljs-base))
745754
;; add anything left to :cljs-base module
746755
(doseq [source sources']
756+
(when (:verbose opts)
757+
(util/debug-prn " adding entry" (:provides source)))
747758
(if-not (deps/-foreign? source)
748759
(.add ^JSModule cljs-base-closure-module
749760
(js-source-file (javascript-name source) source))

0 commit comments

Comments
 (0)