build: Small build improvement for speed and fix CI for CY2023#1223
Merged
cedrik-fuoco-adsk merged 12 commits intoAcademySoftwareFoundation:mainfrom Apr 15, 2026
Merged
Conversation
cedrik-fuoco-adsk
force-pushed
the
ci-speed-up-1
branch
3 times, most recently
from
b5c46da to
0014a31
Compare
Signed-off-by: Cédrik Fuoco <cedrik.fuoco@autodesk.com>
Signed-off-by: Cédrik Fuoco <cedrik.fuoco@autodesk.com>
Signed-off-by: Cédrik Fuoco <cedrik.fuoco@autodesk.com>
Signed-off-by: Cédrik Fuoco <105517825+cedrik-fuoco-adsk@users.noreply.github.com> Signed-off-by: Cédrik Fuoco <cedrik.fuoco@autodesk.com>
Signed-off-by: Cédrik Fuoco <105517825+cedrik-fuoco-adsk@users.noreply.github.com> Signed-off-by: Cédrik Fuoco <cedrik.fuoco@autodesk.com>
Signed-off-by: Cédrik Fuoco <105517825+cedrik-fuoco-adsk@users.noreply.github.com> Signed-off-by: Cédrik Fuoco <cedrik.fuoco@autodesk.com>
Signed-off-by: Cédrik Fuoco <cedrik.fuoco@autodesk.com>
Signed-off-by: Cédrik Fuoco <cedrik.fuoco@autodesk.com>
Signed-off-by: Cédrik Fuoco <cedrik.fuoco@autodesk.com>
Signed-off-by: Cédrik Fuoco <cedrik.fuoco@autodesk.com>
The relocatability test was moving the entire Python install directory to a temp location, causing concurrent builds (e.g. OCIO) to fail with 'Cannot open include file: Python.h' when they tried to access headers from the temporarily-missing directory. Copy instead of move so the original install stays in place. Signed-off-by: Cédrik Fuoco <cedrik.fuoco@autodesk.com>
cedrik-fuoco-adsk
force-pushed
the
ci-speed-up-1
branch
from
37881c9 to
c28eb74
Compare
cedrik-fuoco-adsk
changed the title
cedrik-fuoco-adsk
requested review from
bernie-laberge and
eloisebrosseau
as code owners
Signed-off-by: Cédrik Fuoco <cedrik.fuoco@autodesk.com>
cedrik-fuoco-adsk
force-pushed
the
ci-speed-up-1
branch
from
9391c45 to
4a16c49
Compare
eloisebrosseau
approved these changes
Apr 15, 2026
cedrik-fuoco-adsk
merged commit adb66ba
into
AcademySoftwareFoundation:main
17 checks passed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Small build improvement for speed and fix CI for CY2023
Linked issues
none
Summarize your change.
Describe the reason for the change.
The windows build is very slow and any minor improvement helps overtime.
OpenSSL PDB contention:
Multiple cl.exe processes writing to the same app.pdb under JOM causes sporadic Error 2 failures.
Fixed by setting CL=/Z7 to embed debug info
Python test race condition:
test_python.py was moving the Python install directory to a temp location for relocatability testing.
Any concurrent build (e.g OCIO) needing Python.h would fail with C1083: Cannot open include file.
Fixed by copying instead of moving.
Describe what you have tested and on which operating system.
Windows and CI (all platforms)
Add a list of changes, and note any that might need special attention during the review.
•
.github/actions/build-windows/action.yml: Install JOM via Chocolatey and add it to PATH•
cmake/dependencies/raw.cmake: Use JOM (if available) instead of nmake for LibRaw builds•
cmake/dependencies/ocio.cmake: Re-enable--parallelfor OCIO Windows builds•
cmake/dependencies/oiio.cmake: Skip OCIO dependency for CY2023 (same behavior as before)•
cmake/defaults/CY2023.cmake: Add OIIO 2.5.19.1 version/hash for CY2023•
src/build/make_openssl.py: Use JOM with /FS + /Z7 for parallel OpenSSL builds; add no-tests to configure•
src/build/test_python.py: Use copytree instead of move so the install dir stays in place during testing (fixes thePython.hnot found race condition)•
src/lib/base/stl_ext/CMakeLists.txt: Enable unity build for stl_ext