keytable: fix BPF protocol compilation on mips

Commit Message

Sean Young Nov. 7, 2018, 3:36 p.m. UTC

clang -idirafter /usr/local/include -idirafter
+/usr/lib/llvm-6.0/lib/clang/6.0.1/include -idirafter
+/usr/include/mips64el-linux-gnuabi64 -idirafter /usr/include
+-I../../../include -target bpf -O2 -c grundig.c
> In file included from grundig.c:5:
> In file included from ../../../include/linux/lirc.h:10:
> In file included from /usr/include/linux/types.h:9:
> In file included from /usr/include/linux/posix_types.h:36:
> In file included from
+/usr/include/mips64el-linux-gnuabi64/asm/posix_types.h:13:
> /usr/include/mips64el-linux-gnuabi64/asm/sgidefs.h:19:2: error: Use a Linux
+compiler or give up.
> #error Use a Linux compiler or give up.

This requires __linux__ to be defined.

Signed-off-by: Sean Young <[e-mail]>
---
 utils/keytable/bpf_protocols/Makefile.am | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Gregor Jasny Nov. 7, 2018, 7:45 p.m. UTC | #1

On 11/7/18 4:36 PM, Sean Young wrote:
> clang -idirafter /usr/local/include -idirafter
> +/usr/lib/llvm-6.0/lib/clang/6.0.1/include -idirafter
> +/usr/include/mips64el-linux-gnuabi64 -idirafter /usr/include
> +-I../../../include -target bpf -O2 -c grundig.c
>> In file included from grundig.c:5:
>> In file included from ../../../include/linux/lirc.h:10:
>> In file included from /usr/include/linux/types.h:9:
>> In file included from /usr/include/linux/posix_types.h:36:
>> In file included from
> +/usr/include/mips64el-linux-gnuabi64/asm/posix_types.h:13:
>> /usr/include/mips64el-linux-gnuabi64/asm/sgidefs.h:19:2: error: Use a Linux
> +compiler or give up.
>> #error Use a Linux compiler or give up.
> 
> This requires __linux__ to be defined.

Thanks! I uploaded a new Debian package. Let's see how it goes:
https://buildd.debian.org/status/package.php?p=v4l-utils

I'll pick up your other patches soon.

-Gregor

Patch

diff --git a/utils/keytable/bpf_protocols/Makefile.am b/utils/keytable/bpf_protocols/Makefile.am
index 8887b897..ba79742c 100644
--- a/utils/keytable/bpf_protocols/Makefile.am
+++ b/utils/keytable/bpf_protocols/Makefile.am
@@ -8,7 +8,7 @@  CLANG_SYS_INCLUDES := $(shell $(CLANG) -v -E - </dev/null 2>&1 \
         | sed -n '/<...> search starts here:/,/End of search list./{ s| \(/.*\)|-idirafter \1|p }')
 
 %.o: %.c bpf_helpers.h
-	$(CLANG) $(CLANG_SYS_INCLUDES) -I$(top_srcdir)/include -target bpf -O2 -c $<
+	$(CLANG) $(CLANG_SYS_INCLUDES) -D__linux__ -I$(top_srcdir)/include -target bpf -O2 -c $<
 
 PROTOCOLS = grundig.o pulse_distance.o pulse_length.o rc_mm.o manchester.o