Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Leiningen hook does not work with lein trampoline cljsbuild auto? #1

Open
guns opened this issue Dec 20, 2013 · 4 comments
Open

Leiningen hook does not work with lein trampoline cljsbuild auto? #1

guns opened this issue Dec 20, 2013 · 4 comments

Comments

@guns
Copy link

guns commented Dec 20, 2013

The documentation states:

you can automatically run lein-dalap when you are running cljsbuild auto or once command, just by adding a :hooks to your project.clj file

Unfortunately, adding :hooks [leiningen.dalap] only seems to trigger lein-dalap once on start when running cljsbuild auto. Subsequent file changes trigger cljsbuild without dalap transformations.

Am I doing something wrong? It would be quite nice to avoid running yet another JVM for a lein plugin.

Thank you very much for lein-dalap; Clojure by default is really the sanest way to do a multi-platform preprocessor.

@tavisrudd
Copy link
Member

@guns sorry for the delayed response - had notifications turned off. What version of cljsbuild is this with? It should be re-running each time auto detects change.

@guns
Copy link
Author

guns commented Feb 14, 2014

What version of cljsbuild is this with?

lein-cljsbuild 1.0.0.

https://github.com/clojure/tools.cli/blob/master/project.clj

I just re-tried running cljsbuild auto without a trampoline and it works beautifully! I typically run lein processes in a trampoline, so it seems I never attempted running without one.

So perhaps this is not a bug, but then I don't think I understand why running in a trampoline causes this problem…

@tavisrudd
Copy link
Member

Hmm, I've never tried it with trampoline. Does lein trampoline cljsbuild auto work with normal cljs file?

@guns
Copy link
Author

guns commented Feb 14, 2014

Yes, lein trampoline cljsbuild auto works when just auto-compiling a cljs file.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants