From 3e7fbfdc91b5d825d0d482c8bdc3e660711da578 Mon Sep 17 00:00:00 2001
From: Eric Blake <eblake@redhat.com>
Date: Tue, 5 Oct 2010 08:06:50 -0600
Subject: [PATCH] faccessat: remove unused wrappers
* lib/openat.h (accessat, euidaccesat): Delete, since the mere
presence of these wrappers dragged in -lgen on Solaris.
Reported by Clemens Brogi; fix suggested by Paul Eggert.
Signed-off-by: Eric Blake <eblake@redhat.com>
---
ChangeLog | 7 +++++++
lib/openat.h | 18 +++---------------
2 files changed, 10 insertions(+), 15 deletions(-)
diff --git a/ChangeLog b/ChangeLog
index edc7c8b69b..87559f4b24 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,10 @@
+2010-10-05 Eric Blake <eblake@redhat.com>
+
+ faccessat: remove unused wrappers
+ * lib/openat.h (accessat, euidaccesat): Delete, since the mere
+ presence of these wrappers dragged in -lgen on Solaris.
+ Reported by Clemens Brogi; fix suggested by Paul Eggert.
+
2010-10-05 Jim Meyering <meyering@redhat.com>
tests: require @PRAGMA_COLUMNS@ with each @PRAGMA_SYSTEM_HEADER@
diff --git a/lib/openat.h b/lib/openat.h
index 11a0314d24..1d4432e6ca 100644
--- a/lib/openat.h
+++ b/lib/openat.h
@@ -93,22 +93,10 @@ lstatat (int fd, char const *name, struct stat *st)
return fstatat (fd, name, st, AT_SYMLINK_NOFOLLOW);
}
-#if GNULIB_FACCESSAT
/* For now, there are no wrappers named laccessat or leuidaccessat,
since gnulib doesn't support faccessat(,AT_SYMLINK_NOFOLLOW) and
- since access rights on symlinks are of limited utility. */
-
-static inline int
-accessat (int fd, char const *file, int mode)
-{
- return faccessat (fd, file, mode, 0);
-}
-
-static inline int
-euidaccessat (int fd, char const *file, int mode)
-{
- return faccessat (fd, file, mode, AT_EACCESS);
-}
-#endif
+ since access rights on symlinks are of limited utility. Likewise,
+ wrappers are not provided for accessat or euidaccessat, so as to
+ avoid dragging in -lgen on some platforms. */
#endif /* _GL_HEADER_OPENAT */
--
2.30.2