Skip to content

Commit 4a5abdc

Browse files
committed
common: Get rid of arch-specific include
As @liuqun pointed out on #44, the right way to fix the weird missing include issue on Ubuntu is to install the gcc-multilib package. So add that to setup_instructions, and get rid of the additional arch-specific include path. Signed-off-by: Toke Høiland-Jørgensen <toke@redhat.com>
1 parent a339ed6 commit 4a5abdc

2 files changed

Lines changed: 4 additions & 3 deletions

File tree

common/common.mk

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -40,8 +40,6 @@ EXTRA_DEPS +=
4040
KERN_USER_H ?= $(wildcard common_kern_user.h)
4141

4242
CFLAGS ?= -I$(LIBBPF_DIR)/build/usr/include/ -g
43-
# Extra include for Ubuntu issue #44
44-
CFLAGS += -I/usr/include/x86_64-linux-gnu
4543
CFLAGS += -I../headers/
4644
LDFLAGS ?= -L$(LIBBPF_DIR)
4745

setup_dependencies.org

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -11,12 +11,15 @@ this document and install the needed software packages.
1111
- [[#dependencies][Dependencies]]
1212
- [[#packages-on-fedora][Packages on Fedora]]
1313
- [[#packages-on-debianubuntu][Packages on Debian/Ubuntu]]
14+
- [[#packages-on-opensuse][Packages on openSUSE]]
1415
- [[#kernel-headers-dependency][Kernel headers dependency]]
1516
- [[#packages-on-fedora-1][Packages on Fedora]]
1617
- [[#packages-on-debianubuntu-1][Packages on Debian/Ubuntu]]
18+
- [[#packages-on-opensuse-1][Packages on openSUSE]]
1719
- [[#recommended-tools][Recommended tools]]
1820
- [[#packages-on-fedora-2][Packages on Fedora]]
1921
- [[#packages-on-debianubuntu-2][Packages on Debian/Ubuntu]]
22+
- [[#packages-on-opensuse-2][Packages on openSUSE]]
2023

2124
* Based on libbpf
2225

@@ -82,7 +85,7 @@ Note that you need to do this in the shell you are using to load programs
8285
On Debian and Ubuntu installations, install the dependencies like this:
8386

8487
#+begin_example
85-
$ sudo apt install clang llvm libelf-dev
88+
$ sudo apt install clang llvm libelf-dev gcc-multilib
8689
#+end_example
8790

8891
To install the 'perf' utility, run this on Debian:

0 commit comments

Comments
 (0)