Skip to content

⚡ Preserve generated header timestamps - #537

Merged
burgholzer merged 3 commits into
developfrom
codex/preserve-generated-header-timestamps
Sep 8, 2026
Merged

⚡ Preserve generated header timestamps#537
burgholzer merged 3 commits into
developfrom
codex/preserve-generated-header-timestamps

Conversation

@burgholzer

@burgholzer burgholzer commented Sep 8, 2026

Copy link
Copy Markdown
Contributor

🤖 AI text below 🤖

Description

Unchanged CMake configuration rewrote every prefixed QDMI header, forcing dependent device sources to compile again. Generate the output only when its contents change, while preserving literal @...@ and ${...} text.

The change is limited to header generation and its changelog entry. The added standalone regression test was removed at the maintainer's request. uvx prek run -a passes on this revision; fresh CI is pending.

GPT-6 via Codex implemented and validated this change at the maintainer's request.

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.

Avoid rebuilding device implementations after unchanged CMake configuration. Cover unchanged timestamps, changed content, and literal header placeholders.

Assisted-by: GPT-6 via Codex
@burgholzer burgholzer added the fix Fixes something that isn't working label Sep 8, 2026
@burgholzer burgholzer self-assigned this Sep 8, 2026
Standalone scripts do not inherit the project policy version. CMake 3.x otherwise expands the generated header content twice under CMP0053 OLD. Match the project minimum of CMake 3.24.

Assisted-by: GPT-6 via Codex
Remove the standalone generation test and add the pull request and author to the changelog.

Assisted-by: GPT-6 via Codex
@burgholzer
burgholzer merged commit 7cf3dfd into develop Sep 8, 2026
19 checks passed
@burgholzer
burgholzer deleted the codex/preserve-generated-header-timestamps branch September 8, 2026 00:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

fix Fixes something that isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant