On Mon, Aug 14, 2006 at 03:52:52PM +0300, Dmitry Pryanishnikov wrote: > > Hello! > > On Mon, 14 Aug 2006, Andrey Chernov wrote: > >> - Touch /usr/share/tmac/man.local if it already exists. > > > >It is definitely bad habbit touching the files to marking them not > >obsoleted. Many developers including me use 'INSTALL = install -C' which > >not touch files when not needed, so we have correct date of the actual > >last file change. With that your touching those files will falsely looks > >updated. Please back it out. > > Note that this commit does _not_ touch the file. As I've shown in my > previous letter, ':>>$file' will _not_ change file's modification time. > Maybe, we should avoid spelling ':>>' as a 'touch' in the commit log > to prevent such a confusion? BTW, knowing that ':>>$file' doesn't change > modification time, I'm really curiuos what this ".else / :>> / .endif" > really > does? Isn't this a plain NOOP?! It just opens file for write. Assuming /tmp/1 exists, watch yourself with truss sh -c ':>>/tmp/1'. On the other hand, touch uses utimes(2) to change file access and modification times.
pgpBYsT3MRFMV.pgp
Description: PGP signature
