Skip to content

various: migrate fetchPnpmDeps from fetcherVersion = 1 to fetcherVersion = 3 (part 4)#494402

Merged
gepbird merged 15 commits into
NixOS:masterfrom
qweered:fetcherVersion-3-migration-part4
Apr 25, 2026
Merged

various: migrate fetchPnpmDeps from fetcherVersion = 1 to fetcherVersion = 3 (part 4)#494402
gepbird merged 15 commits into
NixOS:masterfrom
qweered:fetcherVersion-3-migration-part4

Conversation

@qweered

@qweered qweered commented Feb 26, 2026

Copy link
Copy Markdown
Contributor

Migrate various packages using fetchPnpmDeps from fetcherVersion = 1 to fetcherVersion = 3 and update their hashes accordingly.

fetcherVersion = 1 is deprecated and will be removed. Version 3 uses a more robust fetching strategy.

Things done

  • Built on platform:
    • x86_64-linux
    • aarch64-linux
    • x86_64-darwin
    • aarch64-darwin
  • Tested, as applicable:
  • Ran nixpkgs-review on this PR. See nixpkgs-review usage.
  • Tested basic functionality of all binary files, usually in ./result/bin/.
  • Nixpkgs Release Notes
    • Package update: when the change is major or breaking.
  • NixOS Release Notes
    • Module addition: when adding a new NixOS module.
    • Module update: when the change is significant.
  • Fits CONTRIBUTING.md, pkgs/README.md, maintainers/README.md and other READMEs.

@qweered qweered marked this pull request as ready for review February 26, 2026 15:20
@nixpkgs-ci nixpkgs-ci Bot added 10.rebuild-linux: 11-100 This PR causes between 11 and 100 packages to rebuild on Linux. 10.rebuild-darwin: 11-100 This PR causes between 11 and 100 packages to rebuild on Darwin. 9.needs: reviewer This PR currently has no reviewers requested and needs attention. 6.topic: python Python is a high-level, general-purpose programming language. labels Feb 26, 2026
@qweered

qweered commented Feb 26, 2026

Copy link
Copy Markdown
Contributor Author

nixpkgs-review result

Generated using nixpkgs-review-gha

Command: nixpkgs-review pr 494402
Commit: 03e08826fbd685b9e1f3c26e92be41f1a6b187d2 (subsequent changes)
Merge: 2dd4e144281a1fa2cf8f4867edcf1136ff367038

Logs: https://github.com/qweered/nixpkgs-review-gha/actions/runs/22453325493


x86_64-linux

❌ 7 packages failed to build:
  • flood
  • froide
  • froide-govplan
  • froide-govplan.dist
  • froide.dist
  • vikunja
  • vikunja-desktop
✅ 19 packages built:
  • discourse
  • discourseAllPlugins
  • python313Packages.django-filingcabinet
  • python313Packages.django-filingcabinet.dist
  • tailwindcss-language-server
  • taler-challenger
  • taler-mdb
  • taler-merchant
  • taler-sync
  • taler-wallet-core
  • taze
  • textlint
  • tsx
  • vtsls
  • vue-language-server
  • wealthfolio
  • zammad
  • zenn-cli
  • zigbee2mqtt

@qweered qweered marked this pull request as draft February 26, 2026 19:21
@nixpkgs-ci nixpkgs-ci Bot added 2.status: merge conflict This PR has merge conflicts with the target branch and removed 9.needs: reviewer This PR currently has no reviewers requested and needs attention. labels Feb 26, 2026
@qweered qweered force-pushed the fetcherVersion-3-migration-part4 branch from 03e0882 to 9350cef Compare April 24, 2026 18:34
@nixpkgs-ci nixpkgs-ci Bot added 10.rebuild-darwin: 1-10 This PR causes between 1 and 10 packages to rebuild on Darwin. and removed 2.status: merge conflict This PR has merge conflicts with the target branch 10.rebuild-darwin: 11-100 This PR causes between 11 and 100 packages to rebuild on Darwin. labels Apr 24, 2026
@qweered qweered force-pushed the fetcherVersion-3-migration-part4 branch from 9350cef to 4599f37 Compare April 24, 2026 18:49
@nixpkgs-ci nixpkgs-ci Bot added 10.rebuild-darwin: 11-100 This PR causes between 11 and 100 packages to rebuild on Darwin. and removed 10.rebuild-darwin: 1-10 This PR causes between 1 and 10 packages to rebuild on Darwin. labels Apr 24, 2026
@qweered

qweered commented Apr 24, 2026

Copy link
Copy Markdown
Contributor Author

Tested on x86_64, no errrors

@qweered qweered marked this pull request as ready for review April 24, 2026 21:00
@nixpkgs-ci nixpkgs-ci Bot added the 9.needs: reviewer This PR currently has no reviewers requested and needs attention. label Apr 24, 2026
@qweered qweered requested a review from gepbird April 24, 2026 22:15
@nixpkgs-ci nixpkgs-ci Bot removed the 9.needs: reviewer This PR currently has no reviewers requested and needs attention. label Apr 24, 2026
qweered added a commit to qweered/nixpkgs that referenced this pull request Apr 24, 2026
fetcherVersion = 1 was kept around for backwards compatibility while
callers migrated to 2/3. Most of them have; the remainder are being
cleared in NixOS#494402 and NixOS#513204.

Emit a `lib.warnIf` warning pointing at the affected pname whenever v1
is used, gated by `lib.oldestSupportedReleaseIsAtLeast 2611` so the
deprecation message only becomes visible once 26.11 is the oldest
supported release (i.e., the next LTS to EOL). At that point,
`supportedFetcherVersions` should drop 1 and the dead v1 code paths in
default.nix, pnpm-config-hook.sh and serve.nix can be cleaned up.

Also rewrite the manual section to recommend fetcherVersion = 3 for new
packages and mark v1 as deprecated/scheduled-for-removal in the
version history.
qweered added a commit to qweered/nixpkgs that referenced this pull request Apr 24, 2026
fetcherVersion = 1 was kept around for backwards compatibility while
callers migrated to 2/3. Most of them have; the remainder are being
cleared in NixOS#494402 and NixOS#513204.

Emit a `lib.warnIf` warning pointing at the affected pname whenever v1
is used, gated by `lib.oldestSupportedReleaseIsAtLeast 2605` so the
deprecation message activates once 26.05 becomes the oldest supported
release (~Jun 2026, when 25.11 EOLs). That gives users roughly one
release cycle of advance notice before v1 is removed in 26.11.

After 26.11 ships, a follow-up PR should drop 1 from
`supportedFetcherVersions` and clean up the dead v1 code paths in
default.nix, pnpm-config-hook.sh and serve.nix.

Also rewrite the manual section to recommend fetcherVersion = 3 for new
packages and mark v1 as deprecated/scheduled-for-removal in the
version history.
qweered added a commit to qweered/nixpkgs that referenced this pull request Apr 24, 2026
fetcherVersion = 1 was kept around for backwards compatibility while
callers migrated to 2/3. Most of them have; the remainder are being
cleared in NixOS#494402 and NixOS#513204.

Emit a `lib.warnIf` warning pointing at the affected pname whenever v1
is used, gated by `lib.oldestSupportedReleaseIsAtLeast 2605` so the
deprecation message activates once 26.05 becomes the oldest supported
release (~Jun 2026, when 25.11 EOLs). That gives users roughly one
release cycle of advance notice before v1 is removed in 26.11.

After 26.11 ships, a follow-up PR should drop 1 from
`supportedFetcherVersions` and clean up the dead v1 code paths in
default.nix, pnpm-config-hook.sh and serve.nix.

Also rewrite the manual section to recommend fetcherVersion = 3 for new
packages and mark v1 as deprecated/scheduled-for-removal in the
version history.
@gepbird

gepbird commented Apr 25, 2026

Copy link
Copy Markdown
Contributor

nixpkgs-review result

Generated using nixpkgs-review.

Command: nixpkgs-review pr 494402
Commit: 4599f379e8d06fafd0a15e1626206a257837a5ff


x86_64-linux

✅ 29 packages built:
  • discourse
  • discourseAllPlugins
  • element-desktop
  • flood
  • froide
  • froide-govplan
  • froide-govplan.dist
  • froide.dist
  • home-assistant-custom-lovelace-modules.custom-sidebar
  • ocis
  • python313Packages.django-filingcabinet
  • python313Packages.django-filingcabinet.dist
  • python314Packages.django-filingcabinet
  • python314Packages.django-filingcabinet.dist
  • session-desktop
  • tailwindcss-language-server
  • taler-challenger
  • taler-mdb
  • taler-merchant
  • taler-sync
  • taler-wallet-core
  • taze
  • textlint
  • tsx
  • vtsls
  • vue-language-server
  • wealthfolio
  • zenn-cli
  • zigbee2mqtt

@gepbird gepbird left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for coming back to this :)

@gepbird gepbird added this pull request to the merge queue Apr 25, 2026
Merged via the queue into NixOS:master with commit dc66b37 Apr 25, 2026
30 checks passed
@Scrumplex

Copy link
Copy Markdown
Member

nixpkgs-review result

Generated using nixpkgs-review.

Command: nixpkgs-review pr 494402
Commit: 4599f379e8d06fafd0a15e1626206a257837a5ff


x86_64-linux

❌ 2 packages failed to build:
  • element-desktop
  • session-desktop
✅ 27 packages built:
  • discourse
  • discourseAllPlugins
  • flood
  • froide
  • froide-govplan
  • froide-govplan.dist
  • froide.dist
  • home-assistant-custom-lovelace-modules.custom-sidebar
  • ocis
  • python313Packages.django-filingcabinet
  • python313Packages.django-filingcabinet.dist
  • python314Packages.django-filingcabinet
  • python314Packages.django-filingcabinet.dist
  • tailwindcss-language-server
  • taler-challenger
  • taler-mdb
  • taler-merchant
  • taler-sync
  • taler-wallet-core
  • taze
  • textlint
  • tsx
  • vtsls
  • vue-language-server
  • wealthfolio
  • zenn-cli
  • zigbee2mqtt

@Scrumplex

Copy link
Copy Markdown
Member

Oh I am late

@qweered qweered deleted the fetcherVersion-3-migration-part4 branch April 25, 2026 09:13
Scrumplex pushed a commit to qweered/nixpkgs that referenced this pull request Apr 26, 2026
fetcherVersion = 1 was kept around for backwards compatibility while
callers migrated to 2/3. Most of them have; the remainder are being
cleared in NixOS#494402 and NixOS#513204.

Emit a `lib.warnIf` warning pointing at the affected pname whenever v1
is used.

After 26.11 ships, a follow-up PR should drop 1 from
`supportedFetcherVersions` and clean up the dead v1 code paths in
default.nix, pnpm-config-hook.sh and serve.nix.

Also rewrite the manual section to recommend fetcherVersion = 3 for new
packages and mark v1 as deprecated/scheduled-for-removal in the
version history.

Signed-off-by: Sefa Eyeoglu <contact@scrumplex.net>
Scrumplex pushed a commit to qweered/nixpkgs that referenced this pull request Apr 26, 2026
fetcherVersion = 1 was kept around for backwards compatibility while
callers migrated to 2/3. Most of them have; the remainder are being
cleared in NixOS#494402 and NixOS#513204.

Emit a `lib.warnIf` warning pointing at the affected pname whenever v1
is used.

After 26.11 ships, a follow-up PR should drop 1 from
`supportedFetcherVersions` and clean up the dead v1 code paths in
default.nix, pnpm-config-hook.sh and serve.nix.

Also rewrite the manual section to recommend fetcherVersion = 3 for new
packages and mark v1 as deprecated/scheduled-for-removal in the
version history.

Signed-off-by: Sefa Eyeoglu <contact@scrumplex.net>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

6.topic: python Python is a high-level, general-purpose programming language. 10.rebuild-darwin: 11-100 This PR causes between 11 and 100 packages to rebuild on Darwin. 10.rebuild-linux: 11-100 This PR causes between 11 and 100 packages to rebuild on Linux.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants