emezeske / lein-cljsbuild

Leiningen plugin to make ClojureScript development easy.
Other
1.1k stars 152 forks source link

Run shutdown-agents after a cljsbuild once if :notify-command is used #453

Closed danielcompton closed 7 years ago

danielcompton commented 7 years ago

:notify-command uses a future through cljsbuild.util/process-start. If a notify-command is run during a cljsbuild once run, then the cljsbuild process will stay running for another minute until the agents are shut down.

You can reproduce this on the Reagent project by running

$ lein with-profile prod-test do clean, cljsbuild once
miikka commented 7 years ago

lein cljsbuild test has the same problem, because the test commands are run with cljsbuild.util/process-start as well.