-
-
Notifications
You must be signed in to change notification settings - Fork 27
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
The setup-cpp action wrongly installs Windows LLVMs to the same folder #249
Comments
|
That's what I intended to do, and worked well on (ubuntu-22.04, llvm-16.0.6, *, *). Therefore I opened this issue to know whether the behavior of intalling different versions of Windows LLVM to the same folder is intentional.
I thought the separated Could this be fixed by installing Windows LLVMs to different folders, or simply by reporting an error when the clang-tidy and llvm version don't match? |
The code here hands syncing the version for these tools. It can be improved to use the specific version. I think the reason it doesn't work is that Line 59 in 34bb783
We should probably parse the compiler field and inform the sync logic about that. Then later install the parsed information. A tuple of compiler + version and separating this switch case into a separate function should be good enough Line 44 in 34bb783
|
From the log of (windows-2022, llvm-16.0.6, *, *) jobs, for the setup-cpp action specified as
clang-tidy triggers an installation of llvm 17.0.6 (see Setup cpp#472). After that, llvm 16.0.6 is triggered to be installed to the same folder (see Setup cpp#554), which is, however, not installed according to the log.
To further ensure that llvm 16.0.6 is not installed, I directly specify the clang version to 16.0.6 in the corresponding conan profile, but cmake still uses llvm-17.0.6 according to Configure cmake#76.
Upvote & Fund
@aminya is using Polar.sh so you can upvote and help fund this issue. The funding is received once the issue is completed & confirmed by you.
Thank you in advance for helping prioritize & fund our backlog!
The text was updated successfully, but these errors were encountered: