open-telemetry / open-telemetry/opentelemetry-cpp
[CMAKE] Use file sets for header file install manifests and verification
Open
Nobody has claimed this yet.
build and test
do-not-stale
issue:blocked
triage/accepted
- Dominant language
- C++
- Stars
- 1.4k
- Forks
- 632
- Avg merge
- 1d 13h
- Merged PRs (30d)
- 75
Description
[Blocked on upgrading the minimum CMake support to 3.24]
- Use File Sets for headers to define install manifests per component.
- Create a CI test to verify header file sets using CMake's
VERIFY_INTERFACE_HEADER_SETS.
See: https://cmake.org/cmake/help/latest/prop_tgt/VERIFY_INTERFACE_HEADER_SETS.html
Contributor guide
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
The issue names no specific files or tests. First confirm the minimum CMake version and locate the component install manifests and CI configuration; then review CMake file-set documentation and VERIFY_INTERFACE_HEADER_SETS. Done means header file sets define per-component install manifests and CI verifies the interface headers.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- cmake, cpp
- Domain
- build-system, ci-cd
- Issue type
- Feature
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Quiet
- Clarity
- Mostly clear
- Newbie friendliness
- 55/100