Skip to content

Do not unnecessarily install CUDA for ROCm#3837

Closed
ramilbakhshyiev wants to merge 1 commit intomainfrom
export-D62283602
Closed

Do not unnecessarily install CUDA for ROCm#3837
ramilbakhshyiev wants to merge 1 commit intomainfrom
export-D62283602

Conversation

@ramilbakhshyiev
Copy link
Contributor

@ramilbakhshyiev ramilbakhshyiev commented Sep 6, 2024

Summary: ROCm does not require CUDA, this change stops installing it.

Differential Revision: D62283602

Summary: ROCm does not require CUDA, this change stops installing it. This is also likely to fix #3787 which is coming in a separate change.

Differential Revision: D62283602
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D62283602

facebook-github-bot pushed a commit that referenced this pull request Sep 6, 2024
Summary:

ROCm does not require CUDA, this change stops installing it. This is also likely to fix #3787 which is coming in a separate change.

Differential Revision: D62283602
facebook-github-bot pushed a commit that referenced this pull request Sep 6, 2024
Summary:

ROCm does not require CUDA, this change stops installing it. This is also likely to fix #3787 which is coming in a separate change.

Differential Revision: D62283602
facebook-github-bot pushed a commit that referenced this pull request Sep 6, 2024
Summary:

ROCm does not require CUDA, this change stops installing it. This is also likely to fix #3787 which is coming in a separate change.

Differential Revision: D62283602
@ramilbakhshyiev
Copy link
Contributor Author

No need to rebase, Phabricator has already committed this change.

@ramilbakhshyiev ramilbakhshyiev deleted the export-D62283602 branch September 6, 2024 19:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants