Skip to content

nextcloud32.packages.apps.gpoddersync: init at 3.13.1#450804

Merged
dotlambda merged 1 commit into
NixOS:masterfrom
x10an14:master
Oct 11, 2025
Merged

nextcloud32.packages.apps.gpoddersync: init at 3.13.1#450804
dotlambda merged 1 commit into
NixOS:masterfrom
x10an14:master

Conversation

@x10an14

@x10an14 x10an14 commented Oct 10, 2025

Copy link
Copy Markdown
Contributor

Added gpoddersync nextcloud app to NC 32, as discussed here: #446632 (comment)

Here's changelog: https://github.com/thrillfall/nextcloud-gpodder/releases/tag/3.13

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.

Add a 👍 reaction to pull requests you find important.

@x10an14 x10an14 marked this pull request as draft October 10, 2025 21:33
@nixpkgs-ci nixpkgs-ci Bot added 10.rebuild-darwin: 0 This PR does not cause any packages to rebuild on Darwin. 10.rebuild-linux: 0 This PR does not cause any packages to rebuild on Linux. labels Oct 10, 2025
@x10an14

x10an14 commented Oct 10, 2025

Copy link
Copy Markdown
Contributor Author

Awaiting nixpkgs-review rev HEAD with amended commit adding the missing final 2 (not just 1x) \n at the end of file - thus passing nix fmt.

It's been running for 12 mins and counting.

@x10an14

x10an14 commented Oct 10, 2025

Copy link
Copy Markdown
Contributor Author

It's been running for 12 mins and counting.

25+ mins and counting. Headed to bed, will test on NC 32 instance tomorrow.

@x10an14 x10an14 marked this pull request as ready for review October 10, 2025 22:04
@nixpkgs-ci nixpkgs-ci Bot added the 9.needs: reviewer This PR currently has no reviewers requested and needs attention. label Oct 10, 2025

@dotlambda dotlambda left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Please run pkgs/servers/nextcloud/packages/generate.sh.

@nixpkgs-ci nixpkgs-ci Bot removed the 9.needs: reviewer This PR currently has no reviewers requested and needs attention. label Oct 10, 2025
@x10an14

x10an14 commented Oct 11, 2025

Copy link
Copy Markdown
Contributor Author

Please run pkgs/servers/nextcloud/packages/generate.sh.

Done. Next is testing for me, then I'm satisfied.

Let me know if I forgot anything/should do something else/more!

@x10an14

x10an14 commented Oct 11, 2025

Copy link
Copy Markdown
Contributor Author

nixpkgs-review result

Generated using nixpkgs-review.

Command: nixpkgs-review pr 450804
Commit: 55160731f4fa8d2d80920166f96d40d1f02a3dbb

@x10an14

x10an14 commented Oct 11, 2025

Copy link
Copy Markdown
Contributor Author

Done. Next is testing for me, then I'm satisfied.

Tested on my NC instance (v32 + gpoddersync).
Works fine!

Comment thread pkgs/servers/nextcloud/packages/32.json Outdated
@dotlambda

Copy link
Copy Markdown
Member

Why are there two commits?

@x10an14

x10an14 commented Oct 11, 2025

Copy link
Copy Markdown
Contributor Author

Why are there two commits?

Because you asked me to run #450804 (review), which touches other packages.
Without further context, abiding the https://github.com/NixOS/nixpkgs/blob/master/CONTRIBUTING.md (since it touches other packages), that's what made sense to me.

Is there some documented instruction I've missed somewhere?


I can just squash the commits, but I'm awaiting thrillfall/nextcloud-gpodder#206 as requested by you.

@dotlambda

Copy link
Copy Markdown
Member

If you really want two commits it's fine but then you need to make sure that the first adds only the one package exactly as the script would.

@x10an14

x10an14 commented Oct 11, 2025

Copy link
Copy Markdown
Contributor Author

This was a learning experience, but now I hope all is correct.

@dotlambda dotlambda changed the title nextcloud32.packages.apps.gpoddersync: init at 3.13 nextcloud32.packages.apps.gpoddersync: init at 3.13.1 Oct 11, 2025

@dotlambda dotlambda left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I manually built nextcloud31Packages.apps and nextcloud32Packages.apps.
Everything but hmr_enabler built successfully, see #435943 (comment).
I also ran nixosTests.nextcloud.

@dotlambda dotlambda added this pull request to the merge queue Oct 11, 2025
Merged via the queue into NixOS:master with commit 2c72f81 Oct 11, 2025
28 of 32 checks passed
@x10an14

x10an14 commented Oct 12, 2025

Copy link
Copy Markdown
Contributor Author

(...) I also ran nixosTests.nextcloud.

Is there a way to do this with the new nix3 / experimental nix commands?
So far I've only found the nix-build --attrs way to do so, unless I hackily perform a nix eval --apply builtins.attrNames --json and iterate over the results.

Ref: NixOS/nix#7701 (comment)

@dotlambda

Copy link
Copy Markdown
Member

@x10an14 nix build -f . nixosTests.nextcloud

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

Labels

10.rebuild-darwin: 0 This PR does not cause any packages to rebuild on Darwin. 10.rebuild-linux: 0 This PR does not cause any packages to rebuild on Linux.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants