[Issue]: Tarball (migraphx-2.17.0+rocm10.0.0.tar.gz) is missing CMake-generated headers and config files
Nobody has claimed this yet.
- Dominant language
- C++
- Stars
- 333
- Forks
- 150
- Avg merge
- 4d 19h
- Merged PRs (30d)
- 54
Description
Problem Description
The official standalone Linux distribution tarball hosted at https://stable.repo.amd.com/rocm/migraphx/tarball/migraphx-2.17.0%2Brocm10.0.0.tar.gz appears to have packaged the raw source src/include/migraphx/ directory instead of the complete CMake installation output directory (CMAKE_INSTALL_INCLUDEDIR).
As a result, build-generated headers and CMake export modules are completely missing from the archive.
include/migraphx/api/export.h- Included by
include/migraphx/migraphx.h:31(#include <migraphx/api/export.h>). - Causes compilation failure immediately when including
<migraphx/migraphx.h>:/path/to/include/migraphx/migraphx.h:31:10: fatal error: 'migraphx/api/export.h' file not found 31 | #include <migraphx/api/export.h> | ^~~~~~~~~~~~~~~~~~~~~~~
- Included by
include/migraphx/config.h- Included by
include/migraphx/migraphx.h:32(#include <migraphx/config.h>).
- Included by
include/migraphx/version.h- Missing version definition header for external API consumers.
lib/cmake/migraphx/migraphx-config.cmake(and related export targets)- The
cmake/folder in the tarball is empty, breakingfind_package(migraphx REQUIRED CONFIG)and CMake imported targets (migraphx::c).
- The
Operating System
Alma Linux 8
CPU
Github Runner
ROCm Version
ROCm 10.0.0
Contributor guide
No contributing guide indexed for this repository
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start by inspecting the standalone archive at the linked tarball URL and compare its contents with the CMake installation output, especially CMAKE_INSTALL_INCLUDEDIR and lib/cmake/migraphx. Verify that export.h, config.h, version.h, migraphx-config.cmake, and related export targets are packaged, then confirm external inclusion and find_package(migraphx REQUIRED CONFIG) work.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- cmake, cpp
- Domain
- build-system, release
- Issue type
- Bug
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Active
- Clarity
- Mostly clear
- Newbie friendliness
- 63/100