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

pnpm i followed by pnpm i --frozen-lockfile complains about outdated lockfile #5080

Closed
JonnyBurger opened this issue Jul 22, 2022 · 6 comments · Fixed by #5120
Closed

pnpm i followed by pnpm i --frozen-lockfile complains about outdated lockfile #5080

JonnyBurger opened this issue Jul 22, 2022 · 6 comments · Fixed by #5120
Assignees
Milestone

Comments

@JonnyBurger
Copy link

JonnyBurger commented Jul 22, 2022

We have managed to land in a state where pnpm i --frozen-lockfile compains that pnpm-lock.yaml is not up to date, but pnpm i does not fix it. This issue appeared since the latest release.

pnpm version:

v7.6.0

Code to reproduce the issue:

git clone https://github.com/remotion-dev/remotion remotion-pnpm
cd remotion-pnpm
git checkout f76e74cec19d80348813760ca053a4af83ab71bc
pnpm i
pnpm i --frozen-lockfile

Expected behavior:

No error

Actual behavior:

 ERR_PNPM_OUTDATED_LOCKFILE  Cannot install with "frozen-lockfile" because pnpm-lock.yaml is not up-to-date with packages/core/package.json

Additional information:

Corepack is enabled
The original lockfile was generated by v7.2.1

  • node -v prints: v16.16.0
  • Windows, macOS, or Linux?: macOS
@jirayrva
Copy link

I'm having the same issue after upgrading from pnpm@7.3.0 to pnpm@7.6.0. It works fine when downgrading to pnpm@7.3.0 (I didn't try other minor versions)

When using pnpm@7.6.0

The error occurs in the projects that have peerDependencies in the package.json. When I remove peerDependencies, the install succeeds. I even tried this on the remotion repo and saw the same behavior.

Additional information:

Dev machine

  • Corepack: disabled
  • node: 16.5.1
  • OS: Windows

CI:

  • Corepack: disabled
  • 14.20.0
  • OS: Linux

@jirayrva
Copy link

It's working fine up till 7.5.2. It seems to have been broken in 7.6.0

@yankeeinlondon
Copy link

ran into this same issue I believe ... thought I was going mad

willsoto added a commit to willsoto/nestjs-prometheus that referenced this issue Jul 27, 2022
@elevatebart
Copy link

Probably caused by this fix #5028

If you set auto-install-peers to false it stops misbehaving.

@zkochan
Copy link
Member

zkochan commented Jul 29, 2022

I am looking into it.

@JonnyBurger
Copy link
Author

Appreciate the fix! 🙌

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants