Skip to content

Conversation

@fabianschuiki
Copy link
Contributor

Instead of just saying "unknown git version", use the current git commit hash as CIRCT version in non-release builds. This makes the output of --version of our tools more useful, because it points to an actual commit.

With this change, the sv-tests-results website can provide an actual git commit tooltip for the circt-verilog tool being used.

Copy link
Member

@uenoku uenoku left a comment

Choose a reason for hiding this comment

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

Looks reasonable to me.

Instead of just saying "unknown git version", use the current git commit
hash as CIRCT version in non-release builds. This makes the output of
`--version` of our tools more useful, because it points to an actual
commit.

With this change, the [sv-tests-results] website can provide an actual
git commit tooltip for the circt-verilog tool being used.

[sv-tests-results]: https://chipsalliance.github.io/sv-tests-results/
@fabianschuiki fabianschuiki force-pushed the fschuiki/git-hash-version branch from 5f59b09 to 4c4d91c Compare August 8, 2025 22:59
@fabianschuiki fabianschuiki merged commit c9b454f into main Aug 8, 2025
7 checks passed
@fabianschuiki fabianschuiki deleted the fschuiki/git-hash-version branch August 8, 2025 23:16
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.

3 participants