-
Notifications
You must be signed in to change notification settings - Fork 5
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
Install rustc #8
Conversation
We are not going to `apt-get install rustc`, because what is installed using apt-get tends to be too old, compared to what we need (1.60.0+).
This pull request is needed for ruby/ruby#5870 |
Dockerfile
Outdated
|
||
RUN set -ex \ | ||
&& apt-get update \ | ||
&& apt-get install ${packages} \ | ||
libjemalloc-dev openssl ruby tzdata valgrind sudo \ | ||
libcapstone-dev \ |
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.
libcapstone-dev \ |
It should work without this since the Rust crate we use takes care of building libcapstone
from source.
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.
This is for CI. If precompiled library in our docker image reduces build time, I think it's worth.
No plans to merge this for now. Will revisit later if needed again. |
Bump! We now in fact need this. Let us revive. |
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.
Looks fine to me. Is there some way to test this before merging? If not I think it's fine to merge and tweak when we find problems.
Actually this repo's CI is not working as expected. It already pushed images based on this pull request. So yeah, we found a problem already, and was fixed. Thanks for approval. |
We are not going to
apt-get install rustc
, because what is installed using apt-get tends to be too old, compared to what we need (1.60.0+).