Skip to content

Commit a51094c

Browse files
Fix QNN Android Maven release build (pytorch#23234)
The QNN Maven publish for ExecuTorch 1.5.1 failed before AAR assembly because the release workflow invokes `.ci/scripts/build-qnn-sdk.sh`. That script builds the host-only `PyQnnManagerAdaptor` and `quantized_ops_aot_lib`, which are not shipped in the Android artifact and conflict with the newer PyTorch headers in the current CI image. The failure is visible in [run 36488741446](https://github.com/pytorch/executorch/actions/runs/36488741446). This removes that redundant host build. `setup-qnn-deps.sh` still installs the QNN SDK, and `scripts/build_android_library.sh` still builds the QNN runtime included in the AAR. Once merged, the workflow can be dispatched with version `1.5.1`, source ref `v1.5.1`, flavor `qnn`, and Maven upload enabled, so no new ExecuTorch version is required. Validation: YAML parsing, shell syntax checks for the invoked build scripts, and `git diff --check`. This PR was authored with Codex.
1 parent c8e3cbc commit a51094c

1 file changed

Lines changed: 2 additions & 1 deletion

File tree

‎.github/workflows/android-release-artifacts.yml‎

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -162,7 +162,8 @@ jobs:
162162
163163
if [[ "$FLAVOR" == "qnn" || "$FLAVOR" == "all" ]]; then
164164
PYTHON_EXECUTABLE=python bash .ci/scripts/setup-qnn-deps.sh
165-
PYTHON_EXECUTABLE=python bash .ci/scripts/build-qnn-sdk.sh
165+
# build_android_library.sh builds the QNN runtime included in the AAR;
166+
# the host-only PyQnnManagerAdaptor is not part of this artifact.
166167
export EXECUTORCH_BUILD_QNN=ON
167168
source backends/qualcomm/scripts/qnn_config.sh
168169
export QNN_SDK_ROOT="/tmp/qnn/${QNN_VERSION}"

0 commit comments

Comments
 (0)