+tar (1.22-3) UNRELEASED; urgency=low
+
+ * Preserve timestamps of extracted symlinks, closes: #313237
+
+ -- Carl Worth <cworth@cworth.org> Tue, 04 Aug 2009 17:11:18 -0700
+
tar (1.22-2) unstable; urgency=low
* Add Carl Worth as an uploader.
use just futimes (or equivalent) instead of utimes (or equivalent),
and fail if on an old system without futimes (or equivalent).
If TIMESPEC is null, set the time stamps to the current time.
+ If the file is a symlink and IS_SYMLINK is set, then the
+ time stamps of the symlink itself will be updated if
+ possible, (but if not supported by the operating system
+ then no change will occur).
Return 0 on success, -1 (setting errno) on failure. */
int
gl_futimens (int fd ATTRIBUTE_UNUSED,
- char const *file, struct timespec const timespec[2])
+ char const *file, struct timespec const timespec[2],
+ int is_symlink)
{
/* Some Linux-based NFS clients are buggy, and mishandle time stamps
of files in NFS file systems in some cases. We have no
#if HAVE_UTIMENSAT
if (fd < 0)
{
- int result = utimensat (AT_FDCWD, file, timespec, 0);
+ int flags = is_symlink ? AT_SYMLINK_NOFOLLOW : 0;
+ int result = utimensat (AT_FDCWD, file, timespec, flags);
# ifdef __linux__
/* Work around what might be a kernel bug:
http://bugzilla.redhat.com/442352
return result;
}
#endif
+
+ /* Without utimensat we have no way to update a symlink rather than
+ * the target, so just return immediately. */
+ if (is_symlink)
+ return 0;
+
#if HAVE_FUTIMENS
{
int result = futimens (fd, timespec);
}
/* Set the access and modification time stamps of FILE to be
- TIMESPEC[0] and TIMESPEC[1], respectively. */
+ TIMESPEC[0] and TIMESPEC[1], respectively.
+ If the file is a symlink and is_symlink is set, then the
+ time stamps of the symlink itself will be updated if
+ possible, (but if not supported by the operating system
+ then no change will occur). */
int
-utimens (char const *file, struct timespec const timespec[2])
+utimens (char const *file, struct timespec const timespec[2], int is_symlink)
{
- return gl_futimens (-1, file, timespec);
+ return gl_futimens (-1, file, timespec, is_symlink);
}
#include <time.h>
-int gl_futimens (int, char const *, struct timespec const [2]);
-int utimens (char const *, struct timespec const [2]);
+int gl_futimens (int, char const *, struct timespec const [2], int flags);
+int utimens (char const *, struct timespec const [2], int flags);
mode_t invert_permissions, enum permstatus permstatus,
char typeflag)
{
- if (typeflag != SYMTYPE)
+ /* We do the utime before the chmod because some versions of utime are
+ broken and trash the modes of the file. */
+
+ if (! touch_option && permstatus != INTERDIR_PERMSTATUS)
{
- /* We do the utime before the chmod because some versions of utime are
- broken and trash the modes of the file. */
+ /* We set the accessed time to `now', which is really the time we
+ started extracting files, unless incremental_option is used, in
+ which case .st_atime is used. */
- if (! touch_option && permstatus != INTERDIR_PERMSTATUS)
- {
- /* We set the accessed time to `now', which is really the time we
- started extracting files, unless incremental_option is used, in
- which case .st_atime is used. */
-
- /* FIXME: incremental_option should set ctime too, but how? */
-
- struct timespec ts[2];
- if (incremental_option)
- ts[0] = st->atime;
- else
- ts[0] = start_time;
- ts[1] = st->mtime;
-
- if (utimens (file_name, ts) != 0)
- utime_error (file_name);
- else
- {
- check_time (file_name, ts[0]);
- check_time (file_name, ts[1]);
- }
- }
+ /* FIXME: incremental_option should set ctime too, but how? */
- /* Some systems allow non-root users to give files away. Once this
- done, it is not possible anymore to change file permissions.
- However, setting file permissions now would be incorrect, since
- they would apply to the wrong user, and there would be a race
- condition. So, don't use systems that allow non-root users to
- give files away. */
+ struct timespec ts[2];
+ if (incremental_option)
+ ts[0] = st->atime;
+ else
+ ts[0] = start_time;
+ ts[1] = st->mtime;
+
+ if (utimens (file_name, ts, typeflag == SYMTYPE) != 0)
+ utime_error (file_name);
+ else
+ {
+ check_time (file_name, ts[0]);
+ check_time (file_name, ts[1]);
+ }
}
+ /* Some systems allow non-root users to give files away. Once this
+ done, it is not possible anymore to change file permissions.
+ However, setting file permissions now would be incorrect, since
+ they would apply to the wrong user, and there would be a race
+ condition. So, don't use systems that allow non-root users to
+ give files away. */
+
if (0 < same_owner_option && permstatus != INTERDIR_PERMSTATUS)
{
/* When lchown exists, it should be used to change the attributes of
}
#endif
- return gl_futimens (fd, file, timespec);
+ return gl_futimens (fd, file, timespec, 0);
}
/* A description of a working directory. */