Skip to content

Commit 6f2dfd1

Browse files
authored
ci: optimize openexr build time, don't run nightly on forks (#2097)
* When building openexr as a dependency for OSL, there is no need to build the openexr command line utilities, saving a bit of build time. * Fix an error from PR #2074 where we tried to disable nightly CI runs on forks -- we didn't make the right change to the Windows jobs, so they were still running. Signed-off-by: Larry Gritz <lg@larrygritz.com>
1 parent 11e8baa commit 6f2dfd1

File tree

2 files changed

+2
-2
lines changed

2 files changed

+2
-2
lines changed

.github/workflows/ci.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -567,7 +567,7 @@ jobs:
567567

568568

569569
windows:
570-
if: ${{ ! contains(github.ref, 'linux-only') && ! contains(github.ref, 'macos-only') && ! contains(github.ref, 'optix-only') }}
570+
if: ${{ (github.event.repository.fork == false || github.event_name != 'schedule') && ! contains(github.ref, 'linux-only') && ! contains(github.ref, 'macos-only') && ! contains(github.ref, 'optix-only') }}
571571
name: "${{matrix.desc}}"
572572
uses: ./.github/workflows/build-steps.yml
573573
with:

src/build-scripts/build_openexr.bash

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -46,7 +46,7 @@ cmake -S ${OPENEXR_SOURCE_DIR} -B ${OPENEXR_BUILD_DIR} \
4646
-DCMAKE_INSTALL_PREFIX="${OPENEXR_INSTALL_DIR}" \
4747
-DCMAKE_PREFIX_PATH="${CMAKE_PREFIX_PATH}" \
4848
-DBUILD_SHARED_LIBS=${OpenEXR_LOCAL_BUILD_SHARED_LIBS:=ON} \
49-
-DOPENEXR_BUILD_UTILS=0 \
49+
-DOPENEXR_BUILD_TOOLS=0 \
5050
-DBUILD_TESTING=0 \
5151
-DOPENEXR_VIEWERS_ENABLE=0 \
5252
-DINSTALL_OPENEXR_EXAMPLES=0 \

0 commit comments

Comments
 (0)