From owner-cvs-all  Tue Nov 19  8:21:53 2002
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 ABECE37B401; Tue, 19 Nov 2002 08:21:52 -0800 (PST)
Received: from corbulon.video-collage.com (corbulon.video-collage.com [64.35.99.179])
	by mx1.FreeBSD.org (Postfix) with ESMTP
	id 70DEF43E4A; Tue, 19 Nov 2002 08:21:48 -0800 (PST)
	(envelope-from mi+mx@aldan.algebra.com)
Received: from misha.murex.com (250-217.customer.cloud9.net [168.100.250.217])
	by corbulon.video-collage.com (8.12.6/8.12.6) with ESMTP id gAJGLQjJ032607
	(version=TLSv1/SSLv3 cipher=RC4-MD5 bits=128 verify=FAIL);
	Tue, 19 Nov 2002 11:21:31 -0500 (EST)
	(envelope-from mi+mx@aldan.algebra.com)
Content-Type: text/plain;
  charset="iso-8859-1"
From: Mikhail Teterin <mi+mx@aldan.algebra.com>
Organization: Virtual Estates, Inc.
To: Will Andrews <will@csociety.org>, Doug Barton <DougB@FreeBSD.org>
Subject: REINPLACE_CMD and the useless .bak files (Re: cvs commit: ports/net/firedns/files patch-firemake.libs)
Date: Tue, 19 Nov 2002 11:24:20 -0500
User-Agent: KMail/1.4.3
Cc: Edwin Groothuis <edwin@FreeBSD.org>, cvs-committers@FreeBSD.org,
	cvs-all@FreeBSD.org
References: <200211152259.gAFMxJwv052222@repoman.freebsd.org> <3DD9EB57.33F38CDE@FreeBSD.org> <20021119130550.GK79926@procyon.firepipe.net>
In-Reply-To: <20021119130550.GK79926@procyon.firepipe.net>
MIME-Version: 1.0
Content-Transfer-Encoding: 8bit
Message-Id: <200211191124.21000.mi+mx@aldan.algebra.com>
X-Scanned-By: MIMEDefang 2.21 (www . roaringpenguin . com / mimedefang)
Sender: owner-cvs-all@FreeBSD.ORG
Precedence: bulk
List-ID: <cvs-all.FreeBSD.ORG>
List-Archive: <http://docs.freebsd.org/mail/> (Web Archive)
List-Help: <mailto:majordomo@FreeBSD.ORG?subject=help> (List Instructions)
List-Subscribe: <mailto:majordomo@FreeBSD.ORG?subject=subscribe%20cvs-all>
List-Unsubscribe: <mailto:majordomo@FreeBSD.ORG?subject=unsubscribe%20cvs-all>
X-Loop: FreeBSD.ORG

On Tuesday 19 November 2002 08:05 am, Will Andrews wrote:
= Also, REINPLACE_CMD [...] doesn't leave a temp file

That was the intention of REINPLACE_CMD, but by default it does leave
the "turds"[1] around. The only difference will be, that the "old"
configure will be named configure.bak instead of configure.Patched.

Unless the REINPLACE_ARGS is set to ``-i ""'' by the port, that is.

[1] This is the word used by at least one portmgr member and
REINPLACE_CMD (over regular sed(1)) proponent at the time of
REINPLACE_CMD introduction and debate. REINPLACE_CMD's ability NOT to
leave the .bak (or .orig) files around was one of his points -- advanced
with the unpleasantly robust language, I must add.

Now, that the -i "" works reliably, can we turn it on?

	 .if defined(USE_REINPLACE)
	-REINPLACE_ARGS?=        -i.bak
	+REINPLACE_ARGS?=        -i ""

Yours,

	-mi


To Unsubscribe: send mail to majordomo@FreeBSD.org
with "unsubscribe cvs-all" in the body of the message