bug#7142: [PATCH] faccessat: remove unused wrappers (original) (raw)
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
From: | Eric Blake |
---|---|
Subject: | bug#7142: [PATCH] faccessat: remove unused wrappers |
Date: | Tue, 5 Oct 2010 08:08:31 -0600 |
- 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 <address@hidden>
I'm pushing this.
ChangeLog | 7 +++++++ lib/openat.h | 18 +++--------------- 2 files changed, 10 insertions(+), 15 deletions(-)
diff --git a/ChangeLog b/ChangeLog index edc7c8b..87559f4 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,3 +1,10 @@ +2010-10-05 Eric Blake <address@hidden> +
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 <address@hidden>
tests: require @PRAGMA_COLUMNS@ with each @PRAGMA_SYSTEM_HEADER@
diff --git a/lib/openat.h b/lib/openat.h index 11a0314..1d4432e 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 */
1.7.2.3
- bug#7142: coreutils on Solaris 10(x86) 09/10, (continued)
- bug#7142: coreutils on Solaris 10(x86) 09/10, Jim Meyering, 2010/10/01
* Message not available
* bug#7142: coreutils on Solaris 10(x86) 09/10, Jim Meyering, 2010/10/01 - bug#7142: coreutils on Solaris 10(x86) 09/10, Eric Blake, 2010/10/01
* Message not available
* bug#7142: coreutils on Solaris 10(x86) 09/10, Eric Blake, 2010/10/01
* bug#7142: coreutils on Solaris 10(x86) 09/10, Clemens Brogi, 2010/10/04
* bug#7142: coreutils on Solaris 10(x86) 09/10, Eric Blake, 2010/10/04
* bug#7142: coreutils on Solaris 10(x86) 09/10, Paul Eggert, 2010/10/04
* bug#7142: coreutils on Solaris 10(x86) 09/10, Eric Blake, 2010/10/04
* bug#7142: coreutils on Solaris 10(x86) 09/10, Paul Eggert, 2010/10/04
* bug#7142: coreutils on Solaris 10(x86) 09/10, Jim Meyering, 2010/10/05
* bug#7142: [PATCH] faccessat: remove unused wrappers,Eric Blake <=
* bug#7142: [PATCH] faccessat: remove unused wrappers, Jim Meyering, 2010/10/05
* bug#7142: [PATCH] build: fix build on Solaris 10, Eric Blake, 2010/10/05
- bug#7142: coreutils on Solaris 10(x86) 09/10, Jim Meyering, 2010/10/01
- Prev by Date:bug#7142: coreutils on Solaris 10(x86) 09/10
- Next by Date:bug#7142: [PATCH] faccessat: remove unused wrappers
- Previous by thread:bug#7142: coreutils on Solaris 10(x86) 09/10
- Next by thread:bug#7142: [PATCH] faccessat: remove unused wrappers
- Index(es):