cvs commit: src/gnu/usr.bin/groff/tmac Makefile

[Kostik Belousov]

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.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 187 bytes
Desc: not available
Url : http://lists.freebsd.org/pipermail/cvs-src/attachments/20060814/da63ddeb/attachment.pgp