Skip to content
This repository has been archived by the owner on Aug 24, 2021. It is now read-only.

Fixed README.md with libusb dev headers + correct nodejs version #322

Merged
merged 2 commits into from
Nov 11, 2019

Conversation

Someguy123
Copy link
Contributor

Possibly fixes issue #313 and fixes issue #290

I had issues trying to run npm install on Ubuntu 18.04 Server, and after some trial and error I managed to get it working with the following workarounds:

  • You MUST use an older NodeJS version such as v6.17.1 (last LTS for v6), as the current LTS pf 12.13.0 does NOT work. (this is why NVM exists)
    • nvm install v6.17.1
  • You MUST install node-gyp before running npm install, otherwise it will fail to compile some of the dependencies and the install will error.
    • npm install node-gyp
  • At least on Debian/Ubuntu, you MUST install libusb-1.0.0-dev or it will error saying it can't find libusb.h
    • apt install -y libusb-1.0-0-dev

I've updated the README.md and the readme in dapp/README.md to make these installation requirements clear, and save others having to figure out why it's broken.

@denisgranha
Copy link
Contributor

Thank you @Someguy123 you are right!

@denisgranha denisgranha merged commit 79d1c6c into gnosis:develop Nov 11, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants