# HG changeset patch # User Pierre-Yves David # Date 1545419663 -3600 # Node ID c06301c5b3e285ec942a308c21e16330b91d0cc3 # Parent b4ffdeb2afd866ad5393886c37df52e02241914b# Parent add4c12a9c376a26d81ea172e249a7279a097323 test-compat: merge mercurial-4.5 into mercurial-4.4 diff -r b4ffdeb2afd8 -r c06301c5b3e2 .hgtags --- a/.hgtags Tue Nov 27 04:07:36 2018 +0100 +++ b/.hgtags Fri Dec 21 20:14:23 2018 +0100 @@ -74,3 +74,4 @@ c6362c4abd695fb96e2fd63c150c051852303c7e 8.2.1 45d4b49d81d9ed23e40126f72bfc3fb339522356 8.3.0 b90422a11a887c6ff756c2a5622ea0a1e260ff4c 8.3.1 +7edc5c148df0150087832b861966d658df0b601e 8.3.2 diff -r b4ffdeb2afd8 -r c06301c5b3e2 CHANGELOG --- a/CHANGELOG Tue Nov 27 04:07:36 2018 +0100 +++ b/CHANGELOG Fri Dec 21 20:14:23 2018 +0100 @@ -1,8 +1,14 @@ Changelog ========= +8.3.3 - in progress +------------------- -8.3.2 - in progress + * evolve: fix possible crash when the repo changes during evolve (issue-6028) + * test: avoid leaking `hg serve` process + * topic: fix error message for the `ngtip` revset + +8.3.2 -- 2017-11-27 ------------------- * evolve: not longer attempt to translate revision's descriptions (issue6016) diff -r b4ffdeb2afd8 -r c06301c5b3e2 debian/changelog --- a/debian/changelog Tue Nov 27 04:07:36 2018 +0100 +++ b/debian/changelog Fri Dec 21 20:14:23 2018 +0100 @@ -1,3 +1,9 @@ +mercurial-evolve (8.3.2-1) UNRELEASED; urgency=medium + + * new upstream release + + -- Pierre-Yves David Tue, 27 Nov 2018 04:29:19 +0100 + mercurial-evolve (8.3.1) unstable; urgency=medium * new upstream release diff -r b4ffdeb2afd8 -r c06301c5b3e2 hgext3rd/evolve/compat.py --- a/hgext3rd/evolve/compat.py Tue Nov 27 04:07:36 2018 +0100 +++ b/hgext3rd/evolve/compat.py Fri Dec 21 20:14:23 2018 +0100 @@ -171,7 +171,7 @@ import mercurial.utils.dateutil makedate = mercurial.utils.dateutil.makedate parsedate = mercurial.utils.dateutil.parsedate -except ImportError as e: +except ImportError: import mercurial.util makedate = mercurial.util.makedate parsedate = mercurial.util.parsedate diff -r b4ffdeb2afd8 -r c06301c5b3e2 hgext3rd/evolve/evolvecmd.py --- a/hgext3rd/evolve/evolvecmd.py Tue Nov 27 04:07:36 2018 +0100 +++ b/hgext3rd/evolve/evolvecmd.py Fri Dec 21 20:14:23 2018 +0100 @@ -61,7 +61,7 @@ bool: a boolean value indicating whether the instability was solved newnode: if bool is True, then the newnode of the resultant commit formed. newnode can be node, when resolution led to no new - commit. If bool is False, this is ''. + commit. If bool is False, this is ".". """ displayer = None if stacktmplt: @@ -101,7 +101,7 @@ bool: a boolean value indicating whether the instability was solved newnode: if bool is True, then the newnode of the resultant commit formed. newnode can be node, when resolution led to no new - commit. If bool is False, this is ''. + commit. If bool is False, this is ".". """ pctx = orig.p1() keepbranch = orig.p1().branch() != orig.branch() @@ -125,7 +125,7 @@ if not pctx.obsolete(): ui.warn(_("cannot solve instability of %s, skipping\n") % orig) - return (False, '') + return (False, ".") obs = pctx newer = obsutil.successorssets(repo, obs.node()) # search of a parent which is not killed @@ -139,7 +139,7 @@ msg = _("skipping %s: divergent rewriting. can't choose " "destination\n") % obs ui.write_err(msg) - return (False, '') + return (False, ".") targets = newer[0] assert targets if len(targets) > 1: @@ -157,7 +157,7 @@ "ambiguous destination: " "parent split across two branches\n") ui.write_err(msg) - return (False, '') + return (False, ".") target = repo[selectedrev] else: target = repo[heads.first()] @@ -177,7 +177,7 @@ todo = 'hg rebase -r %s -d %s\n' % (orig, target) if dryrun: repo.ui.write(todo) - return (False, '') + return (False, ".") else: repo.ui.note(todo) if progresscb: @@ -201,7 +201,7 @@ bool: a boolean value indicating whether the instability was solved newnode: if bool is True, then the newnode of the resultant commit formed. newnode can be node, when resolution led to no new - commit. If bool is False, this is ''. + commit. If bool is False, this is ".". """ repo = repo.unfiltered() bumped = repo[bumped.rev()] @@ -209,14 +209,14 @@ if len(bumped.parents()) > 1: msg = _('skipping %s : we do not handle merge yet\n') % bumped ui.write_err(msg) - return (False, '') + return (False, ".") prec = repo.set('last(allprecursors(%d) and public())', bumped.rev()).next() # For now we deny target merge if len(prec.parents()) > 1: msg = _('skipping: %s: public version is a merge, ' 'this is not handled yet\n') % prec ui.write_err(msg) - return (False, '') + return (False, ".") if not ui.quiet or confirm: repo.ui.write(_('recreate:'), label='evolve.operation') @@ -232,7 +232,7 @@ repo.ui.write(('hg revert --all --rev %s;\n' % bumped)) repo.ui.write(('hg commit --msg "%s update to %s"\n' % (TROUBLES['PHASEDIVERGENT'], bumped))) - return (False, '') + return (False, ".") if progresscb: progresscb() tmpctx = bumped @@ -343,7 +343,7 @@ bool: a boolean value indicating whether the instability was solved newnode: if bool is True, then the newnode of the resultant commit formed. newnode can be node, when resolution led to no new - commit. If bool is False, this is ''. + commit. If bool is False, this is ".". """ repo = repo.unfiltered() divergent = repo[divergent.rev()] @@ -376,7 +376,7 @@ "| You should contact your local evolution Guru for help.\n" ) % (divergent, TROUBLES['CONTENTDIVERGENT'], othersstr) ui.write_err(msg) - return (False, '') + return (False, ".") other = others[0] evolvestate['other-divergent'] = other.node() evolvestate['base'] = base.node() @@ -390,7 +390,7 @@ "| This probably means redoing the merge and using \n" "| `hg prune` to kill older version.\n") ui.write_err(hint) - return (False, '') + return (False, ".") otherp1 = other.p1().rev() divp1 = divergent.p1().rev() @@ -450,7 +450,7 @@ ) % {'d': divergent, 'o': other} ui.write_err(msg) ui.write_err(hint) - return (False, '') + return (False, ".") if not ui.quiet or confirm: ui.write(_('merge:'), label='evolve.operation') @@ -470,7 +470,7 @@ ui.write(('hg revert --all --rev tip &&\n')) ui.write(('hg commit -m "`hg log -r %s --template={desc}`";\n' % divergent)) - return (False, '') + return (False, ".") evolvestate['resolutionparent'] = resolutionparent # relocate the other divergent if required diff -r b4ffdeb2afd8 -r c06301c5b3e2 hgext3rd/evolve/legacy.py --- a/hgext3rd/evolve/legacy.py Tue Nov 27 04:07:36 2018 +0100 +++ b/hgext3rd/evolve/legacy.py Fri Dec 21 20:14:23 2018 +0100 @@ -32,7 +32,7 @@ try: from mercurial.utils.dateutil import makedate -except ImportError as e: +except ImportError: # compat with hg < 4.6 from mercurial.util import makedate diff -r b4ffdeb2afd8 -r c06301c5b3e2 hgext3rd/evolve/metadata.py --- a/hgext3rd/evolve/metadata.py Tue Nov 27 04:07:36 2018 +0100 +++ b/hgext3rd/evolve/metadata.py Fri Dec 21 20:14:23 2018 +0100 @@ -5,7 +5,7 @@ # This software may be used and distributed according to the terms of the # GNU General Public License version 2 or any later version. -__version__ = '8.3.2.dev' +__version__ = '8.3.3.dev' testedwith = '4.3.2 4.4.2 4.5.2 4.6.2 4.7' minimumhgversion = '4.3' buglink = 'https://bz.mercurial-scm.org/' diff -r b4ffdeb2afd8 -r c06301c5b3e2 hgext3rd/topic/__init__.py --- a/hgext3rd/topic/__init__.py Tue Nov 27 04:07:36 2018 +0100 +++ b/hgext3rd/topic/__init__.py Fri Dec 21 20:14:23 2018 +0100 @@ -177,7 +177,7 @@ 'topic.active': 'green', } -__version__ = '0.12.2.dev' +__version__ = '0.12.3.dev' testedwith = '4.3.3 4.4.2 4.5.2 4.6.2 4.7' minimumhgversion = '4.3' diff -r b4ffdeb2afd8 -r c06301c5b3e2 hgext3rd/topic/revset.py --- a/hgext3rd/topic/revset.py Tue Nov 27 04:07:36 2018 +0100 +++ b/hgext3rd/topic/revset.py Fri Dec 21 20:14:23 2018 +0100 @@ -82,9 +82,9 @@ Name is horrible so that people change it. """ - args = revset.getargs(x, 1, 1, 'topic takes one') + args = revset.getargs(x, 1, 1, 'ngtip takes one argument') # match a specific topic - branch = revset.getstring(args[0], 'ngtip() argument must be a string') + branch = revset.getstring(args[0], 'ngtip requires a string') if branch == '.': branch = repo['.'].branch() return subset & revset.baseset(destination.ngtip(repo, branch)) @@ -95,8 +95,9 @@ This is roughly equivalent to 'topic(.) - obsolete' with a sorting moving unstable changeset after there future parent (as if evolve where already - run).""" - err = 'stack() takes no argument, it works on current topic' + run). + """ + err = 'stack takes no arguments, it works on current topic' revset.getargs(x, 0, 0, err) topic = None branch = None diff -r b4ffdeb2afd8 -r c06301c5b3e2 tests/test-issue-6028.t --- /dev/null Thu Jan 01 00:00:00 1970 +0000 +++ b/tests/test-issue-6028.t Fri Dec 21 20:14:23 2018 +0100 @@ -0,0 +1,123 @@ +This test file test the #6028 issue + +evolve fails with mercurial.error.ProgrammingError: unsupported changeid '' of type + +https://bz.mercurial-scm.org/show_bug.cgi?id=6028 + +Global setup +============ + + $ . $TESTDIR/testlib/common.sh + $ cat >> $HGRCPATH < [ui] + > interactive = true + > [phases] + > publish=False + > [extensions] + > evolve = + > topic = + > EOF + +Test +==== + + $ hg init $TESTTMP/issue-6028 + $ cd $TESTTMP/issue-6028 + +create initial commit + $ echo "0" > 0 + $ hg ci -Am 0 + adding 0 + + + $ hg up default + 0 files updated, 0 files merged, 0 files removed, 0 files unresolved + $ hg topics a + marked working directory as topic: a + $ echo "a" > a + $ hg ci -Am a + adding a + active topic 'a' grew its first changeset + (see 'hg help topics' for more information) + + + $ hg up default + 0 files updated, 0 files merged, 1 files removed, 0 files unresolved + $ hg topics b + marked working directory as topic: b + $ echo "b" > b + $ hg ci -Am b + adding b + active topic 'b' grew its first changeset + (see 'hg help topics' for more information) + + $ hg up default + 0 files updated, 0 files merged, 1 files removed, 0 files unresolved + $ hg branch integration + marked working directory as branch integration + + $ hg merge a + 1 files updated, 0 files merged, 0 files removed, 0 files unresolved + (branch merge, don't forget to commit) + $ hg ci -m "merged a" + + $ hg merge b + 1 files updated, 0 files merged, 0 files removed, 0 files unresolved + (branch merge, don't forget to commit) + $ hg ci -m "merged b" + + $ hg up a + switching to topic a + 0 files updated, 0 files merged, 1 files removed, 0 files unresolved + $ echo "a bad commit" >> a_bad_commit + $ hg add a_bad_commit + $ hg ci -m "a bad commit" + $ hg up integration + 1 files updated, 0 files merged, 1 files removed, 0 files unresolved + $ hg merge a + 1 files updated, 0 files merged, 0 files removed, 0 files unresolved + (branch merge, don't forget to commit) + $ hg ci -m "merged a bad commit" + + $ hg up a + switching to topic a + 0 files updated, 0 files merged, 1 files removed, 0 files unresolved + $ echo "aa" >> a + $ hg ci -m "aa" + $ hg up integration + 2 files updated, 0 files merged, 0 files removed, 0 files unresolved + $ hg merge a + 1 files updated, 0 files merged, 0 files removed, 0 files unresolved + (branch merge, don't forget to commit) + $ hg ci -m "merged aa" + + $ hg up b + switching to topic b + 0 files updated, 0 files merged, 2 files removed, 0 files unresolved + $ echo "bb" >> b + $ hg ci -m "bb" + $ hg up integration + 3 files updated, 0 files merged, 0 files removed, 0 files unresolved + $ hg merge b + 1 files updated, 0 files merged, 0 files removed, 0 files unresolved + (branch merge, don't forget to commit) + $ hg ci -m "merged bb" + +create instability by pruning two changesets, one in a topic, one in a merge + $ hg prune -r 5:6 + 2 changesets pruned + 3 new orphan changesets + + $ hg up 4 + 2 files updated, 0 files merged, 1 files removed, 0 files unresolved + +start the evolve + $ hg evolve + move:[8] merged aa + atop:[4] merged b + working directory is now at c920dd828523 + +evolve creates an obsolete changeset above as 11 + $ hg evolve -r . + cannot solve instability of c920dd828523, skipping + cannot solve instability of c920dd828523, skipping diff -r b4ffdeb2afd8 -r c06301c5b3e2 tests/test-topic-stack.t --- a/tests/test-topic-stack.t Tue Nov 27 04:07:36 2018 +0100 +++ b/tests/test-topic-stack.t Fri Dec 21 20:14:23 2018 +0100 @@ -320,11 +320,11 @@ 5 default {foo} draft c_f $ hg log -r 'stack(foo)' - hg: parse error: stack() takes no argument, it works on current topic + hg: parse error: stack takes no arguments, it works on current topic [255] $ hg log -r 'stack(foobar)' - hg: parse error: stack() takes no argument, it works on current topic + hg: parse error: stack takes no arguments, it works on current topic [255] Case with multiple heads on the topic diff -r b4ffdeb2afd8 -r c06301c5b3e2 tests/test-wireproto.t --- a/tests/test-wireproto.t Tue Nov 27 04:07:36 2018 +0100 +++ b/tests/test-wireproto.t Fri Dec 21 20:14:23 2018 +0100 @@ -230,6 +230,7 @@ $ $RUNTESTDIR/killdaemons.py $DAEMON_PIDS $ hg serve -R server -n test -p $HGPORT -d --pid-file=hg.pid -A access.log -E errors.log --config experimental.evolution='!' + $ cat hg.pid >> $DAEMON_PIDS $ hg debugpushkey http://localhost:$HGPORT/ obsolete (do some extra pulling to be sure)