[PATCH] subrepo: share instead of clone if the parent repo is shared (issue5675) (BC)
Matt Harbison
mharbison72 at gmail.com
Mon Oct 16 13:21:24 UTC 2017
# HG changeset patch
# User Matt Harbison <matt_harbison at yahoo.com>
# Date 1508122082 14400
# Sun Oct 15 22:48:02 2017 -0400
# Node ID d0c2b68fedb27184337af6392ecc1f03dab39522
# Parent 718adb1bf3a9a1ee509a803c7512c14296a1db79
subrepo: share instead of clone if the parent repo is shared (issue5675) (BC)
Previously, only the top level repo was shared, and then any subrepos were
cloned on demand. This is problematic because commits to the parent repo would
write an updated .hgsubstate to the share source, but the corresponding subrepo
commit would be stuck in the local subrepo. That would prevent an update in the
source repo. We already go to great lengths to avoid having inconsistent repos
(e.g., `hg push -r rev` will push _everything_ in a subrepo, even if it isn't
referenced in one of the parent's outgoing commits). Therefore, this seems like
a bug fix, and there's no option to get the old behavior. I can't imagine the
previous behavior was useful to anybody.
The corresponding change to unshare is in progress, but I didn't want to lose a
day in the review process this close to the freeze.
There shouldn't be an issue with svn, since it is centralized. Maybe --git-dir
can be used for git subrepos, but I'll leave that to someone more familiar with
git.
An integer was previously being implicitly returned from commands.share(), which
caused dispatch() to start crashing when changing over to returning the shared
repo. All error paths appear to raise, so this can be hardcoded to success.
The clone command checks for 'is None' in a similar pattern, but since
hg.clone() always returns a tuple, that seems wrong?
diff --git a/hgext/share.py b/hgext/share.py
--- a/hgext/share.py
+++ b/hgext/share.py
@@ -100,8 +100,9 @@
the broken clone to reset it to a changeset that still exists.
"""
- return hg.share(ui, source, dest=dest, update=not noupdate,
- bookmarks=bookmarks, relative=relative)
+ hg.share(ui, source, dest=dest, update=not noupdate,
+ bookmarks=bookmarks, relative=relative)
+ return 0
@command('unshare', [], '')
def unshare(ui, repo):
diff --git a/mercurial/hg.py b/mercurial/hg.py
--- a/mercurial/hg.py
+++ b/mercurial/hg.py
@@ -255,6 +255,7 @@
r = repository(ui, destwvfs.base)
postshare(srcrepo, r, bookmarks=bookmarks, defaultpath=defaultpath)
_postshareupdate(r, update, checkout=checkout)
+ return r
def postshare(sourcerepo, destrepo, bookmarks=True, defaultpath=None):
"""Called after a new shared repo is created.
diff --git a/mercurial/subrepo.py b/mercurial/subrepo.py
--- a/mercurial/subrepo.py
+++ b/mercurial/subrepo.py
@@ -857,28 +857,40 @@
def _get(self, state):
source, revision, kind = state
+ parentrepo = self._repo._subparent
+
if revision in self._repo.unfiltered():
- return True
+ # Allow shared subrepos tracked at null to setup the sharedpath
+ if revision != node.nullhex or not parentrepo.shared():
+ return True
self._repo._subsource = source
srcurl = _abssource(self._repo)
other = hg.peer(self._repo, {}, srcurl)
if len(self._repo) == 0:
- self.ui.status(_('cloning subrepo %s from %s\n')
- % (subrelpath(self), srcurl))
- parentrepo = self._repo._subparent
# use self._repo.vfs instead of self.wvfs to remove .hg only
self._repo.vfs.rmtree()
- other, cloned = hg.clone(self._repo._subparent.baseui, {},
- other, self._repo.root,
- update=False)
- self._repo = cloned.local()
+ if parentrepo.shared():
+ self.ui.status(_('sharing subrepo %s from %s\n')
+ % (subrelpath(self), srcurl))
+ shared = hg.share(self._repo._subparent.baseui,
+ other, self._repo.root,
+ update=False, bookmarks=False)
+ self._repo = shared.local()
+ else:
+ self.ui.status(_('cloning subrepo %s from %s\n')
+ % (subrelpath(self), srcurl))
+ other, cloned = hg.clone(self._repo._subparent.baseui, {},
+ other, self._repo.root,
+ update=False)
+ self._repo = cloned.local()
self._initrepo(parentrepo, source, create=True)
self._cachestorehash(srcurl)
else:
- self.ui.status(_('pulling subrepo %s from %s\n')
- % (subrelpath(self), srcurl))
cleansub = self.storeclean(srcurl)
- exchange.pull(self._repo, other)
+ if not parentrepo.shared():
+ self.ui.status(_('pulling subrepo %s from %s\n')
+ % (subrelpath(self), srcurl))
+ exchange.pull(self._repo, other)
if cleansub:
# keep the repo clean after pull
self._cachestorehash(srcurl)
diff --git a/tests/test-archive.t b/tests/test-archive.t
--- a/tests/test-archive.t
+++ b/tests/test-archive.t
@@ -18,6 +18,57 @@
$ echo "subrepo = subrepo" > .hgsub
$ hg add .hgsub
$ hg ci -m "add subrepo"
+
+ $ cat >> $HGRCPATH <<EOF
+ > [extensions]
+ > share =
+ > EOF
+
+hg subrepos are shared when the parent repo is
+
+ $ cd ..
+ $ hg share test shared1
+ updating working directory
+ sharing subrepo subrepo from $TESTTMP/test/subrepo
+ 5 files updated, 0 files merged, 0 files removed, 0 files unresolved
+ $ cat shared1/subrepo/.hg/sharedpath
+ $TESTTMP/test/subrepo/.hg (no-eol) (glob)
+
+hg subrepos are shared into existence on demand if the parent was shared
+
+ $ hg clone -qr 1 test clone1
+ $ hg share clone1 share2
+ updating working directory
+ 2 files updated, 0 files merged, 0 files removed, 0 files unresolved
+ $ hg -R clone1 -q pull
+ $ hg -R share2 update tip
+ sharing subrepo subrepo from $TESTTMP/test/subrepo
+ 3 files updated, 0 files merged, 0 files removed, 0 files unresolved
+ $ cat share2/subrepo/.hg/sharedpath
+ $TESTTMP/test/subrepo/.hg (no-eol) (glob)
+ $ echo 'mod' > share2/subrepo/sub
+ $ hg -R share2 ci -Sqm 'subrepo mod'
+ $ hg -R clone1 update -C tip
+ cloning subrepo subrepo from $TESTTMP/test/subrepo
+ 3 files updated, 0 files merged, 0 files removed, 0 files unresolved
+ $ rm -rf clone1
+
+ $ hg clone -qr 1 test clone1
+ $ hg share clone1 shared3
+ updating working directory
+ 2 files updated, 0 files merged, 0 files removed, 0 files unresolved
+ $ hg -R clone1 -q pull
+ $ hg -R shared3 archive --config ui.archivemeta=False -r tip -S archive
+ sharing subrepo subrepo from $TESTTMP/test/subrepo
+ $ cat shared3/subrepo/.hg/sharedpath
+ $TESTTMP/test/subrepo/.hg (no-eol) (glob)
+ $ diff -r archive test
+ Only in test: .hg
+ Only in test/subrepo: .hg
+ [1]
+ $ rm -rf archive
+
+ $ cd test
$ echo "[web]" >> .hg/hgrc
$ echo "name = test-archive" >> .hg/hgrc
$ echo "archivesubrepos = True" >> .hg/hgrc
diff --git a/tests/test-subrepo.t b/tests/test-subrepo.t
--- a/tests/test-subrepo.t
+++ b/tests/test-subrepo.t
@@ -1181,10 +1181,32 @@
Check that share works with subrepo
$ hg --config extensions.share= share . ../shared
updating working directory
- cloning subrepo subrepo-2 from $TESTTMP/subrepo-status/subrepo-2
+ sharing subrepo subrepo-1 from $TESTTMP/subrepo-status/subrepo-1
+ sharing subrepo subrepo-2 from $TESTTMP/subrepo-status/subrepo-2
2 files updated, 0 files merged, 0 files removed, 0 files unresolved
- $ test -f ../shared/subrepo-1/.hg/sharedpath
- [1]
+ $ find ../shared/* | sort
+ ../shared/subrepo-1
+ ../shared/subrepo-1/.hg
+ ../shared/subrepo-1/.hg/cache
+ ../shared/subrepo-1/.hg/cache/storehash
+ ../shared/subrepo-1/.hg/cache/storehash/* (glob)
+ ../shared/subrepo-1/.hg/hgrc
+ ../shared/subrepo-1/.hg/requires
+ ../shared/subrepo-1/.hg/sharedpath
+ ../shared/subrepo-2
+ ../shared/subrepo-2/.hg
+ ../shared/subrepo-2/.hg/branch
+ ../shared/subrepo-2/.hg/cache
+ ../shared/subrepo-2/.hg/cache/checkisexec (execbit !)
+ ../shared/subrepo-2/.hg/cache/checklink (symlink !)
+ ../shared/subrepo-2/.hg/cache/checklink-target (symlink !)
+ ../shared/subrepo-2/.hg/cache/storehash
+ ../shared/subrepo-2/.hg/cache/storehash/* (glob)
+ ../shared/subrepo-2/.hg/dirstate
+ ../shared/subrepo-2/.hg/hgrc
+ ../shared/subrepo-2/.hg/requires
+ ../shared/subrepo-2/.hg/sharedpath
+ ../shared/subrepo-2/file
$ hg -R ../shared in
abort: repository default not found!
[255]
More information about the Mercurial-devel
mailing list