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

installing plugins via signalk seems broken #253

Closed
mgrouch opened this issue Oct 13, 2022 · 8 comments
Closed

installing plugins via signalk seems broken #253

mgrouch opened this issue Oct 13, 2022 · 8 comments
Labels
bug Something isn't working bullseye Bullseye has workaround

Comments

@mgrouch
Copy link

mgrouch commented Oct 13, 2022

installing plugins via signalk seems broken

@mgrouch mgrouch added bug Something isn't working bullseye Bullseye labels Oct 13, 2022
@mgrouch mgrouch changed the title installing puligins via signalk seems broken installing plugins via signalk seems broken Oct 13, 2022
@mgrouch
Copy link
Author

mgrouch commented Oct 13, 2022




4797 timing idealTree Completed in 171915ms
4798 timing command:install Completed in 172119ms
4799 verbose stack TypeError: Cannot read properties of null (reading 'matches')
4799 verbose stack     at Link.matches (/usr/lib/node_modules/npm/node_modules/@npmcli/arborist/lib/node.js:1114:41)
4799 verbose stack     at Link.canDedupe (/usr/lib/node_modules/npm/node_modules/@npmcli/arborist/lib/node.js:1068:15)
4799 verbose stack     at PlaceDep.pruneDedupable (/usr/lib/node_modules/npm/node_modules/@npmcli/arborist/lib/place-dep.js:468:14)
4799 verbose stack     at PlaceDep.placeInTree (/usr/lib/node_modules/npm/node_modules/@npmcli/arborist/lib/place-dep.js:329:14)
4799 verbose stack     at PlaceDep.place (/usr/lib/node_modules/npm/node_modules/@npmcli/arborist/lib/place-dep.js:216:10)
4799 verbose stack     at new PlaceDep (/usr/lib/node_modules/npm/node_modules/@npmcli/arborist/lib/place-dep.js:73:10)
4799 verbose stack     at PlaceDep.placeInTree (/usr/lib/node_modules/npm/node_modules/@npmcli/arborist/lib/place-dep.js:365:26)
4799 verbose stack     at PlaceDep.place (/usr/lib/node_modules/npm/node_modules/@npmcli/arborist/lib/place-dep.js:216:10)
4799 verbose stack     at new PlaceDep (/usr/lib/node_modules/npm/node_modules/@npmcli/arborist/lib/place-dep.js:73:10)
4799 verbose stack     at /usr/lib/node_modules/npm/node_modules/@npmcli/arborist/lib/arborist/build-ideal-tree.js:944:31
4800 verbose cwd /home/signalk/.signalk
4801 verbose Linux 5.15.61-v8+
4802 verbose node v16.17.1
4803 verbose npm  v8.19.2
4804 error Cannot read properties of null (reading 'matches')
4805 verbose exit 1

@mgrouch
Copy link
Author

mgrouch commented Oct 13, 2022

See also: npm/cli#4367

@mgrouch
Copy link
Author

mgrouch commented Oct 13, 2022

Something like this should have worked from command line. (but it doesn't due to other dependencies issues)


cd /home/signalk/.signalk
sudo su signalk -c "rm ../.signalk/pnpm-lock.yaml; pnpm config set auto-install-peers true"
sudo su signalk -c "pnpm install \
                signalk-n2k-switching-emulator \
                --unsafe-perm"

@mgrouch
Copy link
Author

mgrouch commented Oct 13, 2022

it worked if I run this:


cd /home/signalk/.signalk

sudo su signalk -c "rm ../.signalk/pnpm-lock.yaml; pnpm config set auto-install-peers true"

sudo su signalk -c "pnpm add --save-peer tslib@^1.10.0"

sudo su signalk -c "pnpm install \
                signalk-n2k-switching-emulator \
                --unsafe-perm"

@mgrouch
Copy link
Author

mgrouch commented Oct 13, 2022

Linked request: SignalK/signalk-server#1484

@mgrouch
Copy link
Author

mgrouch commented Oct 13, 2022

another example:


cd /home/signalk/.signalk
sudo su signalk -c "rm ../.signalk/pnpm-lock.yaml; pnpm config set auto-install-peers true"

sudo su signalk -c "pnpm install signalk-from-batch-format --unsafe-perm --save-peer"

@mgrouch
Copy link
Author

mgrouch commented Oct 13, 2022

for signalk-hwt901b-imu:

cd /home/signalk/.signalk
sudo su signalk -c "rm ../.signalk/pnpm-lock.yaml; pnpm config set auto-install-peers true"

sudo su signalk -c "pnpm install signalk-hwt901b-imu --unsafe-perm --save-peer"


@mgrouch
Copy link
Author

mgrouch commented Oct 14, 2022

BBN Marine OS version 2022-10-14 and up fixes this issue.

@mgrouch mgrouch closed this as completed Oct 14, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working bullseye Bullseye has workaround
Projects
None yet
Development

No branches or pull requests

1 participant