Skip to content

Commit

Permalink
Use env var instead of wheel-build-params for release flag
Browse files Browse the repository at this point in the history
  • Loading branch information
HolyWu committed May 11, 2024
1 parent ec6e951 commit 8915f84
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 0 additions & 2 deletions .github/workflows/build-test-windows.yml
Original file line number Diff line number Diff line change
Expand Up @@ -47,7 +47,6 @@ jobs:
- repository: pytorch/tensorrt
pre-script: packaging/pre_build_script_windows.sh
env-script: packaging/vc_env_helper.bat
wheel-build-params: "--release"
smoke-test-script: packaging/smoke_test_windows.py
package-name: torch_tensorrt
name: Build torch-tensorrt whl package
Expand All @@ -60,7 +59,6 @@ jobs:
build-matrix: ${{ needs.substitute-runner.outputs.matrix }}
pre-script: ${{ matrix.pre-script }}
env-script: ${{ matrix.env-script }}
wheel-build-params: ${{ matrix.wheel-build-params }}
smoke-test-script: ${{ matrix.smoke-test-script }}
package-name: ${{ matrix.package-name }}
trigger-event: ${{ github.event_name }}
Expand Down
2 changes: 2 additions & 0 deletions packaging/pre_build_script_windows.sh
Original file line number Diff line number Diff line change
Expand Up @@ -23,3 +23,5 @@ unzip TensorRT.zip -d C:/
export CUDA_HOME="$(echo ${CUDA_PATH} | sed -e 's#\\#\/#g')"

cat toolchains/ci_workspaces/WORKSPACE.win.release.tmpl | envsubst > WORKSPACE

echo "RELEASE=1" >> ${GITHUB_ENV}

0 comments on commit 8915f84

Please sign in to comment.