From: Kevin Buettner Date: Wed, 24 Feb 2021 18:48:04 +0000 (-0700) Subject: Fix aarch64-linux-hw-point.c build problem X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=665af52ec2a52184d39a76d6e724fa4733dbab3c;p=binutils-gdb.git Fix aarch64-linux-hw-point.c build problem Due to a recent glibc header file change, the file nat/aarch64-linux-hw-point.c no longer builds on Fedora rawhide. An enum for PTRACE_SYSEMU is now provided by . In the past, PTRACE_SYSEMU was defined only in . This is what it looks like... In : #define PTRACE_SYSEMU 31 In : enum __ptrace_request { ... PTRACE_SYSEMU = 31, #define PT_SYSEMU PTRACE_SYSEMU ... } When and are both included in a source file, we run into the following build problem when the former is included before the latter: In file included from nat/aarch64-linux-hw-point.c:26: /usr/include/sys/ptrace.h:86:3: error: expected identifier before numeric constant 86 | PTRACE_SYSEMU = 31, | ^~~~~~~~~~~~~ (There are more errors after this one too.) The file builds without error when is included after . I found that this is already done in nat/aarch64-sve-linux-ptrace.h (which is included by nat/aarch64-linux-ptrace.c). I've tested this change on Fedora rawhide and Fedora 33, both running on an aarch64 machine. gdb/ChangeLog: * nat/aarch64-linux-hw-point.c: Include after . --- diff --git a/gdb/ChangeLog b/gdb/ChangeLog index 00a71fe9025..f7052421935 100644 --- a/gdb/ChangeLog +++ b/gdb/ChangeLog @@ -1,3 +1,8 @@ +2021-02-24 Kevin Buettner + + * nat/aarch64-linux-hw-point.c: Include after + . + 2021-02-24 Andrew Burgess * exec.c (set_section_command): Move variable declarations into diff --git a/gdb/nat/aarch64-linux-hw-point.c b/gdb/nat/aarch64-linux-hw-point.c index 73e4edd68cb..0278ac2bb28 100644 --- a/gdb/nat/aarch64-linux-hw-point.c +++ b/gdb/nat/aarch64-linux-hw-point.c @@ -23,8 +23,8 @@ #include "aarch64-linux-hw-point.h" #include -#include #include +#include #include /* Number of hardware breakpoints/watchpoints the target supports.