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

Bundled dependencies for [email protected] have versions of libnpmexec, libnpmpack, libnpmversion which do not match package.json #7423

Closed
2 tasks done
dominykas opened this issue Apr 26, 2024 · 1 comment · Fixed by #7429
Labels
Bug thing that needs fixing Priority 2 secondary priority issue Release 10.x

Comments

@dominykas
Copy link

Is there an existing issue for this?

  • I have searched the existing issues

This issue exists in the latest npm version

  • I am using the latest npm

Current Behavior

npm ls -g --all results in:

06:03:17 #25 8.386 npm error invalid: [email protected] /usr/local/lib/node_modules/npm/node_modules/libnpmexec
06:03:17 #25 8.386 npm error invalid: [email protected] /usr/local/lib/node_modules/npm/node_modules/libnpmpack
06:03:17 #25 8.386 npm error invalid: [email protected] /usr/local/lib/node_modules/npm/node_modules/libnpmversion

Expected Behavior

No errors.

Steps To Reproduce

  1. Tested inside Ubuntu@22 docker container and on macOS with [email protected] with baseline [email protected]
  2. Run npm i -g npm

Downloading the latest npm tarball via npm pack npm highlights the problem:

~/devel/experiments
❯ cat package/package.json | grep libnpmexec
    "libnpmexec": "^7.0.4",
    "libnpmexec",

~/devel/experiments
❯ cat package/node_modules/libnpmexec/package.json| grep version
  "version": "8.0.0",

Environment

  • npm:
  • Node.js:
  • OS Name:
  • System Model Name:
  • npm config:
; copy and paste output from `npm config ls` here
@dominykas dominykas added Bug thing that needs fixing Needs Triage needs review for next steps Release 10.x labels Apr 26, 2024
@wraithgar wraithgar added Priority 2 secondary priority issue and removed Needs Triage needs review for next steps labels Apr 26, 2024
@wraithgar
Copy link
Member

Looks like the release script isn't bumping the base package.json anymore. This is mostly a cosmetic problem, as npm bundles all of its dependencies.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug thing that needs fixing Priority 2 secondary priority issue Release 10.x
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants