[bug] detect_api.detect_clang_compiler(compiler_exe="clang") returns an incorrect version about conan HOT 3 OPEN

Hi @Todiq

I think this is intended behavior of the API. One api is to detect the full version of clang, in your case is 17.0.

Then, Conan default settings might not include certain versions. To map full compiler versions to the actual versions, there is another api called default_compiler_version, that can be used:

compiler.version={{detect_api.default_compiler_version(compiler, version)}}

This way the api gives power to users that want the full clang version, while allowing to also define the predefined settings.yml default versions with the default Conan compatibility.

Check https://docs.conan.io/2/reference/config_files/profiles.html for more details.

from conan.

Hello @memsharded,

Thanks for your answer. If the API's goal is to detect the full version, shouldn't it then also add the minor version? I am currently running clang 17.0.3, not 17.0.

compiler.version={{detect_api.default_compiler_version(compiler, version)}}

Thanks for the hint, but I cannot use that as is. In fact, when there are more than one compiler installed, I can't be sure of what the result would be. In my case, it returns msvc's version, which is 194.

I guess there would be a way in Jinja2's syntax to remove the .0 of 17.0?

from conan.

Thanks for the hint, but I cannot use that as is. In fact, when there are more than one compiler installed, I can't be sure of what the result would be. In my case, it returns msvc's version, which is 194.

But you can call it with default_compiler_version(clang, clang_version) from your code above:

{% set clang, clang_version, clang_exe = detect_api.detect_clang_compiler(compiler_exe="clang") %}
{% set clang_final_version = default_compiler_version(clang, clang_version) %}
...
myproj*/*:compiler.version = {{clang_final_version}}

Thanks for your answer. If the API's goal is to detect the full version, shouldn't it then also add the minor version? I am currently running clang 17.0.3, not 17.0.

To be accurate, Conan default settings define in some cases, as gcc, both major and major+minor versions:

"10", "10.1", "10.2", "10.3", "10.4", "10.5",
"11", "11.1", "11.2", "11.3", "11.4",

If users want to chose which one of the defaults they use, they can use the one with the minor, or the one returned by default_compiler_version. It is true that we haven't seen anyone so far that wanted to model down to the patch version of the compiler.

from conan.