Fixed failing hunks.
[rsync-patches.git] / atimes.diff
1 After applying this patch and running configure, you MUST run this
2 command before "make":
3
4     make proto
5
6
7 --- orig/batch.c        2005-10-26 16:49:08
8 +++ batch.c     2004-07-03 20:15:41
9 @@ -225,6 +225,8 @@ void show_flist(int index, struct file_s
10                 rprintf(FINFO, "flist->flags=%#x\n", fptr[i]->flags);
11                 rprintf(FINFO, "flist->modtime=%#lx\n",
12                         (long unsigned) fptr[i]->modtime);
13 +               rprintf(FINFO, "flist->atime=%#lx\n",
14 +                       (long unsigned) fptr[i]->atime);
15                 rprintf(FINFO, "flist->length=%.0f\n",
16                         (double) fptr[i]->length);
17                 rprintf(FINFO, "flist->mode=%#o\n", (int) fptr[i]->mode);
18 --- orig/flist.c        2005-11-07 04:29:01
19 +++ flist.c     2005-07-28 00:16:34
20 @@ -50,6 +50,7 @@ extern int preserve_perms;
21  extern int preserve_devices;
22  extern int preserve_uid;
23  extern int preserve_gid;
24 +extern int preserve_atimes;
25  extern int relative_paths;
26  extern int implied_dirs;
27  extern int copy_links;
28 @@ -138,16 +139,18 @@ static void list_file_entry(struct file_
29  
30  #ifdef SUPPORT_LINKS
31         if (preserve_links && S_ISLNK(f->mode)) {
32 -               rprintf(FINFO, "%s %11.0f %s %s -> %s\n",
33 +               rprintf(FINFO, "%s %11.0f %s %s %s -> %s\n",
34                         perms,
35                         (double)f->length, timestring(f->modtime),
36 +                       timestring(f->atime),
37                         safe_fname(f_name(f)), safe_fname(f->u.link));
38         } else
39  #endif
40         {
41 -               rprintf(FINFO, "%s %11.0f %s %s\n",
42 +               rprintf(FINFO, "%s %11.0f %s %s %s\n",
43                         perms,
44                         (double)f->length, timestring(f->modtime),
45 +                       timestring(f->atime),
46                         safe_fname(f_name(f)));
47         }
48  }
49 @@ -309,6 +312,7 @@ void send_file_entry(struct file_struct 
50  {
51         unsigned short flags;
52         static time_t modtime;
53 +       static time_t atime;
54         static mode_t mode;
55         static int64 dev;
56         static dev_t rdev;
57 @@ -324,7 +328,7 @@ void send_file_entry(struct file_struct 
58  
59         if (!file) {
60                 write_byte(f, 0);
61 -               modtime = 0, mode = 0;
62 +               modtime = 0, atime = 0, mode = 0;
63                 dev = 0, rdev = makedev(0, 0);
64                 rdev_major = 0;
65                 uid = 0, gid = 0;
66 @@ -373,6 +377,12 @@ void send_file_entry(struct file_struct 
67                 flags |= XMIT_SAME_TIME;
68         else
69                 modtime = file->modtime;
70 +       if (preserve_atimes && !S_ISDIR(mode)) {
71 +               if (file->atime == atime)
72 +                       flags |= XMIT_SAME_ATIME;
73 +               else
74 +                       atime = file->atime;
75 +       }
76  
77  #ifdef SUPPORT_HARD_LINKS
78         if (file->link_u.idev) {
79 @@ -426,6 +436,8 @@ void send_file_entry(struct file_struct 
80                 write_int(f, modtime);
81         if (!(flags & XMIT_SAME_MODE))
82                 write_int(f, to_wire_mode(mode));
83 +       if (preserve_atimes && !S_ISDIR(mode) && !(flags & XMIT_SAME_ATIME))
84 +               write_int(f, atime);
85         if (preserve_uid && !(flags & XMIT_SAME_UID)) {
86                 if (!numeric_ids)
87                         add_uid(uid);
88 @@ -494,6 +506,7 @@ static struct file_struct *receive_file_
89                                               unsigned short flags, int f)
90  {
91         static time_t modtime;
92 +       static time_t atime;
93         static mode_t mode;
94         static int64 dev;
95         static dev_t rdev;
96 @@ -512,7 +525,7 @@ static struct file_struct *receive_file_
97         struct file_struct *file;
98  
99         if (!flist) {
100 -               modtime = 0, mode = 0;
101 +               modtime = 0, atime = 0, mode = 0;
102                 dev = 0, rdev = makedev(0, 0);
103                 rdev_major = 0;
104                 uid = 0, gid = 0;
105 @@ -568,6 +581,8 @@ static struct file_struct *receive_file_
106                 modtime = (time_t)read_int(f);
107         if (!(flags & XMIT_SAME_MODE))
108                 mode = from_wire_mode(read_int(f));
109 +       if (preserve_atimes && !S_ISDIR(mode) && !(flags & XMIT_SAME_ATIME))
110 +               atime = (time_t)read_int(f);
111  
112         if (preserve_uid && !(flags & XMIT_SAME_UID))
113                 uid = (uid_t)read_int(f);
114 @@ -618,6 +633,7 @@ static struct file_struct *receive_file_
115  
116         file->flags = 0;
117         file->modtime = modtime;
118 +       file->atime = atime;
119         file->length = file_length;
120         file->mode = mode;
121         file->uid = uid;
122 @@ -866,6 +882,7 @@ skip_filters:
123  
124         file->flags = flags;
125         file->modtime = st.st_mtime;
126 +       file->atime = st.st_atime;
127         file->length = st.st_size;
128         if (chmod_modes && am_sender && (S_ISREG(st.st_mode) || S_ISDIR(st.st_mode)))
129                 file->mode = tweak_mode(st.st_mode, chmod_modes);
130 --- orig/generator.c    2005-10-30 22:30:28
131 +++ generator.c 2005-07-28 00:14:43
132 @@ -44,6 +44,7 @@ extern int preserve_uid;
133  extern int preserve_gid;
134  extern int preserve_times;
135  extern int omit_dir_times;
136 +extern int preserve_atimes;
137  extern int delete_before;
138  extern int delete_during;
139  extern int delete_after;
140 @@ -325,9 +326,21 @@ void itemize(struct file_struct *file, i
141                             : S_ISDIR(file->mode) ? !omit_dir_times
142                             : !S_ISLNK(file->mode);
143  
144 +                       if (iflags & (ITEM_TRANSFER|ITEM_LOCAL_CHANGE) && !keep_time
145 +                            && (!(iflags & ITEM_XNAME_FOLLOWS) || *xname)) {
146 +                               iflags |= ITEM_REPORT_TIME;
147 +                               if (!preserve_atimes && !S_ISDIR(file->mode))
148 +                                       iflags |= ITEM_REPORT_ATIME;
149 +                       } else {
150 +                               if (keep_time && cmp_time(file->modtime, st->st_mtime) != 0)
151 +                                       iflags |= ITEM_REPORT_TIME;
152 +                               if (preserve_atimes && !S_ISDIR(file->mode)
153 +                                && cmp_time(file->atime, st->st_atime) != 0)
154 +                                       iflags |= ITEM_REPORT_ATIME;
155 +                       }
156                         if ((iflags & (ITEM_TRANSFER|ITEM_LOCAL_CHANGE) && !keep_time
157                              && (!(iflags & ITEM_XNAME_FOLLOWS) || *xname))
158 -                           || (keep_time && cmp_modtime(file->modtime, st->st_mtime) != 0))
159 +                           || (keep_time && cmp_time(file->modtime, st->st_mtime) != 0))
160                                 iflags |= ITEM_REPORT_TIME;
161                         if (preserve_perms
162                          && (file->mode & CHMOD_BITS) != (st->st_mode & CHMOD_BITS))
163 @@ -379,7 +392,7 @@ static int unchanged_file(char *fn, stru
164         if (ignore_times)
165                 return 0;
166  
167 -       return cmp_modtime(st->st_mtime, file->modtime) == 0;
168 +       return cmp_time(st->st_mtime, file->modtime) == 0;
169  }
170  
171  
172 @@ -539,7 +552,7 @@ static int find_fuzzy(struct file_struct
173                 name = fp->basename;
174  
175                 if (fp->length == file->length
176 -                   && cmp_modtime(fp->modtime, file->modtime) == 0) {
177 +                   && cmp_time(fp->modtime, file->modtime) == 0) {
178                         if (verbose > 4) {
179                                 rprintf(FINFO,
180                                         "fuzzy size/modtime match for %s\n",
181 @@ -891,7 +904,7 @@ static void recv_generator(char *fname, 
182         }
183  
184         if (update_only && statret == 0
185 -           && cmp_modtime(st.st_mtime, file->modtime) > 0) {
186 +           && cmp_time(st.st_mtime, file->modtime) > 0) {
187                 if (verbose > 1)
188                         rprintf(FINFO, "%s is newer\n", safe_fname(fname));
189                 return;
190 --- orig/log.c  2005-10-26 16:49:08
191 +++ log.c       2005-07-28 00:22:30
192 @@ -38,6 +38,7 @@ extern int module_id;
193  extern int msg_fd_out;
194  extern int protocol_version;
195  extern int preserve_times;
196 +extern int preserve_atimes;
197  extern int log_format_has_o_or_i;
198  extern int daemon_log_format_has_o_or_i;
199  extern char *auth_user;
200 @@ -503,11 +504,14 @@ static void log_formatted(enum logcode c
201                         n[4] = !(iflags & ITEM_REPORT_TIME) ? '.'
202                              : !preserve_times || IS_DEVICE(file->mode)
203                                                || S_ISLNK(file->mode) ? 'T' : 't';
204 -                       n[5] = !(iflags & ITEM_REPORT_PERMS) ? '.' : 'p';
205 -                       n[6] = !(iflags & ITEM_REPORT_OWNER) ? '.' : 'o';
206 -                       n[7] = !(iflags & ITEM_REPORT_GROUP) ? '.' : 'g';
207 -                       n[8] = !(iflags & ITEM_REPORT_XATTRS) ? '.' : 'a';
208 -                       n[9] = '\0';
209 +                       n[5] = !(iflags & ITEM_REPORT_ATIME) ? '.'
210 +                            : !preserve_atimes || IS_DEVICE(file->mode)
211 +                                               || S_ISLNK(file->mode) ? 'A' : 'a';
212 +                       n[6] = !(iflags & ITEM_REPORT_PERMS) ? '.' : 'p';
213 +                       n[7] = !(iflags & ITEM_REPORT_OWNER) ? '.' : 'o';
214 +                       n[8] = !(iflags & ITEM_REPORT_GROUP) ? '.' : 'g';
215 +                       n[9] = !(iflags & ITEM_REPORT_XATTRS) ? '.' : 'a';
216 +                       n[10] = '\0';
217  
218                         if (iflags & (ITEM_IS_NEW|ITEM_MISSING_DATA)) {
219                                 char ch = iflags & ITEM_IS_NEW ? '+' : '?';
220 --- orig/options.c      2005-11-07 04:29:01
221 +++ options.c   2005-11-07 04:32:19
222 @@ -50,6 +50,7 @@ int preserve_uid = 0;
223  int preserve_gid = 0;
224  int preserve_times = 0;
225  int omit_dir_times = 0;
226 +int preserve_atimes = 0;
227  int update_only = 0;
228  int cvs_exclude = 0;
229  int dry_run = 0;
230 @@ -292,8 +293,9 @@ void usage(enum logcode F)
231    rprintf(F," -o, --owner                 preserve owner (root only)\n");
232    rprintf(F," -g, --group                 preserve group\n");
233    rprintf(F," -D, --devices               preserve devices (root only)\n");
234 -  rprintf(F," -t, --times                 preserve times\n");
235 -  rprintf(F," -O, --omit-dir-times        omit directories when preserving times\n");
236 +  rprintf(F," -t, --times                 preserve modify times\n");
237 +  rprintf(F," -O, --omit-dir-times        omit directories when preserving modify times\n");
238 +  rprintf(F," -A, --atimes                preserve access times\n");
239    rprintf(F,"     --chmod=CHMOD           change destination permissions\n");
240    rprintf(F," -S, --sparse                handle sparse files efficiently\n");
241    rprintf(F," -n, --dry-run               show what would have been transferred\n");
242 @@ -397,6 +399,9 @@ static struct poptOption long_options[] 
243    {"times",           't', POPT_ARG_VAL,    &preserve_times, 1, 0, 0 },
244    {"no-times",         0,  POPT_ARG_VAL,    &preserve_times, 0, 0, 0 },
245    {"no-t",             0,  POPT_ARG_VAL,    &preserve_times, 0, 0, 0 },
246 +  {"atimes",          'A', POPT_ARG_VAL,    &preserve_atimes, 1, 0, 0 },
247 +  {"no-atimes",        0,  POPT_ARG_VAL,    &preserve_atimes, 0, 0, 0 },
248 +  {"no-A",             0,  POPT_ARG_VAL,    &preserve_atimes, 0, 0, 0 },
249    {"omit-dir-times",  'O', POPT_ARG_VAL,    &omit_dir_times, 2, 0, 0 },
250    {"modify-window",    0,  POPT_ARG_INT,    &modify_window, OPT_MODIFY_WINDOW, 0, 0 },
251    {"owner",           'o', POPT_ARG_VAL,    &preserve_uid, 1, 0, 0 },
252 @@ -1429,6 +1434,8 @@ void server_options(char **args,int *arg
253                 argstr[x++] = 'D';
254         if (preserve_times)
255                 argstr[x++] = 't';
256 +       if (preserve_atimes)
257 +               argstr[x++] = 'A';
258         if (omit_dir_times == 2 && am_sender)
259                 argstr[x++] = 'O';
260         if (preserve_perms)
261 --- orig/rsync.c        2005-07-27 23:31:12
262 +++ rsync.c     2005-07-28 00:17:37
263 @@ -27,6 +27,7 @@ extern int dry_run;
264  extern int daemon_log_format_has_i;
265  extern int preserve_times;
266  extern int omit_dir_times;
267 +extern int preserve_atimes;
268  extern int am_root;
269  extern int am_server;
270  extern int am_sender;
271 @@ -56,6 +57,7 @@ int set_perms(char *fname,struct file_st
272         int updated = 0;
273         STRUCT_STAT st2;
274         int change_uid, change_gid;
275 +       time_t atime, mtime;
276  
277         if (!st) {
278                 if (dry_run)
279 @@ -70,16 +72,29 @@ int set_perms(char *fname,struct file_st
280  
281         if (!preserve_times || (S_ISDIR(st->st_mode) && omit_dir_times))
282                 flags |= PERMS_SKIP_MTIME;
283 +       if (!preserve_atimes || S_ISDIR(st->st_mode))
284 +               flags |= PERMS_SKIP_ATIME;
285         if (!(flags & PERMS_SKIP_MTIME)
286 -           && cmp_modtime(st->st_mtime, file->modtime) != 0) {
287 -               int ret = set_modtime(fname, file->modtime, st->st_mode);
288 +           && cmp_time(st->st_mtime, file->modtime) != 0) {
289 +               mtime = file->modtime;
290 +               updated = 1;
291 +       } else
292 +               mtime = st->st_mtime;
293 +       if (!(flags & PERMS_SKIP_ATIME)
294 +           && cmp_time(st->st_atime, file->atime) != 0) {
295 +               atime = file->atime;
296 +               updated = 1;
297 +       } else
298 +               atime = st->st_atime;
299 +       if (updated) {
300 +               int ret = set_times(fname, mtime, atime, st->st_mode);
301                 if (ret < 0) {
302                         rsyserr(FERROR, errno, "failed to set times on %s",
303                                 full_fname(fname));
304                         return 0;
305                 }
306 -               if (ret == 0) /* ret == 1 if symlink could not be set */
307 -                       updated = 1;
308 +               if (ret > 0) /* ret == 1 if symlink could not be set */
309 +                       updated = 0;
310         }
311  
312         change_uid = am_root && preserve_uid && st->st_uid != file->uid;
313 --- orig/rsync.h        2005-10-14 18:45:50
314 +++ rsync.h     2005-07-28 00:04:51
315 @@ -54,6 +54,7 @@
316  #define XMIT_HAS_IDEV_DATA (1<<9)
317  #define XMIT_SAME_DEV (1<<10)
318  #define XMIT_RDEV_MINOR_IS_SMALL (1<<11)
319 +#define XMIT_SAME_ATIME (1<<12)
320  
321  /* These flags are used in the live flist data. */
322  
323 @@ -119,6 +120,7 @@
324  
325  #define PERMS_REPORT           (1<<0)
326  #define PERMS_SKIP_MTIME       (1<<1)
327 +#define PERMS_SKIP_ATIME       (1<<2)
328  
329  #define FULL_FLUSH     1
330  #define NORMAL_FLUSH   0
331 @@ -140,6 +142,7 @@
332  #define DEL_TERSE              (1<<3)
333  
334  /* For use by the itemize_changes code */
335 +#define ITEM_REPORT_ATIME (1<<0)
336  #define ITEM_REPORT_CHECKSUM (1<<1)
337  #define ITEM_REPORT_SIZE (1<<2)
338  #define ITEM_REPORT_TIME (1<<3)
339 @@ -522,6 +525,7 @@ struct file_struct {
340                 struct hlink *links;
341         } link_u;
342         time_t modtime;
343 +       time_t atime;
344         uid_t uid;
345         gid_t gid;
346         mode_t mode;
347 --- orig/rsync.yo       2005-11-07 04:29:02
348 +++ rsync.yo    2005-11-07 04:34:55
349 @@ -319,8 +319,9 @@ to the detailed description below for a 
350   -o, --owner                 preserve owner (root only)
351   -g, --group                 preserve group
352   -D, --devices               preserve devices (root only)
353 - -t, --times                 preserve times
354 - -O, --omit-dir-times        omit directories when preserving times
355 + -t, --times                 preserve modify times
356 + -O, --omit-dir-times        omit directories when preserving mod-times
357 + -A, --atimes                preserve access times
358       --chmod=CHMOD           change destination permissions
359   -S, --sparse                handle sparse files efficiently
360   -n, --dry-run               show what would have been transferred
361 @@ -698,6 +699,12 @@ it is preserving modification times (see
362  the directories on the receiving side, it is a good idea to use bf(-O).
363  This option is inferred if you use bf(--backup) without bf(--backup-dir).
364  
365 +dit(bf(-A, --atimes)) This tells rsync to set the access times of the
366 +destination files to the same value as the source files.  Note that the
367 +reading of the source file may update the atime of the source files, so
368 +repeated rsync runs with --atimes may be needed if you want to force the
369 +access-time values to be 100% identical on the two systems.
370 +
371  dit(bf(--chmod)) This options tells rsync to apply the listed "chmod" pattern
372  to the permission of the files on the destination.  In addition to the normal
373  parsing rules specified in the chmod manpage, you can specify an item that
374 @@ -1139,7 +1146,7 @@ changes that are being made to each file
375  This is exactly the same as specifying bf(--log-format='%i %n%L').
376  
377  The "%i" escape has a cryptic output that is 9 letters long.  The general
378 -format is like the string bf(UXcstpoga)), where bf(U) is replaced by the
379 +format is like the string bf(UXcstapogx)), where bf(U) is replaced by the
380  kind of update being done, bf(X) is replaced by the file-type, and the
381  other letters represent attributes that may be output if they are being
382  modified.
383 @@ -1178,17 +1185,22 @@ quote(itemize(
384    by the file transfer.
385    it() A bf(t) means the modification time is different and is being updated
386    to the sender's value (requires bf(--times)).  An alternate value of bf(T)
387 -  means that the time will be set to the transfer time, which happens
388 +  means that the modify time will be set to the transfer time, which happens
389    anytime a symlink is transferred, or when a file or device is transferred
390    without bf(--times).
391 +  it() A bf(a) means the access time is different and is being updated to
392 +  the sender's value (requires bf(--atimes)).  An alternate value of bf(A)
393 +  means that the access time  will be set to the transfer time, which happens
394 +  anytime a symlink is transferred, or when a file or device is transferred
395 +  without bf(--atimes).
396    it() A bf(p) means the permissions are different and are being updated to
397    the sender's value (requires bf(--perms)).
398    it() An bf(o) means the owner is different and is being updated to the
399    sender's value (requires bf(--owner) and root privileges).
400    it() A bf(g) means the group is different and is being updated to the
401    sender's value (requires bf(--group) and the authority to set the group).
402 -  it() The bf(a) is reserved for a future enhanced version that supports
403 -  extended file attributes, such as ACLs.
404 +  it() The bf(x) is reserved for a future enhanced version that supports
405 +  extended file attributes.
406  ))
407  
408  One other output is possible:  when deleting files, the "%i" will output
409 --- orig/testsuite/atimes.test  2004-06-30 00:06:23
410 +++ testsuite/atimes.test       2004-06-30 00:06:23
411 @@ -0,0 +1,19 @@
412 +#! /bin/sh
413 +
414 +# Test rsync copying atimes
415 +
416 +. "$suitedir/rsync.fns"
417 +
418 +set -x
419 +
420 +mkdir "$fromdir"
421 +
422 +touch "$fromdir/foo"
423 +touch -a -t 200102031717.42 "$fromdir/foo"
424 +
425 +TLS_ARGS=--atime
426 +
427 +checkit "$RSYNC -rtAgvvv \"$fromdir/\" \"$todir/\"" "$fromdir" "$todir"
428 +
429 +# The script would have aborted on error, so getting here means we've won.
430 +exit 0
431 --- orig/testsuite/itemize.test 2005-07-07 20:35:48
432 +++ testsuite/itemize.test      2005-07-28 00:29:54
433 @@ -44,14 +44,14 @@ ln "$fromdir/foo/config1" "$fromdir/foo/
434  $RSYNC -iplr "$fromdir/" "$todir/" \
435      | tee "$outfile"
436  cat <<EOT >"$chkfile"
437 -cd+++++++ bar/
438 -cd+++++++ bar/baz/
439 ->f+++++++ bar/baz/rsync
440 -cd+++++++ foo/
441 ->f+++++++ foo/config1
442 ->f+++++++ foo/config2
443 ->f+++++++ foo/extra
444 -cL+++++++ foo/sym -> ../bar/baz/rsync
445 +cd++++++++ bar/
446 +cd++++++++ bar/baz/
447 +>f++++++++ bar/baz/rsync
448 +cd++++++++ foo/
449 +>f++++++++ foo/config1
450 +>f++++++++ foo/config2
451 +>f++++++++ foo/extra
452 +cL++++++++ foo/sym -> ../bar/baz/rsync
453  EOT
454  diff $diffopt "$chkfile" "$outfile" || test_fail "test 1 failed"
455  
456 @@ -63,10 +63,10 @@ chmod 601 "$fromdir/foo/config2"
457  $RSYNC -iplrH "$fromdir/" "$todir/" \
458      | tee "$outfile"
459  cat <<EOT >"$chkfile"
460 ->f..T.... bar/baz/rsync
461 ->f..T.... foo/config1
462 ->f.sTp... foo/config2
463 -hf..T.... foo/extra => foo/config1
464 +>f..TA.... bar/baz/rsync
465 +>f..TA.... foo/config1
466 +>f.sTAp... foo/config2
467 +hf..TA.... foo/extra => foo/config1
468  EOT
469  diff $diffopt "$chkfile" "$outfile" || test_fail "test 2 failed"
470  
471 @@ -83,11 +83,11 @@ chmod 777 "$todir/bar/baz/rsync"
472  $RSYNC -iplrtc "$fromdir/" "$todir/" \
473      | tee "$outfile"
474  cat <<EOT >"$chkfile"
475 -.f..tp... bar/baz/rsync
476 -.d..t.... foo/
477 -.f..t.... foo/config1
478 ->fcstp... foo/config2
479 -cL..T.... foo/sym -> ../bar/baz/rsync
480 +.f..t.p... bar/baz/rsync
481 +.d..t..... foo/
482 +.f..t..... foo/config1
483 +>fcst.p... foo/config2
484 +cL..TA.... foo/sym -> ../bar/baz/rsync
485  EOT
486  diff $diffopt "$chkfile" "$outfile" || test_fail "test 3 failed"
487  
488 @@ -112,15 +112,15 @@ $RSYNC -ivvplrtH "$fromdir/" "$todir/" \
489      | tee "$outfile"
490  filter_outfile
491  cat <<EOT >"$chkfile"
492 -.d        ./
493 -.d        bar/
494 -.d        bar/baz/
495 -.f...p... bar/baz/rsync
496 -.d        foo/
497 -.f        foo/config1
498 ->f..t.... foo/config2
499 -hf        foo/extra
500 -.L        foo/sym -> ../bar/baz/rsync
501 +.d         ./
502 +.d         bar/
503 +.d         bar/baz/
504 +.f....p... bar/baz/rsync
505 +.d         foo/
506 +.f         foo/config1
507 +>f..t..... foo/config2
508 +hf         foo/extra
509 +.L         foo/sym -> ../bar/baz/rsync
510  EOT
511  diff $diffopt "$chkfile" "$outfile" || test_fail "test 5 failed"
512  
513 @@ -139,8 +139,8 @@ touch "$todir/foo/config2"
514  $RSYNC -iplrtH "$fromdir/" "$todir/" \
515      | tee "$outfile"
516  cat <<EOT >"$chkfile"
517 -.f...p... foo/config1
518 ->f..t.... foo/config2
519 +.f....p... foo/config1
520 +>f..t..... foo/config2
521  EOT
522  diff $diffopt "$chkfile" "$outfile" || test_fail "test 7 failed"
523  
524 @@ -149,15 +149,15 @@ $RSYNC -ivvplrtH --copy-dest="$lddir" "$
525      | tee "$outfile"
526  filter_outfile
527  cat <<EOT >"$chkfile"
528 -.d..t.... ./
529 -cd+++++++ bar/
530 -cd+++++++ bar/baz/
531 -cf....... bar/baz/rsync
532 -cd+++++++ foo/
533 -cf....... foo/config1
534 -cf....... foo/config2
535 -hf+++++++ foo/extra => foo/config1
536 -cL+++++++ foo/sym -> ../bar/baz/rsync
537 +.d..t..... ./
538 +cd++++++++ bar/
539 +cd++++++++ bar/baz/
540 +cf........ bar/baz/rsync
541 +cd++++++++ foo/
542 +cf........ foo/config1
543 +cf........ foo/config2
544 +hf++++++++ foo/extra => foo/config1
545 +cL++++++++ foo/sym -> ../bar/baz/rsync
546  EOT
547  diff $diffopt "$chkfile" "$outfile" || test_fail "test 8 failed"
548  
549 @@ -165,12 +165,12 @@ rm -rf "$todir"
550  $RSYNC -iplrtH --link-dest="$lddir" "$fromdir/" "$todir/" \
551      | tee "$outfile"
552  cat <<EOT >"$chkfile"
553 -.d..t.... ./
554 -cd+++++++ bar/
555 -cd+++++++ bar/baz/
556 -cd+++++++ foo/
557 -hf+++++++ foo/extra => foo/config1
558 -cL+++++++ foo/sym -> ../bar/baz/rsync
559 +.d..t..... ./
560 +cd++++++++ bar/
561 +cd++++++++ bar/baz/
562 +cd++++++++ foo/
563 +hf++++++++ foo/extra => foo/config1
564 +cL++++++++ foo/sym -> ../bar/baz/rsync
565  EOT
566  diff $diffopt "$chkfile" "$outfile" || test_fail "test 9 failed"
567  
568 --- orig/testsuite/rsync.fns    2005-06-10 21:33:28
569 +++ testsuite/rsync.fns 2005-07-28 00:41:20
570 @@ -50,7 +50,7 @@ printmsg() {
571  
572  
573  rsync_ls_lR() {
574 -    find "$@" -print | sort | sed 's/ /\\ /g' | xargs "$TOOLDIR/tls"
575 +    find "$@" -print | sort | sed 's/ /\\ /g' | xargs "$TOOLDIR/tls" $TLS_ARGS
576  }
577  
578  rsync_getgroups() { 
579 @@ -158,6 +158,10 @@ checkit() {
580      # We can just write everything to stdout/stderr, because the
581      # wrapper hides it unless there is a problem.
582  
583 +    if test x$TLS_ARGS = x--atime; then
584 +       ( cd "$2" && rsync_ls_lR . ) > "$tmpdir/ls-from"
585 +    fi
586 +
587      echo "Running: \"$1\""  
588      eval "$1" 
589      status=$?
590 @@ -165,10 +169,13 @@ checkit() {
591         failed="YES";
592      fi
593  
594 +    if test x$TLS_ARGS != x--atime; then
595 +       ( cd "$2" && rsync_ls_lR . ) > "$tmpdir/ls-from"
596 +    fi
597 +
598      echo "-------------"
599      echo "check how the directory listings compare with diff:"
600      echo ""
601 -    ( cd "$2" && rsync_ls_lR . ) > "$tmpdir/ls-from"
602      ( cd "$3" && rsync_ls_lR . ) > "$tmpdir/ls-to"
603      diff $diffopt "$tmpdir/ls-from" "$tmpdir/ls-to" || failed=YES
604  
605 --- orig/tls.c  2005-09-24 17:40:31
606 +++ tls.c       2005-03-23 17:49:48
607 @@ -39,6 +39,7 @@
608  
609  
610  #include "rsync.h"
611 +#include "popt.h"
612  
613  #define PROGRAM "tls"
614  
615 @@ -48,6 +49,7 @@ int read_only = 1;
616  int list_only = 0;
617  int preserve_perms = 0;
618  
619 +static int display_atime = 0;
620  
621  static void failed(char const *what, char const *where)
622  {
623 @@ -56,14 +58,29 @@ static void failed(char const *what, cha
624         exit(1);
625  }
626  
627 +static void storetime(char *dest, time_t t)
628 +{
629 +       if (t) {
630 +               struct tm *mt = gmtime(&t);
631  
632 +               sprintf(dest, "%04d-%02d-%02d %02d:%02d:%02d ",
633 +                       (int)mt->tm_year + 1900,
634 +                       (int)mt->tm_mon + 1,
635 +                       (int)mt->tm_mday,
636 +                       (int)mt->tm_hour,
637 +                       (int)mt->tm_min,
638 +                       (int)mt->tm_sec);
639 +       } else {
640 +               strcpy(dest, "                    ");
641 +       }
642 +}      
643  
644  static void list_file(const char *fname)
645  {
646         STRUCT_STAT buf;
647         char permbuf[PERMSTRING_SIZE];
648 -       struct tm *mt;
649 -       char datebuf[50];
650 +       char mtimebuf[50];
651 +       char atimebuf[50];
652         char linkbuf[4096];
653  
654         if (do_lstat(fname, &buf) < 0)
655 @@ -96,19 +113,8 @@ static void list_file(const char *fname)
656  
657         permstring(permbuf, buf.st_mode);
658  
659 -       if (buf.st_mtime) {
660 -               mt = gmtime(&buf.st_mtime);
661 -
662 -               sprintf(datebuf, "%04d-%02d-%02d %02d:%02d:%02d",
663 -                       (int)mt->tm_year + 1900,
664 -                       (int)mt->tm_mon + 1,
665 -                       (int)mt->tm_mday,
666 -                       (int)mt->tm_hour,
667 -                       (int)mt->tm_min,
668 -                       (int)mt->tm_sec);
669 -       } else {
670 -               strcpy(datebuf, "                   ");
671 -       }
672 +       storetime(mtimebuf, buf.st_mtime);
673 +       storetime(atimebuf, buf.st_atime);
674  
675         /* TODO: Perhaps escape special characters in fname? */
676  
677 @@ -119,24 +125,55 @@ static void list_file(const char *fname)
678                     (long)minor(buf.st_rdev));
679         } else /* NB: use double for size since it might not fit in a long. */
680                 printf("%12.0f", (double)buf.st_size);
681 -       printf(" %6ld.%-6ld %6ld %s %s%s\n",
682 +       printf(" %6ld.%-6ld %6ld %s%s%s%s\n",
683                (long)buf.st_uid, (long)buf.st_gid, (long)buf.st_nlink,
684 -              datebuf, fname, linkbuf);
685 +              mtimebuf, display_atime && !S_ISDIR(buf.st_mode) ? atimebuf : "",
686 +              fname, linkbuf);
687  }
688  
689 +static struct poptOption long_options[] = {
690 +  /* longName, shortName, argInfo, argPtr, value, descrip, argDesc */
691 +  {"atime",           'u', POPT_ARG_NONE,   &display_atime, 0,   0, 0},
692 +  {"help",            'h', POPT_ARG_NONE,   0,              'h', 0, 0},
693 +  {0,0,0,0,0,0,0}
694 +};
695 +
696 +static void tls_usage(int ret)
697 +{
698 +       fprintf(stderr, "usage: " PROGRAM " [--atime | -u] DIR ...\n"
699 +           "Trivial file listing program for portably checking rsync\n");
700 +       exit(ret);
701 +}
702  
703  int
704  main(int argc, char *argv[])
705  {
706 -       if (argc < 2) {
707 -               fprintf(stderr, "usage: " PROGRAM " DIR ...\n"
708 -                       "Trivial file listing program for portably checking rsync\n");
709 -               return 1;
710 +       poptContext pc;
711 +       const char **extra_args;
712 +       int opt;
713 +
714 +       pc = poptGetContext(PROGRAM, argc, (const char **)argv,
715 +                           long_options, 0);
716 +       while ((opt = poptGetNextOpt(pc)) != -1) {
717 +               switch (opt) {
718 +               case 'h':
719 +                       tls_usage(0);
720 +               default:
721 +                       fprintf(stderr,
722 +                               "%s: %s\n",
723 +                               poptBadOption(pc, POPT_BADOPTION_NOALIAS),
724 +                               poptStrerror(opt));
725 +                       tls_usage(1);
726 +               }
727         }
728  
729 -       for (argv++; *argv; argv++) {
730 -               list_file(*argv);
731 -       }
732 +       extra_args = poptGetArgs(pc);
733 +       if (*extra_args == NULL)
734 +               tls_usage(1);
735 +
736 +       for (; *extra_args; extra_args++)
737 +               list_file(*extra_args);
738 +       poptFreeContext(pc);
739  
740         return 0;
741  }
742 --- orig/util.c 2005-10-16 22:38:40
743 +++ util.c      2005-07-27 23:37:27
744 @@ -129,7 +129,7 @@ void overflow_exit(char *str)
745  
746  
747  
748 -int set_modtime(char *fname, time_t modtime, mode_t mode)
749 +int set_times(char *fname, time_t modtime, time_t atime, mode_t mode)
750  {
751  #if !defined HAVE_LUTIMES || !defined HAVE_UTIMES
752         if (S_ISLNK(mode))
753 @@ -137,9 +137,13 @@ int set_modtime(char *fname, time_t modt
754  #endif
755  
756         if (verbose > 2) {
757 -               rprintf(FINFO, "set modtime of %s to (%ld) %s",
758 +               char mtimebuf[200];
759 +
760 +               strlcpy(mtimebuf, timestring(modtime), sizeof mtimebuf);
761 +               rprintf(FINFO,
762 +                       "set modtime, atime of %s to (%ld) %s, (%ld) %s\n",
763                         safe_fname(fname), (long)modtime,
764 -                       asctime(localtime(&modtime)));
765 +                       mtimebuf, (long)atime, timestring(atime));
766         }
767  
768         if (dry_run)
769 @@ -148,7 +152,7 @@ int set_modtime(char *fname, time_t modt
770         {
771  #ifdef HAVE_UTIMES
772                 struct timeval t[2];
773 -               t[0].tv_sec = time(NULL);
774 +               t[0].tv_sec = atime;
775                 t[0].tv_usec = 0;
776                 t[1].tv_sec = modtime;
777                 t[1].tv_usec = 0;
778 @@ -159,12 +163,12 @@ int set_modtime(char *fname, time_t modt
779                 return utimes(fname, t);
780  #elif defined HAVE_UTIMBUF
781                 struct utimbuf tbuf;
782 -               tbuf.actime = time(NULL);
783 +               tbuf.actime = atime;
784                 tbuf.modtime = modtime;
785                 return utime(fname,&tbuf);
786  #elif defined HAVE_UTIME
787                 time_t t[2];
788 -               t[0] = time(NULL);
789 +               t[0] = atime;
790                 t[1] = modtime;
791                 return utime(fname,t);
792  #else
793 @@ -1171,8 +1175,8 @@ int msleep(int t)
794  
795  
796  /**
797 - * Determine if two file modification times are equivalent (either
798 - * exact or in the modification timestamp window established by
799 + * Determine if two file  times are equivalent (either
800 + * exact or in the timestamp window established by
801   * --modify-window).
802   *
803   * @retval 0 if the times should be treated as the same
804 @@ -1181,7 +1185,7 @@ int msleep(int t)
805   *
806   * @retval -1 if the 2nd is later
807   **/
808 -int cmp_modtime(time_t file1, time_t file2)
809 +int cmp_time(time_t file1, time_t file2)
810  {
811         if (file2 > file1) {
812                 if (file2 - file1 <= modify_window)