Skip to content
Snippets Groups Projects

more in-depth refactoring for obslog --origins and some unused yet functions/keyword arguments

Closed Anton Shestakov requested to merge topic/default/obslog-predecessors-1 into branch/default
All threads resolved!
Files
3
+ 103
53
@@ -108,12 +108,13 @@ TEMPLATE_FIRST_LINE = b"""{if(rev, "%(presentnode)s", "%(missingnode)s")}""" % {
b"presentnode": TEMPLATE_PRESENT_NODE,
b"missingnode": TEMPLATE_MISSING_NODE
}
TEMPLATE_VERB = b"""{label("evolve.verb", verb)}"""
TEMPLATE_PREDNODES = b"""{label("evolve.node", join(prednodes % "{prednode|short}", ", "))}"""
TEMPLATE_SUCCNODES = b"""{label("evolve.node", join(succnodes % "{succnode|short}", ", "))}"""
TEMPLATE_REWRITE = b"""{if(succnodes, "%(verb)s{if(effects, "({join(effects, ", ")})")} as %(succnodes)s", "pruned")}""" % {
b"verb": TEMPLATE_VERB,
TEMPLATE_NODES = b"""{if(prednodes, "from %(prednodes)s")}{if(succnodes, "as %(succnodes)s")}""" % {
b"prednodes": TEMPLATE_PREDNODES,
b"succnodes": TEMPLATE_SUCCNODES
}
TEMPLATE_REWRITE = b"""{label("evolve.verb", verb)}{if(effects, "({join(effects, ", ")})")}"""
TEMPLATE_OPERATIONS = b"""{if(operations, "using {label("evolve.operation", join(operations, ", "))}")}"""
TEMPLATE_USERS = b"""by {label("evolve.user", join(users, ", "))}"""
TEMPLATE_ONE_DATE = b"""({date(max(dates), "%a %b %d %H:%M:%S %Y %1%2")})"""
@@ -125,10 +126,11 @@ TEMPLATE_DATES = b"""{label("evolve.date", ifeq(min(dates), max(dates), "%(oneda
TEMPLATE_NOTES = b"""{if(notes, notes % "\n note: {label("evolve.note", note)}")}"""
TEMPLATE_PATCH = b"""{if(patch, "{patch}")}{if(nopatchreason, "\n(No patch available, {nopatchreason})")}"""
DEFAULT_TEMPLATE = (b"""%(firstline)s
{markers %% " {separate(" ", "%(rewrite)s", "%(operations)s", "%(users)s", "%(dates)s")}%(notes)s{indent(descdiff, " ")}{indent("%(patch)s", " ")}\n"}
{markers %% " {separate(" ", "%(rewrite)s", "%(nodes)s", "%(operations)s", "%(users)s", "%(dates)s")}%(notes)s{indent(descdiff, " ")}{indent("%(patch)s", " ")}\n"}
""") % {
b"firstline": TEMPLATE_FIRST_LINE,
b"rewrite": TEMPLATE_REWRITE,
b"nodes": TEMPLATE_NODES,
b"operations": TEMPLATE_OPERATIONS,
b"users": TEMPLATE_USERS,
b"dates": TEMPLATE_DATES,
@@ -136,6 +138,19 @@ DEFAULT_TEMPLATE = (b"""%(firstline)s
b"patch": TEMPLATE_PATCH,
}
def _nodesandmarkers(repo, ctx, filternonlocal):
if filternonlocal:
r = obsutil.successorsandmarkers(repo, ctx)
if r is None:
r = []
for succset in sorted(r):
if succset[b'markers']:
yield (succset[b'successors'], succset[b'markers'])
else:
markers = repo.obsstore.successors.get(ctx.node(), ())
for marker in sorted(markers):
yield (marker[1], [marker])
class obsmarker_printer(logcmdutil.changesetprinter):
"""show (available) information about a node
@@ -173,28 +188,10 @@ class obsmarker_printer(logcmdutil.changesetprinter):
markerfm = fm.nested(b"markers")
# Succs markers
if self.filter is False:
succs = self.repo.obsstore.successors.get(changenode, ())
succs = sorted(succs)
for successor in succs:
displaymarkers(self.ui, markerfm, successor[1],
[successor], ctx.node(), self.repo,
self._includediff)
else:
r = obsutil.successorsandmarkers(self.repo, ctx)
if r is None:
r = []
for succset in sorted(r):
markers = succset[b"markers"]
if not markers:
continue
successors = succset[b"successors"]
displaymarkers(self.ui, markerfm, successors, markers,
ctx.node(), self.repo, self._includediff)
data = _nodesandmarkers(self.repo, ctx, self.filter)
for nodes, markers in data:
displaymarkers(self.ui, markerfm, nodes, markers, ctx.node(),
self.repo, self._includediff)
markerfm.end()
@@ -208,7+205,7 @@
raise error.ProgrammingError(msg)
def flush(self, ctx):
''' changeset_printer has some logic around buffering data
in self.headers that we don't use
'''
pass
@@ -455,7+452,7 @@
nodec = repo.changelog.node
unfi = repo.unfiltered()
nodes = [nodec(r) for r in revs]
seen = set(nodes)
toshow = []
walksuccessors = opts and opts.get('all')
filternonlocal = opts and opts.get('filternonlocal')
includediff = opts and opts.get('patch')
while nodes:
ctxnode = nodes.pop()
displaynode(fm, unfi, ctxnode)
succs = successors.get(ctxnode, ())
markerfm = fm.nested(b"markers")
for successor in sorted(succs):
includediff = opts and opts.get("patch")
displaymarkers(ui, markerfm, successor[1], [successor], ctxnode,
unfi, includediff)
markerfm.end()
fm.plain(b'\n')
if ctxnode in unfi:
toshow.append(unfi[ctxnode])
else:
if filternonlocal is False:
toshow.append(missingchangectx(unfi, ctxnode))
preds = predecessors.get(ctxnode, ())
for p in sorted(preds):
@@ -479,6 +474,28 @@ def displayrevs(ui, repo, revs, opts):
if p[0] not in seen:
seen.add(p[0])
nodes.append(p[0])
if walksuccessors:
for successor in successors.get(ctxnode, ()):
for s in successor[1]:
if s not in seen:
seen.add(s)
nodes.append(s)
for ctx in toshow:
displaynode(fm, unfi, ctx.node())
markerfm = fm.nested(b"markers")
data = _nodesandmarkers(repo, ctx, filternonlocal)
for nodes_, markers in data:
displaymarkers(ui, markerfm, nodes_, markers, ctx.node(), unfi,
includediff)
markerfm.end()
fm.plain(b'\n')
fm.end()
def displaynode(fm, repo, node):
@@ -512,10 +529,14 @@ def displaymissingctx(fm, nodewithoutctx):
label=b"evolve.node evolve.missing_change_ctx")
fm.plain(b'\n')
def displaymarkers(ui, fm, succnodes, markers, node, repo, includediff=False):
def displaymarkers(ui, fm, nodes, markers, node, repo, includediff=False,
successive=True):
fm.startitem()
verb = _successorsetverb(succnodes, markers)[b"verb"]
if successive:
verb = _successorsetverb(nodes, markers)[b"verb"]
else:
verb = _predecessorsverb(nodes, markers)
fm.data(verb=verb)
@@ -524,10 +545,14 @@ def displaymarkers(ui, fm, succnodes, markers, node, repo, includediff=False):
fmteffects = fm.formatlist(effects, b'effect', sep=b', ')
fm.data(effects=fmteffects)
if len(succnodes) > 0:
hexnodes = (nodemod.hex(succnode) for succnode in sorted(succnodes))
nodes = fm.formatlist(hexnodes, b'succnode')
fm.data(succnodes=nodes)
if len(nodes) > 0:
hexnodes = (nodemod.hex(node) for node in sorted(nodes))
if successive:
nodelist = fm.formatlist(hexnodes, b'succnode')
fm.data(succnodes=nodelist)
else:
nodelist = fm.formatlist(hexnodes, b'prednode')
fm.data(prednodes=nodelist)
# Operations
operations = obsutil.markersoperations(markers)
@@ -549,22 +574,27 @@ def displaymarkers(ui, fm, succnodes, markers, node, repo, includediff=False):
# Patch display
if includediff is True:
_patchavailable = patchavailable(node, repo, succnodes)
_patchavailable = patchavailable(node, repo, nodes,
successive=successive)
if _patchavailable[0] is True:
succ = _patchavailable[1]
diffnode = _patchavailable[1]
basectx = repo[node]
succctx = repo[succ]
if successive:
actx = repo[node]
bctx = repo[diffnode]
else:
actx = repo[diffnode]
bctx = repo[node]
# Description patch
descriptionpatch = getmarkerdescriptionpatch(repo,
basectx.description(),
succctx.description())
actx.description(),
bctx.description())
if descriptionpatch:
# add the diffheader
diffheader = b"diff -r %s -r %s changeset-description\n" %\
(basectx, succctx)
(actx, bctx)
descriptionpatch = diffheader + descriptionpatch
def tolist(text):
@@ -584,8 +614,8 @@ def displaymarkers(ui, fm, succnodes, markers, node, repo, includediff=False):
matchfn = scmutil.matchall(repo)
firstline = True
linestart = True
for chunk, label in patch.diffui(repo, node, succ, matchfn,
opts=diffopts):
for chunk, label in patch.diffui(repo, actx.node(), bctx.node(),
matchfn, opts=diffopts):
if firstline:
ui.write(b'\n')
firstline = False
@@ -718,6 +748,26 @@ def _successorsetverb(successorset, markers):
verb = b'split'
return {b'verb': verb}
def _predecessorsverb(predecessors, markers):
""" Return the verb summarizing a set of predecessors and related markers.
"""
verb = None
if not predecessors:
# we got successors instead of predecessors, and they are empty
# (this is a special case for showing prunes)
verb = b'pruned'
elif len(markers) == 1 and len(markers[0][1]) > 1:
# peeked at the successors to see if this is a split
verb = b'split'
elif len(predecessors) == 1:
verb = _markerspreciseverb(markers)
if verb is None:
verb = b'rewritten'
else:
verb = b'folded'
return verb
# Use a more advanced version of obsfateverb that uses effect-flag
@eh.wrapfunction(obsutil, 'obsfateverb')
def obsfateverb(orig, *args, **kwargs):
Loading