-
Notifications
You must be signed in to change notification settings - Fork 7
Refactoring of Dockerfile #41
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
base: main
Are you sure you want to change the base?
Conversation
…the individual source languages.
…r C but in the sense of continuity it is to have 'its own' Dockerfile env.
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
@@ -0,0 +1 @@ | |||
Dockerfile.cpp_env No newline at end of file |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't think C and C++ need a separate container. It's not even possible to build clang
with support for only one or the other and system packages between them will probably have a large amount of overlap.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
C is only a symbolic link to the C++ container, but just having it in there for continuity's sake s.t. every language has its own container..
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why though? We collect C and C++ from the same builders (spack and Gentoo, although Gentoo needs its own container).
Have you validated that packages still build? The list of dependencies was long in the first place because it increased the percentage of packages that would build. I don't think I ever kept exact notes of what needed what though. |
Refactor the existing Dockerfile into language-specific Dockerfiles, laying the groundwork for adding the other builders. Namely:
Dockerfile.cpp_env
Dockerfile.julia_env
Dockerfile.rust_env
Dockerfile.swift_env
Dockerfile.c_env
points at the C++ Dockerfile, but yet exists for continuity's sake.The list of system-dependencies was pruned in the process.
cc @lwshanbd @jdoerfert