bzr branch
http://gegoxaren.bato24.eu/bzr/brz/remove-bazaar
| 485
by Martin Pool - move commit code into its own module | 1 | # Copyright (C) 2005 Canonical Ltd
 | 
| 1248
by Martin Pool - new weave based cleanup [broken] | 2 | #
 | 
| 485
by Martin Pool - move commit code into its own module | 3 | # This program is free software; you can redistribute it and/or modify
 | 
| 4 | # it under the terms of the GNU General Public License as published by
 | |
| 5 | # the Free Software Foundation; either version 2 of the License, or
 | |
| 6 | # (at your option) any later version.
 | |
| 1248
by Martin Pool - new weave based cleanup [broken] | 7 | #
 | 
| 485
by Martin Pool - move commit code into its own module | 8 | # This program is distributed in the hope that it will be useful,
 | 
| 9 | # but WITHOUT ANY WARRANTY; without even the implied warranty of
 | |
| 10 | # MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
 | |
| 11 | # GNU General Public License for more details.
 | |
| 1248
by Martin Pool - new weave based cleanup [broken] | 12 | #
 | 
| 485
by Martin Pool - move commit code into its own module | 13 | # You should have received a copy of the GNU General Public License
 | 
| 14 | # along with this program; if not, write to the Free Software
 | |
| 15 | # Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307  USA
 | |
| 16 | ||
| 17 | ||
| 1248
by Martin Pool - new weave based cleanup [broken] | 18 | # XXX: Can we do any better about making interrupted commits change
 | 
| 19 | # nothing?  Perhaps the best approach is to integrate commit of
 | |
| 20 | # AtomicFiles with releasing the lock on the Branch.
 | |
| 21 | ||
| 22 | # TODO: Separate 'prepare' phase where we find a list of potentially
 | |
| 23 | # committed files.  We then can then pause the commit to prompt for a
 | |
| 24 | # commit message, knowing the summary will be the same as what's
 | |
| 25 | # actually used for the commit.  (But perhaps simpler to simply get
 | |
| 26 | # the tree status, then use that for a selective commit?)
 | |
| 27 | ||
| 28 | # The newly committed revision is going to have a shape corresponding
 | |
| 29 | # to that of the working inventory.  Files that are not in the
 | |
| 30 | # working tree and that were in the predecessor are reported as
 | |
| 31 | # removed --- this can include files that were either removed from the
 | |
| 32 | # inventory or deleted in the working tree.  If they were only
 | |
| 33 | # deleted from disk, they are removed from the working inventory.
 | |
| 34 | ||
| 35 | # We then consider the remaining entries, which will be in the new
 | |
| 36 | # version.  Directory entries are simply copied across.  File entries
 | |
| 37 | # must be checked to see if a new version of the file should be
 | |
| 38 | # recorded.  For each parent revision inventory, we check to see what
 | |
| 39 | # version of the file was present.  If the file was present in at
 | |
| 40 | # least one tree, and if it was the same version in all the trees,
 | |
| 41 | # then we can just refer to that version.  Otherwise, a new version
 | |
| 42 | # representing the merger of the file versions must be added.
 | |
| 43 | ||
| 44 | # TODO: Update hashcache before and after - or does the WorkingTree
 | |
| 45 | # look after that?
 | |
| 1245
by Martin Pool doc | 46 | |
| 1339
by Martin Pool - doc | 47 | # TODO: Rather than mashing together the ancestry and storing it back,
 | 
| 48 | # perhaps the weave should have single method which does it all in one
 | |
| 49 | # go, avoiding a lot of redundant work.
 | |
| 1335
by Martin Pool doc | 50 | |
| 1341
by Martin Pool - doc | 51 | # TODO: Perhaps give a warning if one of the revisions marked as
 | 
| 52 | # merged is already in the ancestry, and then don't record it as a
 | |
| 53 | # distinct parent.
 | |
| 54 | ||
| 1343
by Martin Pool - fix up test for merge of trees | 55 | # TODO: If the file is newly merged but unchanged from the version it
 | 
| 56 | # merges from, then it should still be reported as newly added
 | |
| 57 | # relative to the basis revision.
 | |
| 58 | ||
| 1185.16.66
by mbp at sourcefrog - todo | 59 | # TODO: Do checks that the tree can be committed *before* running the 
 | 
| 60 | # editor; this should include checks for a pointless commit and for 
 | |
| 61 | # unknown or missing files.
 | |
| 62 | ||
| 63 | # TODO: If commit fails, leave the message in a file somewhere.
 | |
| 64 | ||
| 1194
by Martin Pool - [BROKEN] more progress of commit into weaves | 65 | |
| 1189
by Martin Pool - BROKEN: partial support for commit into weave | 66 | import os | 
| 1390
by Robert Collins pair programming worx... merge integration and weave | 67 | import re | 
| 1189
by Martin Pool - BROKEN: partial support for commit into weave | 68 | import sys | 
| 1188
by Martin Pool - clean up imports in commit code | 69 | import time | 
| 1248
by Martin Pool - new weave based cleanup [broken] | 70 | import pdb | 
| 1194
by Martin Pool - [BROKEN] more progress of commit into weaves | 71 | |
| 1188
by Martin Pool - clean up imports in commit code | 72 | from binascii import hexlify | 
| 1194
by Martin Pool - [BROKEN] more progress of commit into weaves | 73 | from cStringIO import StringIO | 
| 1188
by Martin Pool - clean up imports in commit code | 74 | |
| 1508.1.6
by Robert Collins Move Branch.unknowns() to WorkingTree. | 75 | from bzrlib.atomicfile import AtomicFile | 
| 1442.1.2
by Robert Collins create a config module - there is enough config logic to make this worthwhile, and start testing config processing. | 76 | from bzrlib.osutils import (local_time_offset, | 
| 77 | rand_bytes, compact_date, | |
| 1188
by Martin Pool - clean up imports in commit code | 78 | kind_marker, is_inside_any, quotefn, | 
| 1248
by Martin Pool - new weave based cleanup [broken] | 79 | sha_string, sha_strings, sha_file, isdir, isfile, | 
| 80 | split_lines) | |
| 1442.1.2
by Robert Collins create a config module - there is enough config logic to make this worthwhile, and start testing config processing. | 81 | import bzrlib.config | 
| 1264
by Martin Pool - Raise a better error from commit when a parent is absent | 82 | from bzrlib.errors import (BzrError, PointlessCommit, | 
| 83 | HistoryMissing, | |
| 1185.16.65
by mbp at sourcefrog - new commit --strict option | 84 | ConflictsInTree, | 
| 85 |                            StrictCommitFailed
 | |
| 1264
by Martin Pool - Raise a better error from commit when a parent is absent | 86 |                            )
 | 
| 1442.1.60
by Robert Collins gpg sign commits if the policy says we need to | 87 | import bzrlib.gpg as gpg | 
| 1311
by Martin Pool - remove RevisionReference; just hold parent ids directly | 88 | from bzrlib.revision import Revision | 
| 1442.1.62
by Robert Collins Allow creation of testaments from uncommitted data, and use that to get signatures before committing revisions. | 89 | from bzrlib.testament import Testament | 
| 1264
by Martin Pool - Raise a better error from commit when a parent is absent | 90 | from bzrlib.trace import mutter, note, warning | 
| 1189
by Martin Pool - BROKEN: partial support for commit into weave | 91 | from bzrlib.xml5 import serializer_v5 | 
| 1416
by Robert Collins when committing a specific file, include all its parents | 92 | from bzrlib.inventory import Inventory, ROOT_ID | 
| 1189
by Martin Pool - BROKEN: partial support for commit into weave | 93 | from bzrlib.weave import Weave | 
| 94 | from bzrlib.weavefile import read_weave, write_weave_v5 | |
| 1508.1.6
by Robert Collins Move Branch.unknowns() to WorkingTree. | 95 | from bzrlib.workingtree import WorkingTree | 
| 1189
by Martin Pool - BROKEN: partial support for commit into weave | 96 | |
| 97 | ||
| 1205
by Martin Pool - add bzrlib.commit.commit compatability interface | 98 | def commit(*args, **kwargs): | 
| 99 | """Commit a new revision to a branch. | |
| 100 | ||
| 101 |     Function-style interface for convenience of old callers.
 | |
| 102 | ||
| 103 |     New code should use the Commit class instead.
 | |
| 104 |     """
 | |
| 1276
by Martin Pool - make Branch.commit accept (and ignore) verbose argument | 105 |     ## XXX: Remove this in favor of Branch.commit?
 | 
| 1205
by Martin Pool - add bzrlib.commit.commit compatability interface | 106 | Commit().commit(*args, **kwargs) | 
| 107 | ||
| 108 | ||
| 1189
by Martin Pool - BROKEN: partial support for commit into weave | 109 | class NullCommitReporter(object): | 
| 110 | """I report on progress of a commit.""" | |
| 1185.12.8
by Aaron Bentley Fixed commit so all output comes though CommitReporter | 111 | |
| 112 | def snapshot_change(self, change, path): | |
| 113 |         pass
 | |
| 114 | ||
| 115 | def completed(self, revno, rev_id): | |
| 116 |         pass
 | |
| 117 | ||
| 118 | def deleted(self, file_id): | |
| 119 |         pass
 | |
| 120 | ||
| 121 | def escaped(self, escape_count, message): | |
| 122 |         pass
 | |
| 123 | ||
| 124 | def missing(self, path): | |
| 125 |         pass
 | |
| 1189
by Martin Pool - BROKEN: partial support for commit into weave | 126 | |
| 1508.1.6
by Robert Collins Move Branch.unknowns() to WorkingTree. | 127 | |
| 1189
by Martin Pool - BROKEN: partial support for commit into weave | 128 | class ReportCommitToLog(NullCommitReporter): | 
| 1185.12.8
by Aaron Bentley Fixed commit so all output comes though CommitReporter | 129 | |
| 130 | def snapshot_change(self, change, path): | |
| 131 | note("%s %s", change, path) | |
| 132 | ||
| 133 | def completed(self, revno, rev_id): | |
| 134 | note('committed r%d {%s}', revno, rev_id) | |
| 135 | ||
| 136 | def deleted(self, file_id): | |
| 137 | note('deleted %s', file_id) | |
| 138 | ||
| 139 | def escaped(self, escape_count, message): | |
| 140 | note("replaced %d control characters in message", escape_count) | |
| 141 | ||
| 142 | def missing(self, path): | |
| 143 | note('missing %s', path) | |
| 1189
by Martin Pool - BROKEN: partial support for commit into weave | 144 | |
| 1508.1.6
by Robert Collins Move Branch.unknowns() to WorkingTree. | 145 | |
| 1189
by Martin Pool - BROKEN: partial support for commit into weave | 146 | class Commit(object): | 
| 147 | """Task of committing a new revision. | |
| 148 | ||
| 149 |     This is a MethodObject: it accumulates state as the commit is
 | |
| 150 |     prepared, and then it is discarded.  It doesn't represent
 | |
| 151 |     historical revisions, just the act of recording a new one.
 | |
| 152 | ||
| 153 |             missing_ids
 | |
| 154 |             Modified to hold a list of files that have been deleted from
 | |
| 155 |             the working directory; these should be removed from the
 | |
| 156 |             working inventory.
 | |
| 485
by Martin Pool - move commit code into its own module | 157 |     """
 | 
| 1189
by Martin Pool - BROKEN: partial support for commit into weave | 158 | def __init__(self, | 
| 1442.1.60
by Robert Collins gpg sign commits if the policy says we need to | 159 | reporter=None, | 
| 160 | config=None): | |
| 1189
by Martin Pool - BROKEN: partial support for commit into weave | 161 | if reporter is not None: | 
| 162 | self.reporter = reporter | |
| 163 | else: | |
| 164 | self.reporter = NullCommitReporter() | |
| 1442.1.60
by Robert Collins gpg sign commits if the policy says we need to | 165 | if config is not None: | 
| 166 | self.config = config | |
| 167 | else: | |
| 168 | self.config = None | |
| 1189
by Martin Pool - BROKEN: partial support for commit into weave | 169 | |
| 170 | def commit(self, | |
| 171 | branch, message, | |
| 172 | timestamp=None, | |
| 173 | timezone=None, | |
| 174 | committer=None, | |
| 175 | specific_files=None, | |
| 176 | rev_id=None, | |
| 1276
by Martin Pool - make Branch.commit accept (and ignore) verbose argument | 177 | allow_pointless=True, | 
| 1185.16.65
by mbp at sourcefrog - new commit --strict option | 178 | strict=False, | 
| 1185.16.35
by Martin Pool - stub for revision properties | 179 | verbose=False, | 
| 180 | revprops=None): | |
| 1189
by Martin Pool - BROKEN: partial support for commit into weave | 181 | """Commit working copy as a new revision. | 
| 182 | ||
| 183 |         timestamp -- if not None, seconds-since-epoch for a
 | |
| 184 |              postdated/predated commit.
 | |
| 185 | ||
| 1253
by Martin Pool - test that pointless commits are trapped | 186 |         specific_files -- If true, commit only those files.
 | 
| 1189
by Martin Pool - BROKEN: partial support for commit into weave | 187 | |
| 1253
by Martin Pool - test that pointless commits are trapped | 188 |         rev_id -- If set, use this as the new revision id.
 | 
| 1189
by Martin Pool - BROKEN: partial support for commit into weave | 189 |             Useful for test or import commands that need to tightly
 | 
| 190 |             control what revisions are assigned.  If you duplicate
 | |
| 191 |             a revision id that exists elsewhere it is your own fault.
 | |
| 192 |             If null (default), a time/random revision id is generated.
 | |
| 1253
by Martin Pool - test that pointless commits are trapped | 193 | |
| 194 |         allow_pointless -- If true (default), commit even if nothing
 | |
| 195 |             has changed and no merges are recorded.
 | |
| 1185.16.35
by Martin Pool - stub for revision properties | 196 | |
| 1185.16.65
by mbp at sourcefrog - new commit --strict option | 197 |         strict -- If true, don't allow a commit if the working tree
 | 
| 198 |             contains unknown files.
 | |
| 199 | ||
| 1185.16.35
by Martin Pool - stub for revision properties | 200 |         revprops -- Properties for new revision
 | 
| 1189
by Martin Pool - BROKEN: partial support for commit into weave | 201 |         """
 | 
| 1285
by Martin Pool - fix bug in committing files that are renamed but not modified | 202 | mutter('preparing to commit') | 
| 1189
by Martin Pool - BROKEN: partial support for commit into weave | 203 | |
| 204 | self.branch = branch | |
| 1248
by Martin Pool - new weave based cleanup [broken] | 205 | self.weave_store = branch.weave_store | 
| 1189
by Martin Pool - BROKEN: partial support for commit into weave | 206 | self.rev_id = rev_id | 
| 207 | self.specific_files = specific_files | |
| 1194
by Martin Pool - [BROKEN] more progress of commit into weaves | 208 | self.allow_pointless = allow_pointless | 
| 1185.35.15
by Aaron Bentley Added branch nicks to revisions | 209 | self.revprops = {'branch-nick': branch.nick} | 
| 210 | if revprops: | |
| 211 | self.revprops.update(revprops) | |
| 1508.1.6
by Robert Collins Move Branch.unknowns() to WorkingTree. | 212 | self.work_tree = WorkingTree(branch.base, branch) | 
| 1189
by Martin Pool - BROKEN: partial support for commit into weave | 213 | |
| 1185.22.4
by Michael Ellerman Strict commit was a little .. ah .. too strict, oops :} | 214 | if strict: | 
| 215 |             # raise an exception as soon as we find a single unknown.
 | |
| 1508.1.6
by Robert Collins Move Branch.unknowns() to WorkingTree. | 216 | for unknown in self.work_tree.unknowns(): | 
| 1185.22.4
by Michael Ellerman Strict commit was a little .. ah .. too strict, oops :} | 217 | raise StrictCommitFailed() | 
| 1185.16.65
by mbp at sourcefrog - new commit --strict option | 218 | |
| 1189
by Martin Pool - BROKEN: partial support for commit into weave | 219 | if timestamp is None: | 
| 220 | self.timestamp = time.time() | |
| 221 | else: | |
| 222 | self.timestamp = long(timestamp) | |
| 223 | ||
| 1442.1.60
by Robert Collins gpg sign commits if the policy says we need to | 224 | if self.config is None: | 
| 225 | self.config = bzrlib.config.BranchConfig(self.branch) | |
| 226 | ||
| 1248
by Martin Pool - new weave based cleanup [broken] | 227 | if rev_id is None: | 
| 1442.1.60
by Robert Collins gpg sign commits if the policy says we need to | 228 | self.rev_id = _gen_revision_id(self.config, self.timestamp) | 
| 1248
by Martin Pool - new weave based cleanup [broken] | 229 | else: | 
| 230 | self.rev_id = rev_id | |
| 231 | ||
| 1189
by Martin Pool - BROKEN: partial support for commit into weave | 232 | if committer is None: | 
| 1442.1.60
by Robert Collins gpg sign commits if the policy says we need to | 233 | self.committer = self.config.username() | 
| 1189
by Martin Pool - BROKEN: partial support for commit into weave | 234 | else: | 
| 235 | assert isinstance(committer, basestring), type(committer) | |
| 236 | self.committer = committer | |
| 237 | ||
| 238 | if timezone is None: | |
| 239 | self.timezone = local_time_offset() | |
| 240 | else: | |
| 241 | self.timezone = int(timezone) | |
| 242 | ||
| 1393.4.2
by Harald Meland Cleanup + better test of commit-msg control character escape code. | 243 | if isinstance(message, str): | 
| 244 | message = message.decode(bzrlib.user_encoding) | |
| 245 | assert isinstance(message, unicode), type(message) | |
| 1189
by Martin Pool - BROKEN: partial support for commit into weave | 246 | self.message = message | 
| 1390
by Robert Collins pair programming worx... merge integration and weave | 247 | self._escape_commit_message() | 
| 1189
by Martin Pool - BROKEN: partial support for commit into weave | 248 | |
| 1245
by Martin Pool doc | 249 | self.branch.lock_write() | 
| 1189
by Martin Pool - BROKEN: partial support for commit into weave | 250 | try: | 
| 251 | self.work_inv = self.work_tree.inventory | |
| 252 | self.basis_tree = self.branch.basis_tree() | |
| 253 | self.basis_inv = self.basis_tree.inventory | |
| 254 | ||
| 1223
by Martin Pool - store inventories in weave | 255 | self._gather_parents() | 
| 1344
by Martin Pool - disallow selected-file commit of merges | 256 | if len(self.parents) > 1 and self.specific_files: | 
| 257 | raise NotImplementedError('selected-file commit of merges is not supported yet') | |
| 1284
by Martin Pool - in commit, avoid reading basis inventory twice | 258 | self._check_parents_present() | 
| 259 | ||
| 1251
by Martin Pool - fix up commit in directory with some deleted files | 260 | self._remove_deleted() | 
| 1092.2.22
by Robert Collins text_version and name_version unification looking reasonable | 261 | self._populate_new_inv() | 
| 262 | self._store_snapshot() | |
| 1248
by Martin Pool - new weave based cleanup [broken] | 263 | self._report_deletes() | 
| 1245
by Martin Pool doc | 264 | |
| 1248
by Martin Pool - new weave based cleanup [broken] | 265 | if not (self.allow_pointless | 
| 1279
by Martin Pool - pointless commit of r1 no longer allowed | 266 | or len(self.parents) > 1 | 
| 1249
by Martin Pool - improvements to weave commit [broken] | 267 | or self.new_inv != self.basis_inv): | 
| 1189
by Martin Pool - BROKEN: partial support for commit into weave | 268 | raise PointlessCommit() | 
| 269 | ||
| 1185.14.10
by Aaron Bentley Commit aborts with conflicts in the tree. | 270 | if len(list(self.work_tree.iter_conflicts()))>0: | 
| 271 | raise ConflictsInTree | |
| 272 | ||
| 1189
by Martin Pool - BROKEN: partial support for commit into weave | 273 | self._record_inventory() | 
| 274 | self._make_revision() | |
| 1185.33.59
by Martin Pool [patch] keep a cached basis inventory (Johan Rydberg) | 275 | self.work_tree.set_pending_merges([]) | 
| 1194
by Martin Pool - [BROKEN] more progress of commit into weaves | 276 | self.branch.append_revision(self.rev_id) | 
| 1472
by Robert Collins post commit hook, first pass implementation | 277 | self.reporter.completed(self.branch.revno()+1, self.rev_id) | 
| 278 | if self.config.post_commit() is not None: | |
| 279 | hooks = self.config.post_commit().split(' ') | |
| 280 |                 # this would be nicer with twisted.python.reflect.namedAny
 | |
| 281 | for hook in hooks: | |
| 282 | result = eval(hook + '(branch, rev_id)', | |
| 283 | {'branch':self.branch, | |
| 284 | 'bzrlib':bzrlib, | |
| 285 | 'rev_id':self.rev_id}) | |
| 1189
by Martin Pool - BROKEN: partial support for commit into weave | 286 | finally: | 
| 287 | self.branch.unlock() | |
| 288 | ||
| 289 | def _record_inventory(self): | |
| 1223
by Martin Pool - store inventories in weave | 290 | """Store the inventory for the new revision.""" | 
| 1248
by Martin Pool - new weave based cleanup [broken] | 291 | inv_text = serializer_v5.write_inventory_to_string(self.new_inv) | 
| 292 | self.inv_sha1 = sha_string(inv_text) | |
| 1390
by Robert Collins pair programming worx... merge integration and weave | 293 | s = self.branch.control_weaves | 
| 1352
by Martin Pool - store control weaves in .bzr/, not mixed in with file weaves | 294 | s.add_text('inventory', self.rev_id, | 
| 1417.1.8
by Robert Collins use transactions in the weave store interface, which enables caching for log | 295 | split_lines(inv_text), self.present_parents, | 
| 296 | self.branch.get_transaction()) | |
| 1223
by Martin Pool - store inventories in weave | 297 | |
| 1390
by Robert Collins pair programming worx... merge integration and weave | 298 | def _escape_commit_message(self): | 
| 299 | """Replace xml-incompatible control characters.""" | |
| 1185.11.5
by John Arbash Meinel Merged up-to-date against mainline, still broken. | 300 |         # Python strings can include characters that can't be
 | 
| 301 |         # represented in well-formed XML; escape characters that
 | |
| 302 |         # aren't listed in the XML specification
 | |
| 303 |         # (http://www.w3.org/TR/REC-xml/#NT-Char).
 | |
| 1390
by Robert Collins pair programming worx... merge integration and weave | 304 | self.message, escape_count = re.subn( | 
| 1393.4.2
by Harald Meland Cleanup + better test of commit-msg control character escape code. | 305 | u'[^\x09\x0A\x0D\u0020-\uD7FF\uE000-\uFFFD]+', | 
| 1185.11.5
by John Arbash Meinel Merged up-to-date against mainline, still broken. | 306 | lambda match: match.group(0).encode('unicode_escape'), | 
| 1390
by Robert Collins pair programming worx... merge integration and weave | 307 | self.message) | 
| 1185.11.5
by John Arbash Meinel Merged up-to-date against mainline, still broken. | 308 | if escape_count: | 
| 1185.12.8
by Aaron Bentley Fixed commit so all output comes though CommitReporter | 309 | self.reporter.escaped(escape_count, self.message) | 
| 1223
by Martin Pool - store inventories in weave | 310 | |
| 311 | def _gather_parents(self): | |
| 1092.2.25
by Robert Collins support ghosts in commits | 312 | """Record the parents of a merge for merge detection.""" | 
| 1457.1.14
by Robert Collins Move pending_merges() to WorkingTree. | 313 | pending_merges = self.work_tree.pending_merges() | 
| 1223
by Martin Pool - store inventories in weave | 314 | self.parents = [] | 
| 1408
by Robert Collins we do not need revision_trees in commit, parent inventories are sufficient | 315 | self.parent_invs = [] | 
| 1092.2.25
by Robert Collins support ghosts in commits | 316 | self.present_parents = [] | 
| 1241
by Martin Pool - rename last_patch to last_revision | 317 | precursor_id = self.branch.last_revision() | 
| 1223
by Martin Pool - store inventories in weave | 318 | if precursor_id: | 
| 319 | self.parents.append(precursor_id) | |
| 320 | self.parents += pending_merges | |
| 1092.2.25
by Robert Collins support ghosts in commits | 321 | for revision in self.parents: | 
| 322 | if self.branch.has_revision(revision): | |
| 1408
by Robert Collins we do not need revision_trees in commit, parent inventories are sufficient | 323 | self.parent_invs.append(self.branch.get_inventory(revision)) | 
| 1092.2.25
by Robert Collins support ghosts in commits | 324 | self.present_parents.append(revision) | 
| 1284
by Martin Pool - in commit, avoid reading basis inventory twice | 325 | |
| 326 | def _check_parents_present(self): | |
| 1264
by Martin Pool - Raise a better error from commit when a parent is absent | 327 | for parent_id in self.parents: | 
| 1270
by Martin Pool - fix recording of merged ancestry lines | 328 | mutter('commit parent revision {%s}', parent_id) | 
| 1264
by Martin Pool - Raise a better error from commit when a parent is absent | 329 | if not self.branch.has_revision(parent_id): | 
| 1092.2.25
by Robert Collins support ghosts in commits | 330 | if parent_id == self.branch.last_revision(): | 
| 1185.12.9
by Aaron Bentley Removed crude language | 331 | warning("parent is missing %r", parent_id) | 
| 1092.2.25
by Robert Collins support ghosts in commits | 332 | raise HistoryMissing(self.branch, 'revision', parent_id) | 
| 333 | else: | |
| 334 | mutter("commit will ghost revision %r", parent_id) | |
| 1284
by Martin Pool - in commit, avoid reading basis inventory twice | 335 | |
| 1189
by Martin Pool - BROKEN: partial support for commit into weave | 336 | def _make_revision(self): | 
| 337 | """Record a new revision object for this commit.""" | |
| 338 | self.rev = Revision(timestamp=self.timestamp, | |
| 339 | timezone=self.timezone, | |
| 340 | committer=self.committer, | |
| 341 | message=self.message, | |
| 342 | inventory_sha1=self.inv_sha1, | |
| 1185.16.35
by Martin Pool - stub for revision properties | 343 | revision_id=self.rev_id, | 
| 344 | properties=self.revprops) | |
| 1313
by Martin Pool - rename to Revision.parent_ids to avoid confusion with old usage | 345 | self.rev.parent_ids = self.parents | 
| 1248
by Martin Pool - new weave based cleanup [broken] | 346 | rev_tmp = StringIO() | 
| 1189
by Martin Pool - BROKEN: partial support for commit into weave | 347 | serializer_v5.write_revision(self.rev, rev_tmp) | 
| 580
by Martin Pool - Use explicit lock methods on a branch, rather than doing it | 348 | rev_tmp.seek(0) | 
| 1442.1.62
by Robert Collins Allow creation of testaments from uncommitted data, and use that to get signatures before committing revisions. | 349 | if self.config.signature_needed(): | 
| 350 | plaintext = Testament(self.rev, self.new_inv).as_short_text() | |
| 351 | self.branch.store_revision_signature(gpg.GPGStrategy(self.config), | |
| 352 | plaintext, self.rev_id) | |
| 1393.2.3
by John Arbash Meinel Fixing typos, updating stores, getting tests to pass. | 353 | self.branch.revision_store.add(rev_tmp, self.rev_id) | 
| 1189
by Martin Pool - BROKEN: partial support for commit into weave | 354 | mutter('new revision_id is {%s}', self.rev_id) | 
| 355 | ||
| 356 | def _remove_deleted(self): | |
| 1248
by Martin Pool - new weave based cleanup [broken] | 357 | """Remove deleted files from the working inventories. | 
| 358 | ||
| 359 |         This is done prior to taking the working inventory as the
 | |
| 360 |         basis for the new committed inventory.
 | |
| 361 | ||
| 362 |         This returns true if any files
 | |
| 363 |         *that existed in the basis inventory* were deleted.
 | |
| 364 |         Files that were added and deleted
 | |
| 365 |         in the working copy don't matter.
 | |
| 366 |         """
 | |
| 1254
by Martin Pool - fix handling of selective commit with deleted files | 367 | specific = self.specific_files | 
| 368 | deleted_ids = [] | |
| 369 | for path, ie in self.work_inv.iter_entries(): | |
| 370 | if specific and not is_inside_any(specific, path): | |
| 371 |                 continue
 | |
| 372 | if not self.work_tree.has_filename(path): | |
| 1185.12.8
by Aaron Bentley Fixed commit so all output comes though CommitReporter | 373 | self.reporter.missing(path) | 
| 1185.12.5
by Aaron Bentley Fixed ordering issue in Commit._remove_deleted | 374 | deleted_ids.append((path, ie.file_id)) | 
| 1254
by Martin Pool - fix handling of selective commit with deleted files | 375 | if deleted_ids: | 
| 1185.12.5
by Aaron Bentley Fixed ordering issue in Commit._remove_deleted | 376 | deleted_ids.sort(reverse=True) | 
| 377 | for path, file_id in deleted_ids: | |
| 1248
by Martin Pool - new weave based cleanup [broken] | 378 | del self.work_inv[file_id] | 
| 1457.1.11
by Robert Collins Move _write_inventory to WorkingTree. | 379 | self.work_tree._write_inventory(self.work_inv) | 
| 1248
by Martin Pool - new weave based cleanup [broken] | 380 | |
| 1092.2.22
by Robert Collins text_version and name_version unification looking reasonable | 381 | def _store_snapshot(self): | 
| 382 | """Pass over inventory and record a snapshot. | |
| 1303
by Martin Pool - commit updates entry_version | 383 | |
| 1092.2.21
by Robert Collins convert name_version to revision in inventory entries | 384 |         Entries get a new revision when they are modified in 
 | 
| 385 |         any way, which includes a merge with a new set of
 | |
| 1411
by Robert Collins use weave ancestry to determine inventory entry previous heads, prevent propogating 'I did a merge' merges. | 386 |         parents that have the same entry. 
 | 
| 1092.2.21
by Robert Collins convert name_version to revision in inventory entries | 387 |         """
 | 
| 1303
by Martin Pool - commit updates entry_version | 388 |         # XXX: Need to think more here about when the user has
 | 
| 389 |         # made a specific decision on a particular value -- c.f.
 | |
| 390 |         # mark-merge.  
 | |
| 391 | for path, ie in self.new_inv.iter_entries(): | |
| 1411
by Robert Collins use weave ancestry to determine inventory entry previous heads, prevent propogating 'I did a merge' merges. | 392 | previous_entries = ie.find_previous_heads( | 
| 393 | self.parent_invs, | |
| 1185.33.88
by Martin Pool [patch] read only the table-of-contents of weaves to speed up commit | 394 | self.weave_store.get_weave_prelude_or_empty(ie.file_id, | 
| 1417.1.8
by Robert Collins use transactions in the weave store interface, which enables caching for log | 395 | self.branch.get_transaction())) | 
| 1092.2.23
by Robert Collins move inventory entry centric snapshot taking logic to inventory.py | 396 | if ie.revision is None: | 
| 397 | change = ie.snapshot(self.rev_id, path, previous_entries, | |
| 1417.1.8
by Robert Collins use transactions in the weave store interface, which enables caching for log | 398 | self.work_tree, self.weave_store, | 
| 399 | self.branch.get_transaction()) | |
| 1092.2.23
by Robert Collins move inventory entry centric snapshot taking logic to inventory.py | 400 | else: | 
| 401 | change = "unchanged" | |
| 1185.12.8
by Aaron Bentley Fixed commit so all output comes though CommitReporter | 402 | self.reporter.snapshot_change(change, path) | 
| 1092.2.22
by Robert Collins text_version and name_version unification looking reasonable | 403 | |
| 404 | def _populate_new_inv(self): | |
| 405 | """Build revision inventory. | |
| 406 | ||
| 407 |         This creates a new empty inventory. Depending on
 | |
| 408 |         which files are selected for commit, and what is present in the
 | |
| 409 |         current tree, the new inventory is populated. inventory entries 
 | |
| 410 |         which are candidates for modification have their revision set to
 | |
| 411 |         None; inventory entries that are carried over untouched have their
 | |
| 412 |         revision set to their prior value.
 | |
| 1301
by Martin Pool - more docs in commit code | 413 |         """
 | 
| 1092.2.22
by Robert Collins text_version and name_version unification looking reasonable | 414 | mutter("Selecting files for commit with filter %s", self.specific_files) | 
| 415 | self.new_inv = Inventory() | |
| 1249
by Martin Pool - improvements to weave commit [broken] | 416 | for path, new_ie in self.work_inv.iter_entries(): | 
| 417 | file_id = new_ie.file_id | |
| 418 | mutter('check %s {%s}', path, new_ie.file_id) | |
| 1248
by Martin Pool - new weave based cleanup [broken] | 419 | if self.specific_files: | 
| 420 | if not is_inside_any(self.specific_files, path): | |
| 1249
by Martin Pool - improvements to weave commit [broken] | 421 | mutter('%s not selected for commit', path) | 
| 1416
by Robert Collins when committing a specific file, include all its parents | 422 | self._carry_entry(file_id) | 
| 1249
by Martin Pool - improvements to weave commit [broken] | 423 |                     continue
 | 
| 1416
by Robert Collins when committing a specific file, include all its parents | 424 | else: | 
| 425 |                     # this is selected, ensure its parents are too.
 | |
| 426 | parent_id = new_ie.parent_id | |
| 427 | while parent_id != ROOT_ID: | |
| 428 | if not self.new_inv.has_id(parent_id): | |
| 429 | ie = self._select_entry(self.work_inv[parent_id]) | |
| 430 | mutter('%s selected for commit because of %s', | |
| 431 | self.new_inv.id2path(parent_id), path) | |
| 432 | ||
| 433 | ie = self.new_inv[parent_id] | |
| 434 | if ie.revision is not None: | |
| 435 | ie.revision = None | |
| 436 | mutter('%s selected for commit because of %s', | |
| 437 | self.new_inv.id2path(parent_id), path) | |
| 438 | parent_id = ie.parent_id | |
| 1092.2.22
by Robert Collins text_version and name_version unification looking reasonable | 439 | mutter('%s selected for commit', path) | 
| 1416
by Robert Collins when committing a specific file, include all its parents | 440 | self._select_entry(new_ie) | 
| 441 | ||
| 442 | def _select_entry(self, new_ie): | |
| 443 | """Make new_ie be considered for committing.""" | |
| 444 | ie = new_ie.copy() | |
| 445 | ie.revision = None | |
| 446 | self.new_inv.add(ie) | |
| 447 | return ie | |
| 448 | ||
| 449 | def _carry_entry(self, file_id): | |
| 1285
by Martin Pool - fix bug in committing files that are renamed but not modified | 450 | """Carry the file unchanged from the basis revision.""" | 
| 1249
by Martin Pool - improvements to weave commit [broken] | 451 | if self.basis_inv.has_id(file_id): | 
| 452 | self.new_inv.add(self.basis_inv[file_id].copy()) | |
| 1248
by Martin Pool - new weave based cleanup [broken] | 453 | |
| 454 | def _report_deletes(self): | |
| 455 | for file_id in self.basis_inv: | |
| 456 | if file_id not in self.new_inv: | |
| 1185.12.8
by Aaron Bentley Fixed commit so all output comes though CommitReporter | 457 | self.reporter.deleted(self.basis_inv.id2path(file_id)) | 
| 1248
by Martin Pool - new weave based cleanup [broken] | 458 | |
| 1442.1.6
by Robert Collins first stage major overhaul of configs, giving use BranchConfigs, LocationConfigs and GlobalConfigs | 459 | def _gen_revision_id(config, when): | 
| 1189
by Martin Pool - BROKEN: partial support for commit into weave | 460 | """Return new revision-id.""" | 
| 1442.1.6
by Robert Collins first stage major overhaul of configs, giving use BranchConfigs, LocationConfigs and GlobalConfigs | 461 | s = '%s-%s-' % (config.user_email(), compact_date(when)) | 
| 1189
by Martin Pool - BROKEN: partial support for commit into weave | 462 | s += hexlify(rand_bytes(8)) | 
| 463 | return s |