Skip to content

Conversation

@perheld
Copy link
Collaborator

@perheld perheld commented Sep 8, 2025

Break up the create_setup_path() function and move the path creation code specific for each module into their respective sub script to not clutter up setup.sh.

Functions added in respective script are "setup_path_X" for each sub script.

Signed-off-by: per.held@arm.com
Change-Id: Ie9ab8c5c722a8e450c08b7b519faa34b8b55f699

@perheld perheld requested a review from digantdesai as a code owner September 8, 2025 20:48
@perheld perheld added module: arm Issues related to arm backend release notes: none Do not include this in the release notes labels Sep 8, 2025
@pytorch-bot
Copy link

pytorch-bot bot commented Sep 8, 2025

🔗 Helpful Links

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

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

✅ No Failures

As of commit d0ed5c9 with merge base 9ffcf4b (image):
💚 Looks good so far! There are no failures yet. 💚

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

@meta-cla meta-cla 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 Sep 8, 2025
Break up the create_setup_path() function and move the path creation
code specific for each module into their respective sub script to not
clutter up setup.sh.

Functions added in respective script are "setup_path_X" for each sub
script.

Signed-off-by: per.held@arm.com
Change-Id: Ie9ab8c5c722a8e450c08b7b519faa34b8b55f699
@perheld perheld force-pushed the ph-split-setup-refactor-path branch from 00a7727 to d0ed5c9 Compare September 8, 2025 20:48
@zingo zingo merged commit 3db7900 into pytorch:main Sep 9, 2025
292 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

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: arm Issues related to arm backend release notes: none Do not include this in the release notes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants