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

not found: npm on zowe plugins install #1615

Closed
t1m0thyj opened this issue Jan 27, 2023 Discussed in #1613 · 0 comments · Fixed by zowe/imperative#932
Closed

not found: npm on zowe plugins install #1615

t1m0thyj opened this issue Jan 27, 2023 Discussed in #1613 · 0 comments · Fixed by zowe/imperative#932
Labels
priority-medium Not functioning - next quarter if capacity permits severity-high Bug for which there may be workaround but limits the usage of the Zowe for major use cases

Comments

@t1m0thyj
Copy link
Member

Discussed in #1613

Originally posted by dkelosky January 24, 2023
Hi - a user is getting this error "not found: npm" when running zowe plugins install ...; however, as you can see below, npm version is 9.3.1. He's using zowe 7.9.7

C:\>zowe plugins install @broadcom/ops-for-zowe-cli
Handler Instantiation Failed:
Could not instantiate the handler C:\Users\gl891170\AppData\Roaming\npm\node_modules\@zowe\cli\node_modules\@zowe\imperative\lib\imperative\src\plugins\cmd\install\install.handler for command install
Error Details:
not found: npm

C:\>npm --version
9.3.1

what do


In daemon mode on Windows, we use the spread operator to override process.env:
https://github.com/zowe/imperative/blob/cb175d40b4460bfc0c1fe319f13170b38b7969e2/packages/cmd/src/CommandProcessor.ts#L468

This breaks Node.js magic that aliases process.env.PATH to process.env.Path. We should replace the spread operator with a loop to assign properties.

@t1m0thyj t1m0thyj added priority-medium Not functioning - next quarter if capacity permits severity-high Bug for which there may be workaround but limits the usage of the Zowe for major use cases labels Jan 27, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
priority-medium Not functioning - next quarter if capacity permits severity-high Bug for which there may be workaround but limits the usage of the Zowe for major use cases
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant