Skip to content

Move channels to tarballs - #100

Merged
tomberek merged 5 commits into
NixOS:masterfrom
peterc-s:move-to-channel-tarballs
Sep 11, 2026
Merged

tomberek merged 5 commits into
NixOS:masterfrom
peterc-s:move-to-channel-tarballs

Conversation

@peterc-s

@peterc-s peterc-s commented Feb 9, 2026

Copy link
Copy Markdown
Contributor

See this thread for full context.

TL:DR; the channel tarballs are actually ahead of the GitHub and with the lockable tarball protocol being a thing, they should probably be used.

This PR also removes a URL literal in rust-web-server which have been deprecated.

I'm not certain what to do with full though.

TO-DO

  • Move over easy examples that use github: to channels.nixos.org.
  • Move over examples that use nixpkgs from the registry (as these also link to GitHub).
  • Figure out what to do with full, should it be moved over to the channel tarballs or have an example of it at least?

@GraysonTinker GraysonTinker left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

LGTM (I came here to do the same thing), but we should probably recommend the .tar.zst files that are now available.

@nixos-discourse

Copy link
Copy Markdown

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:

https://discourse.nixos.org/t/psa-use-nixos-org-tarballs-for-your-flake-inputs/79950/27

@peterc-s

Copy link
Copy Markdown
Contributor Author

Pushed a change to swap to .zst, but still have the unresolved TODO about the full template.

@peterc-s
peterc-s requested a review from tomberek as a code owner September 11, 2026 19:15
@peterc-s

Copy link
Copy Markdown
Contributor Author

Added the tarball to the "full" template but it isn't being used. At least it is there as an example I guess.

@tomberek
tomberek merged commit eeb156d into NixOS:master Sep 11, 2026
@tomberek

Copy link
Copy Markdown
Collaborator

Thanks!

Comment thread go-hello/flake.nix

# Nixpkgs / NixOS version to use.
inputs.nixpkgs.url = "nixpkgs/nixos-21.11";
inputs.nixpkgs.url = "https://channels.nixos.org/nixos-21.11/nixexprs.tar.zst";

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Eh, this is wrong. 21.11 doesn't have zstd tarballs, this won't work. Those are only for the newer channels like 26.05 and unstable - the non-EOL ones.

But also, surely the templates should be updated to non-EOL nixpkgs? But that will require quite a bit of work.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

My bad, should've noticed and checked this!

tomberek added a commit to pinpox/nixos-flake-templates that referenced this pull request Sep 14, 2026
The vendorHash, Go version bump, and nixpkgs pin from NixOS#92 were already
fixed on master by NixOS#89, NixOS#100, and NixOS#105; this lands the one still-
relevant piece of that PR: dropping the deprecated defaultPackage
output in favor of packages.default, matching the pattern already used
by c-hello and rust-web-server.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
tomberek added a commit that referenced this pull request Sep 14, 2026
The vendorHash, Go version bump, and nixpkgs pin from #92 were already
fixed on master by #89, #100, and #105; this lands the one still-
relevant piece of that PR: dropping the deprecated defaultPackage
output in favor of packages.default, matching the pattern already used
by c-hello and rust-web-server.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants