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, | 
| 1570.1.2
by Robert Collins Import bzrtools' 'fix' command as 'bzr reconcile.' | 79 | sha_file, isdir, isfile, | 
| 1248
by Martin Pool - new weave based cleanup [broken] | 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 | 
| 1508.1.22
by Robert Collins implement out of date working tree checks in commit. | 82 | import bzrlib.errors as errors | 
| 1264
by Martin Pool - Raise a better error from commit when a parent is absent | 83 | from bzrlib.errors import (BzrError, PointlessCommit, | 
| 84 | HistoryMissing, | |
| 1185.16.65
by mbp at sourcefrog - new commit --strict option | 85 | ConflictsInTree, | 
| 86 |                            StrictCommitFailed
 | |
| 1264
by Martin Pool - Raise a better error from commit when a parent is absent | 87 |                            )
 | 
| 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 | 
| 1534.4.25
by Robert Collins Add a --transport parameter to the test suite to set the default transport to be used in the test suite. | 93 | from bzrlib.symbol_versioning import * | 
| 1508.1.6
by Robert Collins Move Branch.unknowns() to WorkingTree. | 94 | from bzrlib.workingtree import WorkingTree | 
| 1189
by Martin Pool - BROKEN: partial support for commit into weave | 95 | |
| 96 | ||
| 1534.4.25
by Robert Collins Add a --transport parameter to the test suite to set the default transport to be used in the test suite. | 97 | @deprecated_function(zero_seven) | 
| 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): | |
| 1616.1.4
by Martin Pool Verbose commit shouldn't talk about every unchanged file. | 131 | if change == 'unchanged': | 
| 132 |             return
 | |
| 1185.12.8
by Aaron Bentley Fixed commit so all output comes though CommitReporter | 133 | note("%s %s", change, path) | 
| 134 | ||
| 135 | def completed(self, revno, rev_id): | |
| 1607.1.5
by Robert Collins Make commit verbose mode work!. | 136 | note('Committed revision %d.', revno) | 
| 1185.12.8
by Aaron Bentley Fixed commit so all output comes though CommitReporter | 137 | |
| 138 | def deleted(self, file_id): | |
| 139 | note('deleted %s', file_id) | |
| 140 | ||
| 141 | def escaped(self, escape_count, message): | |
| 142 | note("replaced %d control characters in message", escape_count) | |
| 143 | ||
| 144 | def missing(self, path): | |
| 145 | note('missing %s', path) | |
| 1189
by Martin Pool - BROKEN: partial support for commit into weave | 146 | |
| 1508.1.6
by Robert Collins Move Branch.unknowns() to WorkingTree. | 147 | |
| 1189
by Martin Pool - BROKEN: partial support for commit into weave | 148 | class Commit(object): | 
| 149 | """Task of committing a new revision. | |
| 150 | ||
| 151 |     This is a MethodObject: it accumulates state as the commit is
 | |
| 152 |     prepared, and then it is discarded.  It doesn't represent
 | |
| 153 |     historical revisions, just the act of recording a new one.
 | |
| 154 | ||
| 155 |             missing_ids
 | |
| 156 |             Modified to hold a list of files that have been deleted from
 | |
| 157 |             the working directory; these should be removed from the
 | |
| 158 |             working inventory.
 | |
| 485
by Martin Pool - move commit code into its own module | 159 |     """
 | 
| 1189
by Martin Pool - BROKEN: partial support for commit into weave | 160 | def __init__(self, | 
| 1442.1.60
by Robert Collins gpg sign commits if the policy says we need to | 161 | reporter=None, | 
| 162 | config=None): | |
| 1189
by Martin Pool - BROKEN: partial support for commit into weave | 163 | if reporter is not None: | 
| 164 | self.reporter = reporter | |
| 165 | else: | |
| 166 | self.reporter = NullCommitReporter() | |
| 1442.1.60
by Robert Collins gpg sign commits if the policy says we need to | 167 | if config is not None: | 
| 168 | self.config = config | |
| 169 | else: | |
| 170 | self.config = None | |
| 1189
by Martin Pool - BROKEN: partial support for commit into weave | 171 | |
| 172 | def commit(self, | |
| 1534.4.32
by Robert Collins Rename deprecated_nonce to DEPRECATED_PARAMETER | 173 | branch=DEPRECATED_PARAMETER, message=None, | 
| 1189
by Martin Pool - BROKEN: partial support for commit into weave | 174 | timestamp=None, | 
| 175 | timezone=None, | |
| 176 | committer=None, | |
| 177 | specific_files=None, | |
| 178 | rev_id=None, | |
| 1276
by Martin Pool - make Branch.commit accept (and ignore) verbose argument | 179 | allow_pointless=True, | 
| 1185.16.65
by mbp at sourcefrog - new commit --strict option | 180 | strict=False, | 
| 1185.16.35
by Martin Pool - stub for revision properties | 181 | verbose=False, | 
| 1534.4.25
by Robert Collins Add a --transport parameter to the test suite to set the default transport to be used in the test suite. | 182 | revprops=None, | 
| 1587.1.8
by Robert Collins Local commits on unbound branches fail. | 183 | working_tree=None, | 
| 1607.1.5
by Robert Collins Make commit verbose mode work!. | 184 | local=False, | 
| 185 | reporter=None, | |
| 186 | config=None): | |
| 1189
by Martin Pool - BROKEN: partial support for commit into weave | 187 | """Commit working copy as a new revision. | 
| 188 | ||
| 1534.4.25
by Robert Collins Add a --transport parameter to the test suite to set the default transport to be used in the test suite. | 189 |         branch -- the deprecated branch to commit to. New callers should pass in 
 | 
| 190 |                   working_tree instead
 | |
| 191 | ||
| 192 |         message -- the commit message, a mandatory parameter
 | |
| 193 | ||
| 1189
by Martin Pool - BROKEN: partial support for commit into weave | 194 |         timestamp -- if not None, seconds-since-epoch for a
 | 
| 195 |              postdated/predated commit.
 | |
| 196 | ||
| 1253
by Martin Pool - test that pointless commits are trapped | 197 |         specific_files -- If true, commit only those files.
 | 
| 1189
by Martin Pool - BROKEN: partial support for commit into weave | 198 | |
| 1253
by Martin Pool - test that pointless commits are trapped | 199 |         rev_id -- If set, use this as the new revision id.
 | 
| 1189
by Martin Pool - BROKEN: partial support for commit into weave | 200 |             Useful for test or import commands that need to tightly
 | 
| 201 |             control what revisions are assigned.  If you duplicate
 | |
| 202 |             a revision id that exists elsewhere it is your own fault.
 | |
| 203 |             If null (default), a time/random revision id is generated.
 | |
| 1253
by Martin Pool - test that pointless commits are trapped | 204 | |
| 205 |         allow_pointless -- If true (default), commit even if nothing
 | |
| 206 |             has changed and no merges are recorded.
 | |
| 1185.16.35
by Martin Pool - stub for revision properties | 207 | |
| 1185.16.65
by mbp at sourcefrog - new commit --strict option | 208 |         strict -- If true, don't allow a commit if the working tree
 | 
| 209 |             contains unknown files.
 | |
| 210 | ||
| 1185.16.35
by Martin Pool - stub for revision properties | 211 |         revprops -- Properties for new revision
 | 
| 1587.1.8
by Robert Collins Local commits on unbound branches fail. | 212 |         :param local: Perform a local only commit.
 | 
| 1189
by Martin Pool - BROKEN: partial support for commit into weave | 213 |         """
 | 
| 1285
by Martin Pool - fix bug in committing files that are renamed but not modified | 214 | mutter('preparing to commit') | 
| 1189
by Martin Pool - BROKEN: partial support for commit into weave | 215 | |
| 1534.4.25
by Robert Collins Add a --transport parameter to the test suite to set the default transport to be used in the test suite. | 216 | if deprecated_passed(branch): | 
| 217 | warn("Commit.commit (branch, ...): The branch parameter is " | |
| 218 | "deprecated as of bzr 0.8. Please use working_tree= instead.", | |
| 1534.4.26
by Robert Collins Move working tree initialisation out from Branch.initialize, deprecated Branch.initialize to Branch.create. | 219 | DeprecationWarning, stacklevel=2) | 
| 1534.4.25
by Robert Collins Add a --transport parameter to the test suite to set the default transport to be used in the test suite. | 220 | self.branch = branch | 
| 1508.1.19
by Robert Collins Give format3 working trees their own last-revision marker. | 221 | self.work_tree = self.branch.bzrdir.open_workingtree() | 
| 1534.4.25
by Robert Collins Add a --transport parameter to the test suite to set the default transport to be used in the test suite. | 222 | elif working_tree is None: | 
| 223 | raise BzrError("One of branch and working_tree must be passed into commit().") | |
| 224 | else: | |
| 225 | self.work_tree = working_tree | |
| 226 | self.branch = self.work_tree.branch | |
| 227 | if message is None: | |
| 228 | raise BzrError("The message keyword parameter is required for commit().") | |
| 229 | ||
| 1534.4.28
by Robert Collins first cut at merge from integration. | 230 | self.weave_store = self.branch.repository.weave_store | 
| 1505.1.24
by John Arbash Meinel Updated commit to handle bound branches. Included test to handle commit after merge | 231 | self.bound_branch = None | 
| 1587.1.9
by Robert Collins Local commits do no alter or access the master branch. | 232 | self.local = local | 
| 1505.1.24
by John Arbash Meinel Updated commit to handle bound branches. Included test to handle commit after merge | 233 | self.master_branch = None | 
| 1614.1.1
by Aaron Bentley Fixed master locking in commit | 234 | self.master_locked = False | 
| 1189
by Martin Pool - BROKEN: partial support for commit into weave | 235 | self.rev_id = rev_id | 
| 236 | self.specific_files = specific_files | |
| 1194
by Martin Pool - [BROKEN] more progress of commit into weaves | 237 | self.allow_pointless = allow_pointless | 
| 1593.1.1
by Robert Collins Move responsibility for setting branch nickname in commits to the WorkingTree convenience function. | 238 | self.revprops = {} | 
| 239 | if revprops is not None: | |
| 1185.35.15
by Aaron Bentley Added branch nicks to revisions | 240 | self.revprops.update(revprops) | 
| 1189
by Martin Pool - BROKEN: partial support for commit into weave | 241 | |
| 1607.1.5
by Robert Collins Make commit verbose mode work!. | 242 | if reporter is None and self.reporter is None: | 
| 243 | self.reporter = NullCommitReporter() | |
| 1607.1.8
by Robert Collins Integrate latest bzr.dev. | 244 | elif reporter is not None: | 
| 1607.1.5
by Robert Collins Make commit verbose mode work!. | 245 | self.reporter = reporter | 
| 246 | ||
| 1587.1.6
by Robert Collins Update bound branch implementation to 0.8. | 247 | self.work_tree.lock_write() | 
| 1189
by Martin Pool - BROKEN: partial support for commit into weave | 248 | try: | 
| 1587.1.6
by Robert Collins Update bound branch implementation to 0.8. | 249 |             # setup the bound branch variables as needed.
 | 
| 1505.1.24
by John Arbash Meinel Updated commit to handle bound branches. Included test to handle commit after merge | 250 | self._check_bound_branch() | 
| 251 | ||
| 1587.1.6
by Robert Collins Update bound branch implementation to 0.8. | 252 |             # check for out of date working trees
 | 
| 253 |             # if we are bound, then self.branch is the master branch and this
 | |
| 254 |             # test is thus all we need.
 | |
| 1587.1.8
by Robert Collins Local commits on unbound branches fail. | 255 | if self.work_tree.last_revision() != self.master_branch.last_revision(): | 
| 1587.1.6
by Robert Collins Update bound branch implementation to 0.8. | 256 | raise errors.OutOfDateTree(self.work_tree) | 
| 257 | ||
| 258 | if strict: | |
| 259 |                 # raise an exception as soon as we find a single unknown.
 | |
| 260 | for unknown in self.work_tree.unknowns(): | |
| 261 | raise StrictCommitFailed() | |
| 262 | ||
| 263 | if timestamp is None: | |
| 264 | self.timestamp = time.time() | |
| 265 | else: | |
| 266 | self.timestamp = long(timestamp) | |
| 267 | ||
| 268 | if self.config is None: | |
| 269 | self.config = bzrlib.config.BranchConfig(self.branch) | |
| 270 | ||
| 271 | if rev_id is None: | |
| 272 | self.rev_id = _gen_revision_id(self.config, self.timestamp) | |
| 273 | else: | |
| 274 | self.rev_id = rev_id | |
| 275 | ||
| 276 | if committer is None: | |
| 277 | self.committer = self.config.username() | |
| 278 | else: | |
| 279 | assert isinstance(committer, basestring), type(committer) | |
| 280 | self.committer = committer | |
| 281 | ||
| 282 | if timezone is None: | |
| 283 | self.timezone = local_time_offset() | |
| 284 | else: | |
| 285 | self.timezone = int(timezone) | |
| 286 | ||
| 287 | if isinstance(message, str): | |
| 288 | message = message.decode(bzrlib.user_encoding) | |
| 289 | assert isinstance(message, unicode), type(message) | |
| 290 | self.message = message | |
| 291 | self._escape_commit_message() | |
| 292 | ||
| 1189
by Martin Pool - BROKEN: partial support for commit into weave | 293 | self.work_inv = self.work_tree.inventory | 
| 1534.4.35
by Robert Collins Give branch its own basis tree and last_revision methods; deprecated branch.working_tree() | 294 | self.basis_tree = self.work_tree.basis_tree() | 
| 1189
by Martin Pool - BROKEN: partial support for commit into weave | 295 | self.basis_inv = self.basis_tree.inventory | 
| 296 | ||
| 1223
by Martin Pool - store inventories in weave | 297 | self._gather_parents() | 
| 1344
by Martin Pool - disallow selected-file commit of merges | 298 | if len(self.parents) > 1 and self.specific_files: | 
| 299 | raise NotImplementedError('selected-file commit of merges is not supported yet') | |
| 1284
by Martin Pool - in commit, avoid reading basis inventory twice | 300 | self._check_parents_present() | 
| 301 | ||
| 1251
by Martin Pool - fix up commit in directory with some deleted files | 302 | self._remove_deleted() | 
| 1092.2.22
by Robert Collins text_version and name_version unification looking reasonable | 303 | self._populate_new_inv() | 
| 304 | self._store_snapshot() | |
| 1248
by Martin Pool - new weave based cleanup [broken] | 305 | self._report_deletes() | 
| 1245
by Martin Pool doc | 306 | |
| 1248
by Martin Pool - new weave based cleanup [broken] | 307 | if not (self.allow_pointless | 
| 1279
by Martin Pool - pointless commit of r1 no longer allowed | 308 | or len(self.parents) > 1 | 
| 1249
by Martin Pool - improvements to weave commit [broken] | 309 | or self.new_inv != self.basis_inv): | 
| 1189
by Martin Pool - BROKEN: partial support for commit into weave | 310 | raise PointlessCommit() | 
| 311 | ||
| 1534.10.22
by Aaron Bentley Got ConflictList implemented | 312 | if len(self.work_tree.conflicts())>0: | 
| 1185.14.10
by Aaron Bentley Commit aborts with conflicts in the tree. | 313 | raise ConflictsInTree | 
| 314 | ||
| 1570.1.2
by Robert Collins Import bzrtools' 'fix' command as 'bzr reconcile.' | 315 | self.inv_sha1 = self.branch.repository.add_inventory( | 
| 316 | self.rev_id, | |
| 317 | self.new_inv, | |
| 318 | self.present_parents | |
| 319 |                 )
 | |
| 1189
by Martin Pool - BROKEN: partial support for commit into weave | 320 | self._make_revision() | 
| 1587.1.7
by Robert Collins Commit revision data locally first for bound branch commits. | 321 |             # revision data is in the local branch now.
 | 
| 1587.1.6
by Robert Collins Update bound branch implementation to 0.8. | 322 | |
| 1587.1.7
by Robert Collins Commit revision data locally first for bound branch commits. | 323 |             # upload revision data to the master.
 | 
| 324 |             # this will propogate merged revisions too if needed.
 | |
| 325 | if self.bound_branch: | |
| 326 | self.master_branch.repository.fetch(self.branch.repository, | |
| 327 | revision_id=self.rev_id) | |
| 328 |                 # now the master has the revision data
 | |
| 329 |                 # 'commit' to the master first so a timeout here causes the local
 | |
| 330 |                 # branch to be out of date
 | |
| 331 | self.master_branch.append_revision(self.rev_id) | |
| 332 | ||
| 333 |             # and now do the commit locally.
 | |
| 1194
by Martin Pool - [BROKEN] more progress of commit into weaves | 334 | self.branch.append_revision(self.rev_id) | 
| 1587.1.6
by Robert Collins Update bound branch implementation to 0.8. | 335 | |
| 1185.33.59
by Martin Pool [patch] keep a cached basis inventory (Johan Rydberg) | 336 | self.work_tree.set_pending_merges([]) | 
| 1638.1.2
by Robert Collins Change the basis-inventory file to not have the revision-id in the file name. | 337 | self.work_tree.set_last_revision(self.rev_id) | 
| 1587.1.6
by Robert Collins Update bound branch implementation to 0.8. | 338 |             # now the work tree is up to date with the branch
 | 
| 339 | ||
| 1607.1.5
by Robert Collins Make commit verbose mode work!. | 340 | self.reporter.completed(self.branch.revno(), self.rev_id) | 
| 1472
by Robert Collins post commit hook, first pass implementation | 341 | if self.config.post_commit() is not None: | 
| 342 | hooks = self.config.post_commit().split(' ') | |
| 343 |                 # this would be nicer with twisted.python.reflect.namedAny
 | |
| 344 | for hook in hooks: | |
| 345 | result = eval(hook + '(branch, rev_id)', | |
| 346 | {'branch':self.branch, | |
| 347 | 'bzrlib':bzrlib, | |
| 348 | 'rev_id':self.rev_id}) | |
| 1189
by Martin Pool - BROKEN: partial support for commit into weave | 349 | finally: | 
| 1505.1.24
by John Arbash Meinel Updated commit to handle bound branches. Included test to handle commit after merge | 350 | self._cleanup_bound_branch() | 
| 1587.1.6
by Robert Collins Update bound branch implementation to 0.8. | 351 | self.work_tree.unlock() | 
| 1189
by Martin Pool - BROKEN: partial support for commit into weave | 352 | |
| 1505.1.24
by John Arbash Meinel Updated commit to handle bound branches. Included test to handle commit after merge | 353 | def _check_bound_branch(self): | 
| 354 | """Check to see if the local branch is bound. | |
| 355 | ||
| 356 |         If it is bound, then most of the commit will actually be
 | |
| 357 |         done using the remote branch as the target branch.
 | |
| 358 |         Only at the end will the local branch be updated.
 | |
| 359 |         """
 | |
| 1587.1.9
by Robert Collins Local commits do no alter or access the master branch. | 360 | if self.local and not self.branch.get_bound_location(): | 
| 361 | raise errors.LocalRequiresBoundBranch() | |
| 362 | ||
| 363 | if not self.local: | |
| 364 | self.master_branch = self.branch.get_master_branch() | |
| 365 | ||
| 1505.1.24
by John Arbash Meinel Updated commit to handle bound branches. Included test to handle commit after merge | 366 | if not self.master_branch: | 
| 1587.1.8
by Robert Collins Local commits on unbound branches fail. | 367 |             # make this branch the reference branch for out of date checks.
 | 
| 368 | self.master_branch = self.branch | |
| 1505.1.24
by John Arbash Meinel Updated commit to handle bound branches. Included test to handle commit after merge | 369 |             return
 | 
| 370 | ||
| 371 |         # If the master branch is bound, we must fail
 | |
| 372 | master_bound_location = self.master_branch.get_bound_location() | |
| 373 | if master_bound_location: | |
| 1505.1.27
by John Arbash Meinel Adding tests against an sftp branch. | 374 | raise errors.CommitToDoubleBoundBranch(self.branch, | 
| 1505.1.24
by John Arbash Meinel Updated commit to handle bound branches. Included test to handle commit after merge | 375 | self.master_branch, master_bound_location) | 
| 376 | ||
| 377 |         # TODO: jam 20051230 We could automatically push local
 | |
| 378 |         #       commits to the remote branch if they would fit.
 | |
| 379 |         #       But for now, just require remote to be identical
 | |
| 380 |         #       to local.
 | |
| 381 | ||
| 382 |         # Make sure the local branch is identical to the master
 | |
| 383 | master_rh = self.master_branch.revision_history() | |
| 384 | local_rh = self.branch.revision_history() | |
| 385 | if local_rh != master_rh: | |
| 386 | raise errors.BoundBranchOutOfDate(self.branch, | |
| 387 | self.master_branch) | |
| 388 | ||
| 389 |         # Now things are ready to change the master branch
 | |
| 390 |         # so grab the lock
 | |
| 391 | self.bound_branch = self.branch | |
| 392 | self.master_branch.lock_write() | |
| 1614.1.1
by Aaron Bentley Fixed master locking in commit | 393 | self.master_locked = True | 
| 1587.1.7
by Robert Collins Commit revision data locally first for bound branch commits. | 394 | ####        
 | 
| 395 | ####        # Check to see if we have any pending merges. If we do
 | |
| 396 | ####        # those need to be pushed into the master branch
 | |
| 397 | ####        pending_merges = self.work_tree.pending_merges()
 | |
| 398 | ####        if pending_merges:
 | |
| 399 | ####            for revision_id in pending_merges:
 | |
| 400 | ####                self.master_branch.repository.fetch(self.bound_branch.repository,
 | |
| 401 | ####                                                    revision_id=revision_id)
 | |
| 1505.1.24
by John Arbash Meinel Updated commit to handle bound branches. Included test to handle commit after merge | 402 | |
| 403 | def _cleanup_bound_branch(self): | |
| 404 | """Executed at the end of a try/finally to cleanup a bound branch. | |
| 405 | ||
| 406 |         If the branch wasn't bound, this is a no-op.
 | |
| 407 |         If it was, it resents self.branch to the local branch, instead
 | |
| 408 |         of being the master.
 | |
| 409 |         """
 | |
| 410 | if not self.bound_branch: | |
| 411 |             return
 | |
| 1614.1.1
by Aaron Bentley Fixed master locking in commit | 412 | if self.master_locked: | 
| 413 | self.master_branch.unlock() | |
| 1505.1.24
by John Arbash Meinel Updated commit to handle bound branches. Included test to handle commit after merge | 414 | |
| 1390
by Robert Collins pair programming worx... merge integration and weave | 415 | def _escape_commit_message(self): | 
| 416 | """Replace xml-incompatible control characters.""" | |
| 1666.1.6
by Robert Collins Make knit the default format. | 417 |         # FIXME: RBC 20060419 this should be done by the revision
 | 
| 418 |         # serialiser not by commit. Then we can also add an unescaper
 | |
| 419 |         # in the deserializer and start roundtripping revision messages
 | |
| 420 |         # precisely. See repository_implementations/test_repository.py
 | |
| 421 | ||
| 1185.11.5
by John Arbash Meinel Merged up-to-date against mainline, still broken. | 422 |         # Python strings can include characters that can't be
 | 
| 423 |         # represented in well-formed XML; escape characters that
 | |
| 424 |         # aren't listed in the XML specification
 | |
| 425 |         # (http://www.w3.org/TR/REC-xml/#NT-Char).
 | |
| 1390
by Robert Collins pair programming worx... merge integration and weave | 426 | self.message, escape_count = re.subn( | 
| 1393.4.2
by Harald Meland Cleanup + better test of commit-msg control character escape code. | 427 | u'[^\x09\x0A\x0D\u0020-\uD7FF\uE000-\uFFFD]+', | 
| 1185.11.5
by John Arbash Meinel Merged up-to-date against mainline, still broken. | 428 | lambda match: match.group(0).encode('unicode_escape'), | 
| 1390
by Robert Collins pair programming worx... merge integration and weave | 429 | self.message) | 
| 1185.11.5
by John Arbash Meinel Merged up-to-date against mainline, still broken. | 430 | if escape_count: | 
| 1185.12.8
by Aaron Bentley Fixed commit so all output comes though CommitReporter | 431 | self.reporter.escaped(escape_count, self.message) | 
| 1223
by Martin Pool - store inventories in weave | 432 | |
| 433 | def _gather_parents(self): | |
| 1092.2.25
by Robert Collins support ghosts in commits | 434 | """Record the parents of a merge for merge detection.""" | 
| 1457.1.14
by Robert Collins Move pending_merges() to WorkingTree. | 435 | pending_merges = self.work_tree.pending_merges() | 
| 1223
by Martin Pool - store inventories in weave | 436 | self.parents = [] | 
| 1408
by Robert Collins we do not need revision_trees in commit, parent inventories are sufficient | 437 | self.parent_invs = [] | 
| 1092.2.25
by Robert Collins support ghosts in commits | 438 | self.present_parents = [] | 
| 1241
by Martin Pool - rename last_patch to last_revision | 439 | precursor_id = self.branch.last_revision() | 
| 1223
by Martin Pool - store inventories in weave | 440 | if precursor_id: | 
| 441 | self.parents.append(precursor_id) | |
| 442 | self.parents += pending_merges | |
| 1092.2.25
by Robert Collins support ghosts in commits | 443 | for revision in self.parents: | 
| 1185.67.2
by Aaron Bentley Renamed Branch.storage to Branch.repository | 444 | if self.branch.repository.has_revision(revision): | 
| 445 | inventory = self.branch.repository.get_inventory(revision) | |
| 1185.65.1
by Aaron Bentley Refactored out ControlFiles and RevisionStore from _Branch | 446 | self.parent_invs.append(inventory) | 
| 1092.2.25
by Robert Collins support ghosts in commits | 447 | self.present_parents.append(revision) | 
| 1284
by Martin Pool - in commit, avoid reading basis inventory twice | 448 | |
| 449 | def _check_parents_present(self): | |
| 1264
by Martin Pool - Raise a better error from commit when a parent is absent | 450 | for parent_id in self.parents: | 
| 1270
by Martin Pool - fix recording of merged ancestry lines | 451 | mutter('commit parent revision {%s}', parent_id) | 
| 1185.67.2
by Aaron Bentley Renamed Branch.storage to Branch.repository | 452 | if not self.branch.repository.has_revision(parent_id): | 
| 1092.2.25
by Robert Collins support ghosts in commits | 453 | if parent_id == self.branch.last_revision(): | 
| 1185.12.9
by Aaron Bentley Removed crude language | 454 | warning("parent is missing %r", parent_id) | 
| 1092.2.25
by Robert Collins support ghosts in commits | 455 | raise HistoryMissing(self.branch, 'revision', parent_id) | 
| 456 | else: | |
| 457 | mutter("commit will ghost revision %r", parent_id) | |
| 1284
by Martin Pool - in commit, avoid reading basis inventory twice | 458 | |
| 1189
by Martin Pool - BROKEN: partial support for commit into weave | 459 | def _make_revision(self): | 
| 460 | """Record a new revision object for this commit.""" | |
| 1570.1.2
by Robert Collins Import bzrtools' 'fix' command as 'bzr reconcile.' | 461 | rev = Revision(timestamp=self.timestamp, | 
| 462 | timezone=self.timezone, | |
| 463 | committer=self.committer, | |
| 464 | message=self.message, | |
| 465 | inventory_sha1=self.inv_sha1, | |
| 466 | revision_id=self.rev_id, | |
| 467 | properties=self.revprops) | |
| 468 | rev.parent_ids = self.parents | |
| 469 | self.branch.repository.add_revision(self.rev_id, rev, self.new_inv, self.config) | |
| 1189
by Martin Pool - BROKEN: partial support for commit into weave | 470 | |
| 471 | def _remove_deleted(self): | |
| 1248
by Martin Pool - new weave based cleanup [broken] | 472 | """Remove deleted files from the working inventories. | 
| 473 | ||
| 474 |         This is done prior to taking the working inventory as the
 | |
| 475 |         basis for the new committed inventory.
 | |
| 476 | ||
| 477 |         This returns true if any files
 | |
| 478 |         *that existed in the basis inventory* were deleted.
 | |
| 479 |         Files that were added and deleted
 | |
| 480 |         in the working copy don't matter.
 | |
| 481 |         """
 | |
| 1254
by Martin Pool - fix handling of selective commit with deleted files | 482 | specific = self.specific_files | 
| 483 | deleted_ids = [] | |
| 484 | for path, ie in self.work_inv.iter_entries(): | |
| 485 | if specific and not is_inside_any(specific, path): | |
| 486 |                 continue
 | |
| 487 | if not self.work_tree.has_filename(path): | |
| 1185.12.8
by Aaron Bentley Fixed commit so all output comes though CommitReporter | 488 | self.reporter.missing(path) | 
| 1185.12.5
by Aaron Bentley Fixed ordering issue in Commit._remove_deleted | 489 | deleted_ids.append((path, ie.file_id)) | 
| 1254
by Martin Pool - fix handling of selective commit with deleted files | 490 | if deleted_ids: | 
| 1185.12.5
by Aaron Bentley Fixed ordering issue in Commit._remove_deleted | 491 | deleted_ids.sort(reverse=True) | 
| 492 | for path, file_id in deleted_ids: | |
| 1248
by Martin Pool - new weave based cleanup [broken] | 493 | del self.work_inv[file_id] | 
| 1457.1.11
by Robert Collins Move _write_inventory to WorkingTree. | 494 | self.work_tree._write_inventory(self.work_inv) | 
| 1248
by Martin Pool - new weave based cleanup [broken] | 495 | |
| 1092.2.22
by Robert Collins text_version and name_version unification looking reasonable | 496 | def _store_snapshot(self): | 
| 497 | """Pass over inventory and record a snapshot. | |
| 1303
by Martin Pool - commit updates entry_version | 498 | |
| 1092.2.21
by Robert Collins convert name_version to revision in inventory entries | 499 |         Entries get a new revision when they are modified in 
 | 
| 500 |         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. | 501 |         parents that have the same entry. 
 | 
| 1092.2.21
by Robert Collins convert name_version to revision in inventory entries | 502 |         """
 | 
| 1303
by Martin Pool - commit updates entry_version | 503 |         # XXX: Need to think more here about when the user has
 | 
| 504 |         # made a specific decision on a particular value -- c.f.
 | |
| 505 |         # mark-merge.  
 | |
| 506 | 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. | 507 | previous_entries = ie.find_previous_heads( | 
| 1616.1.4
by Martin Pool Verbose commit shouldn't talk about every unchanged file. | 508 | self.parent_invs, | 
| 1596.2.20
by Robert Collins optimise commit to only access weaves for merged, or altered files during commit. | 509 | self.weave_store, | 
| 510 | self.branch.repository.get_transaction()) | |
| 1092.2.23
by Robert Collins move inventory entry centric snapshot taking logic to inventory.py | 511 | if ie.revision is None: | 
| 512 | 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 | 513 | self.work_tree, self.weave_store, | 
| 514 | self.branch.get_transaction()) | |
| 1092.2.23
by Robert Collins move inventory entry centric snapshot taking logic to inventory.py | 515 | else: | 
| 516 | change = "unchanged" | |
| 1185.12.8
by Aaron Bentley Fixed commit so all output comes though CommitReporter | 517 | self.reporter.snapshot_change(change, path) | 
| 1092.2.22
by Robert Collins text_version and name_version unification looking reasonable | 518 | |
| 519 | def _populate_new_inv(self): | |
| 520 | """Build revision inventory. | |
| 521 | ||
| 522 |         This creates a new empty inventory. Depending on
 | |
| 523 |         which files are selected for commit, and what is present in the
 | |
| 524 |         current tree, the new inventory is populated. inventory entries 
 | |
| 525 |         which are candidates for modification have their revision set to
 | |
| 526 |         None; inventory entries that are carried over untouched have their
 | |
| 527 |         revision set to their prior value.
 | |
| 1301
by Martin Pool - more docs in commit code | 528 |         """
 | 
| 1092.2.22
by Robert Collins text_version and name_version unification looking reasonable | 529 | mutter("Selecting files for commit with filter %s", self.specific_files) | 
| 1638.1.2
by Robert Collins Change the basis-inventory file to not have the revision-id in the file name. | 530 | self.new_inv = Inventory(revision_id=self.rev_id) | 
| 1249
by Martin Pool - improvements to weave commit [broken] | 531 | for path, new_ie in self.work_inv.iter_entries(): | 
| 532 | file_id = new_ie.file_id | |
| 533 | mutter('check %s {%s}', path, new_ie.file_id) | |
| 1248
by Martin Pool - new weave based cleanup [broken] | 534 | if self.specific_files: | 
| 535 | if not is_inside_any(self.specific_files, path): | |
| 1249
by Martin Pool - improvements to weave commit [broken] | 536 | mutter('%s not selected for commit', path) | 
| 1416
by Robert Collins when committing a specific file, include all its parents | 537 | self._carry_entry(file_id) | 
| 1249
by Martin Pool - improvements to weave commit [broken] | 538 |                     continue
 | 
| 1416
by Robert Collins when committing a specific file, include all its parents | 539 | else: | 
| 540 |                     # this is selected, ensure its parents are too.
 | |
| 541 | parent_id = new_ie.parent_id | |
| 542 | while parent_id != ROOT_ID: | |
| 543 | if not self.new_inv.has_id(parent_id): | |
| 544 | ie = self._select_entry(self.work_inv[parent_id]) | |
| 545 | mutter('%s selected for commit because of %s', | |
| 546 | self.new_inv.id2path(parent_id), path) | |
| 547 | ||
| 548 | ie = self.new_inv[parent_id] | |
| 549 | if ie.revision is not None: | |
| 550 | ie.revision = None | |
| 551 | mutter('%s selected for commit because of %s', | |
| 552 | self.new_inv.id2path(parent_id), path) | |
| 553 | parent_id = ie.parent_id | |
| 1092.2.22
by Robert Collins text_version and name_version unification looking reasonable | 554 | mutter('%s selected for commit', path) | 
| 1416
by Robert Collins when committing a specific file, include all its parents | 555 | self._select_entry(new_ie) | 
| 556 | ||
| 557 | def _select_entry(self, new_ie): | |
| 558 | """Make new_ie be considered for committing.""" | |
| 559 | ie = new_ie.copy() | |
| 560 | ie.revision = None | |
| 561 | self.new_inv.add(ie) | |
| 562 | return ie | |
| 563 | ||
| 564 | def _carry_entry(self, file_id): | |
| 1285
by Martin Pool - fix bug in committing files that are renamed but not modified | 565 | """Carry the file unchanged from the basis revision.""" | 
| 1249
by Martin Pool - improvements to weave commit [broken] | 566 | if self.basis_inv.has_id(file_id): | 
| 567 | self.new_inv.add(self.basis_inv[file_id].copy()) | |
| 1248
by Martin Pool - new weave based cleanup [broken] | 568 | |
| 569 | def _report_deletes(self): | |
| 570 | for file_id in self.basis_inv: | |
| 571 | if file_id not in self.new_inv: | |
| 1185.12.8
by Aaron Bentley Fixed commit so all output comes though CommitReporter | 572 | self.reporter.deleted(self.basis_inv.id2path(file_id)) | 
| 1248
by Martin Pool - new weave based cleanup [broken] | 573 | |
| 1442.1.6
by Robert Collins first stage major overhaul of configs, giving use BranchConfigs, LocationConfigs and GlobalConfigs | 574 | def _gen_revision_id(config, when): | 
| 1189
by Martin Pool - BROKEN: partial support for commit into weave | 575 | """Return new revision-id.""" | 
| 1442.1.6
by Robert Collins first stage major overhaul of configs, giving use BranchConfigs, LocationConfigs and GlobalConfigs | 576 | s = '%s-%s-' % (config.user_email(), compact_date(when)) | 
| 1189
by Martin Pool - BROKEN: partial support for commit into weave | 577 | s += hexlify(rand_bytes(8)) | 
| 578 | return s |