-
Notifications
You must be signed in to change notification settings - Fork 633
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 NanoVDB CI #1815
base: master
Are you sure you want to change the base?
Fix NanoVDB CI #1815
Conversation
Signed-off-by: apradhana <[email protected]>
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.
looks good to me
- name: Setup cmake | ||
uses: jwlawson/actions-setup-cmake@v2 | ||
with: | ||
cmake-version: '3.28.x' |
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.
I'd rather not use random people's actions here
@apradhana thanks for trying to look into this - but are you saying we only support CMake version up to 3.28 with this change? That doesn't sounds appropriate to me, can you link me to any filed issues with the newer versions of CMake and zlib through vcpkg please |
Hi @Idclip , this is the issue I was looking at: https://gitlab.kitware.com/cmake/cmake/-/issues/25355. |
@Idclip : I also got an email from github explaining "Jobs using the macos-11 YAML workflow label should be updated to macos-14, macos-13, macos-12, or macos-latest.". So I'm trying out the macos-14 runner in case it fixes the macos problem. |
Signed-off-by: apradhana <[email protected]>
@apradhana the issue you linked is talking about a regression in CMake 3.27. Your PR seems to be back-porting CMake from 3.29 to 3.28. I'm not sure why your change is related to this. It also looks like more recent Windows builds have been succeeding so I believe this has been fixed on vcpkg mainline. Currently all our CI runners are offline (I have no idea why). When they are back up I will investigate this |
All of the ASWF CI runners are still offline |
Signed-off-by: apradhana <[email protected]>
This is an attempt to fix NanoVDB CI. For Windows, this boils down to downgrading the CMake to
cmake 3.28
. I follow an example given on this page to downgrade the CMake version. Perhaps I should also update the documentation here: https://www.openvdb.org/documentation/doxygen/dependencies.html#depKnownIssues.I tried the fix that @Idclip suggested for MacOS, but the CI is still failing when the homebrew installs the dependencies.