Skip to content

peertube: 8.0.2 -> 8.1.5; bump nodejs version from 20 to 24#515594

Merged
wegank merged 1 commit into
NixOS:masterfrom
epireyn:update/peertube
May 7, 2026
Merged

peertube: 8.0.2 -> 8.1.5; bump nodejs version from 20 to 24#515594
wegank merged 1 commit into
NixOS:masterfrom
epireyn:update/peertube

Conversation

@epireyn

@epireyn epireyn commented May 1, 2026

Copy link
Copy Markdown
Contributor

Resolves #515284 by bumping nodejs version.

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.

@nixpkgs-ci nixpkgs-ci Bot added 10.rebuild-linux: 1-10 This PR causes between 1 and 10 packages to rebuild on Linux. 10.rebuild-darwin: 0 This PR does not cause any packages to rebuild on Darwin. 10.rebuild-linux: 1 This PR causes 1 package to rebuild on Linux. labels May 1, 2026
@qweered qweered mentioned this pull request May 1, 2026
Open
25 tasks
@eljamm

eljamm commented May 2, 2026

Copy link
Copy Markdown
Contributor

nixpkgs-review result for #515594

Generated using nixpkgs-review-gha

Command: nixpkgs-review pr 515594
Commit: f07457fd1676d548a1f3e7a6fc0d1479430d05d6 (subsequent changes)
Merge: 7d272c885be7da2fdb0a604029492030fd196560

Logs: https://github.com/eljamm/nixpkgs-review-gha/actions/runs/25242199202

x86_64-linux

✅ 3 packages built:
  • peertube
  • peertube.cli
  • peertube.runner

aarch64-linux

✅ 3 packages built:
  • peertube
  • peertube.cli
  • peertube.runner

@eljamm

eljamm commented May 2, 2026

Copy link
Copy Markdown
Contributor

The PeerTube NixOS test is passing, although it would be great if someone with a real instance can verify that it's working properly.

eljamm
eljamm approved these changes May 7, 2026
View reviewed changes

@eljamm eljamm 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.

Tested this release briefly against the NGIpkgs demo and it seems to be fine to me. Version 8.1.0 switched to sharp for image processing which works. It also merges the previews directory into thumbnails, but since the former will still be around for comparability reasons, I guess we don't really have to worry about this for now in the module.

@nixpkgs-ci nixpkgs-ci Bot added 12.approvals: 1 This PR was reviewed and approved by one person. 12.approved-by: package-maintainer This PR was reviewed and approved by a maintainer listed in any of the changed packages. labels May 7, 2026
@wegank wegank added this pull request to the merge queue May 7, 2026
Merged via the queue into NixOS:master with commit 49fe932 May 7, 2026
35 of 37 checks passed
@qweered

qweered commented May 7, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

nodejs_20 still referenced in nixos/modules/services/web-apps/peertube.nix

@wegank wegank mentioned this pull request May 7, 2026
Merged
13 tasks
@epireyn epireyn deleted the update/peertube branch May 8, 2026 14:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@ethancedwards8 ethancedwards8 Awaiting requested review from ethancedwards8

@immae immae Awaiting requested review from immae

@Izorkin Izorkin Awaiting requested review from Izorkin

@phanirithvij phanirithvij Awaiting requested review from phanirithvij

@Prince213 Prince213 Awaiting requested review from Prince213

@wegank wegank Awaiting requested review from wegank

1 more reviewer

@eljamm eljamm eljamm approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

10.rebuild-darwin: 0 This PR does not cause any packages to rebuild on Darwin. 10.rebuild-linux: 1-10 This PR causes between 1 and 10 packages to rebuild on Linux. 10.rebuild-linux: 1 This PR causes 1 package to rebuild on Linux. 12.approvals: 1 This PR was reviewed and approved by one person. 12.approved-by: package-maintainer This PR was reviewed and approved by a maintainer listed in any of the changed packages.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Nodejs 20 EOL

4 participants

@epireyn @eljamm @qweered @wegank