[bitbake-devel] [PATCH 4/4] fetch2/git.py: Optimize clone fall back when it is local
Richard Purdie
richard.purdie at linuxfoundation.org
Thu Jun 14 14:26:33 UTC 2012
On Thu, 2012-06-14 at 08:33 -0500, Jason Wessel wrote:
> On 06/14/2012 08:03 AM, Richard Purdie wrote:
> > On Wed, 2012-06-13 at 22:09 -0500, Jason Wessel wrote:
> >> A file:// url should use "clone -s" to greatly speed
> >> up the clone in the case of a kernel when it is local.
> >>
> >> Signed-off-by: Jason Wessel <jason.wessel at windriver.com>
> >> ---
> >> lib/bb/fetch2/git.py | 8 +++++++-
> >> 1 file changed, 7 insertions(+), 1 deletion(-)
> >>
> >> diff --git a/lib/bb/fetch2/git.py b/lib/bb/fetch2/git.py
> >> index 1ad9213..f5a3983 100644
> >> --- a/lib/bb/fetch2/git.py
> >> +++ b/lib/bb/fetch2/git.py
> >> @@ -192,7 +192,13 @@ class Git(FetchMethod):
> >>
> >> # If the repo still doesn't exist, fallback to cloning it
> >> if not os.path.exists(ud.clonedir):
> >> - clone_cmd = "%s clone --bare --mirror %s %s" % (ud.basecmd, repourl, ud.clonedir)
> >> + if repourl.startswith("file://"):
> >> + use_s = "-s"
> >> + repo = repourl[7:]
> >> + else:
> >> + use_s = ""
> >> + repo = repourl
> >> + clone_cmd = "%s clone %s --bare --mirror %s %s" % (ud.basecmd, use_s, repo, ud.clonedir)
> >> if ud.proto.lower() != 'file':
> >> bb.fetch2.check_network_access(d, clone_cmd)
> >> runfetchcmd(clone_cmd, d)
> >
I think you can then simplify this to:
@@ -192,7 +192,9 @@ class Git(FetchMethod):
# If the repo still doesn't exist, fallback to cloning it
if not os.path.exists(ud.clonedir):
- clone_cmd = "%s clone --bare --mirror %s %s" % (ud.basecmd, repourl, ud.clonedir)
+ if repourl.startswith("file://"):
+ repo = repourl[7:]
+ clone_cmd = "%s clone -l --bare --mirror %s %s" % (ud.basecmd, repo, ud.clonedir)
if ud.proto.lower() != 'file':
bb.fetch2.check_network_access(d, clone_cmd)
runfetchcmd(clone_cmd, d)
Cheers,
Richard
More information about the bitbake-devel
mailing list