Skip to content

Conversation

@jathu
Copy link
Contributor

@jathu jathu commented May 16, 2025

Summary

In this diff we create a pybind preset that is copied with configurations from setup.py. In upcoming diffs, we will completely gut setup.py and point directly to this preset.

Test plan

$ cmake --list-presets
Available configure presets:

  "macos-arm64"  - Build everything buildable on macOS arm64
  "pybind" - Build pybindings exported in the wheel
$ cmake --preset pybind && cmake --build cmake-out --parallel

cc @larryliu0820

@jathu jathu added module: build/install Issues related to the cmake and buck2 builds, and to installing ExecuTorch ciflow/trunk ciflow/binaries release notes: build Changes related to build, including dependency upgrades, build flags, optimizations, etc. labels May 16, 2025
@pytorch-bot
Copy link

pytorch-bot bot commented May 16, 2025

🔗 Helpful Links

🧪 See artifacts and rendered test results at hud.pytorch.org/pr/pytorch/executorch/10932

Note: Links to docs will display an error until the docs builds have been completed.

✅ You can merge normally! (1 Unrelated Failure)

As of commit 0b0a91e with merge base 5c6d4e5 (image):

BROKEN TRUNK - The following job failed but were present on the merge base:

👉 Rebase onto the `viable/strict` branch to avoid these failures

This comment was automatically generated by Dr. CI and updates every 15 minutes.

@facebook-github-bot facebook-github-bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label May 16, 2025
@jathu jathu force-pushed the jathu/preset--pybind-macos branch 5 times, most recently from a978c01 to 0346f80 Compare May 16, 2025 01:27
@jathu jathu changed the title Create a macOS pybind preset Create a pybind preset May 16, 2025
@jathu jathu force-pushed the jathu/preset--pybind-macos branch 4 times, most recently from ce4776f to de1fc49 Compare May 16, 2025 03:13
@jathu jathu force-pushed the jathu/preset--pybind-macos branch from de1fc49 to 0b0a91e Compare May 16, 2025 15:19
@jathu jathu marked this pull request as ready for review May 16, 2025 16:49
@jathu jathu merged commit 502db64 into main May 16, 2025
195 of 196 checks passed
@jathu jathu deleted the jathu/preset--pybind-macos branch May 16, 2025 19:42
hinriksnaer pushed a commit to hinriksnaer/executorch that referenced this pull request May 19, 2025
### Summary

In this diff we create a `pybind` preset that is copied with
configurations from
[setup.py](https://github.com/pytorch/executorch/blob/7175ca420dc5a173f8635da976457bf6f17bbbc1/setup.py).
In upcoming diffs, we will completely gut setup.py and point directly to
this preset.

### Test plan

```
$ cmake --list-presets
Available configure presets:

  "macos-arm64"  - Build everything buildable on macOS arm64
  "pybind" - Build pybindings exported in the wheel
```
```
$ cmake --preset pybind && cmake --build cmake-out --parallel
```

cc @larryliu0820
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ciflow/binaries ciflow/trunk CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. module: build/install Issues related to the cmake and buck2 builds, and to installing ExecuTorch release notes: build Changes related to build, including dependency upgrades, build flags, optimizations, etc.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants