Skip to content

aws-vault: 7.7.5 -> 7.6.5#460421

Merged
numinit merged 1 commit intoNixOS:masterfrom
er0k:aws-vault-unbreak
Nov 11, 2025
Merged

aws-vault: 7.7.5 -> 7.6.5#460421
numinit merged 1 commit intoNixOS:masterfrom
er0k:aws-vault-unbreak

Conversation

@er0k
Copy link
Member

@er0k er0k commented Nov 10, 2025

There was a breaking change introduced in 7.7.0 on darwin platforms using the keychain backend, as reported by @retrry. This reverts to the last known working version, 7.6.5.

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.

There was a breaking change introduced in 7.7.0 on darwin platforms
using the keychain backend, as reported by @retrry. This reverts to the
last known working version, 7.6.5.
@nixpkgs-ci nixpkgs-ci bot requested a review from zimbatm November 10, 2025 18:58
@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: 1-10 This PR causes between 1 and 10 packages to rebuild on Darwin. 11.by: package-maintainer This PR was created by a maintainer of all the package it changes. 10.rebuild-darwin: 1 This PR causes 1 package to rebuild on Darwin. 10.rebuild-linux: 1 This PR causes 1 package to rebuild on Linux. labels Nov 10, 2025
@er0k
Copy link
Member Author

er0k commented Nov 10, 2025

nixpkgs-review result

Generated using nixpkgs-review-gha

Command: nixpkgs-review pr 460421
Commit: 735b827525b296a13617680c367169d20ee11e51 (subsequent changes)
Merge: f0753de8fa409af60affaa0ba2e98e6c939b3731

Logs: https://github.com/er0k/nixpkgs-review-gha/actions/runs/19243192109


x86_64-linux

✅ 1 package built:
  • aws-vault

aarch64-linux

✅ 1 package built:
  • aws-vault

x86_64-darwin (sandbox = relaxed)

✅ 1 package built:
  • aws-vault

aarch64-darwin (sandbox = relaxed)

✅ 1 package built:
  • aws-vault

Copy link
Contributor

@numinit numinit left a comment

Choose a reason for hiding this comment

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

Thanks for this, builds and works.

nixpkgs-review result

Generated using nixpkgs-review.

Command: nixpkgs-review pr 460421
Commit: 735b827525b296a13617680c367169d20ee11e51


x86_64-linux

✅ 1 package built:
  • aws-vault

@numinit numinit added this pull request to the merge queue Nov 11, 2025
@nixpkgs-ci nixpkgs-ci bot added 12.approvals: 1 This PR was reviewed and approved by one person. 2.status: merge-bot eligible This PR can be merged by commenting "@NixOS/nixpkgs-merge-bot merge". labels Nov 11, 2025
Merged via the queue into NixOS:master with commit f865b65 Nov 11, 2025
40 of 42 checks passed
@retrry
Copy link
Contributor

retrry commented Nov 11, 2025

@er0k thanks for this! Just tested it - works again.

@er0k er0k deleted the aws-vault-unbreak branch November 11, 2025 19:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

2.status: merge-bot eligible This PR can be merged by commenting "@NixOS/nixpkgs-merge-bot merge". 10.rebuild-darwin: 1-10 This PR causes between 1 and 10 packages to rebuild on Darwin. 10.rebuild-darwin: 1 This PR causes 1 package 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. 11.by: package-maintainer This PR was created by a maintainer of all the package it changes. 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