-
Notifications
You must be signed in to change notification settings - Fork 3.2k
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
[BUG] npm ls --all
throws ELSPROBLEMS with tree created by npm
#6856
Comments
npm install itself is giving issues:- npm ERR! errno -4048 npm ERR! A complete log of this run can be found in: |
Hi, I am trying to solve the issue can I have some help from the community. I believe there is an issue with Arborist package used by Npm to create dependancy tree. What are your thoughts on to this. |
Did npm install working for u after cloning the repo? |
Why |
Yes. |
|
|
I don't know why it's only with |
I suppose that's true; I'm just confused why you'd be using |
It seems the goal was just to print the entire tree, not to check its validity. |
I believe there is an issue with creating a tree. It creates and invalid tree and installs packages with wrong dependancies. The problem is with the install command -> Arborist package. |
npm ls will not print the entire dependency of package |
I have cloned this code git clone https://github.com/nodejs/citgm.git and npm install, after that checked this command npm ls --all i got error npm ERR! A complete log of this run can be found in: So i tried to manual install, after that i check this command npm ls --all there is no error. |
Hi there, I encountered ELSPROBLEMS with $ npm ls --all
...
│ │ ├── react-dom@18.2.0 deduped invalid: "^16.8.3" from node_modules/react-vis
│ │ ├── react@18.2.0 deduped invalid: "^16.8.3" from node_modules/react-vis My package.json has the overrides like:
ELSPROBLEMS does not happen with npm@8. So perhaps npm@10 does not evaluate |
I created a repo for the repro. https://github.com/koooge/npm-cli-6856 |
|
Hi @targos I tried to reproduce in mention npm version 10.1.0 as well as in latest. I don't see this is reproducible. |
Unfortunately, I think it's not reproducible because the dependency tree has changed since I opened this issue. We need to find another reproduction. |
closing this, as its not reproducible. |
It is probably reproducible, just not with the initial steps. |
It still happens.
Try this with npm@10.5.1. (Not node@10 ) |
Fixed my ELSPROBLEMS with
|
Is there an existing issue for this?
This issue exists in the latest npm version
Current Behavior
With a dependency tree freshly created with
npm install
, runningnpm ls --all
throws an error for one of the subdependencies.The reason is that the tree deduplication made a mistake. The wrong major version of that dependency is installed relative to another subdependency.
Expected Behavior
npm ls --all
should not error for a dependency tree optimized by npm itself.Steps To Reproduce
git clone https://github.com/nodejs/citgm.git
cd citgm
npm install
npm ls --all
ornpm ls wrap-ansi
The problem is that
node_modules/tap/node_modules/cliui
wantswrap-ansi@^7.0.0
, but npm installedwrap-ansi@6.2.0
innode_modules/tap/node_modules/wrap-ansi
.See nodejs/citgm#990 for more context.
Environment
The text was updated successfully, but these errors were encountered: