Skip to content

feat: add format_version_with_commit() for foundry to print ssolc version#14

Merged
samlaf merged 2 commits intoseismicfrom
feat--return-compiler-name-with-full-commit-info
Feb 19, 2026
Merged

feat: add format_version_with_commit() for foundry to print ssolc version#14
samlaf merged 2 commits intoseismicfrom
feat--return-compiler-name-with-full-commit-info

Conversation

@samlaf
Copy link
Contributor

@samlaf samlaf commented Feb 19, 2026

Right now we only see "compiling with solc 0.8.31" or whatever... which is confusing because we don't know which exact build its coming from. We don't use semver yet for our ssolc fork so its better to have commit from where it was built.

When compiling we'll thus see the commit version (tested this in foundry using this commit)

$ ../../seismic-foundry/target/debug/sforge build -vvv
[⠊] Compiling...
[⠑] Compiling 44 files with ssolc 0.8.31 (676bdec)
[⠘] ssolc 0.8.31 (676bdec) finished in 615.21ms

Before we'd see Compiler ... with Solc 0.8.31 only.

Right now we only see "compiling with solc 0.8.31" or whatever... which is confusing because we don't know which exact build its coming from. We don't use semver yet for our ssolc fork so its better to have commit from where it was built.
@samlaf samlaf requested a review from cdrappi as a code owner February 19, 2026 22:16
@samlaf samlaf changed the title feat: add format_version_with_commit for foundry to print ssolc version feat: add format_version_with_commit() for foundry to print ssolc version Feb 19, 2026
@samlaf samlaf merged commit d24431f into seismic Feb 19, 2026
4 checks passed
@samlaf samlaf deleted the feat--return-compiler-name-with-full-commit-info branch February 19, 2026 22:34
samlaf added a commit to SeismicSystems/seismic-foundry that referenced this pull request Feb 19, 2026
Uses SeismicSystems/seismic-compilers#14

When compiling with -vvv we can now see the commit version:
```
$ sforge build -vvv
[⠊] Compiling...
[⠑] Compiling 44 files with ssolc 0.8.31 (676bdec)
[⠘] ssolc 0.8.31 (676bdec) finished in 615.21ms
```

Before we'd see `Compiler ... with Solc 0.8.31` only.
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.

1 participant