Skip to content

Commit

Permalink
Fix Windows and Mac weekly CI failure: set env correctly (onnx#5582)
Browse files Browse the repository at this point in the history
### Description
<!-- - Describe your changes. -->
Set env for onnx-weekly correctly.

### Motivation and Context
<!-- - Why is this change required? What problem does it solve? -->
<!-- - If it fixes an open issue, please link to the issue here. -->
Fix Windows and Mac weekly CI failure: set env correctly:
- https://github.com/onnx/onnx/actions/runs/6140254770
- https://github.com/onnx/onnx/actions/runs/6140288376

---------

Signed-off-by: Chun-Wei Chen <jacky82226@gmail.com>
  • Loading branch information
jcwchen committed Sep 11, 2023
1 parent fdefbe8 commit 02a41bf
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 6 deletions.
5 changes: 2 additions & 3 deletions .github/workflows/release_mac.yml
Original file line number Diff line number Diff line change
Expand Up @@ -64,10 +64,9 @@ jobs:
source workflow_scripts/protobuf/build_protobuf_unix.sh $NUM_CORES $(pwd)/protobuf/protobuf_install
if [ '${{ github.event_name }}' == 'schedule' ]; then
sed -i 's/name = "onnx"/name = "onnx-weekly"/' 'pyproject.toml'
ONNX_PREVIEW_BUILD=1 python -m build --wheel
else
python -m build --wheel
export ONNX_PREVIEW_BUILD=1
fi
python -m build --wheel
for file in dist/*.whl; do
python -m pip install --upgrade $file;
Expand Down
5 changes: 2 additions & 3 deletions .github/workflows/release_win.yml
Original file line number Diff line number Diff line change
Expand Up @@ -72,10 +72,9 @@ jobs:
if ('${{ github.event_name }}' -eq 'schedule') {
echo "Build weekly PyPI package"
(Get-Content -Path 'pyproject.toml') | ForEach-Object { $_ -replace 'name = "onnx"', 'name = "onnx-weekly"' } | Set-Content -Path 'pyproject.toml'
ONNX_PREVIEW_BUILD=1 python -m build --wheel
} else {
python -m build --wheel
set ONNX_PREVIEW_BUILD=1
}
python -m build --wheel
Get-ChildItem -Path dist/*.whl | foreach {python -m pip install --upgrade $_.fullname}
- name: Test the installed wheel
Expand Down

0 comments on commit 02a41bf

Please sign in to comment.