-
Notifications
You must be signed in to change notification settings - Fork 536
Restore CMake configure comment #2723
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
🔗 Helpful Links🧪 See artifacts and rendered test results at hud.pytorch.org/pr/pytorch/executorch/2723
Note: Links to docs will display an error until the docs builds have been completed. ✅ No FailuresAs of commit 3178e80 with merge base 04d568d ( This comment was automatically generated by Dr. CI and updates every 15 minutes. |
82e3052
to
ee71e4c
Compare
@GregoryComer has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator. |
ee71e4c
to
251a893
Compare
@GregoryComer has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator. |
251a893
to
3178e80
Compare
@GregoryComer has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator. |
@GregoryComer merged this pull request in 9922c54. |
Summary: A comment at the top of the top-level CMakeLists.txt file was removed previously as part of an update to automatically download buck. I removed more than was needed. This change restores the comment (minus the explicit buck path). Test Plan: This is a comment-only change. I did configure and build on linux x86-64 to validate syntax. Reviewed By: mcr229 Differential Revision: D55445749 Pulled By: GregoryComer fbshipit-source-id: ff8cb601e71be4231f4e853b993bad9f54ea7144 [ghstack-poisoned]
Summary: A comment at the top of the top-level CMakeLists.txt file was removed previously as part of an update to automatically download buck. I removed more than was needed. This change restores the comment (minus the explicit buck path). Test Plan: This is a comment-only change. I did configure and build on linux x86-64 to validate syntax. Reviewed By: mcr229 Differential Revision: D55445749 Pulled By: GregoryComer fbshipit-source-id: ff8cb601e71be4231f4e853b993bad9f54ea7144 ghstack-source-id: f7130b55ae88264c10eec973ce4498e0bcedb6a4 Pull Request resolved: #2743
@pytorchbot cherry-pick --onto release/0.2 -c docs |
Summary: A comment at the top of the top-level CMakeLists.txt file was removed previously as part of an update to automatically download buck. I removed more than was needed. This change restores the comment (minus the explicit buck path). Pull Request resolved: #2723 Test Plan: This is a comment-only change. I did configure and build on linux x86-64 to validate syntax. Reviewed By: mcr229 Differential Revision: D55445749 Pulled By: GregoryComer fbshipit-source-id: ff8cb601e71be4231f4e853b993bad9f54ea7144 (cherry picked from commit 9922c54)
Cherry picking #2723The cherry pick PR is at #2749 Details for Dev Infra teamRaised by workflow job |
A comment at the top of the top-level CMakeLists.txt file was removed previously as part of an update to automatically download buck. I removed more than was needed. This change restores the comment (minus the explicit buck path).
Test Plan:
This is a comment-only change. I did configure and build on linux x86-64 to validate syntax.