From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <gentoo-dev+bounces-59237-garchives=archives.gentoo.org@lists.gentoo.org>
Received: from lists.gentoo.org (pigeon.gentoo.org [208.92.234.80])
	by finch.gentoo.org (Postfix) with ESMTP id 33EC5138010
	for <garchives@archives.gentoo.org>; Sun, 24 Mar 2013 07:47:52 +0000 (UTC)
Received: from pigeon.gentoo.org (localhost [127.0.0.1])
	by pigeon.gentoo.org (Postfix) with SMTP id 3DB51E0774;
	Sun, 24 Mar 2013 07:47:49 +0000 (UTC)
Received: from a1www.kph.uni-mainz.de (a1iwww1.kph.uni-mainz.de [134.93.134.1])
	(using TLSv1 with cipher ECDHE-RSA-AES256-SHA (256/256 bits))
	(No client certificate requested)
	by pigeon.gentoo.org (Postfix) with ESMTPS id 31576E076B
	for <gentoo-dev@lists.gentoo.org>; Sun, 24 Mar 2013 07:47:47 +0000 (UTC)
Received: from a1i15.kph.uni-mainz.de (a1i15.kph.uni-mainz.de [134.93.134.92])
	by a1www.kph.uni-mainz.de (8.14.4/8.13.4) with ESMTP id r2O7lgxW018224;
	Sun, 24 Mar 2013 08:47:42 +0100
Received: from a1i15.kph.uni-mainz.de (localhost [127.0.0.1])
	by a1i15.kph.uni-mainz.de (8.14.6/8.14.2) with ESMTP id r2O7lfN3002095;
	Sun, 24 Mar 2013 08:47:41 +0100
Received: (from ulm@localhost)
	by a1i15.kph.uni-mainz.de (8.14.6/8.14.6/Submit) id r2O7lf8U002093;
	Sun, 24 Mar 2013 08:47:41 +0100
Precedence: bulk
List-Post: <mailto:gentoo-dev@lists.gentoo.org>
List-Help: <mailto:gentoo-dev+help@lists.gentoo.org>
List-Unsubscribe: <mailto:gentoo-dev+unsubscribe@lists.gentoo.org>
List-Subscribe: <mailto:gentoo-dev+subscribe@lists.gentoo.org>
List-Id: Gentoo Linux mail <gentoo-dev.gentoo.org>
X-BeenThere: gentoo-dev@lists.gentoo.org
Reply-to: gentoo-dev@lists.gentoo.org
MIME-Version: 1.0
Content-Type: text/plain; charset=iso-8859-2
Content-Transfer-Encoding: 8bit
Message-ID: <20814.44957.706923.881677@a1i15.kph.uni-mainz.de>
Date: Sun, 24 Mar 2013 08:47:41 +0100
To: =?iso-8859-2?Q?Micha=B3_G=F3rny?= <mgorny@gentoo.org>
Cc: gentoo-dev@lists.gentoo.org
Subject: Re: [gentoo-dev] [PATCH 1/2] Introduce multibuild_merge_root() to
 merge interim installs.
In-Reply-To: <20130323182858.1846ba8d@pomiocik.lan>
References: <20130323172532.1b1100e2@pomiocik.lan>
	<1364055998-23674-1-git-send-email-mgorny@gentoo.org>
	<20813.57273.945749.854552@a1i15.kph.uni-mainz.de>
	<20130323182858.1846ba8d@pomiocik.lan>
X-Mailer: VM 8.2.0b under 24.3.1 (x86_64-pc-linux-gnu)
From: Ulrich Mueller <ulm@gentoo.org>
X-Archives-Salt: 14c299f9-eea5-4a47-8c23-610c3e11addf
X-Archives-Hash: ce90dccd4a4a4f39d92936fae919e9fc

>>>>> On Sat, 23 Mar 2013, Michał Górny wrote:

>> > +multibuild_merge_root() {
>> > + [...]
>> > +	if type -P lockf &>/dev/null; then
>> > +		# On BSD, we have 'lockf' wrapper.
>> > +		tar -C "${src}" -f - -c . \
>> > +			| lockf "${lockfile}" tar -x -f - -C "${dest}"
>> > +	else
>> > + [...]
>> > +		cp -a -l -n "${src}"/. "${dest}"/
>> 
>> You assume that cp is GNU, if there's no lockf. Is it guaranteed that
>> this is true on all platforms?

> They're all just assumptions. If it breaks somewhere, I'll improve
> the function.

Maybe explicitly testing for USERLAND would be less fragile? Or use
tar everywhere?

Ulrich