Minor build and CI fixes#1599
Conversation
hodoulp
left a comment
There was a problem hiding this comment.
Good job. I made a comment to better explain the new unit test.
|
@remia What is the status of this pull request? |
doug-walker
left a comment
There was a problem hiding this comment.
Thanks Remi!
There seems to be an analysis build failure on Windows related to OpenEXRCore, but I didn't try to figure out why it's happening on one Windows build but not another. Any thoughts on what went wrong?
Hi @doug-walker, this is the issue I mentioned in the PR description, OpenEXR doesn't compile on Windows MSVC 2022 currently, while the issue has now been fixed upstream, it's not released yet. I expect the analysis build will be fixed when that is the case. We shouldn't wait for this however. |
Signed-off-by: Rémi Achard <remiachard@gmail.com>
Signed-off-by: Rémi Achard <remiachard@gmail.com>
Signed-off-by: Rémi Achard <remiachard@gmail.com>
Signed-off-by: Rémi Achard <remiachard@gmail.com>
Signed-off-by: Rémi Achard <remiachard@gmail.com>
Signed-off-by: Rémi Achard <remiachard@gmail.com>
* opencolorio: add v2.4.2 * Update xmake.lua * Update xmake.lua * Update xmake.lua * fix yaml-cpp * Fix GCC 15 according to AcademySoftwareFoundation/OpenColorIO#1599 * fix macosx minizip-ng * limit mingw --------- Co-authored-by: Saikari <lin@sz.cn.eu.org>
Fix the analysis workflow
The install location for built dependencies in the analysis workflow is now explicit in Linux and macOS (Windows already use a custom location). This is because OpenImageIO now started to override the install location to point in the build tree when none is specified, instead of using standard Unix paths.
Currently failing on OpenEXR Windows build, see related issue AcademySoftwareFoundation/openexr#1236
Fix issue in static builds on Windows #1581
Fix #1593
Improve documentation for OFX build #1607