ROCm / rocm_smi_lib

ROCm SMI LIB
https://rocm.docs.amd.com/projects/rocm_smi_lib/en/latest/
MIT License
111 stars 48 forks source link

[Issue]: library has version 7.0.0.0 in 6.0.0 tarball and 2.8.0.37 in 6.0.x branch #157

Open imwints opened 4 months ago

imwints commented 4 months ago

Problem Description

We want to detect the installed rocm version with version.major, but on ArchLinux, which installs from the 6.0.0 tarball, the reported major version is 7. I checked the 6.0.x branch and there the major version is 2.

Also in CMakeLists.txt, in both the tarball (6.0.0) and the git tag (6.0.x) the get_package_version_number passes "7.0.0" as the default version for parsing (which makes little sense to me, this is not version 7 but 6 [and how on earth does the get_package_version_number produces 2.8.0.37???]).

Operating System

Irrelevant

CPU

Irrelevant

GPU

AMD Radeon VII

ROCm Version

ROCm 6.0.0

ROCm Component

rocm_smi_lib

Steps to Reproduce

git checkout rocm-6.0.x cmake -B build -G Ninja The configure step will already print the version number

Do the same for the 6.0.0 release tarball

(Optional for Linux users) Output of /opt/rocm/bin/rocminfo --support

No response

Additional Information

No response

imwints commented 4 months ago

This doesn't happen when you do a shallow clone with git clone --depth 1 ..., then the version is 7.0.0.0 again instead of 2.8.0.37 (and the configure output states that git rev-parse failed)