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

Update base image and upgrade GCC version #548

Merged
merged 8 commits into from
May 31, 2024
Merged

Update base image and upgrade GCC version #548

merged 8 commits into from
May 31, 2024

Conversation

snnn
Copy link
Member

@snnn snnn commented May 31, 2024

  1. Use an internal prebuilt base image instead of a public image, so that we do not need to rebuild the manylinux part again and again.
  2. The new CUDA 11 base image is based on almalinux 8 instead of ubi8, so that we can get GCC 11. ubi8 only has GCC 8 and GCC 12, but GCC 12 is compatible with CUDA 11. So before this change we use GCC 8 in CUDA 11 build. After this change we will use GCC 11 instead.
  3. Drop the support for GCC 10 and below.

This PR provides another solution for #257 .

@snnn snnn changed the title Update base image Update base image and upgrade GCC version May 31, 2024
@snnn snnn merged commit 1c630d4 into main May 31, 2024
44 checks passed
@snnn snnn deleted the snnn/new_docker branch May 31, 2024 22:04
baijumeswani pushed a commit that referenced this pull request Jun 12, 2024
1. Use an internal prebuilt base image instead of a public image, so
that we do not need to rebuild the manylinux part again and again.
2. The new CUDA 11 base image is based on almalinux 8 instead of ubi8,
so that we can get GCC 11. ubi8 only has GCC 8 and GCC 12, but GCC 12 is
compatible with CUDA 11. So before this change we use GCC 8 in CUDA 11
build. After this change we will use GCC 11 instead.
3. Drop the support for GCC 10 and below.

This PR provides another solution for #257 .
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants