Fork me on GitHub
#lumo
<
2017-09-04
>
hlolli18:09:06

Seems as if I'm not getting the command line args anymore, I'm running 1.7 and the changelog mentioned this as a bug. I print cljs.core/*command-line-args* and get nil, from (.-argv js/process) I get #js ["/usr/lib/node_modules/lumo-cljs/bin/lumo" "nexe.js" "/tmp/csound-QrUcal.ext" "/tmp/csound-gI6bZL.sco"] where those .ext and .sco files are the command-line-args I'd expect to see, maybe I'm doing something wrong?

richiardiandrea19:09:27

@hlolli you need master there is a patch for that

hlolli19:09:54

ah ok, reading the changelog more carefully I see this is not fixed in the release. Ok thanks for that pointer @richiardiandrea

anmonteiro19:09:21

I'll release a patch version soon

anmonteiro19:09:23

With the fix

anmonteiro19:09:48

Totally my fault 馃檲

hlolli20:09:08

echo "(println \"Hello World\")" | lumo - also doesn't work anymore... hm

hlolli20:09:10

I'll look at it in case we can fit that into the patch fix

hlolli21:09:26

running boot dev gives me java.io.FileNotFoundException: Could not locate clojure/tools/reader__init.class or clojure/tools/reader.clj on classpath. hmm

hlolli21:09:39

ah it's a boot problem or something, the jar file in my maven directory for this specific tools.reader version is 0bytes.

anmonteiro23:09:44

@hlolli thanks for your PR

anmonteiro23:09:52

can you fix the lint issues and delete the yarn.lock changes?

hlolli23:09:02

ah yes, took the intentation helpers off, at my workplace I lost the fight of 2 spaces in js files for 4 spaces (they better get me wider monitor 馃檪 ), so I didnt pay attention to linting.

anmonteiro23:09:15

@hlolli I think it鈥檚 not an indentation problem

anmonteiro23:09:24

also just run yarn prettier after you鈥檙e done and it鈥檒l fix it for you

anmonteiro23:09:42

or you could just add prettier on save in emacs like I have

hlolli23:09:31

hmm good idea, well, in my case I'd do it manually, due to conflict of my taste for my own project and company's taste. I already start using dir locals more for that. Anyway, I fix it.