Skip to content

fetchurl: make pname and version overridable via <pkg>.overrideAttrs#458183

Merged
philiptaron merged 1 commit intoNixOS:masterfrom
ShamrockLee:fetchurl-overriding-pname-version
Nov 5, 2025
Merged

fetchurl: make pname and version overridable via <pkg>.overrideAttrs#458183
philiptaron merged 1 commit intoNixOS:masterfrom
ShamrockLee:fetchurl-overriding-pname-version

Conversation

@ShamrockLee
Copy link
Contributor

@ShamrockLee ShamrockLee commented Nov 3, 2025

Make name references finalAttrs.pname and finalAttrs.version instead of args.pname and args.version.

This PR depends on:

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.

@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. 6.topic: fetch Fetchers (e.g. fetchgit, fetchsvn, ...) labels Nov 3, 2025
@ShamrockLee ShamrockLee force-pushed the fetchurl-overriding-pname-version branch from 0ee0b05 to 7e212bf Compare November 4, 2025 07:11
@ShamrockLee ShamrockLee marked this pull request as ready for review November 4, 2025 07:12
@nix-owners nix-owners bot requested a review from philiptaron November 4, 2025 07:16
@philiptaron philiptaron added this pull request to the merge queue Nov 5, 2025
@philiptaron
Copy link
Contributor

I'm not so sure about the backport.

@ShamrockLee
Copy link
Contributor Author

I'm not so sure about the backport.

That's just a miskake. I only mean to backport the androidenv fix, but labled the wrong PR.

Merged via the queue into NixOS:master with commit 49af835 Nov 5, 2025
30 of 32 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

6.topic: fetch Fetchers (e.g. fetchgit, fetchsvn, ...) 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.

2 participants