-
Notifications
You must be signed in to change notification settings - Fork 6.4k
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
[openmvs] Update to 1.0.1 #9563
Conversation
the automatic check for linux is wrong, the build fails (as reported above), but the check still passes |
@cdcseacave, thanks for the PR! openmvs failed on linux but doesn't fail on CI testing, that due to it also failed on baseline, so it's not a regressions. Additionally 'openmvs:x64-linux=fail 'added to ci.baseline.txt, when the failures on linux fixed, the 'openmvs:x64-linux=fail ' should be removed. |
@PhoebeHui can you pls help with the linux build? |
/azp run |
@cdcseacave, sure, I will try to fix linux build issue in separate PR. |
@PhoebeHui That is fine with me, pls feel free to merge, thx |
This change updates OpenMVS library to v1.0.1. I've tested it on Windows and works as expected, but on Ubuntu 18.04 it fails with an error that does not seem related to my library; can you pls help solve this?