Internet-monitor module Not Starting Mirror

[sdetweil]

@ebourne which authors module are you using
show

git remote -v

[ebourne]

@sdetweil brianhepler

[KristjanESPERANTO]

I also happened to be working on the problem just now. The speedtest-net package is apparently no longer compatible with the current node version. I created a PR to fix this: https://github.com/BrianHepler/internet-monitor/pull/2

Let’s see when @bhepler merge it.

[sdetweil]

@KristjanESPERANTO not node version, electron version

so, the steps to fix I think (tested, failed before, doesn’t fail now)

#!/bin/bash
cd ~/MagicMirror
# this tool should be installed in MM
if [ ! -e node_modules/.bin/electron-rebuild ]; then
    npm install electron/rebuild 
fi
cd modules/internet-monitor
../../node_modules/.bin/electron-rebuild
cd ~/MagicMirror

this should be a postinstall step in the package.json

this is the tested postinstall
and works…

and the fixed package.json

{
  "name": "internet-speed",
  "version": "1.0.0",
  "description": "Displays ping, download and upload speed on your smart mirror.",
  "main": "index.js",
  "scripts": {
    "test": "echo \"Error: no test specified\" && exit 1",
    "postinstall":"./postinstall"
  },
  "keywords": [
    "internet",
    "speed"
  ],
  "author": "Debayan Deb",
  "license": "MIT",
  "dependencies": {
    "justgage": "^1.3.5",
    "round-to": "^1.1.0",
    "speedtest-net": "^2.2.0",
    "raphael": "^2.2.8"
  }
}

have to make postinstall executable

chmod +x postinstall

[KristjanESPERANTO]

@sdetweil Interesting solution! I’m still not quite getting to grips with the rebuild of electron.

My approach in the PR is to use a version of speedtest-net that is compatible with the current version. That way, no rebuild is necessary.

[sdetweil]

@KristjanESPERANTO its not a rebuild of electron … its rebuilding binaries for the current electron version

[KristjanESPERANTO]

@KristjanESPERANTO said in Internet-monitor module Not Starting Mirror:

I also happened to be working on the problem just now. The speedtest-net package is apparently no longer compatible with the current node version. I created a PR to fix this: https://github.com/BrianHepler/internet-monitor/pull/2

Let’s see when @bhepler merge it.

@ebourne The maintainer of the module, @bhepler, accepted my Pull Request. Please update your module and check if it’s working 🙂

[ebourne]

@KristjanESPERANTO just wanna say thanks
it works