Skip to content

馃摑 Document CMake consumption - #538

Open
marcelwa wants to merge 3 commits into
developfrom
codex/document-cmake-consumption
Open

馃摑 Document CMake consumption#538
marcelwa wants to merge 3 commits into
developfrom
codex/document-cmake-consumption

Conversation

@marcelwa

Copy link
Copy Markdown
Collaborator

馃 AI text below 馃

Description

QDMI already installs a CMake package, but consumers have to infer the minimal installation recipe and release-tag convention from the build files. Add a focused guide for installing the header-only interface, consuming qdmi::qdmi with find_package, and embedding a pinned source release with FetchContent. Link it from the README and documentation home page and include it in Doxygen.

The guide distinguishes v1.3.3 release tags from 1.3.3 CMake versions and explains patch compatibility, compiler requirements, and the separate need for a driver/device implementation. This documents existing behavior; it adds no binary artifacts or API changes. Motivated by openQSE/QFw#52.

Validation on Windows with GCC 16.2 and CMake 4.4:

  • Minimal configuration and installation with tests, examples, templates, and docs disabled passed.
  • External C11 consumers built successfully against the installed package and through FetchContent with a local source override, using this checkout (1.3.4-dev).
  • The Doxygen build passed with version 1.17.0. It reports the existing obsolete DOT_MULTI_TARGETS configuration warning.
  • uvx prek run -a and git diff --cached --check passed after formatting.

GPT-6 Astra via Codex drafted the documentation and ran the local checks. Human reviewers retain acceptance decisions and responsibility for reviewing and understanding the AI-assisted changes.

Checklist

  • The pull request only contains commits that are focused and relevant to this change.
  • I have added appropriate tests that cover the new/changed functionality.
  • I have updated the documentation to reflect these changes.
  • I have added entries to the changelog for any noteworthy additions, changes, fixes, or removals.
  • I have added migration instructions to the upgrade guide (if needed).
  • The changes follow the project's style guidelines and introduce no new warnings.
  • The changes are fully tested and pass the CI checks.
  • I have reviewed my own code changes.

If PR contains AI-assisted content:

  • I have disclosed the use of AI tools in the PR description as per our AI Usage Guidelines.
  • AI-assisted commits include an Assisted-by: [Model Name] via [Tool Name] footer.
  • I confirm that I have personally reviewed and understood all AI-generated content, and accept full responsibility for it.

No behavior changed, so no permanent tests or migration instructions were added. Hosted CI and human-review attestations remain unchecked.

Explain release tags, minimal header installation, and installed or embedded CMake consumption.

Assisted-by: GPT-6 Astra via Codex
@marcelwa marcelwa added the documentation Improvements or additions to documentation label Sep 10, 2026
@github-actions

github-actions Bot commented Sep 10, 2026

Copy link
Copy Markdown
Contributor
PR Preview Action v1.8.1

QR code for preview link

馃殌 View preview at
https://Munich-Quantum-Software-Stack.github.io/QDMI/pr-preview/pr-538/

Built to branch gh-pages at 2026-09-10 10:06 UTC.
Preview will be ready when the GitHub Pages deployment is complete.

Assisted-by: GPT-6 Astra via Codex
@marcelwa marcelwa self-assigned this Sep 10, 2026
Preserve both changelog PR references when integrating current develop.

Assisted-by: GPT-6 Astra via Codex
@marcelwa
marcelwa requested a review from burgholzer September 10, 2026 10:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant