Conversation
|
If we use fmt and pugixml from apt, we should not include them as submodules. |
|
We include the submodules as a backup so that people are not required to have them installed already. However, if they do have them installed and |
|
I am running this locally and getting a warning and error. I think the warning was there before the PR but is anyone else getting The CI doesn't test this docker build till it is pushed to develop so this is why I was building locally |
|
Thanks for testingit out @shimwell! Apparently |
Description
This PR makes a few updates to our Dockerfile to speed up build times:
The Embree change alone should shave off 15-20 min of build time.
Checklist
I have run clang-format (version 15) on any C++ source files (if applicable)I have followed the style guidelines for Python source files (if applicable)I have made corresponding changes to the documentation (if applicable)I have added tests that prove my fix is effective or that my feature works (if applicable)