lumo

:lumo: Standalone ClojureScript environment. Currently at version 1.9.0
richiardiandrea 2017-09-05T18:18:53.000157Z

is there a way to build lumo with source maps on for the compiler part? I am trying to debug #239

anmonteiro 2017-09-05T18:19:42.000441Z

uh

anmonteiro 2017-09-05T18:19:57.000414Z

maybe set :source-map true in the boot task

anmonteiro 2017-09-05T18:20:01.000198Z

dunno

anmonteiro 2017-09-05T18:20:11.000281Z

that’s probably a hard one to look into

richiardiandrea 2017-09-05T18:20:14.000327Z

kk will try that

anmonteiro 2017-09-05T18:20:22.000122Z

https://github.com/anmonteiro/lumo/tree/compiler-tweaks

anmonteiro 2017-09-05T18:20:29.000136Z

can you repro it against that branch?

anmonteiro 2017-09-05T18:20:38.000445Z

probably has a better error message

richiardiandrea 2017-09-05T18:20:43.000758Z

will do

richiardiandrea 2017-09-05T18:21:28.000308Z

I'll report on the issue, if we solve that, I will be a very happy fella 😄

anmonteiro 2017-09-05T23:23:00.000019Z

@hlolli does your PR fix #231?

anmonteiro 2017-09-05T23:23:04.000017Z

https://github.com/anmonteiro/lumo/issues/231

anmonteiro 2017-09-05T23:23:09.000075Z

I had logged that before

2017-09-05T23:25:39.000310Z

Depends what is ment by that, stream is as far as i can see a file that is not saved. Or did you mean something else?

2017-09-05T23:26:59.000042Z

Id say the hard part would be cat ing many files and see if lumo can handle that, tought never attempted.

anmonteiro 2017-09-05T23:29:08.000258Z

that’s what I mean

anmonteiro 2017-09-05T23:29:12.000161Z

but I haven’t looked at your PR yet

anmonteiro 2017-09-05T23:29:28.000168Z

@hlolli btw the windows failure is totally not your fault

anmonteiro 2017-09-05T23:29:33.000143Z

I’m fixing it in https://github.com/anmonteiro/lumo/pull/253

2017-09-05T23:30:13.000010Z

Yes saw the message, good to know.

2017-09-05T23:33:44.000089Z

I may want to lool a bit better at the question stdin ad file. If its a question of cleverly give lumo one file stream that understands many files and namespaces or would we want to extend lumo to umderstand more before its opened etc. The limitation is that it can only be one continuous stream per pipe.

2017-09-05T23:37:49.000054Z

And since the return value is always printed in clojure the stdout is given, for the build api its not, that maybe a different more long term work there. Building js to stdout would open up lumo to gulp pipelines

anmonteiro 2017-09-05T23:39:46.000138Z

I see

anmonteiro 2017-09-05T23:39:50.000180Z

there’s one problem with your PR

anmonteiro 2017-09-05T23:40:08.000187Z

@hlolli echo '(println "Hello World") (println "foo")' | lumo ...

anmonteiro 2017-09-05T23:40:15.000017Z

^ this doesn’t work anymore

anmonteiro 2017-09-05T23:40:29.000315Z

please use executeScript so we can support multiple forms

anmonteiro 2017-09-05T23:41:03.000169Z

I’m OK using the buffer approach, but I’m still getting the EOF errors

anmonteiro 2017-09-05T23:41:31.000240Z

so as far as I can tell your PR doesn’t change anything at all

2017-09-05T23:41:46.000025Z

What, lol, im at my phone right now, but its exacly the other wat around on my computer. With string I was getting EOF errors

2017-09-05T23:42:14.000223Z

Thats why I made this PR in the first place

2017-09-05T23:45:09.000020Z

Youre on machintosh? At work Ill give my pr a try on a mac at my workplace. So if youre getting EOF then maybe I need to find a different way to solve this

anmonteiro 2017-09-05T23:48:33.000108Z

ok

anmonteiro 2017-09-05T23:59:50.000008Z

I also can’t remember what I meant by treat stdin as file