chore: [CG-10573] make clang dependency more explicit for debian linux to preven… #211
Conversation
…t initial clang error
|
|
Codecov ReportAll modified and coverable lines are covered by tests ✅ ✅ All tests successful. No failed tests found. Additional details and impacted files |
| rm uv.lock && uv sync --dev | ||
| ``` | ||
|
|
||
| - If `uv sync` fails due to a compilation error, ensure `clang` is installed and then rerun: |
There was a problem hiding this comment.
It can be clang or g++ on linux, not entirely sure
There was a problem hiding this comment.
I can test this
There was a problem hiding this comment.
Without clang and the latest version of g++, I still get the error requiring clang.
There was a problem hiding this comment.
yeah, you need clang or change CC var (export CC=gcc)
…ides-for-sdk-contributing
…ides-for-sdk-contributing
…t initial clang error
Motivation
Prevents getting an error when following the installation steps in contributions for the first time without seeing the clang tip at the bottom of the steps. The error may is embedded in uv logs and may not be immediately obvious.
Content
Add debian/ubuntu section to contribution section to prevent the first time setup error.
Testing
Please check the following before marking your PR as ready for review
Tested the installation steps on an Ubuntu machine.