Fork me on GitHub
#cljsjs
<
2017-02-26
>
pez16:02:50

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

juhoteperi16:02:06

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

pez16:02:29

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”)))

juhoteperi16:02:42

You have gulp installed globally on your machine

juhoteperi16:02:17

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

pez16:02:30

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.

juhoteperi16:02:05

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

pez16:02:14

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?

juhoteperi16:02:31

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

pez16:02:32

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

pez16:02:18

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

juhoteperi16:02:19

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

pez16:02:36

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

juhoteperi16:02:10

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

pez16:02:11

Ah, great. I’ll fix it.

pez17:02:26

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

juhoteperi18:02:31

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