D9285: strip: move into core
valentin.gatienbaron (Valentin Gatien-Baron)
phabricator at mercurial-scm.org
Sun Nov 8 23:31:58 UTC 2020
valentin.gatienbaron created this revision.
Herald added a reviewer: hg-reviewers.
Herald added a subscriber: mercurial-patches.
REVISION SUMMARY
As discussed at the 5.2 sprint, replace strip extension by a core
command, debugstrip. Obviously, the extension stays for backwards
compatibility.
As an implementation note, I moved the strip file as is into core,
which is not done elsewhere, AFAIK. I could have inlined it into
debugcommands, but that doesn't sound great.
REPOSITORY
rHG Mercurial
BRANCH
default
REVISION DETAIL
https://phab.mercurial-scm.org/D9285
AFFECTED FILES
hgext/mq.py
hgext/strip.py
mercurial/debugcommands.py
mercurial/registrar.py
mercurial/strip.py
relnotes/next
tests/test-completion.t
tests/test-help.t
tests/test-strip.t
CHANGE DETAILS
diff --git a/tests/test-strip.t b/tests/test-strip.t
--- a/tests/test-strip.t
+++ b/tests/test-strip.t
@@ -10,7 +10,7 @@
> hg up -C $1
> echo % before update $1, strip $2
> hg log -G -T '{rev}:{node}'
- > hg --traceback strip $2
+ > hg --traceback debugstrip $2
> echo % after update $1, strip $2
> hg log -G -T '{rev}:{node}'
> restore
@@ -765,13 +765,11 @@
Make sure no one adds back a -b option:
$ hg strip -b tip
- hg strip: option -b not recognized
- hg strip [-k] [-f] [-B bookmark] [-r] REV...
+ hg debugstrip: option -b not recognized
+ hg debugstrip [-k] [-f] [-B bookmark] [-r] REV...
strip changesets and all their descendants from the repository
- (use 'hg help -e strip' to show help for the strip extension)
-
options ([+] can be repeated):
-r --rev REV [+] strip specified revision (optional, can specify
@@ -783,7 +781,7 @@
-B --bookmark BOOKMARK [+] remove revs only reachable from given bookmark
--mq operate on patch repository
- (use 'hg strip -h' to show more help)
+ (use 'hg debugstrip -h' to show more help)
[255]
$ cd ..
diff --git a/tests/test-help.t b/tests/test-help.t
--- a/tests/test-help.t
+++ b/tests/test-help.t
@@ -380,7 +380,6 @@
relink recreates hardlinks between repository clones
schemes extend schemes with shortcuts to repository swarms
share share a common history between several working directories
- strip strip changesets and their descendants from history
transplant command to transplant changesets from another branch
win32mbcs allow the use of MBCS paths with problematic encodings
zeroconf discover and advertise repositories on the local network
@@ -1067,6 +1066,7 @@
debugsidedata
dump the side data for a cl/manifest/file revision
debugssl test a secure connection to a server
+ debugstrip strip changesets and all their descendants from the repository
debugsub (no help text available)
debugsuccessorssets
show set of successors for revision
diff --git a/tests/test-completion.t b/tests/test-completion.t
--- a/tests/test-completion.t
+++ b/tests/test-completion.t
@@ -132,6 +132,7 @@
debugsetparents
debugsidedata
debugssl
+ debugstrip
debugsub
debugsuccessorssets
debugtagscache
@@ -320,6 +321,7 @@
debugsetparents:
debugsidedata: changelog, manifest, dir
debugssl:
+ debugstrip: rev, force, no-backup, nobackup, , keep, bookmark, soft
debugsub: rev
debugsuccessorssets: closest
debugtagscache:
diff --git a/relnotes/next b/relnotes/next
--- a/relnotes/next
+++ b/relnotes/next
@@ -8,6 +8,9 @@
compares to the behavior of `hg purge` when the purge extension is
loaded.
+ * `hg strip`, from the strip extension, is now a core command, `hg
+ debugstrip`. The extension remains for compatibility.
+
== New Experimental Features ==
diff --git a/hgext/strip.py b/mercurial/strip.py
copy from hgext/strip.py
copy to mercurial/strip.py
--- a/hgext/strip.py
+++ b/mercurial/strip.py
@@ -1,13 +1,8 @@
-"""strip changesets and their descendants from history
-
-This extension allows you to strip changesets and all their descendants from the
-repository. See the command help for details.
-"""
from __future__ import absolute_import
-from mercurial.i18n import _
-from mercurial.pycompat import getattr
-from mercurial import (
+from .i18n import _
+from .pycompat import getattr
+from . import (
bookmarks as bookmarksmod,
cmdutil,
error,
@@ -27,11 +22,6 @@
cmdtable = {}
command = registrar.command(cmdtable)
-# Note for extension authors: ONLY specify testedwith = 'ships-with-hg-core' for
-# extensions which SHIP WITH MERCURIAL. Non-mainline extensions should
-# be specifying the version(s) of Mercurial they are tested with, or
-# leave the attribute unspecified.
-testedwith = b'ships-with-hg-core'
def checklocalchanges(repo, force=False):
@@ -101,7 +91,7 @@
@command(
- b"strip",
+ b"debugstrip",
[
(
b'r',
@@ -146,10 +136,10 @@
_(b"simply drop changesets from visible history (EXPERIMENTAL)"),
),
],
- _(b'hg strip [-k] [-f] [-B bookmark] [-r] REV...'),
+ _(b'hg debugstrip [-k] [-f] [-B bookmark] [-r] REV...'),
helpcategory=command.CATEGORY_MAINTENANCE,
)
-def stripcmd(ui, repo, *revs, **opts):
+def debugstrip(ui, repo, *revs, **opts):
"""strip changesets and all their descendants from the repository
The strip command removes the specified changesets and all their
diff --git a/mercurial/registrar.py b/mercurial/registrar.py
--- a/mercurial/registrar.py
+++ b/mercurial/registrar.py
@@ -234,6 +234,11 @@
self._table[name] = func, list(options)
return func
+ def rename(self, old, new):
+ """rename a command. Used to add aliases, debugstrip ->
+ debugstrip|strip
+ """
+ self._table[new] = self._table.pop(old)
INTENT_READONLY = b'readonly'
diff --git a/mercurial/debugcommands.py b/mercurial/debugcommands.py
--- a/mercurial/debugcommands.py
+++ b/mercurial/debugcommands.py
@@ -78,6 +78,7 @@
sshpeer,
sslutil,
streamclone,
+ strip,
tags as tagsmod,
templater,
treediscovery,
@@ -105,8 +106,9 @@
release = lockmod.release
-command = registrar.command()
-
+table = {}
+table.update(strip.command._table)
+command = registrar.command(table)
@command(b'debugancestor', [], _(b'[INDEX] REV1 REV2'), optionalrepo=True)
def debugancestor(ui, repo, *args):
diff --git a/hgext/strip.py b/hgext/strip.py
--- a/hgext/strip.py
+++ b/hgext/strip.py
@@ -1,287 +1,24 @@
-"""strip changesets and their descendants from history
+"""strip changesets and their descendants from history (DEPRECATED)
+
+The functionality of this extension has been included in core Mercurial
+since version 5.7. Please use :hg:`debugstrip ...` instead.
This extension allows you to strip changesets and all their descendants from the
repository. See the command help for details.
"""
from __future__ import absolute_import
-from mercurial.i18n import _
-from mercurial.pycompat import getattr
from mercurial import (
- bookmarks as bookmarksmod,
- cmdutil,
- error,
- hg,
- lock as lockmod,
- mergestate as mergestatemod,
- node as nodemod,
- pycompat,
- registrar,
- repair,
- scmutil,
- util,
+ commands,
)
-nullid = nodemod.nullid
-release = lockmod.release
-
-cmdtable = {}
-command = registrar.command(cmdtable)
# Note for extension authors: ONLY specify testedwith = 'ships-with-hg-core' for
# extensions which SHIP WITH MERCURIAL. Non-mainline extensions should
# be specifying the version(s) of Mercurial they are tested with, or
# leave the attribute unspecified.
testedwith = b'ships-with-hg-core'
-
-def checklocalchanges(repo, force=False):
- s = repo.status()
- if not force:
- cmdutil.checkunfinished(repo)
- cmdutil.bailifchanged(repo)
- else:
- cmdutil.checkunfinished(repo, skipmerge=True)
- return s
-
-
-def _findupdatetarget(repo, nodes):
- unode, p2 = repo.changelog.parents(nodes[0])
- currentbranch = repo[None].branch()
-
- if (
- util.safehasattr(repo, b'mq')
- and p2 != nullid
- and p2 in [x.node for x in repo.mq.applied]
- ):
- unode = p2
- elif currentbranch != repo[unode].branch():
- pwdir = b'parents(wdir())'
- revset = b'max(((parents(%ln::%r) + %r) - %ln::%r) and branch(%s))'
- branchtarget = repo.revs(
- revset, nodes, pwdir, pwdir, nodes, pwdir, currentbranch
- )
- if branchtarget:
- cl = repo.changelog
- unode = cl.node(branchtarget.first())
-
- return unode
-
-
-def strip(
- ui,
- repo,
- revs,
- update=True,
- backup=True,
- force=None,
- bookmarks=None,
- soft=False,
-):
- with repo.wlock(), repo.lock():
-
- if update:
- checklocalchanges(repo, force=force)
- urev = _findupdatetarget(repo, revs)
- hg.clean(repo, urev)
- repo.dirstate.write(repo.currenttransaction())
-
- if soft:
- repair.softstrip(ui, repo, revs, backup)
- else:
- repair.strip(ui, repo, revs, backup)
-
- repomarks = repo._bookmarks
- if bookmarks:
- with repo.transaction(b'strip') as tr:
- if repo._activebookmark in bookmarks:
- bookmarksmod.deactivate(repo)
- repomarks.applychanges(repo, tr, [(b, None) for b in bookmarks])
- for bookmark in sorted(bookmarks):
- ui.write(_(b"bookmark '%s' deleted\n") % bookmark)
-
-
- at command(
- b"strip",
- [
- (
- b'r',
- b'rev',
- [],
- _(
- b'strip specified revision (optional, '
- b'can specify revisions without this '
- b'option)'
- ),
- _(b'REV'),
- ),
- (
- b'f',
- b'force',
- None,
- _(
- b'force removal of changesets, discard '
- b'uncommitted changes (no backup)'
- ),
- ),
- (b'', b'no-backup', None, _(b'do not save backup bundle')),
- (b'', b'nobackup', None, _(b'do not save backup bundle (DEPRECATED)'),),
- (b'n', b'', None, _(b'ignored (DEPRECATED)')),
- (
- b'k',
- b'keep',
- None,
- _(b"do not modify working directory during strip"),
- ),
- (
- b'B',
- b'bookmark',
- [],
- _(b"remove revs only reachable from given bookmark"),
- _(b'BOOKMARK'),
- ),
- (
- b'',
- b'soft',
- None,
- _(b"simply drop changesets from visible history (EXPERIMENTAL)"),
- ),
- ],
- _(b'hg strip [-k] [-f] [-B bookmark] [-r] REV...'),
- helpcategory=command.CATEGORY_MAINTENANCE,
-)
-def stripcmd(ui, repo, *revs, **opts):
- """strip changesets and all their descendants from the repository
-
- The strip command removes the specified changesets and all their
- descendants. If the working directory has uncommitted changes, the
- operation is aborted unless the --force flag is supplied, in which
- case changes will be discarded.
-
- If a parent of the working directory is stripped, then the working
- directory will automatically be updated to the most recent
- available ancestor of the stripped parent after the operation
- completes.
-
- Any stripped changesets are stored in ``.hg/strip-backup`` as a
- bundle (see :hg:`help bundle` and :hg:`help unbundle`). They can
- be restored by running :hg:`unbundle .hg/strip-backup/BUNDLE`,
- where BUNDLE is the bundle file created by the strip. Note that
- the local revision numbers will in general be different after the
- restore.
-
- Use the --no-backup option to discard the backup bundle once the
- operation completes.
-
- Strip is not a history-rewriting operation and can be used on
- changesets in the public phase. But if the stripped changesets have
- been pushed to a remote repository you will likely pull them again.
-
- Return 0 on success.
- """
- opts = pycompat.byteskwargs(opts)
- backup = True
- if opts.get(b'no_backup') or opts.get(b'nobackup'):
- backup = False
-
- cl = repo.changelog
- revs = list(revs) + opts.get(b'rev')
- revs = set(scmutil.revrange(repo, revs))
-
- with repo.wlock():
- bookmarks = set(opts.get(b'bookmark'))
- if bookmarks:
- repomarks = repo._bookmarks
- if not bookmarks.issubset(repomarks):
- raise error.Abort(
- _(b"bookmark '%s' not found")
- % b','.join(sorted(bookmarks - set(repomarks.keys())))
- )
-
- # If the requested bookmark is not the only one pointing to a
- # a revision we have to only delete the bookmark and not strip
- # anything. revsets cannot detect that case.
- nodetobookmarks = {}
- for mark, node in pycompat.iteritems(repomarks):
- nodetobookmarks.setdefault(node, []).append(mark)
- for marks in nodetobookmarks.values():
- if bookmarks.issuperset(marks):
- rsrevs = scmutil.bookmarkrevs(repo, marks[0])
- revs.update(set(rsrevs))
- if not revs:
- with repo.lock(), repo.transaction(b'bookmark') as tr:
- bmchanges = [(b, None) for b in bookmarks]
- repomarks.applychanges(repo, tr, bmchanges)
- for bookmark in sorted(bookmarks):
- ui.write(_(b"bookmark '%s' deleted\n") % bookmark)
-
- if not revs:
- raise error.Abort(_(b'empty revision set'))
-
- descendants = set(cl.descendants(revs))
- strippedrevs = revs.union(descendants)
- roots = revs.difference(descendants)
-
- # if one of the wdir parent is stripped we'll need
- # to update away to an earlier revision
- update = any(
- p != nullid and cl.rev(p) in strippedrevs
- for p in repo.dirstate.parents()
- )
-
- rootnodes = {cl.node(r) for r in roots}
-
- q = getattr(repo, 'mq', None)
- if q is not None and q.applied:
- # refresh queue state if we're about to strip
- # applied patches
- if cl.rev(repo.lookup(b'qtip')) in strippedrevs:
- q.applieddirty = True
- start = 0
- end = len(q.applied)
- for i, statusentry in enumerate(q.applied):
- if statusentry.node in rootnodes:
- # if one of the stripped roots is an applied
- # patch, only part of the queue is stripped
- start = i
- break
- del q.applied[start:end]
- q.savedirty()
-
- revs = sorted(rootnodes)
- if update and opts.get(b'keep'):
- urev = _findupdatetarget(repo, revs)
- uctx = repo[urev]
-
- # only reset the dirstate for files that would actually change
- # between the working context and uctx
- descendantrevs = repo.revs(b"only(., %d)", uctx.rev())
- changedfiles = []
- for rev in descendantrevs:
- # blindly reset the files, regardless of what actually changed
- changedfiles.extend(repo[rev].files())
-
- # reset files that only changed in the dirstate too
- dirstate = repo.dirstate
- dirchanges = [f for f in dirstate if dirstate[f] != b'n']
- changedfiles.extend(dirchanges)
-
- repo.dirstate.rebuild(urev, uctx.manifest(), changedfiles)
- repo.dirstate.write(repo.currenttransaction())
-
- # clear resolve state
- mergestatemod.mergestate.clean(repo)
-
- update = False
-
- strip(
- ui,
- repo,
- revs,
- backup=backup,
- update=update,
- force=opts.get(b'force'),
- bookmarks=bookmarks,
- soft=opts[b'soft'],
- )
-
- return 0
+# This is a bit ugly, but a uisetup function that defines strip as an
+# alias for debugstrip would override any user alias for strip,
+# including aliases like "strip = strip --no-backup".
+commands.command.rename(old=b'debugstrip', new=b'debugstrip|strip')
diff --git a/hgext/mq.py b/hgext/mq.py
--- a/hgext/mq.py
+++ b/hgext/mq.py
@@ -100,6 +100,7 @@
revsetlang,
scmutil,
smartset,
+ strip,
subrepoutil,
util,
vfs as vfsmod,
@@ -138,7 +139,7 @@
# force load strip extension formerly included in mq and import some utility
try:
- stripext = extensions.find(b'strip')
+ extensions.find(b'strip')
except KeyError:
# note: load is lazy so we could avoid the try-except,
# but I (marmoute) prefer this explicit code.
@@ -149,9 +150,9 @@
def log(self, event, msgfmt, *msgargs, **opts):
pass
- stripext = extensions.load(dummyui(), b'strip', b'')
-
-strip = stripext.strip
+ extensions.load(dummyui(), b'strip', b'')
+
+strip = strip.strip
def checksubstate(repo, baserev=None):
To: valentin.gatienbaron, #hg-reviewers
Cc: mercurial-patches, mercurial-devel
More information about the Mercurial-devel
mailing list