From mboxrd@z Thu Jan 1 00:00:00 1970 X-GM-THRID: 6498671579909586944 X-Received: by 10.28.12.206 with SMTP id 197mr242248wmm.10.1513090913879; Tue, 12 Dec 2017 07:01:53 -0800 (PST) X-BeenThere: isar-users@googlegroups.com Received: by 10.223.193.73 with SMTP id w9ls2462504wre.7.gmail; Tue, 12 Dec 2017 07:01:53 -0800 (PST) X-Google-Smtp-Source: ACJfBou/mMlAmXnHVE+cO+6FKFVk4cTfVniSkr66vvO68hBF4bas7ApT3qXYFh4NKf1dzwEzdHdT X-Received: by 10.28.189.85 with SMTP id n82mr219905wmf.23.1513090913536; Tue, 12 Dec 2017 07:01:53 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1513090913; cv=none; d=google.com; s=arc-20160816; b=Plbg1YJ9RVmiN//dQS0oFD1IJhYMWRTlvu6NUPlQzW44y7GyRoNQ05P0WJL58zY6mF 3MsW80DaYJoTposmjvKtRQPOUSHipoVcsi2IAXca541sbTb+btykHqAHL3D9FZ536+Sr ZQ/QbCjmoMVLgrih9yTmVD89Kf+zDViEhyAzpRNIbOyzJctsmYiWC7x2ovigj36WmSc5 SFubat3rjMqSmLAaJwXCcCMGjOzvBQ1aghXqlmBdDMjj92UPSgLEta1///wNkXISYq7s 7hbptIxY4fdAVzBS8jab2myBbhjytdJlfvfncdSw5sS1MC29ZJi064FOa5A5pZuU0Ns9 goJA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:subject:cc:to:from:date:arc-authentication-results; bh=Y9m1eqhIT/8hbdXaKGAM6+0XfwQB6mR1lq6UUxnEUXg=; b=u9Ekuk0lWTARhpt0jfFiAitg0je1ckY3Y18UajroMtBhQ4rOuFLUisSoOxKhvUdCJe znLaVCiBlOfF0Q+B2JLZ1Wv0NhYtCMgFom7MclBA4Gu0SeuvH5nTTvXV9npf3P+mqSAd QxBBshWulcfl1cjyKUl22J65ChG47m5T+CR2Wb3v+BAe1ZAa2NciZQd7O/JiZgBjeZCa XsmzwofTr7AR/rRIisOVpQGADmiYjzn7KpzPQufDSiox6KexJ18JwYbdMnDSogl2p2HB 80ilIhd3+hJsBIV/MEBndzyOPFdUGtaQ/MWj+/DxwyTUqNWy4RO8nEcD0e8uZ0YUNj6d MjYA== ARC-Authentication-Results: i=1; gmr-mx.google.com; spf=pass (google.com: domain of henning.schild@siemens.com designates 192.35.17.2 as permitted sender) smtp.mailfrom=henning.schild@siemens.com Return-Path: Received: from thoth.sbs.de (thoth.sbs.de. [192.35.17.2]) by gmr-mx.google.com with ESMTPS id r70si1076892wmg.2.2017.12.12.07.01.53 for (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Tue, 12 Dec 2017 07:01:53 -0800 (PST) Received-SPF: pass (google.com: domain of henning.schild@siemens.com designates 192.35.17.2 as permitted sender) client-ip=192.35.17.2; Authentication-Results: gmr-mx.google.com; spf=pass (google.com: domain of henning.schild@siemens.com designates 192.35.17.2 as permitted sender) smtp.mailfrom=henning.schild@siemens.com Received: from mail1.siemens.de (mail1.siemens.de [139.23.33.14]) by thoth.sbs.de (8.15.2/8.15.2) with ESMTPS id vBCF1rKL031455 (version=TLSv1.2 cipher=DHE-RSA-AES256-GCM-SHA384 bits=256 verify=OK); Tue, 12 Dec 2017 16:01:53 +0100 Received: from mmd1pvb1c.ad001.siemens.net (md1pvb1c.ad001.siemens.net [139.25.68.40] (may be forged)) by mail1.siemens.de (8.15.2/8.15.2) with ESMTP id vBCF1qkZ013069; Tue, 12 Dec 2017 16:01:52 +0100 Date: Tue, 12 Dec 2017 16:01:52 +0100 From: Henning Schild To: Cc: "Christian Storm" , Alexander Smirnov Subject: Re: [PATCH] fetch/git: add support for disabling shared clones on unpack Message-ID: <20171212160152.7a4af55e@mmd1pvb1c.ad001.siemens.net> In-Reply-To: <20171212144443.30618-1-henning.schild@siemens.com> References: <20171212144443.30618-1-henning.schild@siemens.com> X-Mailer: Claws Mail 3.15.0-dirty (GTK+ 2.24.31; x86_64-pc-linux-gnu) MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit X-TUID: gT0g6ia61wPl Let us see what upsteam thinks about that patch. If they accept it we could probably cherry-pick the commit down into our copy of bitbake. Henning Am Tue, 12 Dec 2017 15:44:43 +0100 schrieb Henning Schild : > By default the unpacker will create a "shared" clone when cloning from > the DL_DIR to the WORKDIR. This patch introduces an option to control > that behaviour. Probably something that hardly anyone would want to > do. > > Imagine some recipe steps are executed in a namespace that is > different from the one your downloader and unpacker ran in. (chroot) > Because a "shared" clone has an absolute reference to its "alternate" > you now have to make that "alternate" visible in that new namespace > (chroot) at the exact place. > > With this patch you can unpack "noshared" and get a stand-alone copy. > This copy will also work if the "alternate" is not visible or > existant. > > Signed-off-by: Henning Schild > --- > lib/bb/fetch2/git.py | 13 ++++++++++++- > 1 file changed, 12 insertions(+), 1 deletion(-) > > diff --git a/lib/bb/fetch2/git.py b/lib/bb/fetch2/git.py > index 5ef8cd69..30b88c83 100644 > --- a/lib/bb/fetch2/git.py > +++ b/lib/bb/fetch2/git.py > @@ -53,6 +53,13 @@ Supported SRC_URI options are: > For local git:// urls to use the current branch HEAD as the > revision for use with AUTOREV. Implies nobranch. > > +- noshared > + When unpacking do not clone with the parameter "--shared". This > option will > + allow the unpacked copy to work stand-alone i.e. if your recipe > runs in a > + chroot where the "alternate" can not be found. Setting this will > increase > + the unpack-time and the disk-usage. > + The default is "0", set noshared=1 if needed. > + > """ > > #Copyright (C) 2005 Richard Purdie > @@ -159,6 +166,8 @@ class Git(FetchMethod): > > ud.nobranch = ud.parm.get("nobranch","0") == "1" > > + ud.noshared = ud.parm.get("noshared","0") == "1" > + > # usehead implies nobranch > ud.usehead = ud.parm.get("usehead","0") == "1" > if ud.usehead: > @@ -176,7 +185,9 @@ class Git(FetchMethod): > if len(branches) != len(ud.names): > raise bb.fetch2.ParameterError("The number of name and > branch parameters is not balanced", ud.url) > - ud.cloneflags = "-s -n" > + cloneflags = "-n" > + if not ud.noshared: > + cloneflags += " -s" > if ud.bareclone: > ud.cloneflags += " --mirror" >