Skip to content

Commit 7f8d4a5

Browse files
committed
CI : Don't install before running tests
For two reasons : - This is how things are intended to be used : you should be able to build and run tests before committing to installing. - If we install before testing, then USD is picking up two copies of the SceneCacheFileFormat somehow, causing a test failure.
1 parent 224df0c commit 7f8d4a5

File tree

1 file changed

+2
-1
lines changed

1 file changed

+2
-1
lines changed

.github/workflows/main.yml

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -164,7 +164,7 @@ jobs:
164164
165165
- name: Build
166166
run: |
167-
scons -j 2 install BUILD_TYPE=${{ matrix.buildType }} OPTIONS=${{ matrix.options }} BUILD_CACHEDIR=sconsCache
167+
scons -j 2 BUILD_TYPE=${{ matrix.buildType }} OPTIONS=${{ matrix.options }} BUILD_CACHEDIR=sconsCache
168168
# Copy the config log for use in the "Debug Failures" step, because it
169169
# gets clobbered by the `scons test*` call below.
170170
cp config.log buildConfig.log
@@ -178,6 +178,7 @@ jobs:
178178

179179
- name: Build Package
180180
run: |
181+
scons install BUILD_TYPE=${{ matrix.buildType }} OPTIONS=${{ matrix.options }} BUILD_CACHEDIR=sconsCache
181182
${{ env.PACKAGE_COMMAND }} ${{ env.CORTEX_BUILD_NAME }}.${{env.PACKAGE_EXTENSION}} ${{ env.CORTEX_BUILD_NAME }}
182183
if: matrix.publish
183184

0 commit comments

Comments
 (0)