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