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

Docs: update recommended Clang version #1130

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

merlinND
Copy link
Member

Hi,

@jhoydis pointed out recently that the current build guide's instructions:

sudo apt install clang-10 libc++-10-dev libc++abi-10-dev cmake ninja-build

no longer work on e.g. Ubuntu 22.04 (and most likely more recent versions as well), because the libc++-10-dev and libc++abi-10-dev packages no longer exist.

I understand that Clang 10 is used in the CI and it makes sense to recommend the same compiler version that is tested, but since it's not working anymore, would switching to Clang 15 in the docs be okay?

@njroussel
Copy link
Member

Hi @merlinND

Sorry for the delay. We're changing the environments used by our CI. I'm all for having a more standard & up-to-date Linux environment and build instruction. What you suggest seems fine, I'd just prefer that it perfectly matches what our CI does. I'll report back - this will happen with the nanobind redlease.

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.

2 participants