Skip to content

Commit 9ef3995

Browse files
committed
reshuffle build step order in build only to make sure the sdist artifact is available for install and the repo is subsequently used for testing
Signed-off-by: Eric Reinecke <[email protected]>
1 parent 5e0ce4d commit 9ef3995

File tree

1 file changed

+9
-10
lines changed

1 file changed

+9
-10
lines changed

.github/workflows/python-package.yml

Lines changed: 9 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -165,11 +165,6 @@ jobs:
165165
OTIO_CXX_BUILD_TMP_DIR: ${{ github.workspace }}/build
166166

167167
steps:
168-
- name: Get sdist source
169-
uses: actions/download-artifact@v5
170-
with:
171-
name: sdist
172-
path: ./sdist
173168
- name: Set up MSYS2
174169
if: matrix.python-version == 'mingw64'
175170
uses: msys2/setup-msys2@v2
@@ -187,18 +182,22 @@ jobs:
187182
uses: actions/[email protected]
188183
with:
189184
python-version: ${{ matrix.python-version }}
190-
- name: Install python build dependencies
185+
- name: Get sdist source
186+
uses: actions/download-artifact@v5
187+
with:
188+
name: sdist
189+
path: ./sdist
190+
- name: Build and Install OpenTimelineIO
191+
run: |
192+
python -m pip install sdist/opentimelineio-*.tar.gz -v --break-system-packages
193+
- name: Install python dev dependencies
191194
run: |
192195
python -m pip install --upgrade pip setuptools wheel "flake8>=3.5" check-manifest
193196
- name: Get source
194197
uses: actions/checkout@v4
195198
- name: Install test dependencies
196199
run: |
197200
python -m pip install --break-system-packages --upgrade -r tests/requirements.txt
198-
- name: Build and Install OpenTimelineIO
199-
run: |
200-
ls -1
201-
pkg=`ls -1 sdist/opentimelineio-*.tar.gz` python -m pip install $pkg -v --break-system-packages
202201
- name: Run tests w/ python coverage
203202
run: make ci-postbuild
204203

0 commit comments

Comments
 (0)