Fork me on GitHub
#lumo
<
2017-04-08
>
pesterhazy19:04:57

sudo npm install -g lumo-cljs
fails for me on ubuntu

pesterhazy19:04:58

not sure why

hlolli19:04:23

@pesterhazy are you using Node7.8?

anmonteiro19:04:32

@pesterhazy does it fail consistently?

anmonteiro19:04:45

Node 7.8 shouldn't be necessary

pesterhazy19:04:59

I tried a few times and it failed each time

anmonteiro19:04:07

that's so weird

anmonteiro19:04:48

do you have node or nodejs as the command?

pesterhazy19:04:11

I do have 1.2.0 already installed there if that matters (and it works fine)

pesterhazy19:04:55

well at least as a dependency of unravel-repl...

anmonteiro19:04:57

I'm getting a failure too:

> [email protected] install /Users/anmonteiro/Desktop/svelte-playground/node_modules/lumo-cljs
> node scripts/npm_install.js


 Downloading [========================================] :rate/bps 100% 0.1s
Download failed.

anmonteiro19:04:03

is that your problem?

anmonteiro19:04:43

I think we only allow 30sec to download the Lumo binary

anmonteiro19:04:55

on slow connections this will error out maybe

anmonteiro19:04:42

^ needs to wait for the next release though

pesterhazy19:04:16

that sounds about right

pesterhazy19:04:33

I don't have a particularly slow connection though!

anmonteiro19:04:04

might be GitHub CDN throttling some downloads?

pesterhazy19:04:12

yeah it fails with 8s left to go

pesterhazy19:04:44

I'd bump it to much more, 180s or so

anmonteiro19:04:53

I can do that

pesterhazy19:04:58

otherwise installation of binaries via npm is super convenient, good job on that!

pesterhazy19:04:20

it's also great that I can just specify lumo as a dep of unravel

anmonteiro19:04:34

@pesterhazy I wonder if we can make it faster so that it doesn't have to download the binary

anmonteiro19:04:57

at the cost of space (having all 3 binaries in the NPM package)

anmonteiro19:04:13

I think that's how Flowtype and Bucklescript do their NPM installation but I need to do further research

dominicm20:04:23

Why have a timeout at all?

dominicm20:04:29

Guessing connections is hard

anmonteiro21:04:06

that's a good point

anmonteiro21:04:11

I haven't actually written the current code using request

anmonteiro21:04:28

I wonder if we can just omit the timeout handling

anmonteiro21:04:06

❤️ negative diffs

anmonteiro22:04:26

@pesterhazy that'll be in the next release

anmonteiro22:04:10

I've been working more on Lumo the past few days / weeks so 1.4 should be out sooner than it took to release 1.3

anmonteiro22:04:39

If you haven't been following closely, I added pretty printing through Fipp a couple days ago 🙂

pesterhazy22:04:27

That's great

pesterhazy22:04:44

I might steal that for unravel :)