search

Steveorevo / node-red-contrib-nbrowser

Provides a virtual web browser (a.k.a. "headless browser") appearing as a node.
35 stars 13 forks source link

Error on run "npm i node-red-contrib-nbrowser" #8

Closed chameleonbr closed 6 years ago

chameleonbr commented 6 years ago

I've tried with and without --unsafe-perm option and received same error.

npm ERR! /usr/bin/git ls-remote -h -t git://github.com/segmentio/nightmare.git
npm ERR! 
npm ERR! fatal: Cannot change to '/root/.node-red/..': Permission denied
npm ERR! 
npm ERR! exited with error code: 128

npm ERR! A complete log of this run can be found in:
npm ERR!     /root/.npm/_logs/2018-02-01T13_14_52_816Z-debug.log
chameleonbr commented 6 years ago

If I change the dependencies all works ok.

"dependencies": {
    "nightmare": "^2.10.0",
    "nightmare-download-manager": "^0.2.4",
    "nightmare-upload": "^0.1.1",
    "unfluff": "^3.0.0"
  },
Steveorevo commented 6 years ago

Interesting. We'll need further steps to reproduce the issue; OS platform/version?

chameleonbr commented 6 years ago

I'm behind HTTP proxy, git:// not working with HTTP proxy. It is also common practice for the dependencies to be bound by the npm itself and versioned. So if someone changes the working of a library does not break your package. Look at NodeRED package.json (https://github.com/node-red/node-red/blob/master/package.json) as example.

Steveorevo commented 6 years ago

Ahh, yes I see a few packages that could conflict without a forward compatible identifier ^. Will make the change and test further.

Steveorevo commented 6 years ago

Resolved in 9463b885534358e29ce304c741cf6f568521c26f. New version 1.1.2 pushed. flows.nodered.org should update shortly.

chameleonbr commented 6 years ago

Thanks for fix. It's working now.