Skip to content
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

refactor: split monolithic BUILD.local_cuda into template fragments #284

Merged
merged 2 commits into from
Dec 22, 2024

Conversation

cloudhan
Copy link
Collaborator

@cloudhan cloudhan commented Oct 16, 2024

Tracking issue: #283

This breaks the monolithic BUILD.local_cuda into smaller fragments.

For non hermatic local_cuda repo with local CTK, the local_cuda repo will be instantiate by automatically pasting the fragments back again into a single BUILD file.

For hermatic local_cuda repo (in this case, local to the host env) with deliverable components in latter comming PRs, the local_cuda repo is our entry point to access cuda toolkit. Each component will instantiate a repo for that component only. The BUILD file only contains the fragment for the very component. The targets will then be aliased back to the local_cuda repo as an user interface.

@cloudhan cloudhan force-pushed the cloudhan/hermetic-ctk-1 branch 3 times, most recently from b7887dd to 9953aba Compare December 14, 2024 15:04
@cloudhan cloudhan marked this pull request as ready for review December 14, 2024 15:22
@cloudhan cloudhan force-pushed the cloudhan/hermetic-ctk-1 branch 3 times, most recently from b886a49 to 255a4df Compare December 19, 2024 15:45
@cloudhan cloudhan force-pushed the cloudhan/hermetic-ctk-1 branch from 255a4df to 31495ea Compare December 22, 2024 17:07
@cloudhan cloudhan merged commit 3b6cc27 into main Dec 22, 2024
17 checks passed
@cloudhan cloudhan deleted the cloudhan/hermetic-ctk-1 branch December 22, 2024 17:15
@cloudhan cloudhan mentioned this pull request Dec 30, 2024
12 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant