Fork me on GitHub
#lumo
<
2017-09-05
>
richiardiandrea18:09:53

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

anmonteiro18:09:57

maybe set :source-map true in the boot task

anmonteiro18:09:11

that’s probably a hard one to look into

richiardiandrea18:09:14

kk will try that

anmonteiro18:09:29

can you repro it against that branch?

anmonteiro18:09:38

probably has a better error message

richiardiandrea18:09:28

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

anmonteiro23:09:00

@hlolli does your PR fix #231?

anmonteiro23:09:09

I had logged that before

hlolli23:09:39

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?

hlolli23:09:59

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

anmonteiro23:09:08

that’s what I mean

anmonteiro23:09:12

but I haven’t looked at your PR yet

anmonteiro23:09:28

@hlolli btw the windows failure is totally not your fault

hlolli23:09:13

Yes saw the message, good to know.

hlolli23:09:44

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.

hlolli23:09:49

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

anmonteiro23:09:50

there’s one problem with your PR

anmonteiro23:09:08

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

anmonteiro23:09:15

^ this doesn’t work anymore

anmonteiro23:09:29

please use executeScript so we can support multiple forms

anmonteiro23:09:03

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

anmonteiro23:09:31

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

hlolli23:09:46

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

hlolli23:09:14

Thats why I made this PR in the first place

hlolli23:09:09

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

anmonteiro23:09:50

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