-
Notifications
You must be signed in to change notification settings - Fork 120
[Dockerfile]: add omp-dev and yosys into docker image #2572
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
|
I am surprised you need to apt-install yosys, since bazel manages the sources with an explicit URL. For openmp, you may prefer to run the script that we use in CI to set up a specific clang+openmp: You can run it as ./.github/install_clang_version.sh 19Let me know if that works, and then the dockerfile can stay up to date with the repo by default. |
|
Not sure; will try. |
|
When I try your suggestion it gives some problems; getting a correct clang++ version is not my problem yet - can be handled just the same with Dockerfile commands already in there. |
Code-cleanup
|
I added a small change (code cleanup as well) |
[Dockerfile]: add omp-dev and yosys into docker image