Skip to content

Fix vkloadtests build with cmake 3.29+#1010

Closed
MarkCallow wants to merge 3 commits intomainfrom
fix_vkloadtests_build_with_cmake_329
Closed

Fix vkloadtests build with cmake 3.29+#1010
MarkCallow wants to merge 3 commits intomainfrom
fix_vkloadtests_build_with_cmake_329

Conversation

@MarkCallow
Copy link
Copy Markdown
Collaborator

Fixes an issue with vkloadtests for iOS not running, caused by a change a little while ago.

Fixes a build issue with vkloadtests for macOS with recent versions of CMake that recognize .xcframeworks.

@MathiasMagnus you may need to merge this into your GitHub actions PR in order to have successful builds, depending on what version of CMake is on the CI runners.

Fixes vkloadtests RPATH on iOS.
3.29+ recognize .xcframeworks so FindVulkan was finding MoltenVK.xcframework
which has only a static library not suitable for use in the app bundle.

Removal of some tabs from CMakeLists.txt is along for the ride.
MarkCallow added a commit that referenced this pull request Apr 14, 2025
Port Linux, macOS and web workflows from Travis to GitHub Actions.
Splits the Travis workflow into separate Linux, iOS & macOS, web
and check-reuse workflows. Adds a new `manual` workflow for
requesting build of all platforms.

Incorporates PR #1010 for fixes to build CMake 3.29+ as runners
have recent CMake versions.

Co-authored-by: Mark Callow <github@callow.im>
@MarkCallow
Copy link
Copy Markdown
Collaborator Author

Incorporated in PR #1004.

@MarkCallow MarkCallow closed this Apr 14, 2025
@MarkCallow MarkCallow deleted the fix_vkloadtests_build_with_cmake_329 branch April 14, 2025 11:53
richgel999 pushed a commit to BinomialLLC/KTX-Software-Binomial-Fork that referenced this pull request Mar 9, 2026
Port Linux, macOS and web workflows from Travis to GitHub Actions.
Splits the Travis workflow into separate Linux, iOS & macOS, web
and check-reuse workflows. Adds a new `manual` workflow for
requesting build of all platforms.

Incorporates PR KhronosGroup#1010 for fixes to build CMake 3.29+ as runners
have recent CMake versions.

Co-authored-by: Mark Callow <github@callow.im>
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