lumo

:lumo: Standalone ClojureScript environment. Currently at version 1.9.0
futuro 2017-07-09T17:44:49.840658Z

I'm curious if anyone can repro the following issue with lumo 1.6.0

futuro 2017-07-09T17:44:57.841244Z

one sec, that was the wrong error

futuro 2017-07-09T17:46:09.847319Z

cljs.user=> (js/require "fs")
Cannot read property 'cljs$lang$ctorStr' of undefined
         Object.cljs.core.pr_writer_impl (NO_SOURCE_FILE <embedded>:1703:280)
         cljs.core.pr_writer (NO_SOURCE_FILE <embedded>:1705:415)
         (NO_SOURCE_FILE <embedded>:1725:340)
         Object.cljs.core.pr_sequential_writer (NO_SOURCE_FILE <embedded>:1692:265)
         Object.cljs.core.print_prefix_map (NO_SOURCE_FILE <embedded>:1725:65)
         Object.cljs.core.print_map (NO_SOURCE_FILE <embedded>:1726:359)
         Object.cljs.core.pr_writer_impl (NO_SOURCE_FILE <embedded>:1700:126)
         Object.cljs.core.pr_writer (NO_SOURCE_FILE <embedded>:1705:415)
         Object.cljs.core.pr_seq_writer (NO_SOURCE_FILE <embedded>:1706:51)
         Object.cljs.core.pr_sb_with_opts (NO_SOURCE_FILE <embedded>:1707:156)

cljs.user=>

futuro 2017-07-09T17:47:48.854927Z

if you (def fs (js/require "fs")) you can use it

2017-07-09T17:48:52.860023Z

maybe it's the pretty-printer that's failing

futuro 2017-07-09T17:49:12.861690Z

(js/console.log (js/require "fs")) works

futuro 2017-07-09T17:49:16.862027Z

yeah, that's my assumption

futuro 2017-07-09T17:50:27.868126Z

(.. js/Object (entries fs) (forEach (fn [[key value] index arr] (println value))))

futuro 2017-07-09T17:50:39.869155Z

that throws the same error

futuro 2017-07-09T17:50:43.869458Z

println with the keys works

futuro 2017-07-09T17:51:03.871015Z

and (js/console.log value) works

anmonteiro 2017-07-09T17:51:25.872797Z

Probably a printing error

anmonteiro 2017-07-09T17:51:30.873232Z

Can you file an issue?

futuro 2017-07-09T17:51:34.873526Z

yeah

anmonteiro 2017-07-09T17:52:19.877231Z

Btw: ClojureScript got a cljs.core/*command-line-args* variable today

👍 2
anmonteiro 2017-07-09T17:52:43.879039Z

Going forward we'll be using this instead of lumo.core's

anmonteiro 2017-07-09T17:52:59.880377Z

This will probably be a breaking change

futuro 2017-07-09T17:53:07.881041Z

Neat!

futuro 2017-07-09T17:53:11.881374Z

I'm glad that's part of core

futuro 2017-07-09T17:54:44.888839Z

Issue submitted, please let me know if there's any other information you'd like