-
Notifications
You must be signed in to change notification settings - Fork 235
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鈥檒l occasionally send you account related emails.
Already on GitHub? Sign in to your account
[Core] make remove installation directories optional #11540
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
please use more modern CMake syntax
@jrubiogonzalez just out of curiosity:
|
Co-authored-by: Philipp Bucher <philipp.bucher@tum.de>
We are compiling around 17 applications total between public and private and adding few extra libraries, including mkl. That sums up about 700Mb in win and far more in linux. If we add our environment anti-virus that is quite intrusive (specially in windows) we go in a 0 build (no changes at all) in my workstation from 15s to 5s. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice, thx!
馃摑 Description
This PR makes removing KratosMultiphysics and libs directory optional. This might be dangerous if some applications get stuck, but on our case as we have plenty of external dll's and several applications this reduces the build time for small changes significantly.