Skip to content

Commit d5cbc09

Browse files
committed
Don't recurse submodules when building wheels
Change the build-wheels workflow to only fetch the first layer of submodules. ExecuTorch only needs the first layer of submodules to build its pip package, but the `build_wheels_*.yaml` workflows will recursively fetch all submodules by default. Fetching all submodules can also cause `buck2` to fail because it will try to watch too many files. This change makes `buck2` work on the CI runners, speeds up the jobs, and reduces disk/network usage.
1 parent 1352d4e commit d5cbc09

File tree

2 files changed

+8
-0
lines changed

2 files changed

+8
-0
lines changed

.github/workflows/build-wheels-linux.yml

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -50,6 +50,10 @@ jobs:
5050
test-infra-repository: pytorch/test-infra
5151
test-infra-ref: release/2.3
5252
build-matrix: ${{ needs.generate-matrix.outputs.matrix }}
53+
# ExecuTorch only needs the first layer of submodules; override the
54+
# "recursive" default to do less work, and to give the buck daemon fewer
55+
# files to look at.
56+
submodules: true
5357
pre-script: ${{ matrix.pre-script }}
5458
post-script: ${{ matrix.post-script }}
5559
package-name: ${{ matrix.package-name }}

.github/workflows/build-wheels-m1.yml

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -50,6 +50,10 @@ jobs:
5050
test-infra-repository: pytorch/test-infra
5151
test-infra-ref: release/2.3
5252
build-matrix: ${{ needs.generate-matrix.outputs.matrix }}
53+
# ExecuTorch only needs the first layer of submodules; override the
54+
# "recursive" default to do less work, and to give the buck daemon fewer
55+
# files to look at.
56+
submodules: true
5357
pre-script: ${{ matrix.pre-script }}
5458
post-script: ${{ matrix.post-script }}
5559
package-name: ${{ matrix.package-name }}

0 commit comments

Comments
 (0)