bookmarks.py
525 lines
| 16.5 KiB
| text/x-python
|
PythonLexer
/ mercurial / bookmarks.py
Matt Mackall
|
r13350 | # Mercurial bookmark support code | ||
# | ||||
# Copyright 2008 David Soria Parra <dsp@php.net> | ||||
# | ||||
# This software may be used and distributed according to the terms of the | ||||
# GNU General Public License version 2 or any later version. | ||||
Pierre-Yves David
|
r23360 | import os | ||
Matt Mackall
|
r13350 | from mercurial.i18n import _ | ||
FUJIWARA Katsunori
|
r20025 | from mercurial.node import hex, bin | ||
Pierre-Yves David
|
r22667 | from mercurial import encoding, error, util, obsolete, lock as lockmod | ||
FUJIWARA Katsunori
|
r19896 | import errno | ||
Matt Mackall
|
r13350 | |||
Augie Fackler
|
r17922 | class bmstore(dict): | ||
"""Storage for bookmarks. | ||||
This object should do all bookmark reads and writes, so that it's | ||||
fairly simple to replace the storage underlying bookmarks without | ||||
having to clone the logic surrounding bookmarks. | ||||
This particular bmstore implementation stores bookmarks as | ||||
{hash}\s{name}\n (the same format as localtags) in | ||||
.hg/bookmarks. The mapping is stored as {name: nodeid}. | ||||
This class does NOT handle the "current" bookmark state at this | ||||
time. | ||||
""" | ||||
Matt Mackall
|
r13351 | |||
Augie Fackler
|
r17922 | def __init__(self, repo): | ||
dict.__init__(self) | ||||
self._repo = repo | ||||
try: | ||||
Ryan McElroy
|
r23458 | bkfile = self.getbkfile(repo) | ||
Pierre-Yves David
|
r23360 | for line in bkfile: | ||
Augie Fackler
|
r17922 | line = line.strip() | ||
if not line: | ||||
continue | ||||
if ' ' not in line: | ||||
repo.ui.warn(_('malformed line in .hg/bookmarks: %r\n') | ||||
% line) | ||||
continue | ||||
sha, refspec = line.split(' ', 1) | ||||
refspec = encoding.tolocal(refspec) | ||||
try: | ||||
self[refspec] = repo.changelog.lookup(sha) | ||||
except LookupError: | ||||
pass | ||||
except IOError, inst: | ||||
if inst.errno != errno.ENOENT: | ||||
raise | ||||
Ryan McElroy
|
r23458 | def getbkfile(self, repo): | ||
bkfile = None | ||||
if 'HG_PENDING' in os.environ: | ||||
try: | ||||
bkfile = repo.vfs('bookmarks.pending') | ||||
except IOError, inst: | ||||
if inst.errno != errno.ENOENT: | ||||
raise | ||||
if bkfile is None: | ||||
bkfile = repo.vfs('bookmarks') | ||||
return bkfile | ||||
Pierre-Yves David
|
r22665 | def recordchange(self, tr): | ||
"""record that bookmarks have been changed in a transaction | ||||
The transaction is then responsible for updating the file content.""" | ||||
tr.addfilegenerator('bookmarks', ('bookmarks',), self._write, | ||||
Pierre-Yves David
|
r23317 | location='plain') | ||
Pierre-Yves David
|
r22941 | tr.hookargs['bookmark_moved'] = '1' | ||
Pierre-Yves David
|
r22665 | |||
Augie Fackler
|
r17922 | def write(self): | ||
'''Write bookmarks | ||||
Write the given bookmark => hash dictionary to the .hg/bookmarks file | ||||
in a format equal to those of localtags. | ||||
We also store a backup of the previous state in undo.bookmarks that | ||||
can be copied back on rollback. | ||||
''' | ||||
repo = self._repo | ||||
Ryan McElroy
|
r23469 | self._writerepo(repo) | ||
def _writerepo(self, repo): | ||||
"""Factored out for extensibility""" | ||||
Augie Fackler
|
r17922 | if repo._bookmarkcurrent not in self: | ||
Siddharth Agarwal
|
r20098 | unsetcurrent(repo) | ||
Augie Fackler
|
r17922 | |||
wlock = repo.wlock() | ||||
try: | ||||
file = repo.vfs('bookmarks', 'w', atomictemp=True) | ||||
Pierre-Yves David
|
r22664 | self._write(file) | ||
Augie Fackler
|
r17922 | file.close() | ||
# touch 00changelog.i so hgweb reloads bookmarks (no lock needed) | ||||
Benoit Boissinot
|
r14027 | try: | ||
FUJIWARA Katsunori
|
r19896 | repo.svfs.utime('00changelog.i', None) | ||
Augie Fackler
|
r17922 | except OSError: | ||
Benoit Boissinot
|
r14027 | pass | ||
Augie Fackler
|
r17922 | |||
finally: | ||||
wlock.release() | ||||
Matt Mackall
|
r13351 | |||
Pierre-Yves David
|
r22664 | def _write(self, fp): | ||
for name, node in self.iteritems(): | ||||
fp.write("%s %s\n" % (hex(node), encoding.fromlocal(name))) | ||||
Matt Mackall
|
r13351 | def readcurrent(repo): | ||
'''Get the current bookmark | ||||
Kevin Bullock
|
r18043 | If we use gittish branches we have a current bookmark that | ||
Matt Mackall
|
r13351 | we are on. This function returns the name of the bookmark. It | ||
is stored in .hg/bookmarks.current | ||||
''' | ||||
mark = None | ||||
Benoit Boissinot
|
r14027 | try: | ||
Angel Ezquerra
|
r23877 | file = repo.vfs('bookmarks.current') | ||
Benoit Boissinot
|
r14027 | except IOError, inst: | ||
if inst.errno != errno.ENOENT: | ||||
raise | ||||
return None | ||||
try: | ||||
Mads Kiilerich
|
r17425 | # No readline() in osutil.posixfile, reading everything is cheap | ||
David Soria Parra
|
r13381 | mark = encoding.tolocal((file.readlines() or [''])[0]) | ||
Benoit Boissinot
|
r13627 | if mark == '' or mark not in repo._bookmarks: | ||
Matt Mackall
|
r13351 | mark = None | ||
Benoit Boissinot
|
r14027 | finally: | ||
Matt Mackall
|
r13351 | file.close() | ||
return mark | ||||
Matt Mackall
|
r13350 | def setcurrent(repo, mark): | ||
'''Set the name of the bookmark that we are currently on | ||||
Set the name of the bookmark that we are on (hg update <bookmark>). | ||||
The name is recorded in .hg/bookmarks.current | ||||
''' | ||||
Siddharth Agarwal
|
r20100 | if mark not in repo._bookmarks: | ||
raise AssertionError('bookmark %s does not exist!' % mark) | ||||
Matt Mackall
|
r13350 | current = repo._bookmarkcurrent | ||
if current == mark: | ||||
return | ||||
Matt Mackall
|
r15908 | wlock = repo.wlock() | ||
Matt Mackall
|
r13350 | try: | ||
Angel Ezquerra
|
r23877 | file = repo.vfs('bookmarks.current', 'w', atomictemp=True) | ||
LUO Zheng
|
r14559 | file.write(encoding.fromlocal(mark)) | ||
Greg Ward
|
r15057 | file.close() | ||
Matt Mackall
|
r13350 | finally: | ||
Matt Mackall
|
r15908 | wlock.release() | ||
Matt Mackall
|
r13350 | repo._bookmarkcurrent = mark | ||
Matt Mackall
|
r13352 | |||
Idan Kamara
|
r16191 | def unsetcurrent(repo): | ||
wlock = repo.wlock() | ||||
try: | ||||
Gilles Moris
|
r16194 | try: | ||
FUJIWARA Katsunori
|
r19895 | repo.vfs.unlink('bookmarks.current') | ||
Gilles Moris
|
r16194 | repo._bookmarkcurrent = None | ||
except OSError, inst: | ||||
if inst.errno != errno.ENOENT: | ||||
raise | ||||
Idan Kamara
|
r16191 | finally: | ||
wlock.release() | ||||
Kevin Bullock
|
r18471 | def iscurrent(repo, mark=None, parents=None): | ||
'''Tell whether the current bookmark is also active | ||||
I.e., the bookmark listed in .hg/bookmarks.current also points to a | ||||
parent of the working directory. | ||||
''' | ||||
if not mark: | ||||
mark = repo._bookmarkcurrent | ||||
if not parents: | ||||
parents = [p.node() for p in repo[None].parents()] | ||||
marks = repo._bookmarks | ||||
return (mark in marks and marks[mark] in parents) | ||||
David Soria Parra
|
r13663 | def updatecurrentbookmark(repo, oldnode, curbranch): | ||
try: | ||||
Brodie Rao
|
r16719 | return update(repo, oldnode, repo.branchtip(curbranch)) | ||
except error.RepoLookupError: | ||||
David Soria Parra
|
r13663 | if curbranch == "default": # no default branch! | ||
Kevin Bullock
|
r15621 | return update(repo, oldnode, repo.lookup("tip")) | ||
David Soria Parra
|
r13663 | else: | ||
raise util.Abort(_("branch %s not found") % curbranch) | ||||
Siddharth Agarwal
|
r18513 | def deletedivergent(repo, deletefrom, bm): | ||
'''Delete divergent versions of bm on nodes in deletefrom. | ||||
Return True if at least one bookmark was deleted, False otherwise.''' | ||||
deleted = False | ||||
marks = repo._bookmarks | ||||
divergent = [b for b in marks if b.split('@', 1)[0] == bm.split('@', 1)[0]] | ||||
for mark in divergent: | ||||
Matt Mackall
|
r21843 | if mark == '@' or '@' not in mark: | ||
# can't be divergent by definition | ||||
continue | ||||
Siddharth Agarwal
|
r18513 | if mark and marks[mark] in deletefrom: | ||
if mark != bm: | ||||
del marks[mark] | ||||
deleted = True | ||||
return deleted | ||||
Kevin Bullock
|
r19523 | def calculateupdate(ui, repo, checkout): | ||
'''Return a tuple (targetrev, movemarkfrom) indicating the rev to | ||||
check out and where to move the active bookmark from, if needed.''' | ||||
movemarkfrom = None | ||||
if checkout is None: | ||||
curmark = repo._bookmarkcurrent | ||||
if iscurrent(repo): | ||||
movemarkfrom = repo['.'].node() | ||||
elif curmark: | ||||
ui.status(_("updating to active bookmark %s\n") % curmark) | ||||
checkout = curmark | ||||
return (checkout, movemarkfrom) | ||||
Matt Mackall
|
r13352 | def update(repo, parents, node): | ||
Sean Farley
|
r19110 | deletefrom = parents | ||
Matt Mackall
|
r13352 | marks = repo._bookmarks | ||
update = False | ||||
David Soria Parra
|
r16706 | cur = repo._bookmarkcurrent | ||
if not cur: | ||||
return False | ||||
Siddharth Agarwal
|
r18513 | if marks[cur] in parents: | ||
new = repo[node] | ||||
Sean Farley
|
r19110 | divs = [repo[b] for b in marks | ||
if b.split('@', 1)[0] == cur.split('@', 1)[0]] | ||||
anc = repo.changelog.ancestors([new.rev()]) | ||||
deletefrom = [b.node() for b in divs if b.rev() in anc or b == new] | ||||
Sean Farley
|
r20281 | if validdest(repo, repo[marks[cur]], new): | ||
Siddharth Agarwal
|
r18513 | marks[cur] = new.node() | ||
update = True | ||||
Sean Farley
|
r19110 | if deletedivergent(repo, deletefrom, cur): | ||
Siddharth Agarwal
|
r18513 | update = True | ||
Matt Mackall
|
r13352 | if update: | ||
Augie Fackler
|
r17922 | marks.write() | ||
Kevin Bullock
|
r15621 | return update | ||
Matt Mackall
|
r13353 | |||
def listbookmarks(repo): | ||||
# We may try to list bookmarks on a repo type that does not | ||||
# support it (e.g., statichttprepository). | ||||
Augie Fackler
|
r14946 | marks = getattr(repo, '_bookmarks', {}) | ||
Matt Mackall
|
r13353 | |||
d = {} | ||||
Kevin Bullock
|
r18496 | hasnode = repo.changelog.hasnode | ||
Augie Fackler
|
r14946 | for k, v in marks.iteritems(): | ||
Matt Mackall
|
r15613 | # don't expose local divergent bookmarks | ||
Kevin Bullock
|
r18496 | if hasnode(v) and ('@' not in k or k.endswith('@')): | ||
Matt Mackall
|
r15613 | d[k] = hex(v) | ||
Matt Mackall
|
r13353 | return d | ||
def pushbookmark(repo, key, old, new): | ||||
Pierre-Yves David
|
r22667 | w = l = tr = None | ||
Matt Mackall
|
r13353 | try: | ||
Pierre-Yves David
|
r22667 | w = repo.wlock() | ||
l = repo.lock() | ||||
tr = repo.transaction('bookmarks') | ||||
Matt Mackall
|
r13353 | marks = repo._bookmarks | ||
Durham Goode
|
r22364 | existing = hex(marks.get(key, '')) | ||
if existing != old and existing != new: | ||||
Matt Mackall
|
r13353 | return False | ||
if new == '': | ||||
del marks[key] | ||||
else: | ||||
if new not in repo: | ||||
return False | ||||
marks[key] = repo[new].node() | ||||
Pierre-Yves David
|
r22667 | marks.recordchange(tr) | ||
tr.close() | ||||
Matt Mackall
|
r13353 | return True | ||
finally: | ||||
Pierre-Yves David
|
r22667 | lockmod.release(tr, l, w) | ||
Matt Mackall
|
r13354 | |||
FUJIWARA Katsunori
|
r20024 | def compare(repo, srcmarks, dstmarks, | ||
srchex=None, dsthex=None, targets=None): | ||||
'''Compare bookmarks between srcmarks and dstmarks | ||||
This returns tuple "(addsrc, adddst, advsrc, advdst, diverge, | ||||
differ, invalid)", each are list of bookmarks below: | ||||
:addsrc: added on src side (removed on dst side, perhaps) | ||||
:adddst: added on dst side (removed on src side, perhaps) | ||||
:advsrc: advanced on src side | ||||
:advdst: advanced on dst side | ||||
:diverge: diverge | ||||
:differ: changed, but changeset referred on src is unknown on dst | ||||
:invalid: unknown on both side | ||||
Gregory Szorc
|
r23081 | :same: same on both side | ||
FUJIWARA Katsunori
|
r20024 | |||
Each elements of lists in result tuple is tuple "(bookmark name, | ||||
changeset ID on source side, changeset ID on destination | ||||
side)". Each changeset IDs are 40 hexadecimal digit string or | ||||
None. | ||||
Changeset IDs of tuples in "addsrc", "adddst", "differ" or | ||||
"invalid" list may be unknown for repo. | ||||
This function expects that "srcmarks" and "dstmarks" return | ||||
changeset ID in 40 hexadecimal digit string for specified | ||||
bookmark. If not so (e.g. bmstore "repo._bookmarks" returning | ||||
binary value), "srchex" or "dsthex" should be specified to convert | ||||
into such form. | ||||
If "targets" is specified, only bookmarks listed in it are | ||||
examined. | ||||
''' | ||||
if not srchex: | ||||
srchex = lambda x: x | ||||
if not dsthex: | ||||
dsthex = lambda x: x | ||||
if targets: | ||||
bset = set(targets) | ||||
else: | ||||
srcmarkset = set(srcmarks) | ||||
dstmarkset = set(dstmarks) | ||||
Gregory Szorc
|
r23081 | bset = srcmarkset | dstmarkset | ||
FUJIWARA Katsunori
|
r20024 | |||
Gregory Szorc
|
r23081 | results = ([], [], [], [], [], [], [], []) | ||
FUJIWARA Katsunori
|
r20024 | addsrc = results[0].append | ||
adddst = results[1].append | ||||
advsrc = results[2].append | ||||
advdst = results[3].append | ||||
diverge = results[4].append | ||||
differ = results[5].append | ||||
invalid = results[6].append | ||||
Gregory Szorc
|
r23081 | same = results[7].append | ||
FUJIWARA Katsunori
|
r20024 | |||
for b in sorted(bset): | ||||
if b not in srcmarks: | ||||
if b in dstmarks: | ||||
adddst((b, None, dsthex(dstmarks[b]))) | ||||
else: | ||||
invalid((b, None, None)) | ||||
elif b not in dstmarks: | ||||
addsrc((b, srchex(srcmarks[b]), None)) | ||||
else: | ||||
scid = srchex(srcmarks[b]) | ||||
dcid = dsthex(dstmarks[b]) | ||||
Gregory Szorc
|
r23081 | if scid == dcid: | ||
same((b, scid, dcid)) | ||||
elif scid in repo and dcid in repo: | ||||
FUJIWARA Katsunori
|
r20024 | sctx = repo[scid] | ||
dctx = repo[dcid] | ||||
if sctx.rev() < dctx.rev(): | ||||
if validdest(repo, sctx, dctx): | ||||
advdst((b, scid, dcid)) | ||||
else: | ||||
diverge((b, scid, dcid)) | ||||
else: | ||||
if validdest(repo, dctx, sctx): | ||||
advsrc((b, scid, dcid)) | ||||
else: | ||||
diverge((b, scid, dcid)) | ||||
else: | ||||
# it is too expensive to examine in detail, in this case | ||||
differ((b, scid, dcid)) | ||||
return results | ||||
FUJIWARA Katsunori
|
r24355 | def _diverge(ui, b, path, localmarks, remotenode): | ||
FUJIWARA Katsunori
|
r24353 | '''Return appropriate diverged bookmark for specified ``path`` | ||
This returns None, if it is failed to assign any divergent | ||||
bookmark name. | ||||
FUJIWARA Katsunori
|
r24355 | |||
This reuses already existing one with "@number" suffix, if it | ||||
refers ``remotenode``. | ||||
FUJIWARA Katsunori
|
r24353 | ''' | ||
FUJIWARA Katsunori
|
r20025 | if b == '@': | ||
b = '' | ||||
# try to use an @pathalias suffix | ||||
# if an @pathalias already exists, we overwrite (update) it | ||||
Matt Mackall
|
r22629 | if path.startswith("file:"): | ||
path = util.url(path).path | ||||
FUJIWARA Katsunori
|
r20025 | for p, u in ui.configitems("paths"): | ||
Matt Mackall
|
r22629 | if u.startswith("file:"): | ||
u = util.url(u).path | ||||
FUJIWARA Katsunori
|
r20025 | if path == u: | ||
FUJIWARA Katsunori
|
r24354 | return '%s@%s' % (b, p) | ||
# assign a unique "@number" suffix newly | ||||
for x in range(1, 100): | ||||
n = '%s@%d' % (b, x) | ||||
FUJIWARA Katsunori
|
r24355 | if n not in localmarks or localmarks[n] == remotenode: | ||
FUJIWARA Katsunori
|
r24354 | return n | ||
return None | ||||
FUJIWARA Katsunori
|
r20025 | |||
Pierre-Yves David
|
r22666 | def updatefromremote(ui, repo, remotemarks, path, trfunc, explicit=()): | ||
David Soria Parra
|
r13646 | ui.debug("checking for updated bookmarks\n") | ||
Augie Fackler
|
r17922 | localmarks = repo._bookmarks | ||
Gregory Szorc
|
r23081 | (addsrc, adddst, advsrc, advdst, diverge, differ, invalid, same | ||
FUJIWARA Katsunori
|
r20025 | ) = compare(repo, remotemarks, localmarks, dsthex=hex) | ||
Matt Mackall
|
r15614 | |||
Pierre-Yves David
|
r22644 | status = ui.status | ||
warn = ui.warn | ||||
if ui.configbool('ui', 'quietbookmarkmove', False): | ||||
status = warn = ui.debug | ||||
Pierre-Yves David
|
r22659 | explicit = set(explicit) | ||
FUJIWARA Katsunori
|
r20025 | changed = [] | ||
for b, scid, dcid in addsrc: | ||||
if scid in repo: # add remote bookmarks for changes we already have | ||||
Pierre-Yves David
|
r22644 | changed.append((b, bin(scid), status, | ||
FUJIWARA Katsunori
|
r20025 | _("adding remote bookmark %s\n") % (b))) | ||
for b, scid, dcid in advsrc: | ||||
Pierre-Yves David
|
r22644 | changed.append((b, bin(scid), status, | ||
FUJIWARA Katsunori
|
r20025 | _("updating bookmark %s\n") % (b))) | ||
Pierre-Yves David
|
r22659 | # remove normal movement from explicit set | ||
explicit.difference_update(d[0] for d in changed) | ||||
FUJIWARA Katsunori
|
r20025 | for b, scid, dcid in diverge: | ||
Pierre-Yves David
|
r22659 | if b in explicit: | ||
explicit.discard(b) | ||||
changed.append((b, bin(scid), status, | ||||
Pierre-Yves David
|
r23199 | _("importing bookmark %s\n") % (b))) | ||
Pierre-Yves David
|
r22659 | else: | ||
FUJIWARA Katsunori
|
r24355 | snode = bin(scid) | ||
db = _diverge(ui, b, path, localmarks, snode) | ||||
FUJIWARA Katsunori
|
r24353 | if db: | ||
FUJIWARA Katsunori
|
r24355 | changed.append((db, snode, warn, | ||
FUJIWARA Katsunori
|
r24353 | _("divergent bookmark %s stored as %s\n") % | ||
(b, db))) | ||||
else: | ||||
warn(_("warning: failed to assign numbered name " | ||||
"to divergent bookmark %s\n") % (b)) | ||||
Pierre-Yves David
|
r22659 | for b, scid, dcid in adddst + advdst: | ||
if b in explicit: | ||||
explicit.discard(b) | ||||
changed.append((b, bin(scid), status, | ||||
Pierre-Yves David
|
r23199 | _("importing bookmark %s\n") % (b))) | ||
Pierre-Yves David
|
r22659 | |||
David Soria Parra
|
r13646 | if changed: | ||
Pierre-Yves David
|
r22666 | tr = trfunc() | ||
FUJIWARA Katsunori
|
r20025 | for b, node, writer, msg in sorted(changed): | ||
localmarks[b] = node | ||||
writer(msg) | ||||
Pierre-Yves David
|
r22666 | localmarks.recordchange(tr) | ||
David Soria Parra
|
r13646 | |||
FUJIWARA Katsunori
|
r24397 | def incoming(ui, repo, other): | ||
'''Show bookmarks incoming from other to repo | ||||
''' | ||||
ui.status(_("searching for changed bookmarks\n")) | ||||
r = compare(repo, other.listkeys('bookmarks'), repo._bookmarks, | ||||
dsthex=hex) | ||||
addsrc, adddst, advsrc, advdst, diverge, differ, invalid, same = r | ||||
incomings = [] | ||||
if ui.debugflag: | ||||
getid = lambda id: id | ||||
else: | ||||
getid = lambda id: id[:12] | ||||
def add(b, id): | ||||
incomings.append(" %-25s %s\n" % (b, getid(id))) | ||||
for b, scid, dcid in addsrc: | ||||
add(b, scid) | ||||
if not incomings: | ||||
ui.status(_("no changed bookmarks found\n")) | ||||
return 1 | ||||
for s in sorted(incomings): | ||||
ui.write(s) | ||||
return 0 | ||||
FUJIWARA Katsunori
|
r24398 | def outgoing(ui, repo, other): | ||
'''Show bookmarks outgoing from repo to other | ||||
''' | ||||
ui.status(_("searching for changed bookmarks\n")) | ||||
r = compare(repo, repo._bookmarks, other.listkeys('bookmarks'), | ||||
srchex=hex) | ||||
addsrc, adddst, advsrc, advdst, diverge, differ, invalid, same = r | ||||
outgoings = [] | ||||
if ui.debugflag: | ||||
getid = lambda id: id | ||||
else: | ||||
getid = lambda id: id[:12] | ||||
def add(b, id): | ||||
outgoings.append(" %-25s %s\n" % (b, getid(id))) | ||||
for b, scid, dcid in addsrc: | ||||
add(b, scid) | ||||
if not outgoings: | ||||
ui.status(_("no changed bookmarks found\n")) | ||||
return 1 | ||||
for s in sorted(outgoings): | ||||
ui.write(s) | ||||
return 0 | ||||
FUJIWARA Katsunori
|
r24400 | def summary(repo, other): | ||
'''Compare bookmarks between repo and other for "hg summary" output | ||||
This returns "(# of incoming, # of outgoing)" tuple. | ||||
''' | ||||
r = compare(repo, other.listkeys('bookmarks'), repo._bookmarks, | ||||
dsthex=hex) | ||||
addsrc, adddst, advsrc, advdst, diverge, differ, invalid, same = r | ||||
return (len(addsrc), len(adddst)) | ||||
Pierre-Yves David
|
r17550 | def validdest(repo, old, new): | ||
"""Is the new bookmark destination a valid update from the old one""" | ||||
Pierre-Yves David
|
r18008 | repo = repo.unfiltered() | ||
Pierre-Yves David
|
r17551 | if old == new: | ||
# Old == new -> nothing to update. | ||||
FUJIWARA Katsunori
|
r17625 | return False | ||
Pierre-Yves David
|
r17551 | elif not old: | ||
# old is nullrev, anything is valid. | ||||
# (new != nullrev has been excluded by the previous check) | ||||
FUJIWARA Katsunori
|
r17625 | return True | ||
Pierre-Yves David
|
r17551 | elif repo.obsstore: | ||
Pierre-Yves David
|
r18984 | return new.node() in obsolete.foreground(repo, [old.node()]) | ||
Pierre-Yves David
|
r17551 | else: | ||
Mads Kiilerich
|
r24180 | # still an independent clause as it is lazier (and therefore faster) | ||
FUJIWARA Katsunori
|
r17627 | return old.descendant(new) | ||