Re: cvs commit: src/bin/cpdup cpdup.c cpdup.h hclink.c

[Matthew Dillon <dillon@xxxxxxxxxxxxxxxxxxxx>]

:While you're working on cpdup ...  There's one thing which
:I've always missed in cpdup:  The mtimes of directories are
:not correctly duplicated.  Instead, the mtime of all
:directories is set to the current time.  I guess that's
:because cpdup updates the contents of directories and
:doesn't care to fix the directory's mtime afterwards.
:
:Is there a chance it could be fixed?  I don't think that
:it's difficult, but I must confess that I haven't looked
:at the code, actually.
:
:Currently I use the following workaround:  After each time
:cpdup is used, I run the following command (via a shell
:script wrapper):
:
:cd $SRC; find -d . -type d | cpio -dump $DEST
:
:It fixes the mtimes of all directories.  Most of the time,
:the meta data of the directory tree is still cached from
:the cpdup run, so the performance isn't extremely bad.
:It's still an ugly hack, though, and I would prefer it
:to be fixed in cpdup itself.
:
:(If you agree that it's easy to fix, I could even have a
:look at the code myself.)
:
:Best regards
:   Oliver

     Yup, that's an easy fix, I'll commit it now.

					-Matt