This page is not created by, affiliated with, or supported by Slack Technologies, Inc.
2017-04-14
Channels
- # beginners (53)
- # boot (93)
- # cider (13)
- # cljs-dev (17)
- # cljsrn (20)
- # clojars (1)
- # clojure (349)
- # clojure-austin (1)
- # clojure-gamedev (5)
- # clojure-italy (1)
- # clojure-nl (16)
- # clojure-poland (1)
- # clojure-russia (26)
- # clojure-spec (57)
- # clojure-uk (6)
- # clojurebridge (5)
- # clojurescript (145)
- # code-reviews (2)
- # core-async (88)
- # cursive (1)
- # datomic (3)
- # defnpodcast (10)
- # events (7)
- # hoplon (20)
- # instaparse (1)
- # jobs-discuss (15)
- # keechma (26)
- # lein-figwheel (2)
- # leiningen (1)
- # liberator (11)
- # lumo (40)
- # off-topic (54)
- # om (32)
- # onyx (11)
- # pedestal (6)
- # perun (4)
- # planck (6)
- # re-frame (4)
- # reagent (12)
- # ring (3)
- # ring-swagger (10)
- # rum (1)
- # testing (4)
- # timbre (1)
- # unrepl (20)
- # untangled (111)
- # vim (1)
Seems to be a general reader error, being too eager to evaluate a regex. You can tickle this on the clojure side as well 馃檪 by putting a regex that is unacceptable to java.util.regex.Pattern
in the :cljs regex e.g. \( without a matching close \)
@gmercer Regexes behave differently in the JVM and JS engines
just talked with @martinklepsch, he had a problem with installing lumo over npm (by way of unravel)
strangely he got a bin/lumo
which invoked lumo.exe
so I'm guessing there he got the file for windows, even though he's on osx
we don鈥檛 mess with lein
anywhere
uninstalling and reinstalling fixed it, so maybe some indeterministic problem?
I meant lumo
, not lein
my brain stores words as "4-letter word starting with l"
anyway nothing major, but I suspect there's something that can go wrong in the download script
that worries me
https://github.com/anmonteiro/lumo/blob/master/packages/lumo/scripts/npm_install.js#L7-L17
here鈥檚 that logic
do you see anything wrong with it?
so bin/lumo
defaults to the windows version
and gets overwritten by the platform-dependant one
so if for some reason the dl fails, you're stuck with https://github.com/anmonteiro/lumo/blob/master/packages/lumo/bin/lumo
@pesterhazy that makes sense
so we could write bin/lumo
(not lein
馃槀 ) as part of the installation script too
only for the Windows case, and if the download succeeds
PR welcome. if you don鈥檛 have time just open an issue, please
ok I will
is npm ok with the bin
script not existing in the source?
I don鈥檛 know
I think so
don't think I'll manage a PR as I don't have a windows box to test on
you can try it with npm install .
inside the directory
but will make an attempt
thanks
I鈥檒l make sure to get to that before the next release
@anmonteiro I grokked the different regex languages per platform I was wondering why the reader was eager on resolving the regex, maybe it is difficult to skip over the characters in order to find all the :clj, :cljs or :cljr cases