Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Scrypto coverage, llvm no recognized #1707

Open
a137x opened this issue Feb 1, 2024 · 2 comments
Open

Scrypto coverage, llvm no recognized #1707

a137x opened this issue Feb 1, 2024 · 2 comments

Comments

@a137x
Copy link
Contributor

a137x commented Feb 1, 2024

scrypto coverage command produces error, not recognizing installed llvm of proper version.

❯ scrypto coverage
Missing command: llvm-cov-17. Please install LLVM version matching rustc LLVM version, which is 17.
For more information, check https://apt.llvm.org/
Error: CoverageError(MissingLLVM)

test environment:
Apple Silicon, MacOS 14.2.1, llvm installed via homebrew

❯ llvm-config --version
17.0.6

❯ scrypto -V
scrypto 1.1.0

❯ rustup -V
rustup 1.26.0 (2023-04-05)
info: This is the version for the rustup toolchain manager, not the rustc compiler.
info: The currently active `rustc` version is `rustc 1.74.1 (a28077b28 2023-12-04)`
@a137x
Copy link
Contributor Author

a137x commented Feb 1, 2024

Apparently the culprit lies in spelling of llvm-cov command (at least to my specified OS)
https://github.com/radixdlt/radixdlt-scrypto/blob/main/simulator/src/scrypto/cmd_coverage.rs#L104

@bbarwik
Copy link
Contributor

bbarwik commented Feb 2, 2024

Next week I'll add PR with the fix, it will check version of llvm before trying llvm-17.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

No branches or pull requests

2 participants