Skip to content
This repository has been archived by the owner on Feb 15, 2022. It is now read-only.

Forex.Analytics is breaking docker builds #423

Closed
cmroche opened this issue Jul 29, 2017 · 1 comment
Closed

Forex.Analytics is breaking docker builds #423

cmroche opened this issue Jul 29, 2017 · 1 comment

Comments

@cmroche
Copy link
Contributor

cmroche commented Jul 29, 2017

On both Windows and OSX (testing with Docker Toolbox and Docker for Windows). Any attempt to run docker-compose build results in the following error with forex.analytics

npm WARN zenbot4@4.0.5 No repository field.
npm WARN zenbot4@4.0.5 No license field.
npm ERR! Linux 4.9.36-moby
npm ERR! argv "/usr/local/bin/node" "/usr/local/bin/npm" "install"
npm ERR! node v6.11.1
npm ERR! npm v3.10.10
npm ERR! code ELIFECYCLE

npm ERR! forex.analytics@0.0.14 postinstall: node-gyp configure && node-gyp build
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the forex.analytics@0.0.14 postinstall script 'node-gyp configure && node-gyp build'.
npm ERR! Make sure you have the latest version of node.js and npm installed.
npm ERR! If you do, this is most likely a problem with the forex.analytics package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR! node-gyp configure && node-gyp build
npm ERR! You can get information on how to open an issue for this project with:
npm ERR! npm bugs forex.analytics
npm ERR! Or if that isn't available, you can get their info via:
npm ERR! npm owner ls forex.analytics
npm ERR! There is likely additional logging output above.

npm ERR! Please include the following file with any support request:
npm ERR! /app/npm-debug.log
ERROR: Service 'server' failed to build: The command '/bin/sh -c npm install -g node-gyp && npm install' returned a non-zero code: 1

Also note the node-gyp is missing from packages currently, you'll need to run npm install node-gyp --save to get this far.

@cmroche
Copy link
Contributor Author

cmroche commented Jul 29, 2017

There are two issues preventing forex.analytics from building in the docker image.

  1. The node-gyp package was missing, this was added to the dockerfile since the forex.analytics github page wants it installed globally and not locally.

  2. The --unsafe-perm option must now be passed to npm install since docker runs the dockerfile instructions as root and npm won't otherwise run the preinstall scripts for forex.analytics with the correct permissions (root).

Pull request: #424

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant