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

Windows build: improvement to DLL copying #1282

Merged
merged 1 commit into from
Jan 4, 2025

Conversation

agalasso
Copy link
Contributor

@agalasso agalasso commented Jan 4, 2025

Copy the Windows DLLs with a single call to cmake -E rather than one call per DLL. This slightly speeds up the build for a better experience in iterative development on Windows.

Also moves the DLL copying to POST_BUILD rather then PRE_BUILD as there's no point in copying the DLLs if there's going to be a compile error during iterative development.

@agalasso agalasso requested a review from bwdev01 January 4, 2025 20:00
@agalasso
Copy link
Contributor Author

agalasso commented Jan 4, 2025

re: clang-format check failure -- it's unrelated to this pr and got introduced by #1276 and is fixed by #1278

Copy the Windows DLL with a single call to `cmake -E` rather than one call per
DLL. This slightly speeds up the build for a better experience in iterative
developement on Windows.

Also moves the DLL copying to POST_BUILD rather then PRE_BUILD as there's
no point in copying the DLLs if there's going to be a compile error
during iterative development.
@agalasso agalasso force-pushed the andy/Windows-cmake-DLL-copying-improvement branch from 7340b90 to fe549e5 Compare January 4, 2025 22:17
@agalasso agalasso merged commit c194aa7 into master Jan 4, 2025
4 checks passed
@agalasso agalasso deleted the andy/Windows-cmake-DLL-copying-improvement branch January 4, 2025 22:27
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