Skip to content

remove explicit version from conan profile - use system#520

Merged
SavenkovIgor merged 7 commits intomainfrom
simplify-env
Mar 15, 2026
Merged

remove explicit version from conan profile - use system#520
SavenkovIgor merged 7 commits intomainfrom
simplify-env

Conversation

@SavenkovIgor
Copy link
Owner

No description provided.

Copilot AI review requested due to automatic review settings March 14, 2026 23:10
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the Conan base Clang profile to stop referencing version-suffixed compiler executables, intending to rely on the system-default clang/clang++ instead.

Changes:

  • Switch CC from clang-20 to clang.
  • Switch CXX from clang++-20 to clang++.
Comments suppressed due to low confidence (1)

conanfiles/profile/base/clang:7

  • CC/CXX were changed to clang/clang++, but this profile still pins compiler.version=20. Unless the environment guarantees that clang resolves to Clang 20 (e.g., via PATH/update-alternatives), Conan will believe it is building with Clang 20 while actually invoking a different version, which can break builds and/or produce incorrect package IDs. Either keep clang-20/clang++-20, or remove/align the pinned compiler version and ensure CI/dev environments set the intended default compiler.
CXX=clang++

[settings]
compiler=clang
compiler.version=20

@SavenkovIgor SavenkovIgor enabled auto-merge (squash) March 15, 2026 00:03
@SavenkovIgor SavenkovIgor merged commit 0ef004d into main Mar 15, 2026
4 checks passed
@SavenkovIgor SavenkovIgor deleted the simplify-env branch March 15, 2026 00:10
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