-
Notifications
You must be signed in to change notification settings - Fork 431
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
Adsk Contrib - Better manage Imath dependency #1478
Conversation
Signed-off-by: Patrick Hodoul <Patrick.Hodoul@autodesk.com>
I did my tests on |
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.
lgtm
The PR is a candidate for RB-2.1 |
Signed-off-by: hodoulp <Patrick.Hodoul@autodesk.com>
Signed-off-by: Patrick Hodoul <Patrick.Hodoul@autodesk.com>
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.
Approving immediately since this bug fix is needed to unbreak the CI builds.
* Adsk Contrib - Better manage Imath dependency Signed-off-by: Patrick Hodoul <Patrick.Hodoul@autodesk.com> * Add the OpenImageIO_Util dependency to OSL unit test framework Signed-off-by: hodoulp <Patrick.Hodoul@autodesk.com> * Fix oiio typo Signed-off-by: Patrick Hodoul <Patrick.Hodoul@autodesk.com>
* Adsk Contrib - Better manage Imath dependency Signed-off-by: Patrick Hodoul <Patrick.Hodoul@autodesk.com> * Add the OpenImageIO_Util dependency to OSL unit test framework Signed-off-by: hodoulp <Patrick.Hodoul@autodesk.com> * Fix oiio typo Signed-off-by: Patrick Hodoul <Patrick.Hodoul@autodesk.com>
Signed-off-by: Patrick Hodoul Patrick.Hodoul@autodesk.com
Using
Imath
in the library could sometime conflict with theImath
fromOpenImageIO
. When you try on ubuntu 20.04 using the systemOpenImageIO
the compilation ofOpenColorIO
fails because of symbol clashes onImath
(i.e. 3.1.x from OpenColorIO 2.1.x vs. 3.0.x expected by OpenImageIO 2.1.x).The pull request changes the code to use the OpenImageIO one for apps using OpenImageIO. I also removed some useless
Imath
dependencies.