-
Notifications
You must be signed in to change notification settings - Fork 60
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
Add necessary package installation info for debian based distro #172
base: main
Are you sure you want to change the base?
Conversation
✅ Deploy Preview for aya-rs ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
de03338
to
90e7e82
Compare
@bboyleonp666 Thanks for your contribution! Can you rebase? After rebase, I'm expecting markdownlint to complain about the length of lines, so please fix accordingly. 🙂 |
@vadorovsky, thanks for considering this PR. |
The CI builds for new contributors have to be approved by a maintainer. I've just done it and there is a build error. Could you also remove the "merge" commit and do a rebase instead? |
bpf-linker installation on debian distro requires additional packages, which only updated in https://github.com/aya-rs/bpf-linker yet to the book.
a9acb9e
to
46a9a66
Compare
@vadorovsky Done! I have rebased the commits and force-pushed a newer version. If I do it wrongly, please kindly let me know. |
@vadorovsky should I do some further steps to get this PR merged? |
bpf-linker installation on debian distro requires additional packages, which only updated in https://github.com/aya-rs/bpf-linker yet to the book.