Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix Windows DLL test configuration #3807

Merged

Conversation

jeremy-lunarg
Copy link
Contributor

@jeremy-lunarg jeremy-lunarg commented Dec 10, 2024

Enable Windows DLL test configuration.
Configure tests to pull in DLL dependencies using TARGET_RUNTIME_DLL_DIRS.
Bump cmake version to 3.27 to use TARGET_RUNTIME_DLL_DIRS.
Remove Ubuntu 20.04 test configuration.

Copy link
Contributor

@dneto0 dneto0 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please use cmake-3.31.2 in the kokoro scripts.

@@ -39,7 +39,7 @@ set -e # Fail on any error.

set -x # Display commands being run.

using cmake-3.17.2
using cmake-3.27
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This needs to correspond to a cmake that has been put into the kokoro image.
It should be cmake-3.31.2

@jeremy-lunarg jeremy-lunarg force-pushed the hayes-fix-windows-dll-ci branch from 6d499a8 to cee57f8 Compare December 12, 2024 19:23
Copy link
Contributor

@dneto0 dneto0 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks!

@jeremy-lunarg jeremy-lunarg force-pushed the hayes-fix-windows-dll-ci branch from cee57f8 to 8c87d49 Compare December 12, 2024 19:28
Enable Windows DLL test configuration.
Configure tests to pull in DLL dependencies using TARGET_RUNTIME_DLL_DIRS.
Bump cmake version to 3.27 to use TARGET_RUNTIME_DLL_DIRS.
Remove Ubuntu 20.04 test configuration.
@jeremy-lunarg jeremy-lunarg force-pushed the hayes-fix-windows-dll-ci branch from 8c87d49 to 90e1405 Compare December 12, 2024 20:22
@jeremy-lunarg jeremy-lunarg merged commit 340bf88 into KhronosGroup:main Dec 12, 2024
29 checks passed
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.

2 participants