-
Notifications
You must be signed in to change notification settings - Fork 459
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
Update to yaml 0.7.0 #1466
Comments
Note: Have a look to #1451 to improve/fix it. |
I am also getting an exception when trying to run OCIO v2.0.1 installed via VCPKG. The exception is happening in both YAML-cpp 0.7.0 and 0.6.3 |
Trying yaml 0.7.0 on mac OS 11.5.2, master branch, I didn't hit any major issues. Fixes have to be made on the unit test suite related to multi-lines comment fields no longer having spaces on empty lines, but that's pretty much it. |
I didn't have this issue on OSX with the same code base. Only issue is on Windows 10. |
@riots100 Do you have anything new on this error ? If not, can you give more details on how to reproduce it ? Thanks. |
merged in |
Following the fix for vcpkg (#1449), I found that vcpkg compiles
OpenColorIO 2.0.1
againstyaml 0.7.0
. That's the latest version of yaml, and we never validated that combination.For
2.1.x
we should think about updating yaml.For
2.0.x
I suggest to also update yaml. To be discussed at the TSC meeting i.e. what is the policy for the mandatory third-party library update (those not part of the VFX Reference platform)?Links:
https://github.com/jbeder/yaml-cpp/releases/tag/yaml-cpp-0.7.0
jbeder/yaml-cpp@yaml-cpp-0.6.3...yaml-cpp-0.7.0
The text was updated successfully, but these errors were encountered: