This is an archive of the discontinued Mercurial Phabricator instance.

I haven't looked at the remainder of the series, but it feels weird to basically reinvent a special case of hg log as a debug command. I do see there is a commit introducing a revset for stack. Perhaps we could move it up in the series so we don't need what might be a not-very-useful debug command?

lothiraldan updated this revision to Diff 6213.Feb 28 2018, 6:59 PM

indygreg added a subscriber: yuja.Mar 19 2018, 3:59 PM

indygreg accepted this revision.

indygreg added inline comments.

mercurial/revset.py
1534 ↗	(On Diff #6213)	I'm not sure if the predicate should be prefixed with an underscore to mark it as internal. My reading of the help code is that the lack of a docstring will hide it from the docs. I /think/ this is OK. But I wouldn't be surprised if @yuja or someone told me it needs changed. I'll queue this for now. We can always fix it up later if it is a problem.

This revision is now accepted and ready to land.Mar 19 2018, 3:59 PM

Closed by commit rHG407934a97bc7: stack: import Evolve stack test file (authored by lothiraldan). · Explain WhyMar 19 2018, 4:06 PM

This revision was automatically updated to reflect the committed changes.

martinvonz added a subscriber: martinvonz.Mar 20 2018, 3:33 PM

martinvonz added inline comments.

tests/test-stack.t
82	What if this was @ in the hg repo and only commits 4 and 5 were my own, then I would not want it to be part of my stack. It feels like a definition of upstream would be very helpful. Perhaps a way of providing a separate upstream per commit would be useful. Let's say upstream() was a revset, then I feel like something like "only(., upstream(.))" would be my stack. Of course, it gets weird if I have a history like we have here and I have defined the upstream of 4 to be 2 and the upstream of 5 to be 3 (or the other way around). I guess only commits that share an upstream can be part of the same stack.
97–109	I think I would have preferred to see all of these in my stack rather than just commit 6. Would you?

yuja added inline comments.Mar 21 2018, 10:14 AM

mercurial/revset.py
1534 ↗	(On Diff #6213)	If it's internal (i.e. shouldn't be accessible by user), `_stack()` is preferred. If it's just experimental, it can be hidden by "(EXPERIMENTAL)" in docstring.
1541 ↗	(On Diff #7125)	Nit: I prefer `stackmod.` and name the revset function as `stack()`.

yuja added inline comments.Mar 21 2018, 10:29 AM

mercurial/revset.py
1549 ↗	(On Diff #7125)	This breaks the order of `tip:0 & stack()` for example, and is moot since 68fcc5503ec5, which makes stacks follow the standard order.

lothiraldan added inline comments.Apr 5 2018, 11:35 AM

tests/test-stack.t
97–109	The goal of this series is to define a clean stack definition API. We started by reusing the existing bits from histsedit and show stack. I've added a first way to configure the stack definition through configuration. I think seeing all the drafts is a good idea and we will works in a later series to have a way to include them in the stack.

lothiraldan marked 3 inline comments as done.Apr 5 2018, 12:24 PM

lothiraldan added inline comments.

mercurial/revset.py
1549 ↗	(On Diff #7125)	I have send a follow-up https://phab.mercurial-scm.org/D3136

Revision Contents
Changeset List

		Path
M		mercurial/debugcommands.py (7 lines)
M		tests/test-completion.t (2 lines)
M		tests/test-help.t (1 line)
A	M	tests/test-stack.t (213 lines)

Diff	ID	Description	Created	Lint	Unit
Base		Base
Diff 1	6010		Feb 23 2018, 5:36 AM	★	★
Diff 2	6213		Feb 28 2018, 6:59 PM	★	★
Diff 3	7125	rHG407934a97bc78e9ab3feb3349ae1a458b07b17cd	Jan 19 2018, 10:52 AM	★	★

Status	Author	Revision
Closed	lothiraldan	D3137 histedit: simplify desthistedit
Closed	lothiraldan	D3136 stack: follow-up on the stack revset
Abandoned	lothiraldan	D2403 stack: remove destutil.stackbase
Abandoned	lothiraldan	D2402 stack: introduce an option to limit the selection on the current branch
Abandoned	lothiraldan	D2401 stack: introduce an option to disable the restriction on ancestor
Abandoned	lothiraldan	D2400 stack: begin to make the stack revset configurable
Closed	lothiraldan	D2399 stack: return a sorted smartrev by default
Closed	lothiraldan	D2398 histedit: use the new stack definition for histedit
Closed	lothiraldan	D2397 show: use the new stack definition for show stack
Closed	lothiraldan	D2396 stack: import Evolve stack test file
Closed	lothiraldan	D2395 stack: add a new module for stack-related commands

Diff 6010

mercurial/debugcommands.py

	revlog,			revlog,
	revset,			revset,
	revsetlang,			revsetlang,
	scmutil,			scmutil,
	setdiscovery,			setdiscovery,
	simplemerge,			simplemerge,
	smartset,			smartset,
	sslutil,			sslutil,
				stack,
	streamclone,			streamclone,
	templater,			templater,
	treediscovery,			treediscovery,
	upgrade,			upgrade,
	url as urlmod,			url as urlmod,
	util,			util,
	vfs as vfsmod,			vfs as vfsmod,
	)			)
	ui.status(_('failed.\n'))			ui.status(_('failed.\n'))
	else:			else:
	ui.status(_('done.\n'))			ui.status(_('done.\n'))
	else:			else:
	ui.status(_('full certificate chain is available\n'))			ui.status(_('full certificate chain is available\n'))
	finally:			finally:
	s.close()			s.close()

				@command('debugstack', [])
				def debugstack(ui, repo):
				revs = stack.getstack(repo)
				for c in revs:
				ui.write("%s\n" % c)

	@command('debugsub',			@command('debugsub',
	[('r', 'rev', '',			[('r', 'rev', '',
	_('revision to check'), _('REV'))],			_('revision to check'), _('REV'))],
	_('[-r REV] [REV]'))			_('[-r REV] [REV]'))
	def debugsub(ui, repo, rev=None):			def debugsub(ui, repo, rev=None):
	ctx = scmutil.revsingle(repo, rev, None)			ctx = scmutil.revsingle(repo, rev, None)
	for k, v in sorted(ctx.substate.items()):			for k, v in sorted(ctx.substate.items()):
	ui.write(('path %s\n') % k)			ui.write(('path %s\n') % k)

tests/test-completion.t

	debugpvec			debugpvec
	debugrebuilddirstate			debugrebuilddirstate
	debugrebuildfncache			debugrebuildfncache
	debugrename			debugrename
	debugrevlog			debugrevlog
	debugrevspec			debugrevspec
	debugsetparents			debugsetparents
	debugssl			debugssl
				debugstack
	debugsub			debugsub
	debugsuccessorssets			debugsuccessorssets
	debugtemplate			debugtemplate
	debugupdatecaches			debugupdatecaches
	debugupgraderepo			debugupgraderepo
	debugwalk			debugwalk
	debugwireargs			debugwireargs

	debugpvec:			debugpvec:
	debugrebuilddirstate: rev, minimal			debugrebuilddirstate: rev, minimal
	debugrebuildfncache:			debugrebuildfncache:
	debugrename: rev			debugrename: rev
	debugrevlog: changelog, manifest, dir, dump			debugrevlog: changelog, manifest, dir, dump
	debugrevspec: optimize, show-revs, show-set, show-stage, no-optimized, verify-optimized			debugrevspec: optimize, show-revs, show-set, show-stage, no-optimized, verify-optimized
	debugsetparents:			debugsetparents:
	debugssl:			debugssl:
				debugstack:
	debugsub: rev			debugsub: rev
	debugsuccessorssets: closest			debugsuccessorssets: closest
	debugtemplate: rev, define			debugtemplate: rev, define
	debugupdatecaches:			debugupdatecaches:
	debugupgraderepo: optimize, run			debugupgraderepo: optimize, run
	debugwalk: include, exclude			debugwalk: include, exclude
	debugwireargs: three, four, five, ssh, remotecmd, insecure			debugwireargs: three, four, five, ssh, remotecmd, insecure
	files: rev, print0, include, exclude, template, subrepos			files: rev, print0, include, exclude, template, subrepos

tests/test-help.t

	debugrebuildfncache			debugrebuildfncache
	rebuild the fncache file			rebuild the fncache file
	debugrename dump rename information			debugrename dump rename information
	debugrevlog show data and statistics about a revlog			debugrevlog show data and statistics about a revlog
	debugrevspec parse and apply a revision specification			debugrevspec parse and apply a revision specification
	debugsetparents			debugsetparents
	manually set the parents of the current working directory			manually set the parents of the current working directory
	debugssl test a secure connection to a server			debugssl test a secure connection to a server
				debugstack (no help text available)
	debugsub (no help text available)			debugsub (no help text available)
	debugsuccessorssets			debugsuccessorssets
	show set of successors for revision			show set of successors for revision
	debugtemplate			debugtemplate
	parse and apply a template			parse and apply a template
	debugupdatecaches			debugupdatecaches
	warm all known caches in the repository			warm all known caches in the repository
	debugupgraderepo			debugupgraderepo

tests/test-stack.t

This file was added.


				This test test the low-level definition of stack, agnostic from all formatting

				Initial setup

				$ cat << EOF >> $HGRCPATH
				> [ui]
				> logtemplate = {rev} {branch} {phase} {desc\|firstline}\n
				> [extensions]
				> rebase=
				> [experimental]
				> evolution=createmarkers,exchange,allowunstable
				> EOF

				$ hg init main
				$ cd main
				$ hg branch other
				marked working directory as branch other
				(branches are permanent and global, did you want a bookmark?)
				$ echo aaa > aaa
				$ hg add aaa
				$ hg commit -m c_a
				$ echo aaa > bbb
				$ hg add bbb
				$ hg commit -m c_b
				$ hg branch foo
				marked working directory as branch foo
				$ echo aaa > ccc
				$ hg add ccc
				$ hg commit -m c_c
				$ echo aaa > ddd
				$ hg add ddd
				$ hg commit -m c_d
				$ echo aaa > eee
				$ hg add eee
				$ hg commit -m c_e
				$ echo aaa > fff
				$ hg add fff
				$ hg commit -m c_f
				$ hg log -G
				@ 5 foo draft c_f
				\|
				o 4 foo draft c_e
				\|
				o 3 foo draft c_d
				\|
				o 2 foo draft c_c
				\|
				o 1 other draft c_b
				\|
				o 0 other draft c_a


				Check that stack doesn't include public changesets
				--------------------------------------------------

				$ hg up other
				0 files updated, 0 files merged, 4 files removed, 0 files unresolved
				$ hg debugstack
				1
				0
				$ hg phase --public 'branch("other")'
				$ hg debugstack
				$ hg up foo
				4 files updated, 0 files merged, 0 files removed, 0 files unresolved

				Simple test
				-----------

				'hg debugstack' list all changeset in the branch

				$ hg branch
				foo
				$ hg debugstack
				5
				4
				3
				2

				Case with some of the branch unstable
				------------------------------------

				martinvonzUnsubmitted Not Done What if this was @ in the hg repo and only commits 4 and 5 were my own, then I would not want it to be part of my stack. It feels like a definition of upstream would be very helpful. Perhaps a way of providing a separate upstream per commit would be useful. Let's say upstream() was a revset, then I feel like something like "only(., upstream(.))" would be my stack. Of course, it gets weird if I have a history like we have here and I have defined the upstream of 4 to be 2 and the upstream of 5 to be 3 (or the other way around). I guess only commits that share an upstream can be part of the same stack. martinvonz: What if this was @ in the hg repo and only commits 4 and 5 were my own, then I would not want…
				$ hg up 3
				0 files updated, 0 files merged, 2 files removed, 0 files unresolved
				$ echo bbb > ddd
				$ hg commit --amend
				2 new orphan changesets
				$ hg log -G
				@ 6 foo draft c_d
				\|
				\| * 5 foo draft c_f
				\| \|
				\| * 4 foo draft c_e
				\| \|
				\| x 3 foo draft c_d
				\|/
				o 2 foo draft c_c
				\|
				o 1 other public c_b
				\|
				o 0 other public c_a

				$ hg debugstack
				6
				$ hg up -r "desc(c_e)"
				2 files updated, 0 files merged, 0 files removed, 0 files unresolved
				$ hg debugstack
				4
				3
				martinvonzUnsubmitted Not Done I think I would have preferred to see all of these in my stack rather than just commit 6. Would you? martinvonz: I think I would have preferred to see all of these in my stack rather than just commit 6. Would…
				lothiraldanAuthorUnsubmitted Not Done The goal of this series is to define a clean stack definition API. We started by reusing the existing bits from histsedit and show stack. I've added a first way to configure the stack definition through configuration. I think seeing all the drafts is a good idea and we will works in a later series to have a way to include them in the stack. lothiraldan: The goal of this series is to define a clean stack definition API. We started by reusing the…
				$ hg up -r "desc(c_d)"
				1 files updated, 0 files merged, 1 files removed, 0 files unresolved

				$ hg debugstack
				6

				Case with multiple topological heads
				------------------------------------

				Make things linear again

				$ hg rebase -s 'desc(c_e)' -d 'desc(c_d) - obsolete()'
				rebasing 4:4f2a69f6d380 "c_e"
				rebasing 5:913c298d8b0a "c_f"
				$ hg log -G
				o 8 foo draft c_f
				\|
				o 7 foo draft c_e
				\|
				@ 6 foo draft c_d
				\|
				o 2 foo draft c_c
				\|
				o 1 other public c_b
				\|
				o 0 other public c_a


				Create the second branch

				$ hg up 'desc(c_d)'
				0 files updated, 0 files merged, 0 files removed, 0 files unresolved
				$ echo aaa > ggg
				$ hg add ggg
				$ hg commit -m c_g
				created new head
				$ echo aaa > hhh
				$ hg add hhh
				$ hg commit -m c_h
				$ hg log -G
				@ 10 foo draft c_h
				\|
				o 9 foo draft c_g
				\|
				\| o 8 foo draft c_f
				\| \|
				\| o 7 foo draft c_e
				\|/
				o 6 foo draft c_d
				\|
				o 2 foo draft c_c
				\|
				o 1 other public c_b
				\|
				o 0 other public c_a


				Test output

				$ hg debugstack
				10
				9
				$ hg up 'desc(c_f)'
				2 files updated, 0 files merged, 2 files removed, 0 files unresolved
				$ hg debugstack
				8
				7

				Case with multiple heads with unstability involved
				--------------------------------------------------

				We amend the message to make sure the display base pick the right changeset

				$ hg up 'desc(c_d)'
				0 files updated, 0 files merged, 2 files removed, 0 files unresolved
				$ echo ccc > ddd
				$ hg commit --amend -m 'c_D'
				4 new orphan changesets
				$ hg rebase -d . -s 'desc(c_g)'
				rebasing 9:2ebb6e48ab8a "c_g"
				rebasing 10:634f38e27a1d "c_h"
				$ hg log -G
				o 13 foo draft c_h
				\|
				o 12 foo draft c_g
				\|
				@ 11 foo draft c_D
				\|
				\| * 8 foo draft c_f
				\| \|
				\| * 7 foo draft c_e
				\| \|
				\| x 6 foo draft c_d
				\|/
				o 2 foo draft c_c
				\|
				o 1 other public c_b
				\|
				o 0 other public c_a


				We should improve stack definition to also show 12 and 13 here
				$ hg debugstack
				11

This is an archive of the discontinued Mercurial Phabricator instance.

stack: import Evolve stack test fileClosedPublic

Details

Diff Detail

Event Timeline