cljsjs

pez 2017-02-26T16:28:50.001131Z

That package fails to build in CircleCI. https://circleci.com/gh/cljsjs/packages/1670#tests/containers/0 complaining:

Cannot run program "gulp" (in directory "/home/ubuntu/.boot/cache/tmp/home/ubuntu/packages/simplex-solver/g3a/873hv6/simplex-solver-8e55b56d499dc7d64ac22a5051f2eeb4a22a2534"): error=2, No such file or directory

juhoteperi 2017-02-26T16:32:06.001133Z

You need to install Gulp and use path to node_modules/bin

pez 2017-02-26T16:32:29.001134Z

It builds fine on my own machine. Which file is missing in that error message? The directory or gulp? The previous output suggests gulp installs as it should with npm install. Which makes the error message even stranger. The relevant part of the boot script is:

(binding [*sh-dir* (str (io/file tmp-lib))]
        ((sh "npm" "install"))
        ((sh "gulp" “js-lib”)))

juhoteperi 2017-02-26T16:32:42.001135Z

You have gulp installed globally on your machine

juhoteperi 2017-02-26T16:33:17.001136Z

npm install doesn't install the binary to a directory in PATH

pez 2017-02-26T16:34:30.001137Z

Hmmm, I am a newbie on these things, but without npm install it fails on my own machine complaining about it not being installed locally.

juhoteperi 2017-02-26T16:35:05.001140Z

you have probably installed gulp with npm install -g gulp which installs the gulp to /usr/local/bin or such

pez 2017-02-26T16:36:14.001144Z

Indeed I have. I’ll uninstall it and try locally again. What should I do in the boot script to install it correctly, if you happen to know?

juhoteperi 2017-02-26T16:37:31.001145Z

The installation is correct now, you just need to need to refer to the binary with proper path: ((sh "./node_modules/.bin/gulp"))

pez 2017-02-26T16:38:04.001146Z

Thanks!

pez 2017-02-26T16:38:32.001147Z

I just confirmed I got the same error message on my machine w/o gulp installed globally, btw.

pez 2017-02-26T16:50:18.001150Z

All checks has passed. Lots of thanks, @juhoteperi!

juhoteperi 2017-02-26T16:51:34.001154Z

@pez ^

pez 2017-02-26T16:52:04.001155Z

Thanks.

juhoteperi 2017-02-26T16:52:19.001156Z

The important part is to use +lib-version+ instead of commit sha

pez 2017-02-26T16:57:36.001161Z

@juhoteperi: I did try to use the +lib-versioin+ zip file, but there was none available...

juhoteperi 2017-02-26T16:58:10.001163Z

@pez It is available, but you need to prepend v before the number

juhoteperi 2017-02-26T16:58:43.001165Z

https://github.com/SamDuvall/simplex-solver/tags

pez 2017-02-26T16:59:11.001167Z

Ah, great. I’ll fix it.

pez 2017-02-26T17:02:26.001169Z

What is the correct way of resubmitting the pull request? Close it and make a new one?

juhoteperi 2017-02-26T18:28:31.001170Z

@pez Just push commits into the same branch you created PR from. The PR will be updated.

👍 1