From 367f99c9e4286a4a42468ec22bf12a472a392bdb Mon Sep 17 00:00:00 2001 From: Someguy123 Date: Wed, 6 Nov 2019 04:48:36 +0000 Subject: [PATCH 1/2] Fix README.md with note about libusb and node version --- README.md | 14 ++++++++++++++ 1 file changed, 14 insertions(+) diff --git a/README.md b/README.md index 7ceab372..a496abb7 100644 --- a/README.md +++ b/README.md @@ -5,6 +5,8 @@ Ethereum Multisignature Wallet The purpose of multisig wallets is to increase security by requiring multiple parties to agree on transactions before execution. Transactions can be executed only when confirmed by a predefined number of owners. A web user interface can be found [here](/dapp). +**NOTE:** Not compatible with current NodeJS LTS. Recommended NodeJS version is v6.17.1 (last LTS for v6). + Features ------------- @@ -30,8 +32,20 @@ Being used by Install ------------- ``` +# For Ubuntu/Debian you need to install libusb development headers +apt install -y libusb-1.0-0-dev + git clone https://github.com/gnosis/MultiSigWallet.git cd MultiSigWallet + +# Latest NodeJS (v12.13.0) does NOT appear to work correctly. +# You should use NVM and install Node v6.17.1 for best results: https://github.com/nvm-sh/nvm +# Tested by @Privex on 2019-Nov-06 with v6.17.1 with success +nvm install v6.17.1 + +# node-gyp is required for 'npm install' to work correctly +npm install node-gyp + npm install ``` From a78c3bd5a9735311836cd3f7620cf1de562d3068 Mon Sep 17 00:00:00 2001 From: Someguy123 Date: Wed, 6 Nov 2019 04:50:47 +0000 Subject: [PATCH 2/2] Update dapp/README.md with libusb & node ver --- dapp/Readme.md | 18 ++++++++++++++++-- 1 file changed, 16 insertions(+), 2 deletions(-) diff --git a/dapp/Readme.md b/dapp/Readme.md index 112d426c..7ec58652 100644 --- a/dapp/Readme.md +++ b/dapp/Readme.md @@ -5,12 +5,26 @@ A web user interface for the [MultiSigWallet](https://github.com/gnosis/MultiSig Requirements ------------- -* Node v5+ +* Node v5+ (Confirmed working on v6.17.1 - will not work on current LTS v12.13.0) * [Grunt-cli](http://gruntjs.com/getting-started#installing-the-cli) Install ------------- ``` +# For Ubuntu/Debian you need to install libusb development headers +apt install -y libusb-1.0-0-dev + +# Latest NodeJS (v12.13.0) does NOT appear to work correctly. +# You should use NVM and install Node v6.17.1 for best results: https://github.com/nvm-sh/nvm +# Tested by @Privex on 2019-Nov-06 with v6.17.1 with success +nvm install v6.17.1 + +git clone https://github.com/gnosis/MultiSigWallet.git +cd MultiSigWallet/dapp + +# node-gyp is required for 'npm install' to work correctly +npm install node-gyp + npm install -g grunt-cli npm install grunt @@ -35,4 +49,4 @@ Desktop version ``` npm run build-libs-electron -``` \ No newline at end of file +```