63
62
# TODO: If commit fails, leave the message in a file somewhere.
64
# TODO: Change the parameter 'rev_id' to 'revision_id' to be consistent with
65
# the rest of the code; add a deprecation of the old name.
72
from binascii import hexlify
73
72
from cStringIO import StringIO
75
from bzrlib.atomicfile import AtomicFile
76
from bzrlib.osutils import (local_time_offset,
77
rand_bytes, compact_date,
78
kind_marker, is_inside_any, quotefn,
79
sha_file, isdir, isfile,
81
74
import bzrlib.config
82
75
import bzrlib.errors as errors
83
76
from bzrlib.errors import (BzrError, PointlessCommit,
88
from bzrlib.revision import Revision
80
from bzrlib.osutils import (kind_marker, isdir,isfile, is_inside_any,
81
is_inside_or_parent_of_any,
82
quotefn, sha_file, split_lines)
89
83
from bzrlib.testament import Testament
90
84
from bzrlib.trace import mutter, note, warning
91
85
from bzrlib.xml5 import serializer_v5
92
from bzrlib.inventory import Inventory, ROOT_ID
93
from bzrlib.symbol_versioning import *
86
from bzrlib.inventory import Inventory, ROOT_ID, InventoryEntry
87
from bzrlib import symbol_versioning
88
from bzrlib.symbol_versioning import (deprecated_passed,
94
91
from bzrlib.workingtree import WorkingTree
97
@deprecated_function(zero_seven)
98
def commit(*args, **kwargs):
99
"""Commit a new revision to a branch.
101
Function-style interface for convenience of old callers.
103
New code should use the Commit class instead.
105
## XXX: Remove this in favor of Branch.commit?
106
Commit().commit(*args, **kwargs)
109
94
class NullCommitReporter(object):
110
95
"""I report on progress of a commit."""
245
237
self.reporter = reporter
247
239
self.work_tree.lock_write()
240
self.pb = bzrlib.ui.ui_factory.nested_progress_bar()
242
# Cannot commit with conflicts present.
243
if len(self.work_tree.conflicts())>0:
244
raise ConflictsInTree
249
246
# setup the bound branch variables as needed.
250
247
self._check_bound_branch()
252
249
# 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.
255
if self.work_tree.last_revision() != self.master_branch.last_revision():
251
first_tree_parent = self.work_tree.get_parent_ids()[0]
253
# if there are no parents, treat our parent as 'None'
254
# this is so that we still consier the master branch
255
# - in a checkout scenario the tree may have no
256
# parents but the branch may do.
257
first_tree_parent = None
258
master_last = self.master_branch.last_revision()
259
if (master_last is not None and
260
master_last != first_tree_parent):
256
261
raise errors.OutOfDateTree(self.work_tree)
259
264
# raise an exception as soon as we find a single unknown.
260
265
for unknown in self.work_tree.unknowns():
261
266
raise StrictCommitFailed()
263
if timestamp is None:
264
self.timestamp = time.time()
266
self.timestamp = long(timestamp)
268
268
if self.config is None:
269
self.config = bzrlib.config.BranchConfig(self.branch)
272
self.rev_id = _gen_revision_id(self.config, self.timestamp)
276
if committer is None:
277
self.committer = self.config.username()
279
assert isinstance(committer, basestring), type(committer)
280
self.committer = committer
283
self.timezone = local_time_offset()
285
self.timezone = int(timezone)
269
self.config = self.branch.get_config()
287
271
if isinstance(message, str):
288
272
message = message.decode(bzrlib.user_encoding)
289
273
assert isinstance(message, unicode), type(message)
293
277
self.work_inv = self.work_tree.inventory
294
278
self.basis_tree = self.work_tree.basis_tree()
295
279
self.basis_inv = self.basis_tree.inventory
280
# one to finish, one for rev and inventory, and one for each
281
# inventory entry, and the same for the new inventory.
282
# note that this estimate is too long when we do a partial tree
283
# commit which excludes some new files from being considered.
284
# The estimate is corrected when we populate the new inv.
285
self.pb_total = len(self.work_inv) + 5
297
288
self._gather_parents()
298
289
if len(self.parents) > 1 and self.specific_files:
299
raise NotImplementedError('selected-file commit of merges is not supported yet')
300
self._check_parents_present()
290
raise NotImplementedError('selected-file commit of merges is not supported yet: files %r',
293
self.builder = self.branch.get_commit_builder(self.parents,
294
self.config, timestamp, timezone, committer, revprops, rev_id)
302
296
self._remove_deleted()
303
297
self._populate_new_inv()
304
self._store_snapshot()
305
298
self._report_deletes()
307
if not (self.allow_pointless
308
or len(self.parents) > 1
309
or self.new_inv != self.basis_inv):
310
raise PointlessCommit()
312
if len(self.work_tree.conflicts())>0:
313
raise ConflictsInTree
315
self.inv_sha1 = self.branch.repository.add_inventory(
320
self._make_revision()
300
self._check_pointless()
302
self._emit_progress_update()
303
# TODO: Now the new inventory is known, check for conflicts and
304
# prompt the user for a commit message.
305
# ADHB 2006-08-08: If this is done, populate_new_inv should not add
306
# weave lines, because nothing should be recorded until it is known
307
# that commit will succeed.
308
self.builder.finish_inventory()
309
self._emit_progress_update()
310
self.rev_id = self.builder.commit(self.message)
311
self._emit_progress_update()
321
312
# revision data is in the local branch now.
323
314
# upload revision data to the master.
324
# this will propogate merged revisions too if needed.
315
# this will propagate merged revisions too if needed.
325
316
if self.bound_branch:
326
317
self.master_branch.repository.fetch(self.branch.repository,
327
318
revision_id=self.rev_id)
391
401
self.bound_branch = self.branch
392
402
self.master_branch.lock_write()
393
403
self.master_locked = True
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)
406
"""Cleanup any open locks, progress bars etc."""
407
cleanups = [self._cleanup_bound_branch,
408
self.work_tree.unlock,
410
found_exception = None
411
for cleanup in cleanups:
414
# we want every cleanup to run no matter what.
415
# so we have a catchall here, but we will raise the
416
# last encountered exception up the stack: and
417
# typically this will be useful enough.
420
if found_exception is not None:
421
# don't do a plan raise, because the last exception may have been
422
# trashed, e is our sure-to-work exception even though it loses the
423
# full traceback. XXX: RBC 20060421 perhaps we could check the
424
# exc_info and if its the same one do a plain raise otherwise
425
# 'raise e' as we do now.
403
428
def _cleanup_bound_branch(self):
404
429
"""Executed at the end of a try/finally to cleanup a bound branch.
428
458
def _gather_parents(self):
429
459
"""Record the parents of a merge for merge detection."""
430
pending_merges = self.work_tree.pending_merges()
460
# TODO: Make sure that this list doesn't contain duplicate
461
# entries and the order is preserved when doing this.
462
self.parents = self.work_tree.get_parent_ids()
432
463
self.parent_invs = []
433
self.present_parents = []
434
precursor_id = self.branch.last_revision()
436
self.parents.append(precursor_id)
437
self.parents += pending_merges
438
464
for revision in self.parents:
439
465
if self.branch.repository.has_revision(revision):
466
mutter('commit parent revision {%s}', revision)
440
467
inventory = self.branch.repository.get_inventory(revision)
441
468
self.parent_invs.append(inventory)
442
self.present_parents.append(revision)
444
def _check_parents_present(self):
445
for parent_id in self.parents:
446
mutter('commit parent revision {%s}', parent_id)
447
if not self.branch.repository.has_revision(parent_id):
448
if parent_id == self.branch.last_revision():
449
warning("parent is missing %r", parent_id)
450
raise HistoryMissing(self.branch, 'revision', parent_id)
452
mutter("commit will ghost revision %r", parent_id)
454
def _make_revision(self):
455
"""Record a new revision object for this commit."""
456
rev = Revision(timestamp=self.timestamp,
457
timezone=self.timezone,
458
committer=self.committer,
459
message=self.message,
460
inventory_sha1=self.inv_sha1,
461
revision_id=self.rev_id,
462
properties=self.revprops)
463
rev.parent_ids = self.parents
464
self.branch.repository.add_revision(self.rev_id, rev, self.new_inv, self.config)
470
mutter('commit parent ghost revision {%s}', revision)
466
472
def _remove_deleted(self):
467
473
"""Remove deleted files from the working inventories.
477
483
specific = self.specific_files
485
deleted_paths = set()
479
486
for path, ie in self.work_inv.iter_entries():
487
if is_inside_any(deleted_paths, path):
488
# The tree will delete the required ids recursively.
480
490
if specific and not is_inside_any(specific, path):
482
492
if not self.work_tree.has_filename(path):
493
deleted_paths.add(path)
483
494
self.reporter.missing(path)
484
deleted_ids.append((path, ie.file_id))
486
deleted_ids.sort(reverse=True)
487
for path, file_id in deleted_ids:
488
del self.work_inv[file_id]
489
self.work_tree._write_inventory(self.work_inv)
491
def _store_snapshot(self):
492
"""Pass over inventory and record a snapshot.
494
Entries get a new revision when they are modified in
495
any way, which includes a merge with a new set of
496
parents that have the same entry.
498
# XXX: Need to think more here about when the user has
499
# made a specific decision on a particular value -- c.f.
501
for path, ie in self.new_inv.iter_entries():
502
previous_entries = ie.find_previous_heads(
505
self.branch.repository.get_transaction())
506
if ie.revision is None:
507
change = ie.snapshot(self.rev_id, path, previous_entries,
508
self.work_tree, self.weave_store,
509
self.branch.get_transaction())
512
self.reporter.snapshot_change(change, path)
495
deleted_ids.append(ie.file_id)
496
self.work_tree.unversion(deleted_ids)
514
498
def _populate_new_inv(self):
515
499
"""Build revision inventory.
521
505
None; inventory entries that are carried over untouched have their
522
506
revision set to their prior value.
508
# ESEPARATIONOFCONCERNS: this function is diffing and using the diff
509
# results to create a new inventory at the same time, which results
510
# in bugs like #46635. Any reason not to use/enhance Tree.changes_from?
524
512
mutter("Selecting files for commit with filter %s", self.specific_files)
525
self.new_inv = Inventory(revision_id=self.rev_id)
526
for path, new_ie in self.work_inv.iter_entries():
513
entries = self.work_inv.iter_entries()
514
if not self.builder.record_root_entry:
515
symbol_versioning.warn('CommitBuilders should support recording'
516
' the root entry as of bzr 0.10.', DeprecationWarning,
518
self.builder.new_inventory.add(self.basis_inv.root.copy())
520
self._emit_progress_update()
521
for path, new_ie in entries:
522
self._emit_progress_update()
527
523
file_id = new_ie.file_id
528
mutter('check %s {%s}', path, new_ie.file_id)
529
if self.specific_files:
530
if not is_inside_any(self.specific_files, path):
531
mutter('%s not selected for commit', path)
532
self._carry_entry(file_id)
524
# mutter('check %s {%s}', path, file_id)
525
if (not self.specific_files or
526
is_inside_or_parent_of_any(self.specific_files, path)):
527
# mutter('%s selected for commit', path)
531
# mutter('%s not selected for commit', path)
532
if self.basis_inv.has_id(file_id):
533
ie = self.basis_inv[file_id].copy()
535
# this entry is new and not being committed
535
# this is selected, ensure its parents are too.
536
parent_id = new_ie.parent_id
537
while parent_id != ROOT_ID:
538
if not self.new_inv.has_id(parent_id):
539
ie = self._select_entry(self.work_inv[parent_id])
540
mutter('%s selected for commit because of %s',
541
self.new_inv.id2path(parent_id), path)
543
ie = self.new_inv[parent_id]
544
if ie.revision is not None:
546
mutter('%s selected for commit because of %s',
547
self.new_inv.id2path(parent_id), path)
548
parent_id = ie.parent_id
549
mutter('%s selected for commit', path)
550
self._select_entry(new_ie)
552
def _select_entry(self, new_ie):
553
"""Make new_ie be considered for committing."""
559
def _carry_entry(self, file_id):
560
"""Carry the file unchanged from the basis revision."""
561
if self.basis_inv.has_id(file_id):
562
self.new_inv.add(self.basis_inv[file_id].copy())
538
self.builder.record_entry_contents(ie, self.parent_invs,
539
path, self.work_tree)
540
# describe the nature of the change that has occurred relative to
541
# the basis inventory.
542
if (self.basis_inv.has_id(ie.file_id)):
543
basis_ie = self.basis_inv[ie.file_id]
546
change = ie.describe_change(basis_ie, ie)
547
if change in (InventoryEntry.RENAMED,
548
InventoryEntry.MODIFIED_AND_RENAMED):
549
old_path = self.basis_inv.id2path(ie.file_id)
550
self.reporter.renamed(change, old_path, path)
552
self.reporter.snapshot_change(change, path)
554
if not self.specific_files:
557
# ignore removals that don't match filespec
558
for path, new_ie in self.basis_inv.iter_entries():
559
if new_ie.file_id in self.work_inv:
561
if is_inside_any(self.specific_files, path):
565
self.builder.record_entry_contents(ie, self.parent_invs, path,
568
def _emit_progress_update(self):
569
"""Emit an update to the progress bar."""
570
self.pb.update("Committing", self.pb_count, self.pb_total)
564
573
def _report_deletes(self):
565
for file_id in self.basis_inv:
566
if file_id not in self.new_inv:
567
self.reporter.deleted(self.basis_inv.id2path(file_id))
569
def _gen_revision_id(config, when):
570
"""Return new revision-id."""
571
s = '%s-%s-' % (config.user_email(), compact_date(when))
572
s += hexlify(rand_bytes(8))
574
for path, ie in self.basis_inv.iter_entries():
575
if ie.file_id not in self.builder.new_inventory:
576
self.reporter.deleted(path)
added
removed
bzrlib/commit.py
