Skip to content

Mark ufcxConfigVersion.cmake architecture-independent - #924

Merged
jhale merged 4 commits into
mainfrom
jhale/ufcx-arch-independent
Oct 18, 2026
Merged

jhale merged 4 commits into
mainfrom
jhale/ufcx-arch-independent

Conversation

@jhale

@jhale jhale commented Oct 18, 2026

Copy link
Copy Markdown
Member

Stacked on #922. ufcx is a header-only INTERFACE library, but its version file records the installing machine's CMAKE_SIZEOF_VOID_P. A consumer with a different pointer size is rejected as incompatible for no reason. ARCH_INDEPENDENT drops that check. The generated ufcxConfigVersion.cmake goes from three CMAKE_SIZEOF_VOID_P checks to none.

🤖 Generated with Claude Code

https://claude.ai/code/session_01JudYk8UecTQyirGsLsHUN2

jhale and others added 3 commits October 18, 2026 15:18
AnyNewerVersion lets find_package(ufcx 0.12) accept ufcx 0.13, whose
ufcx.h interface differs.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01JudYk8UecTQyirGsLsHUN2
SameMinorVersion supports version ranges from CMake 3.21.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01JudYk8UecTQyirGsLsHUN2
ufcx is header-only, so the pointer-size check rejects consumers for no
reason.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01JudYk8UecTQyirGsLsHUN2
Base automatically changed from jhale/cmake-same-minor-version to main October 18, 2026 16:28
@jhale
jhale added this pull request to the merge queue Oct 18, 2026
@github-merge-queue
github-merge-queue Bot removed this pull request from the merge queue due to no response for status checks Oct 18, 2026
@jhale
jhale added this pull request to the merge queue Oct 18, 2026
Merged via the queue into main with commit 9f3da98 Oct 18, 2026
18 checks passed
@jhale
jhale deleted the jhale/ufcx-arch-independent branch October 18, 2026 20:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant