From owner-cvs-all@FreeBSD.ORG  Mon Aug 14 12:53:04 2006
Return-Path: <owner-cvs-all@FreeBSD.ORG>
X-Original-To: cvs-all@freebsd.org
Delivered-To: cvs-all@freebsd.org
Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125])
	by hub.freebsd.org (Postfix) with ESMTP id AAEB616A4DD;
	Mon, 14 Aug 2006 12:53:04 +0000 (UTC)
	(envelope-from dmitry@atlantis.dp.ua)
Received: from postman.atlantis.dp.ua (postman.atlantis.dp.ua [193.108.47.1])
	by mx1.FreeBSD.org (Postfix) with ESMTP id F325043D68;
	Mon, 14 Aug 2006 12:53:03 +0000 (GMT)
	(envelope-from dmitry@atlantis.dp.ua)
Received: from smtp.atlantis.dp.ua (smtp.atlantis.dp.ua [193.108.46.231])
	by postman.atlantis.dp.ua (8.13.1/8.13.1) with ESMTP id k7ECqqpm062250; 
	Mon, 14 Aug 2006 15:52:52 +0300 (EEST)
	(envelope-from dmitry@atlantis.dp.ua)
Date: Mon, 14 Aug 2006 15:52:52 +0300 (EEST)
From: Dmitry Pryanishnikov <dmitry@atlantis.dp.ua>
To: Andrey Chernov <ache@freebsd.org>
In-Reply-To: <20060814124528.GB28932@nagual.pp.ru>
Message-ID: <20060814154640.R28464@atlantis.atlantis.dp.ua>
References: <200608141221.k7ECLwHq069066@repoman.freebsd.org>
	<20060814124528.GB28932@nagual.pp.ru>
MIME-Version: 1.0
Content-Type: TEXT/PLAIN; charset=US-ASCII; format=flowed
Cc: cvs-src@freebsd.org, src-committers@freebsd.org,
	Ruslan Ermilov <ru@freebsd.org>, cvs-all@freebsd.org
Subject: Re: cvs commit: src/gnu/usr.bin/groff/tmac Makefile
X-BeenThere: cvs-all@freebsd.org
X-Mailman-Version: 2.1.5
Precedence: list
List-Id: CVS commit messages for the entire tree <cvs-all.freebsd.org>
List-Unsubscribe: <http://lists.freebsd.org/mailman/listinfo/cvs-all>,
	<mailto:cvs-all-request@freebsd.org?subject=unsubscribe>
List-Archive: <http://lists.freebsd.org/pipermail/cvs-all>
List-Post: <mailto:cvs-all@freebsd.org>
List-Help: <mailto:cvs-all-request@freebsd.org?subject=help>
List-Subscribe: <http://lists.freebsd.org/mailman/listinfo/cvs-all>,
	<mailto:cvs-all-request@freebsd.org?subject=subscribe>
X-List-Received-Date: Mon, 14 Aug 2006 12:53:04 -0000


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?!

Sincerely, Dmitry
-- 
Atlantis ISP, System Administrator
e-mail:  dmitry@atlantis.dp.ua
nic-hdl: LYNX-RIPE