public inbox for gdb-cvs@sourceware.org help / color / mirror / Atom feed
From: Michael Frysinger <vapier@sourceware.org> To: gdb-cvs@sourceware.org Subject: [binutils-gdb] sim: assume sys/stat.h always exists (via gnulib) Date: Mon, 16 Jan 2023 09:47:36 +0000 (GMT) [thread overview] Message-ID: <20230116094736.44D913858D3C@sourceware.org> (raw) https://sourceware.org/git/gitweb.cgi?p=binutils-gdb.git;h=da8c9663995deb7abe63ca45251fc0264e031550 commit da8c9663995deb7abe63ca45251fc0264e031550 Author: Mike Frysinger <vapier@gentoo.org> Date: Mon Jan 16 04:42:47 2023 -0500 sim: assume sys/stat.h always exists (via gnulib) We have many uses of sys/stat.h that are unprotected by HAVE_SYS_STAT_H, so this is more formalizing the reality that we require this header. Since we switched to gnulib, it guarantees that a sys/stat.h exists for us to include, so we're doubly OK. Diff: --- sim/common/sim-memopt.c | 2 -- sim/configure | 33 ++------------------------------- sim/cris/traps.c | 2 -- sim/h8300/compile.c | 2 +- sim/m4/sim_ac_platform.m4 | 5 +---- sim/ppc/emul_unix.c | 4 ---- sim/sh/interp.c | 2 -- 7 files changed, 4 insertions(+), 46 deletions(-) diff --git a/sim/common/sim-memopt.c b/sim/common/sim-memopt.c index 2e5016c52ab..7d46a864afc 100644 --- a/sim/common/sim-memopt.c +++ b/sim/common/sim-memopt.c @@ -30,9 +30,7 @@ along with this program. If not, see <http://www.gnu.org/licenses/>. */ #ifdef HAVE_SYS_MMAN_H #include <sys/mman.h> #endif -#ifdef HAVE_SYS_STAT_H #include <sys/stat.h> -#endif #include "sim-main.h" #include "sim-assert.h" diff --git a/sim/configure b/sim/configure index 104751477c7..fd9cdff54fb 100755 --- a/sim/configure +++ b/sim/configure @@ -2810,7 +2810,6 @@ as_fn_append ac_header_list " sys/mount.h" as_fn_append ac_header_list " sys/param.h" as_fn_append ac_header_list " sys/resource.h" as_fn_append ac_header_list " sys/socket.h" -as_fn_append ac_header_list " sys/stat.h" as_fn_append ac_header_list " sys/statfs.h" as_fn_append ac_header_list " sys/termio.h" as_fn_append ac_header_list " sys/termios.h" @@ -5262,8 +5261,6 @@ done - - @@ -8425,9 +8422,7 @@ fi ac_fn_c_check_member "$LINENO" "struct stat" "st_dev" "ac_cv_member_struct_stat_st_dev" "#ifdef HAVE_SYS_TYPES_H #include <sys/types.h> #endif -#ifdef HAVE_SYS_STAT_H #include <sys/stat.h> -#endif " if test "x$ac_cv_member_struct_stat_st_dev" = xyes; then : @@ -8440,9 +8435,7 @@ fi ac_fn_c_check_member "$LINENO" "struct stat" "st_ino" "ac_cv_member_struct_stat_st_ino" "#ifdef HAVE_SYS_TYPES_H #include <sys/types.h> #endif -#ifdef HAVE_SYS_STAT_H #include <sys/stat.h> -#endif " if test "x$ac_cv_member_struct_stat_st_ino" = xyes; then : @@ -8455,9 +8448,7 @@ fi ac_fn_c_check_member "$LINENO" "struct stat" "st_mode" "ac_cv_member_struct_stat_st_mode" "#ifdef HAVE_SYS_TYPES_H #include <sys/types.h> #endif -#ifdef HAVE_SYS_STAT_H #include <sys/stat.h> -#endif " if test "x$ac_cv_member_struct_stat_st_mode" = xyes; then : @@ -8470,9 +8461,7 @@ fi ac_fn_c_check_member "$LINENO" "struct stat" "st_nlink" "ac_cv_member_struct_stat_st_nlink" "#ifdef HAVE_SYS_TYPES_H #include <sys/types.h> #endif -#ifdef HAVE_SYS_STAT_H #include <sys/stat.h> -#endif " if test "x$ac_cv_member_struct_stat_st_nlink" = xyes; then : @@ -8485,9 +8474,7 @@ fi ac_fn_c_check_member "$LINENO" "struct stat" "st_uid" "ac_cv_member_struct_stat_st_uid" "#ifdef HAVE_SYS_TYPES_H #include <sys/types.h> #endif -#ifdef HAVE_SYS_STAT_H #include <sys/stat.h> -#endif " if test "x$ac_cv_member_struct_stat_st_uid" = xyes; then : @@ -8500,9 +8487,7 @@ fi ac_fn_c_check_member "$LINENO" "struct stat" "st_gid" "ac_cv_member_struct_stat_st_gid" "#ifdef HAVE_SYS_TYPES_H #include <sys/types.h> #endif -#ifdef HAVE_SYS_STAT_H #include <sys/stat.h> -#endif " if test "x$ac_cv_member_struct_stat_st_gid" = xyes; then : @@ -8515,9 +8500,7 @@ fi ac_fn_c_check_member "$LINENO" "struct stat" "st_rdev" "ac_cv_member_struct_stat_st_rdev" "#ifdef HAVE_SYS_TYPES_H #include <sys/types.h> #endif -#ifdef HAVE_SYS_STAT_H #include <sys/stat.h> -#endif " if test "x$ac_cv_member_struct_stat_st_rdev" = xyes; then : @@ -8530,9 +8513,7 @@ fi ac_fn_c_check_member "$LINENO" "struct stat" "st_size" "ac_cv_member_struct_stat_st_size" "#ifdef HAVE_SYS_TYPES_H #include <sys/types.h> #endif -#ifdef HAVE_SYS_STAT_H #include <sys/stat.h> -#endif " if test "x$ac_cv_member_struct_stat_st_size" = xyes; then : @@ -8545,9 +8526,7 @@ fi ac_fn_c_check_member "$LINENO" "struct stat" "st_blksize" "ac_cv_member_struct_stat_st_blksize" "#ifdef HAVE_SYS_TYPES_H #include <sys/types.h> #endif -#ifdef HAVE_SYS_STAT_H #include <sys/stat.h> -#endif " if test "x$ac_cv_member_struct_stat_st_blksize" = xyes; then : @@ -8560,9 +8539,7 @@ fi ac_fn_c_check_member "$LINENO" "struct stat" "st_blocks" "ac_cv_member_struct_stat_st_blocks" "#ifdef HAVE_SYS_TYPES_H #include <sys/types.h> #endif -#ifdef HAVE_SYS_STAT_H #include <sys/stat.h> -#endif " if test "x$ac_cv_member_struct_stat_st_blocks" = xyes; then : @@ -8575,9 +8552,7 @@ fi ac_fn_c_check_member "$LINENO" "struct stat" "st_atime" "ac_cv_member_struct_stat_st_atime" "#ifdef HAVE_SYS_TYPES_H #include <sys/types.h> #endif -#ifdef HAVE_SYS_STAT_H #include <sys/stat.h> -#endif " if test "x$ac_cv_member_struct_stat_st_atime" = xyes; then : @@ -8590,9 +8565,7 @@ fi ac_fn_c_check_member "$LINENO" "struct stat" "st_mtime" "ac_cv_member_struct_stat_st_mtime" "#ifdef HAVE_SYS_TYPES_H #include <sys/types.h> #endif -#ifdef HAVE_SYS_STAT_H #include <sys/stat.h> -#endif " if test "x$ac_cv_member_struct_stat_st_mtime" = xyes; then : @@ -8605,9 +8578,7 @@ fi ac_fn_c_check_member "$LINENO" "struct stat" "st_ctime" "ac_cv_member_struct_stat_st_ctime" "#ifdef HAVE_SYS_TYPES_H #include <sys/types.h> #endif -#ifdef HAVE_SYS_STAT_H #include <sys/stat.h> -#endif " if test "x$ac_cv_member_struct_stat_st_ctime" = xyes; then : @@ -12286,7 +12257,7 @@ else lt_dlunknown=0; lt_dlno_uscore=1; lt_dlneed_uscore=2 lt_status=$lt_dlunknown cat > conftest.$ac_ext <<_LT_EOF -#line 12289 "configure" +#line 12260 "configure" #include "confdefs.h" #if HAVE_DLFCN_H @@ -12392,7 +12363,7 @@ else lt_dlunknown=0; lt_dlno_uscore=1; lt_dlneed_uscore=2 lt_status=$lt_dlunknown cat > conftest.$ac_ext <<_LT_EOF -#line 12395 "configure" +#line 12366 "configure" #include "confdefs.h" #if HAVE_DLFCN_H diff --git a/sim/cris/traps.c b/sim/cris/traps.c index f5f169f1c81..29b1b7cfdb3 100644 --- a/sim/cris/traps.c +++ b/sim/cris/traps.c @@ -39,9 +39,7 @@ along with this program. If not, see <http://www.gnu.org/licenses/>. */ #ifdef HAVE_SYS_PARAM_H #include <sys/param.h> #endif -#ifdef HAVE_SYS_STAT_H #include <sys/stat.h> -#endif /* For PATH_MAX, originally. */ #ifdef HAVE_LIMITS_H #include <limits.h> diff --git a/sim/h8300/compile.c b/sim/h8300/compile.c index 467eeafde61..1e303a92eca 100644 --- a/sim/h8300/compile.c +++ b/sim/h8300/compile.c @@ -26,11 +26,11 @@ #ifdef HAVE_SYS_PARAM_H #include <sys/param.h> #endif +#include <sys/stat.h> #include "bfd.h" #include "sim-main.h" #include "sim/sim-h8300.h" -#include "sys/stat.h" #include "sys/types.h" #include "sim-options.h" #include "sim-signal.h" diff --git a/sim/m4/sim_ac_platform.m4 b/sim/m4/sim_ac_platform.m4 index 665c1bd7866..7331ef3a38c 100644 --- a/sim/m4/sim_ac_platform.m4 +++ b/sim/m4/sim_ac_platform.m4 @@ -37,7 +37,6 @@ AC_CHECK_HEADERS_ONCE(m4_flatten([ sys/param.h sys/resource.h sys/socket.h - sys/stat.h sys/statfs.h sys/termio.h sys/termios.h @@ -126,9 +125,7 @@ AC_CHECK_MEMBERS([[struct stat.st_dev], [struct stat.st_ino], [[#ifdef HAVE_SYS_TYPES_H #include <sys/types.h> #endif -#ifdef HAVE_SYS_STAT_H -#include <sys/stat.h> -#endif]]) +#include <sys/stat.h>]]) AC_CHECK_TYPES([__int128]) AC_CHECK_TYPES(socklen_t, [], [], diff --git a/sim/ppc/emul_unix.c b/sim/ppc/emul_unix.c index 2872d236bf3..be9e8385f52 100644 --- a/sim/ppc/emul_unix.c +++ b/sim/ppc/emul_unix.c @@ -1055,7 +1055,6 @@ typedef uint32_t solaris_ino_t; typedef uint32_t solaris_mode_t; typedef uint32_t solaris_nlink_t; -#ifdef HAVE_SYS_STAT_H #define SOLARIS_ST_FSTYPSZ 16 /* array size for file system type name */ /* AIX 7.1 defines st_pad[123] to st_[amc]tim.tv_pad, respectively */ @@ -1145,7 +1144,6 @@ convert_to_solaris_stat(unsigned_word addr, emul_write_buffer(&target, addr, sizeof(target), processor, cia); } -#endif /* HAVE_SYS_STAT_H */ #ifndef HAVE_STAT #define do_solaris_stat 0 @@ -2011,7 +2009,6 @@ typedef int32_t linux_time_t; typedef int32_t linux_clock_t; typedef int32_t linux_daddr_t; -#ifdef HAVE_SYS_STAT_H /* For the PowerPC, don't both with the 'old' stat structure, since there should be no extant binaries with that structure. */ @@ -2082,7 +2079,6 @@ convert_to_linux_stat(unsigned_word addr, emul_write_buffer(&target, addr, sizeof(target), processor, cia); } -#endif /* HAVE_SYS_STAT_H */ #ifndef HAVE_STAT #define do_linux_stat 0 diff --git a/sim/sh/interp.c b/sim/sh/interp.c index 5e0e8c47569..c067f0f2bea 100644 --- a/sim/sh/interp.c +++ b/sim/sh/interp.c @@ -38,9 +38,7 @@ #include <string.h> #include <stdlib.h> -#ifdef HAVE_SYS_STAT_H #include <sys/stat.h> -#endif #include <time.h> #include <sys/time.h> #ifdef HAVE_UTIME_H
reply other threads:[~2023-01-16 9:47 UTC|newest] Thread overview: [no followups] expand[flat|nested] mbox.gz Atom feed
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=20230116094736.44D913858D3C@sourceware.org \ --to=vapier@sourceware.org \ --cc=gdb-cvs@sourceware.org \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).