Skip to content
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

Update libbpf to 0.0.8 release #126

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

zebaz
Copy link

@zebaz zebaz commented May 12, 2020

Update libbpf and pin to specific release (0.0.8).
Starting from 0.0.7 libbpf requires zlib as a build dependency.
This PR adds -lz to LIBS in common.mk and adds zlib-devel as a libbpf build dependency in the ansible playbook.

Starting from 0.0.7 libbpf requires zlib as a build dependency.
This commit adds -lz to LIBS in common.mk and adds zlib-devel
as a libbpf build dependency in the ansible playbook.

Signed-off-by: Zebaz <schalbroeck@gmail.com>
@tohojo
Copy link
Member

tohojo commented May 12, 2020

Thank you for the pull request! I'd like to hold off on merging it until I've had a chance to go over the whole tutorial for any updates; is there any urgent reason you need this updated now?

@zebaz
Copy link
Author

zebaz commented May 12, 2020

No urgency whatsoever, I was doing some experimenting with bpf helpers introduced in newer versions of libbpf and noticed that the repository missed zlib dependency to work with this version. Just thought I’d push my fixes upstream. Fully understandable you’d want to go over the examples, thanks for your quick reply!

@tohojo
Copy link
Member

tohojo commented May 12, 2020 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants