localrepo: fix unpushable repos when using bookmarks (
issue3317)
bookmarks is copied to journal.bookmarks differently from how dirstate is
copied to journal.dirstate. The different way is less robust, which can render
the repo unpushable by other users if the first pushing user aborts their
transaction.
The underlying cause is that the copyfile method attempts an unnecessary chmod,
which fails if the user is not the owner of the journal.bookmarks file.
This patch makes the bookmarks journaling more consistent with the rest of the
journaling, and will allow users to update lingering journal.bookmarks files
that they're not the owners of.
--- a/mercurial/localrepo.py Mon Mar 12 09:39:30 2012 +0100
+++ b/mercurial/localrepo.py Fri Mar 09 15:34:21 2012 -0500
@@ -772,11 +772,12 @@
self.opener.write("journal.desc",
"%d\n%s\n" % (len(self), desc))
- bkname = self.join('bookmarks')
- if os.path.exists(bkname):
- util.copyfile(bkname, self.join('journal.bookmarks'))
- else:
- self.opener.write('journal.bookmarks', '')
+ try:
+ bk = self.opener.read("bookmarks")
+ except IOError:
+ bk = ""
+ self.opener.write("journal.bookmarks", bk)
+
phasesname = self.sjoin('phaseroots')
if os.path.exists(phasesname):
util.copyfile(phasesname, self.sjoin('journal.phaseroots'))