This repository was archived by the owner on Jul 11, 2023. It is now read-only.
Upgrade to LLVM 10 #66
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
(Forgot to mention the reasoning, Rust 1.45 is using LLVM 10 now which is the merge commit from this PR)
The code change should be trivial.
I think we might want to clean up the README a bit, for Ubuntu dependencies:
curl
wget
orgnupg2
at all?ca-certificates-java
either?libelf
is enough (I just realized I didn't havelibelf-dev
- though installing it doesn't solve the load issue I have, and it certainly didn't affectcargo bpf build
)zlib1g
is needed (forgot about the error message but it's the first error I hit when runningcargo bpf build
)linux-headers
need to pick an explicit version, otherwiseapt
(apt-get
too) complainsE: Package 'linux-headers' has no installation candidate
It might also be related to Ubuntu releases, I'm running on 20.04 (Desktop).