Skip to content

Fix Windows Build#3

Open
matthew-mccall wants to merge 4 commits intoopen-algebra:mainfrom
matthew-mccall:fix-windows
Open

Fix Windows Build#3
matthew-mccall wants to merge 4 commits intoopen-algebra:mainfrom
matthew-mccall:fix-windows

Conversation

@matthew-mccall
Copy link
Copy Markdown
Contributor

PR Classification

Build system and project configuration improvements for better cross-platform and IDE support.

PR Summary

This pull request enhances build configuration and project setup for improved compatibility and developer experience across platforms and IDEs.

  • .gitignore: Added rules for Visual Studio and related tools to ignore more build and user-specific files.
  • CMakeLists.txt: Ensured OASIS builds as a static library and split its FetchContent logic.
  • CMakeLists.txt: Explicitly linked RARE target with SDL3::SDL3.
  • CMakeLists.txt: Added a post-build step to copy runtime DLLs for RARETests, fixing test execution on Windows.

Expanded .gitignore to exclude Visual Studio-specific files, build outputs, cache, test results, and other artifacts. This prevents unnecessary or sensitive files from being tracked and improves support for Visual Studio users.
- Fetch OASIS separately and force static build with BUILD_SHARED_LIBS OFF
- Link SDL3 explicitly to RARE target
- Add post-build step to copy runtime DLLs for RARETests
Use generator expressions to only copy runtime DLLs if present.
Echo a message instead when no DLLs are found, preventing
unnecessary commands and potential errors during post-build.
Replaces RARE_BUILD_PARANOID with RARE_BUILD_STRICT in both Linux and Windows CMake configurations. Updates the Windows CI workflow to build and run RARETests instead of OasisTests.
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