ibdknox/brepl

CLOJURESCRIPT_HOME

Closed this issue · 5 comments

Hi, great job!
Minor issue: I think it'd be better to drop the "export CLOJURESCRIPT_HOME" from your script, because:

  • Users have to define this anyways for cljsc to work
  • Requiring to edit your script creates some overhead when pulling your updates.

What do you think?

Regards Benjamin

I agree. If nothing else, commenting it out by default would be great and still provide documentation for folks who haven't followed all the steps on the ClojureScript wiki.

done. :)

Nice - but I guess you accidentally changed the file mode, so brepl isn't executable anymore..

whoops! Fixed.

Thank you!