Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Minorly upgrading project to run with recent node, slight dep change,… #120

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Pysis868
Copy link

… and integration explicitly turned on.

Eliminates primordials and boot stuck errors.

… and integration explicitly turned on.

Eliminates primordials and boot stuck errors.
@Pysis868
Copy link
Author

Pysis868 commented Jun 23, 2020

My workflow:

Had node v10...
Installed new node on Windows via msi.

> npm i
Was getting primordials error this way.
Had to clean original node installation dir and repair install.

> npm cache clean --force

> rm -rf node_modules package-lock.json

> node -v
v14.4.0

> npm -v
6.14.5

> electron .

App threw an error during load
ReferenceError: primordials is not defined
at fs.js:27:26
at req_ (F:\Mirror\D\JStuff\Games\Consoles\Wii U\Emulators and Files\CemUI\node_modules\natives\index.js:143:24)
at Object.req [as require] (F:\Mirror\D\JStuff\Games\Consoles\Wii U\Emulators and Files\CemUI\node_modules\natives\index.js:55:10)
at Object. (F:\Mirror\D\JStuff\Games\Consoles\Wii U\Emulators and Files\CemUI\node_modules\fstream\node_modules\graceful-fs\fs.js:1:37)

> npm ls 'graceful-fs' | grep 'graceful-fs' | sed -r 's|^\W*||g' | cut -d' ' -f1 | sort | uniq
graceful-fs@3.0.12
graceful-fs@4.2.4

> read query; npm ls "$query"
read> graceful-fs@3.0.12
cemui@2.3.3 F:\Mirror\D\JStuff\Games\Consoles\Wii U\Emulators and Files\CemUI
`-- unzip@0.1.11
`-- fstream@0.1.31
`-- graceful-fs@3.0.12

mafintosh/prebuildify-ci#5 (comment)

> npm remove unzip

> npm add unzipper

> grep -ER --exclude-dir='node_modules' "require.*?unzip" .
./app.js: unzip = require('unzip'),

.../app/wrapper/index.html
Uncaught ReferenceError: require is not defined
at main.js:1

https://stackoverflow.com/a/55908510/1091943
Needs security / more upgrading: https://stackoverflow.com/a/59888788/1091943

@Pysis868
Copy link
Author

Gets most of the way booting now. Probably still more errors like after the wizard going back to manage cemu installs, maybe even the first time since it seems mine was not saved.

@Arnauw
Copy link

Arnauw commented Jun 30, 2020

Yo, very nice of you updating this 👍.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants