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

setup-build-env: install llvm development package #137

Merged
merged 1 commit into from
Aug 27, 2024

Conversation

eddyz87
Copy link
Contributor

@eddyz87 eddyz87 commented Aug 23, 2024

Recently kernel BPF selftests gained an optional dependency on LLVM development libraries [1] in order to support tests that disassemble jit produced code. When necessary dependencies are not available such tests are skipped.

This commit adds LLVM development libraries to the build environment, thus enabling jit disassembly tests on the CI.

[1] https://lore.kernel.org/bpf/[email protected]/

Recently kernel BPF selftests gained an optional dependency on LLVM
development libraries [1] in order to support tests that disassemble
jit produced code. When necessary dependencies are not available such
tests are skipped.

This commit adds LLVM development libraries to the build environment,
thus enabling jit disassembly tests on the CI.

[1] https://lore.kernel.org/bpf/[email protected]/
@eddyz87
Copy link
Contributor Author

eddyz87 commented Aug 23, 2024

Something strange happens with CI checks for this pull request (here):

  • done in 3s
  • reports success while: Notice: Success: 29/23, Skipped: 3, Failed: 28

@@ -55,6 +55,8 @@ SELF_OPTS=$(cat <<EOF
-C ${REPO_ROOT}/${REPO_PATH}/tools/testing/selftests/bpf
EOF
)
export LLVM_CONFIG=llvm-config-${LLVM_VERSION}
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

what is relying on this? this looks a bit random

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

tools/build/feature/Makefile uses this variable

@anakryiko anakryiko merged commit b6501f8 into libbpf:main Aug 27, 2024
2 checks passed
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