Skip to content

Commit

Permalink
build: remove llvm-dev dependency from libbcc
Browse files Browse the repository at this point in the history
In iovisor#4737 I added depedency on llvm-config.h in order to discover what LLVM version
is being used. This was only meant to be used at build time.
By adding llvm-config.h in bpf_module.h, a public header of libbcc, I introduced
a dependency on llvm-dev to libbcc which is unnecessary.

This change removes the include from `bpf_module.h`, which should be enough.

It does not seem I need to add the include in any other files.

Fixes iovisor#4997

Signed-off-by: Manu Bretelle <chantr4@gmail.com>
  • Loading branch information
chantra committed May 29, 2024
1 parent c93a19a commit c4e87fe
Showing 1 changed file with 0 additions and 2 deletions.
2 changes: 0 additions & 2 deletions src/cc/bpf_module.h
Original file line number Diff line number Diff line change
Expand Up @@ -22,8 +22,6 @@
#include <string>
#include <vector>

#include <llvm/Config/llvm-config.h>

#include "bcc_exception.h"
#include "table_storage.h"

Expand Down

0 comments on commit c4e87fe

Please sign in to comment.