-
Notifications
You must be signed in to change notification settings - Fork 869
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
NodeInstaller is not copying the npm binaries next to the node exec #1152
Comments
ia3andy
changed the title
NodeInstaller is not copying the binaries next to the node exec
NodeInstaller is not copying the npm binaries next to the node exec
Jun 7, 2024
melloware
added a commit
to melloware/frontend-maven-plugin
that referenced
this issue
Jun 7, 2024
atl-mk
pushed a commit
to atlassian-forks/frontend-maven-plugin
that referenced
this issue
Sep 19, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
This causes
package.json
script usingnpm
to yield:npm command not found
, because it is not in the PATH.There is a comment saying it does, but it's is not actually done:
https://github.com/eirslett/frontend-maven-plugin/blob/master/frontend-plugin-core/src/main/java/com/github/eirslett/maven/plugins/frontend/lib/NodeInstaller.java#L192
It works when using the NPMInstaller because it actually does it:
https://github.com/eirslett/frontend-maven-plugin/blob/master/frontend-plugin-core/src/main/java/com/github/eirslett/maven/plugins/frontend/lib/NPMInstaller.java#L182-L205
The text was updated successfully, but these errors were encountered: