Skip to content

verible: 0.0.3956 -> 0.0.4023#445793

Merged
NickCao merged 1 commit intoNixOS:masterfrom
GaetanLepage:update/verible
Sep 26, 2025
Merged

verible: 0.0.3956 -> 0.0.4023#445793
NickCao merged 1 commit intoNixOS:masterfrom
GaetanLepage:update/verible

Conversation

@GaetanLepage
Copy link
Contributor

Things done

Diff: chipsalliance/verible@v0.0-3956-ge12a194d...v0.0-4023-gc1271a00

cc @hzeller @newAM

  • 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-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. 10.rebuild-darwin: 1 This PR causes 1 package to rebuild on Darwin. labels Sep 24, 2025
@nix-owners nix-owners bot requested review from hzeller and newAM September 24, 2025 13:29
@GaetanLepage GaetanLepage mentioned this pull request Sep 26, 2025
13 tasks
Copy link
Contributor

@hzeller hzeller left a comment

Choose a reason for hiding this comment

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

Thanks.
(locally tested and built x86 Linux successfully)

@nixpkgs-ci nixpkgs-ci bot added 12.approvals: 1 This PR was reviewed and approved by one person. 12.approved-by: package-maintainer This PR was reviewed and approved by a maintainer listed in any of the changed packages. labels Sep 26, 2025
@qweered
Copy link
Contributor

qweered commented Sep 26, 2025

nixpkgs-review result

Generated using nixpkgs-review-gha

Command: nixpkgs-review pr 445793
Commit: 5381cfa83b27cf4d26e67e7596110c8e11f01e73 (subsequent changes)
Merge: 2cd06b97a09b7a12d980d707eb3b740feb7c599c

Logs: https://github.com/qweered/nixpkgs-review-gha/actions/runs/18036846588


x86_64-linux

✅ 2 packages built:
  • verible
  • veridian

aarch64-linux

✅ 2 packages built:
  • verible
  • veridian

x86_64-darwin

No rebuilds


aarch64-darwin (sandbox = true)

❌ 1 package failed to build:
  • verible

@hzeller
Copy link
Contributor

hzeller commented Sep 26, 2025

The failure on darwin seems to be related to building bazel 7 not working on nix currently, so not related to this change.

Copy link
Member

@newAM newAM 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 the update!

@nixpkgs-ci nixpkgs-ci bot added 12.approvals: 2 This PR was reviewed and approved by two persons. and removed 12.approvals: 1 This PR was reviewed and approved by one person. labels Sep 26, 2025
@NickCao NickCao added this pull request to the merge queue Sep 26, 2025
Merged via the queue into NixOS:master with commit d2eea6d Sep 26, 2025
35 of 37 checks passed
@GaetanLepage GaetanLepage deleted the update/verible branch September 26, 2025 20:41
@wegank wegank mentioned this pull request Jan 31, 2026
13 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

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. 12.approvals: 2 This PR was reviewed and approved by two persons. 12.approved-by: package-maintainer This PR was reviewed and approved by a maintainer listed in any of the changed packages.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants