Skip to content

unstableGitUpdater: pass --print-changes to update-source-version#490004

Merged
jian-lin merged 1 commit into
NixOS:masterfrom
linj-fork:pr/unstableGitUpdater-commit
Feb 24, 2026
Merged

unstableGitUpdater: pass --print-changes to update-source-version#490004
jian-lin merged 1 commit into
NixOS:masterfrom
linj-fork:pr/unstableGitUpdater-commit

Conversation

@jian-lin
Copy link
Copy Markdown
Contributor

@jian-lin jian-lin commented Feb 13, 2026

This enables git commit message customization when writing a thin-wrapper around unstableGitUpdater with "commit" in supportedFeatures.

Passing --print-changes unconditionally is consistent with other updaters such as genericUpdater, which is used by many updaters.

We also redirect all non-json stdout to stderr so that the whole
stdout contains a valid json.

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-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. 9.needs: reviewer This PR currently has no reviewers requested and needs attention. 6.topic: updaters Tooling for (semi-)automated updating of packages labels Feb 13, 2026
@nixpkgs-ci nixpkgs-ci Bot removed the 9.needs: reviewer This PR currently has no reviewers requested and needs attention. label Feb 13, 2026
Comment thread pkgs/common-updater/unstable-updater.nix Outdated
@jtojnar jtojnar requested a review from romildo February 13, 2026 09:14
This enables git commit message customization when writing a
thin-wrapper around unstableGitUpdater with "commit" in
supportedFeatures.

Passing --print-changes unconditionally is consistent with other
updaters such as genericUpdater, which is used by many updaters.

We also redirect all non-json stdout to stderr so that the whole
stdout contains a valid json.
@jian-lin jian-lin force-pushed the pr/unstableGitUpdater-commit branch from d23af0c to ca16356 Compare February 13, 2026 11:34
@jian-lin jian-lin requested a review from jtojnar February 13, 2026 11:42
@jian-lin
Copy link
Copy Markdown
Contributor Author

ping :)

@jian-lin
Copy link
Copy Markdown
Contributor Author

I plan to merge this in a few days if no more review.

Copy link
Copy Markdown
Contributor

@romildo romildo left a comment

Choose a reason for hiding this comment

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

LGTM

@nixpkgs-ci nixpkgs-ci Bot added the 12.approvals: 1 This PR was reviewed and approved by one person. label Feb 23, 2026
@jian-lin
Copy link
Copy Markdown
Contributor Author

Thanks for all your review!

@jian-lin jian-lin added this pull request to the merge queue Feb 24, 2026
Merged via the queue into NixOS:master with commit 2983b2c Feb 24, 2026
32 of 34 checks passed
@jian-lin jian-lin deleted the pr/unstableGitUpdater-commit branch February 24, 2026 04:53
adamcstephens pushed a commit to adamcstephens/nixpkgs that referenced this pull request Feb 24, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

6.topic: updaters Tooling for (semi-)automated updating of packages 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. 12.approvals: 1 This PR was reviewed and approved by one person.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants