After update 2.23.0 crash

swvalenti

@swvalenti said in After update 2.23.0 crash:

@JerryP said in After update 2.23.0 crash:

pi@raspberrypi:~/MagicMirror $ ls
 CHANGELOG.md   Collaboration.md   core  'DISPLAY=:0'   index.html   jest.config.js   jsconfig.json   magicmirror@2.23.0   module-types.ts   oud         run-start.sh   splashscreen   th.json        vendor
 clientonly     config             css    fonts         installers   js               LICENSE.md      modules              node_modules      README.md   serveronly     tests          translations
pi@raspberrypi:~/MagicMirror $ npm install
npm ERR! code ENOTEMPTY
npm ERR! syscall rename
npm ERR! path /home/pi/MagicMirror/node_modules/acorn
npm ERR! dest /home/pi/MagicMirror/node_modules/.acorn-pnTZ3fig
npm ERR! errno -39
npm ERR! ENOTEMPTY: directory not empty, rename '/home/pi/MagicMirror/node_modules/acorn' -> '/home/pi/MagicMirror/node_modules/.acorn-pnTZ3fig'

npm ERR! A complete log of this run can be found in:
npm ERR!     /home/pi/.npm/_logs/2023-04-04T20_43_20_803Z-debug-0.log

That didn’t work, no worries I will revert

The package.json showed no issue with that command. It worked when I reverted no issue.

sdetweil

@swvalenti how did you revert?

swvalenti

@sdetweil said in After update 2.23.0 crash:

@swvalenti how did you revert?

Copied original file and renamed before using your script.

sdetweil

@swvalenti got it…

swvalenti

@sdetweil said in After update 2.23.0 crash:

@swvalenti got it…

Thought maybe the script would catch it. Ha. Oh well thanks for all your help and work.

sdetweil

@swvalenti the script DID catch for request, but not jsdom…

these are the undocumented libs used

OWGR.js:const request = require('request');
OWGR.js:const jsdom = require("jsdom");

I just added that to the upgrade script list of missing libs

swvalenti

@sdetweil said in After update 2.23.0 crash:

@swvalenti the script DID catch for request, but not jsdom…

these are the undocumented libs used
OWGR.js:const request = require('request');
OWGR.js:const jsdom = require("jsdom");
I just added that to the upgrade script list of missing libs

Cool so if I update it will work?

sdetweil

@swvalenti should…

swvalenti

@sdetweil said in After update 2.23.0 crash:

@swvalenti should…

Thanks Sam I’ll report back

sdetweil

@swvalenti nope it wont work as it is… looks only in the node_helper… oops…

working on fix…

pushed a fix