-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
visualization_install_tests vs macOS python3.12 upgrade #21023
Comments
@mwoehlke-kitware with the venv, I believe we can probably revert the #21024 now. Could you give it a try? |
The mac job for the PR made to revert this one fails. Therefore it will be closed. |
It doesn't make any sense why Provisioned and Unprovisioned jobs would have a different result. That needs some investigation. |
Specifically, the PR does not change any It may very well be that this issue is still unfixed and the test cases are true positive failures, indicating that we need further changes to our code. However, we should first understand why the provisioned snapshot seems to be inauthentic, before proceeding further. |
In both of the unprovisioned job attempts there was a failure with //:py/install_test. In addition they mention a failed result of |
The
visualization_install_tests
are somewhat brittle w.r.t. how we choose macOS python versions.For now, I'm going to disable them on macOS to unbreak CI.
The problem is that the installed programs use the shebang line
#!/usr/bin/env python3
, which in Unprovisioned macOS builds is not/opt/homebrew/bin/python3.11
but something else ... maybe Apple Python? Anyway, it doesn't work.Probably to fix these tests, we need to set a
$PATH
with a magic symlink tosys.executable
.The text was updated successfully, but these errors were encountered: