1773.4.1
by Martin Pool
Add pyflakes makefile target; fix many warnings |
1 |
# Copyright (C) 2005, 2006 Canonical Ltd
|
1685.1.80
by Wouter van Heyst
more code cleanup |
2 |
#
|
70
by mbp at sourcefrog
Prepare for smart recursive add. |
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.
|
|
1685.1.80
by Wouter van Heyst
more code cleanup |
7 |
#
|
70
by mbp at sourcefrog
Prepare for smart recursive add. |
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.
|
|
1685.1.80
by Wouter van Heyst
more code cleanup |
12 |
#
|
70
by mbp at sourcefrog
Prepare for smart recursive add. |
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 |
||
1773.4.1
by Martin Pool
Add pyflakes makefile target; fix many warnings |
17 |
import errno |
18 |
import os |
|
19 |
from os.path import dirname |
|
1185.85.12
by John Arbash Meinel
Refactoring AddAction to allow redirecting to an encoding file. |
20 |
import sys |
1185.1.53
by Robert Collins
raise a specific error on unsupported branches so that they can be distinguished from generic errors |
21 |
|
2094.3.5
by John Arbash Meinel
Fix imports to ensure modules are loaded before they are used |
22 |
import bzrlib.bzrdir |
1185.1.53
by Robert Collins
raise a specific error on unsupported branches so that they can be distinguished from generic errors |
23 |
import bzrlib.errors as errors |
1399.1.1
by Robert Collins
move checks for versionability of file kinds into InventoryEntry |
24 |
from bzrlib.inventory import InventoryEntry |
1128
by Martin Pool
- smart_add routine now doesn't print filenames directly, but rather |
25 |
from bzrlib.trace import mutter, note, warning |
1092.1.27
by Robert Collins
two bugfixes to smart_add - do not add paths from nested trees to the parent tree, and do not mutate the user supplied file list |
26 |
from bzrlib.errors import NotBranchError |
1508.1.10
by Robert Collins
bzrlib.add.smart_add_branch is now smart_add_tree. (Robert Collins) |
27 |
import bzrlib.osutils |
28 |
from bzrlib.workingtree import WorkingTree |
|
70
by mbp at sourcefrog
Prepare for smart recursive add. |
29 |
|
1685.1.80
by Wouter van Heyst
more code cleanup |
30 |
|
686
by Martin Pool
- glob expand add arguments on win32 |
31 |
def glob_expand_for_win32(file_list): |
1185.23.2
by Aaron Bentley
win32 glob expansion fix from Belchenko |
32 |
if not file_list: |
33 |
return
|
|
686
by Martin Pool
- glob expand add arguments on win32 |
34 |
import glob |
35 |
expanded_file_list = [] |
|
36 |
for possible_glob in file_list: |
|
37 |
glob_files = glob.glob(possible_glob) |
|
38 |
||
39 |
if glob_files == []: |
|
40 |
# special case to let the normal code path handle
|
|
41 |
# files that do not exists
|
|
42 |
expanded_file_list.append(possible_glob) |
|
43 |
else: |
|
44 |
expanded_file_list += glob_files |
|
45 |
return expanded_file_list |
|
46 |
||
1128
by Martin Pool
- smart_add routine now doesn't print filenames directly, but rather |
47 |
|
1092.1.29
by Robert Collins
break smart_add into smart_add and smart_add_branch which will accept a branch parameter |
48 |
def _prepare_file_list(file_list): |
49 |
"""Prepare a file list for use by smart_add_*."""
|
|
50 |
if sys.platform == 'win32': |
|
51 |
file_list = glob_expand_for_win32(file_list) |
|
52 |
if not file_list: |
|
1185.33.66
by Martin Pool
[patch] use unicode literals for all hardcoded paths (Alexander Belchenko) |
53 |
file_list = [u'.'] |
1092.1.29
by Robert Collins
break smart_add into smart_add and smart_add_branch which will accept a branch parameter |
54 |
file_list = list(file_list) |
55 |
return file_list |
|
56 |
||
1159
by Martin Pool
- clean up parameters to smart_add and smart_add_branch |
57 |
|
1185.85.12
by John Arbash Meinel
Refactoring AddAction to allow redirecting to an encoding file. |
58 |
class AddAction(object): |
59 |
"""A class which defines what action to take when adding a file."""
|
|
60 |
||
1757.2.6
by Robert Collins
Steps towards a nicer smart add - unwind the conditional add logic - having parents not in the inventory was overly complicating the rest of the code. |
61 |
def __init__(self, to_file=None, should_print=None): |
1185.85.12
by John Arbash Meinel
Refactoring AddAction to allow redirecting to an encoding file. |
62 |
self._to_file = to_file |
63 |
if to_file is None: |
|
64 |
self._to_file = sys.stdout |
|
1685.1.69
by Wouter van Heyst
merge bzr.dev 1740 |
65 |
self.should_print = False |
1185.85.12
by John Arbash Meinel
Refactoring AddAction to allow redirecting to an encoding file. |
66 |
if should_print is not None: |
67 |
self.should_print = should_print |
|
68 |
||
1757.2.15
by Robert Collins
Consolidate writes of adds to one write per file - drops encoding overhead by 1/3. |
69 |
def __call__(self, inv, parent_ie, path, kind, _quote=bzrlib.osutils.quotefn): |
1185.85.12
by John Arbash Meinel
Refactoring AddAction to allow redirecting to an encoding file. |
70 |
"""Add path to inventory.
|
71 |
||
72 |
The default action does nothing.
|
|
73 |
||
74 |
:param inv: The inventory we are working with.
|
|
1757.2.2
by Robert Collins
Merge basename call eliminate for smart_add. |
75 |
:param path: The FastPath being added
|
1185.85.12
by John Arbash Meinel
Refactoring AddAction to allow redirecting to an encoding file. |
76 |
:param kind: The kind of the object being added.
|
77 |
"""
|
|
1911.3.1
by John Arbash Meinel
Updated smart_add so that the AddAction can return a custom id. |
78 |
if self.should_print: |
79 |
self._to_file.write('added %s\n' % _quote(path.raw_path)) |
|
80 |
return None |
|
1185.85.12
by John Arbash Meinel
Refactoring AddAction to allow redirecting to an encoding file. |
81 |
|
82 |
||
1911.3.2
by John Arbash Meinel
Adding the AddFromBaseAction, which tries to reuse file ids from another tree |
83 |
class AddFromBaseAction(AddAction): |
84 |
"""This class will try to extract file ids from another tree."""
|
|
85 |
||
86 |
def __init__(self, base_tree, base_path, to_file=None, should_print=None): |
|
87 |
super(AddFromBaseAction, self).__init__(to_file=to_file, |
|
88 |
should_print=should_print) |
|
89 |
self.base_tree = base_tree |
|
90 |
self.base_path = base_path |
|
91 |
||
92 |
def __call__(self, inv, parent_ie, path, kind): |
|
93 |
# Place the parent call
|
|
94 |
# Now check to see if we can extract an id for this file
|
|
95 |
file_id, base_path = self._get_base_file_id(path, parent_ie) |
|
96 |
if file_id is not None: |
|
97 |
if self.should_print: |
|
98 |
self._to_file.write('added %s w/ file id from %s\n' |
|
99 |
% (path.raw_path, base_path)) |
|
100 |
else: |
|
101 |
# we aren't doing anything special, so let the default
|
|
102 |
# reporter happen
|
|
103 |
file_id = super(AddFromBaseAction, self).__call__( |
|
104 |
inv, parent_ie, path, kind) |
|
105 |
return file_id |
|
106 |
||
107 |
def _get_base_file_id(self, path, parent_ie): |
|
108 |
"""Look for a file id in the base branch.
|
|
109 |
||
110 |
First, if the base tree has the parent directory,
|
|
111 |
we look for a file with the same name in that directory.
|
|
112 |
Else, we look for an entry in the base tree with the same path.
|
|
113 |
"""
|
|
114 |
||
115 |
if (parent_ie.file_id in self.base_tree): |
|
116 |
base_parent_ie = self.base_tree.inventory[parent_ie.file_id] |
|
117 |
base_child_ie = base_parent_ie.children.get(path.base_path) |
|
118 |
if base_child_ie is not None: |
|
119 |
return (base_child_ie.file_id, |
|
120 |
self.base_tree.id2path(base_child_ie.file_id)) |
|
121 |
full_base_path = bzrlib.osutils.pathjoin(self.base_path, path.raw_path) |
|
122 |
# This may return None, but it is our last attempt
|
|
123 |
return self.base_tree.path2id(full_base_path), full_base_path |
|
124 |
||
125 |
||
1185.85.12
by John Arbash Meinel
Refactoring AddAction to allow redirecting to an encoding file. |
126 |
# TODO: jam 20050105 These could be used for compatibility
|
127 |
# however, they bind against the current stdout, not the
|
|
128 |
# one which exists at the time they are called, so they
|
|
129 |
# don't work for the test suite.
|
|
130 |
# deprecated
|
|
1757.2.6
by Robert Collins
Steps towards a nicer smart add - unwind the conditional add logic - having parents not in the inventory was overly complicating the rest of the code. |
131 |
add_action_add = AddAction() |
132 |
add_action_null = add_action_add |
|
133 |
add_action_add_and_print = AddAction(should_print=True) |
|
134 |
add_action_print = add_action_add_and_print |
|
135 |
||
136 |
||
137 |
def smart_add(file_list, recurse=True, action=None, save=True): |
|
598
by Martin Pool
doc |
138 |
"""Add files to version, optionally recursing into directories.
|
70
by mbp at sourcefrog
Prepare for smart recursive add. |
139 |
|
140 |
This is designed more towards DWIM for humans than API simplicity.
|
|
141 |
For the specific behaviour see the help for cmd_add().
|
|
1159
by Martin Pool
- clean up parameters to smart_add and smart_add_branch |
142 |
|
143 |
Returns the number of files added.
|
|
1757.2.6
by Robert Collins
Steps towards a nicer smart add - unwind the conditional add logic - having parents not in the inventory was overly complicating the rest of the code. |
144 |
Please see smart_add_tree for more detail.
|
70
by mbp at sourcefrog
Prepare for smart recursive add. |
145 |
"""
|
1092.1.29
by Robert Collins
break smart_add into smart_add and smart_add_branch which will accept a branch parameter |
146 |
file_list = _prepare_file_list(file_list) |
1508.1.10
by Robert Collins
bzrlib.add.smart_add_branch is now smart_add_tree. (Robert Collins) |
147 |
tree = WorkingTree.open_containing(file_list[0])[0] |
1930.1.2
by John Arbash Meinel
Alexander Belchenko: Fix add --dry-run |
148 |
return smart_add_tree(tree, file_list, recurse, action=action, save=save) |
1185.85.12
by John Arbash Meinel
Refactoring AddAction to allow redirecting to an encoding file. |
149 |
|
150 |
||
1757.2.2
by Robert Collins
Merge basename call eliminate for smart_add. |
151 |
class FastPath(object): |
152 |
"""A path object with fast accessors for things like basename."""
|
|
153 |
||
154 |
__slots__ = ['raw_path', 'base_path'] |
|
155 |
||
156 |
def __init__(self, path, base_path=None): |
|
157 |
"""Construct a FastPath from path."""
|
|
158 |
if base_path is None: |
|
159 |
self.base_path = bzrlib.osutils.basename(path) |
|
160 |
else: |
|
161 |
self.base_path = base_path |
|
162 |
self.raw_path = path |
|
163 |
||
1850.2.1
by John Arbash Meinel
Fix bug #52578, smart-add wasn't recursing all supplied directories. |
164 |
def __cmp__(self, other): |
165 |
return cmp(self.raw_path, other.raw_path) |
|
166 |
||
167 |
def __hash__(self): |
|
168 |
return hash(self.raw_path) |
|
169 |
||
1757.2.2
by Robert Collins
Merge basename call eliminate for smart_add. |
170 |
|
1757.2.6
by Robert Collins
Steps towards a nicer smart add - unwind the conditional add logic - having parents not in the inventory was overly complicating the rest of the code. |
171 |
def smart_add_tree(tree, file_list, recurse=True, action=None, save=True): |
2255.2.62
by John Arbash Meinel
add a workingtree_implementations test that makes sure smart_add_tree orks properly |
172 |
tree.lock_tree_write() |
173 |
try: |
|
174 |
return _smart_add_tree(tree=tree, file_list=file_list, recurse=recurse, |
|
175 |
action=action, save=save) |
|
176 |
finally: |
|
177 |
tree.unlock() |
|
178 |
||
179 |
def _smart_add_tree(tree, file_list, recurse=True, action=None, save=True): |
|
1092.1.29
by Robert Collins
break smart_add into smart_add and smart_add_branch which will accept a branch parameter |
180 |
"""Add files to version, optionally recursing into directories.
|
181 |
||
182 |
This is designed more towards DWIM for humans than API simplicity.
|
|
183 |
For the specific behaviour see the help for cmd_add().
|
|
1128
by Martin Pool
- smart_add routine now doesn't print filenames directly, but rather |
184 |
|
1508.1.11
by Robert Collins
Documentation. |
185 |
This calls reporter with each (path, kind, file_id) of added files.
|
1159
by Martin Pool
- clean up parameters to smart_add and smart_add_branch |
186 |
|
187 |
Returns the number of files added.
|
|
1757.2.6
by Robert Collins
Steps towards a nicer smart add - unwind the conditional add logic - having parents not in the inventory was overly complicating the rest of the code. |
188 |
|
189 |
:param save: Save the inventory after completing the adds. If False this
|
|
190 |
provides dry-run functionality by doing the add and not saving the
|
|
191 |
inventory. Note that the modified inventory is left in place, allowing
|
|
192 |
further dry-run tasks to take place. To restore the original inventory
|
|
193 |
call tree.read_working_inventory().
|
|
1092.1.29
by Robert Collins
break smart_add into smart_add and smart_add_branch which will accept a branch parameter |
194 |
"""
|
1159
by Martin Pool
- clean up parameters to smart_add and smart_add_branch |
195 |
assert isinstance(recurse, bool) |
1185.85.12
by John Arbash Meinel
Refactoring AddAction to allow redirecting to an encoding file. |
196 |
if action is None: |
1757.2.6
by Robert Collins
Steps towards a nicer smart add - unwind the conditional add logic - having parents not in the inventory was overly complicating the rest of the code. |
197 |
action = AddAction() |
1713.1.9
by Robert Collins
Paired performance tuning of bzr add. (Robert Collins, Martin Pool). |
198 |
|
1713.1.14
by Robert Collins
Review feedback. |
199 |
prepared_list = _prepare_file_list(file_list) |
200 |
mutter("smart add of %r, originally %r", prepared_list, file_list) |
|
1497
by Robert Collins
Move Branch.read_working_inventory to WorkingTree. |
201 |
inv = tree.read_working_inventory() |
1185.46.8
by Aaron Bentley
bzr add reports ignored patterns. |
202 |
added = [] |
203 |
ignored = {} |
|
1757.2.4
by Robert Collins
Teach file_kind about NoSuchFile, reducing duplicate code, and add user files before entering the main loop in smart_add. |
204 |
dirs_to_add = [] |
1757.2.5
by Robert Collins
De-dup the add list so we only walk subtrees once for add. |
205 |
user_dirs = set() |
86
by mbp at sourcefrog
first cut at recursive add |
206 |
|
1713.1.11
by Robert Collins
refactor smart_add to pass around the parent inventory entry and use that, resulting in another 100bzrlib/inventory.py performance improvement, and making inventory writing the dominating factory in add. (Robert Collins) |
207 |
# validate user file paths and convert all paths to tree
|
208 |
# relative : its cheaper to make a tree relative path an abspath
|
|
209 |
# than to convert an abspath to tree relative.
|
|
1713.1.14
by Robert Collins
Review feedback. |
210 |
for filepath in prepared_list: |
1757.2.2
by Robert Collins
Merge basename call eliminate for smart_add. |
211 |
rf = FastPath(tree.relpath(filepath)) |
1713.1.9
by Robert Collins
Paired performance tuning of bzr add. (Robert Collins, Martin Pool). |
212 |
# validate user parameters. Our recursive code avoids adding new files
|
213 |
# that need such validation
|
|
1757.2.2
by Robert Collins
Merge basename call eliminate for smart_add. |
214 |
if tree.is_control_filename(rf.raw_path): |
2279.6.1
by Alexander Belchenko
Instead of __str__ method for FastPath object we use .raw_path attribute (note from Aaron). |
215 |
raise errors.ForbiddenControlFileError(filename=rf.raw_path) |
1757.2.4
by Robert Collins
Teach file_kind about NoSuchFile, reducing duplicate code, and add user files before entering the main loop in smart_add. |
216 |
|
217 |
abspath = tree.abspath(rf.raw_path) |
|
218 |
kind = bzrlib.osutils.file_kind(abspath) |
|
219 |
if kind == 'directory': |
|
1757.2.6
by Robert Collins
Steps towards a nicer smart add - unwind the conditional add logic - having parents not in the inventory was overly complicating the rest of the code. |
220 |
# schedule the dir for scanning
|
1850.2.1
by John Arbash Meinel
Fix bug #52578, smart-add wasn't recursing all supplied directories. |
221 |
user_dirs.add(rf) |
1757.2.4
by Robert Collins
Teach file_kind about NoSuchFile, reducing duplicate code, and add user files before entering the main loop in smart_add. |
222 |
else: |
223 |
if not InventoryEntry.versionable_kind(kind): |
|
1773.4.1
by Martin Pool
Add pyflakes makefile target; fix many warnings |
224 |
raise errors.BadFileKindError(filename=abspath, kind=kind) |
1757.2.6
by Robert Collins
Steps towards a nicer smart add - unwind the conditional add logic - having parents not in the inventory was overly complicating the rest of the code. |
225 |
# ensure the named path is added, so that ignore rules in the later directory
|
226 |
# walk dont skip it.
|
|
227 |
# we dont have a parent ie known yet.: use the relatively slower inventory
|
|
228 |
# probing method
|
|
229 |
versioned = inv.has_filename(rf.raw_path) |
|
230 |
if versioned: |
|
231 |
continue
|
|
1757.2.7
by Robert Collins
Split out the recursive add and add of a single entry in add.py. |
232 |
added.extend(__add_one_and_parent(tree, inv, None, rf, kind, action)) |
1757.2.6
by Robert Collins
Steps towards a nicer smart add - unwind the conditional add logic - having parents not in the inventory was overly complicating the rest of the code. |
233 |
|
234 |
if not recurse: |
|
235 |
# no need to walk any directories at all.
|
|
236 |
if len(added) > 0 and save: |
|
237 |
tree._write_inventory(inv) |
|
238 |
return added, ignored |
|
1757.2.4
by Robert Collins
Teach file_kind about NoSuchFile, reducing duplicate code, and add user files before entering the main loop in smart_add. |
239 |
|
1757.2.5
by Robert Collins
De-dup the add list so we only walk subtrees once for add. |
240 |
# only walk the minimal parents needed: we have user_dirs to override
|
241 |
# ignores.
|
|
242 |
prev_dir = None |
|
1850.2.1
by John Arbash Meinel
Fix bug #52578, smart-add wasn't recursing all supplied directories. |
243 |
|
244 |
is_inside = bzrlib.osutils.is_inside_or_parent_of_any |
|
1757.2.5
by Robert Collins
De-dup the add list so we only walk subtrees once for add. |
245 |
for path in sorted(user_dirs): |
1850.2.1
by John Arbash Meinel
Fix bug #52578, smart-add wasn't recursing all supplied directories. |
246 |
if (prev_dir is None or not is_inside([prev_dir], path.raw_path)): |
247 |
dirs_to_add.append((path, None)) |
|
248 |
prev_dir = path.raw_path |
|
1757.2.5
by Robert Collins
De-dup the add list so we only walk subtrees once for add. |
249 |
|
1757.2.4
by Robert Collins
Teach file_kind about NoSuchFile, reducing duplicate code, and add user files before entering the main loop in smart_add. |
250 |
# this will eventually be *just* directories, right now it starts off with
|
251 |
# just directories.
|
|
252 |
for directory, parent_ie in dirs_to_add: |
|
253 |
# directory is tree-relative
|
|
254 |
abspath = tree.abspath(directory.raw_path) |
|
255 |
||
1757.2.7
by Robert Collins
Split out the recursive add and add of a single entry in add.py. |
256 |
# get the contents of this directory.
|
257 |
||
1757.2.4
by Robert Collins
Teach file_kind about NoSuchFile, reducing duplicate code, and add user files before entering the main loop in smart_add. |
258 |
# find the kind of the path being added.
|
259 |
kind = bzrlib.osutils.file_kind(abspath) |
|
260 |
||
261 |
if not InventoryEntry.versionable_kind(kind): |
|
262 |
warning("skipping %s (can't add file of kind '%s')", abspath, kind) |
|
263 |
continue
|
|
426
by Martin Pool
- Skip symlinks during recursive add (path from aaron) |
264 |
|
1713.1.11
by Robert Collins
refactor smart_add to pass around the parent inventory entry and use that, resulting in another 100bzrlib/inventory.py performance improvement, and making inventory writing the dominating factory in add. (Robert Collins) |
265 |
if parent_ie is not None: |
1757.2.4
by Robert Collins
Teach file_kind about NoSuchFile, reducing duplicate code, and add user files before entering the main loop in smart_add. |
266 |
versioned = directory.base_path in parent_ie.children |
1713.1.11
by Robert Collins
refactor smart_add to pass around the parent inventory entry and use that, resulting in another 100bzrlib/inventory.py performance improvement, and making inventory writing the dominating factory in add. (Robert Collins) |
267 |
else: |
268 |
# without the parent ie, use the relatively slower inventory
|
|
269 |
# probing method
|
|
1757.2.4
by Robert Collins
Teach file_kind about NoSuchFile, reducing duplicate code, and add user files before entering the main loop in smart_add. |
270 |
versioned = inv.has_filename(directory.raw_path) |
86
by mbp at sourcefrog
first cut at recursive add |
271 |
|
1092.1.27
by Robert Collins
two bugfixes to smart_add - do not add paths from nested trees to the parent tree, and do not mutate the user supplied file list |
272 |
if kind == 'directory': |
273 |
try: |
|
1713.1.11
by Robert Collins
refactor smart_add to pass around the parent inventory entry and use that, resulting in another 100bzrlib/inventory.py performance improvement, and making inventory writing the dominating factory in add. (Robert Collins) |
274 |
sub_branch = bzrlib.bzrdir.BzrDir.open(abspath) |
1092.1.27
by Robert Collins
two bugfixes to smart_add - do not add paths from nested trees to the parent tree, and do not mutate the user supplied file list |
275 |
sub_tree = True |
276 |
except NotBranchError: |
|
277 |
sub_tree = False |
|
1185.1.53
by Robert Collins
raise a specific error on unsupported branches so that they can be distinguished from generic errors |
278 |
except errors.UnsupportedFormatError: |
279 |
sub_tree = True |
|
1092.1.27
by Robert Collins
two bugfixes to smart_add - do not add paths from nested trees to the parent tree, and do not mutate the user supplied file list |
280 |
else: |
281 |
sub_tree = False |
|
282 |
||
1757.2.4
by Robert Collins
Teach file_kind about NoSuchFile, reducing duplicate code, and add user files before entering the main loop in smart_add. |
283 |
if directory.raw_path == '': |
1713.1.9
by Robert Collins
Paired performance tuning of bzr add. (Robert Collins, Martin Pool). |
284 |
# mutter("tree root doesn't need to be added")
|
1092.1.27
by Robert Collins
two bugfixes to smart_add - do not add paths from nested trees to the parent tree, and do not mutate the user supplied file list |
285 |
sub_tree = False |
91
by mbp at sourcefrog
special handling of root directory for recursive add |
286 |
elif versioned: |
1713.1.9
by Robert Collins
Paired performance tuning of bzr add. (Robert Collins, Martin Pool). |
287 |
pass
|
1713.1.11
by Robert Collins
refactor smart_add to pass around the parent inventory entry and use that, resulting in another 100bzrlib/inventory.py performance improvement, and making inventory writing the dominating factory in add. (Robert Collins) |
288 |
# mutter("%r is already versioned", abspath)
|
1092.1.27
by Robert Collins
two bugfixes to smart_add - do not add paths from nested trees to the parent tree, and do not mutate the user supplied file list |
289 |
elif sub_tree: |
2255.2.217
by Martin Pool
docs |
290 |
# XXX: This is wrong; people *might* reasonably be trying to add
|
291 |
# subtrees as subtrees. This should probably only be done in formats
|
|
292 |
# which can represent subtrees, and even then perhaps only when
|
|
2255.2.236
by Martin Pool
Review cleanups: mostly updating or removing todo comments. |
293 |
# the user asked to add subtrees. At the moment you can add them
|
294 |
# specially through 'join --reference', which is perhaps
|
|
295 |
# reasonable: adding a new reference is a special operation and
|
|
296 |
# can have a special behaviour. mbp 20070306
|
|
1713.1.11
by Robert Collins
refactor smart_add to pass around the parent inventory entry and use that, resulting in another 100bzrlib/inventory.py performance improvement, and making inventory writing the dominating factory in add. (Robert Collins) |
297 |
mutter("%r is a nested bzr tree", abspath) |
70
by mbp at sourcefrog
Prepare for smart recursive add. |
298 |
else: |
1757.2.7
by Robert Collins
Split out the recursive add and add of a single entry in add.py. |
299 |
__add_one(tree, inv, parent_ie, directory, kind, action) |
300 |
added.append(directory.raw_path) |
|
87
by mbp at sourcefrog
- clean up smart_add code, and make it commit the inventory |
301 |
|
1757.2.6
by Robert Collins
Steps towards a nicer smart add - unwind the conditional add logic - having parents not in the inventory was overly complicating the rest of the code. |
302 |
if kind == 'directory' and not sub_tree: |
303 |
if parent_ie is not None: |
|
304 |
# must be present:
|
|
305 |
this_ie = parent_ie.children[directory.base_path] |
|
306 |
else: |
|
307 |
# without the parent ie, use the relatively slower inventory
|
|
308 |
# probing method
|
|
309 |
this_id = inv.path2id(directory.raw_path) |
|
310 |
if this_id is None: |
|
311 |
this_ie = None |
|
1713.1.11
by Robert Collins
refactor smart_add to pass around the parent inventory entry and use that, resulting in another 100bzrlib/inventory.py performance improvement, and making inventory writing the dominating factory in add. (Robert Collins) |
312 |
else: |
1757.2.6
by Robert Collins
Steps towards a nicer smart add - unwind the conditional add logic - having parents not in the inventory was overly complicating the rest of the code. |
313 |
this_ie = inv[this_id] |
1713.1.11
by Robert Collins
refactor smart_add to pass around the parent inventory entry and use that, resulting in another 100bzrlib/inventory.py performance improvement, and making inventory writing the dominating factory in add. (Robert Collins) |
314 |
|
1910.10.1
by Andrew Bennetts
Sort os.listdir results in add.py. |
315 |
for subf in sorted(os.listdir(abspath)): |
1713.1.9
by Robert Collins
Paired performance tuning of bzr add. (Robert Collins, Martin Pool). |
316 |
# here we could use TreeDirectory rather than
|
317 |
# string concatenation.
|
|
1757.2.4
by Robert Collins
Teach file_kind about NoSuchFile, reducing duplicate code, and add user files before entering the main loop in smart_add. |
318 |
subp = bzrlib.osutils.pathjoin(directory.raw_path, subf) |
1713.1.9
by Robert Collins
Paired performance tuning of bzr add. (Robert Collins, Martin Pool). |
319 |
# TODO: is_control_filename is very slow. Make it faster.
|
320 |
# TreeDirectory.is_control_filename could also make this
|
|
321 |
# faster - its impossible for a non root dir to have a
|
|
322 |
# control file.
|
|
1534.5.5
by Robert Collins
Move is_control_file into WorkingTree.is_control_filename and test. |
323 |
if tree.is_control_filename(subp): |
1185.31.4
by John Arbash Meinel
Fixing mutter() calls to not have to do string processing. |
324 |
mutter("skip control directory %r", subp) |
1757.2.6
by Robert Collins
Steps towards a nicer smart add - unwind the conditional add logic - having parents not in the inventory was overly complicating the rest of the code. |
325 |
elif subf in this_ie.children: |
326 |
# recurse into this already versioned subdir.
|
|
327 |
dirs_to_add.append((FastPath(subp, subf), this_ie)) |
|
87
by mbp at sourcefrog
- clean up smart_add code, and make it commit the inventory |
328 |
else: |
1757.2.5
by Robert Collins
De-dup the add list so we only walk subtrees once for add. |
329 |
# user selection overrides ignoes
|
1713.1.9
by Robert Collins
Paired performance tuning of bzr add. (Robert Collins, Martin Pool). |
330 |
# ignore while selecting files - if we globbed in the
|
331 |
# outer loop we would ignore user files.
|
|
1185.46.8
by Aaron Bentley
bzr add reports ignored patterns. |
332 |
ignore_glob = tree.is_ignored(subp) |
333 |
if ignore_glob is not None: |
|
1713.1.9
by Robert Collins
Paired performance tuning of bzr add. (Robert Collins, Martin Pool). |
334 |
# mutter("skip ignored sub-file %r", subp)
|
1830.3.4
by John Arbash Meinel
A couple no-op cleanups |
335 |
ignored.setdefault(ignore_glob, []).append(subp) |
1185.46.8
by Aaron Bentley
bzr add reports ignored patterns. |
336 |
else: |
1713.1.9
by Robert Collins
Paired performance tuning of bzr add. (Robert Collins, Martin Pool). |
337 |
#mutter("queue to add sub-file %r", subp)
|
1757.2.4
by Robert Collins
Teach file_kind about NoSuchFile, reducing duplicate code, and add user files before entering the main loop in smart_add. |
338 |
dirs_to_add.append((FastPath(subp, subf), this_ie)) |
1185.46.8
by Aaron Bentley
bzr add reports ignored patterns. |
339 |
|
1757.2.6
by Robert Collins
Steps towards a nicer smart add - unwind the conditional add logic - having parents not in the inventory was overly complicating the rest of the code. |
340 |
if len(added) > 0 and save: |
1508.1.10
by Robert Collins
bzrlib.add.smart_add_branch is now smart_add_tree. (Robert Collins) |
341 |
tree._write_inventory(inv) |
1185.46.8
by Aaron Bentley
bzr add reports ignored patterns. |
342 |
return added, ignored |
1185.3.3
by Martin Pool
- patch from mpe to automatically add parent directories |
343 |
|
1685.1.80
by Wouter van Heyst
more code cleanup |
344 |
|
1757.2.7
by Robert Collins
Split out the recursive add and add of a single entry in add.py. |
345 |
def __add_one_and_parent(tree, inv, parent_ie, path, kind, action): |
1713.1.14
by Robert Collins
Review feedback. |
346 |
"""Add a new entry to the inventory and automatically add unversioned parents.
|
347 |
||
348 |
:param inv: Inventory which will receive the new entry.
|
|
349 |
:param parent_ie: Parent inventory entry if known, or None. If
|
|
350 |
None, the parent is looked up by name and used if present, otherwise
|
|
351 |
it is recursively added.
|
|
352 |
:param kind: Kind of new entry (file, directory, etc)
|
|
353 |
:param action: callback(inv, parent_ie, path, kind); return ignored.
|
|
354 |
:returns: A list of paths which have been added.
|
|
355 |
"""
|
|
1185.3.3
by Martin Pool
- patch from mpe to automatically add parent directories |
356 |
# Nothing to do if path is already versioned.
|
1508.1.10
by Robert Collins
bzrlib.add.smart_add_branch is now smart_add_tree. (Robert Collins) |
357 |
# This is safe from infinite recursion because the tree root is
|
1185.3.3
by Martin Pool
- patch from mpe to automatically add parent directories |
358 |
# always versioned.
|
1713.1.11
by Robert Collins
refactor smart_add to pass around the parent inventory entry and use that, resulting in another 100bzrlib/inventory.py performance improvement, and making inventory writing the dominating factory in add. (Robert Collins) |
359 |
if parent_ie is not None: |
360 |
# we have a parent ie already
|
|
361 |
added = [] |
|
362 |
else: |
|
363 |
# slower but does not need parent_ie
|
|
1757.2.2
by Robert Collins
Merge basename call eliminate for smart_add. |
364 |
if inv.has_filename(path.raw_path): |
1713.1.11
by Robert Collins
refactor smart_add to pass around the parent inventory entry and use that, resulting in another 100bzrlib/inventory.py performance improvement, and making inventory writing the dominating factory in add. (Robert Collins) |
365 |
return [] |
1757.2.2
by Robert Collins
Merge basename call eliminate for smart_add. |
366 |
# its really not there : add the parent
|
367 |
# note that the dirname use leads to some extra str copying etc but as
|
|
368 |
# there are a limited number of dirs we can be nested under, it should
|
|
369 |
# generally find it very fast and not recurse after that.
|
|
1757.2.7
by Robert Collins
Split out the recursive add and add of a single entry in add.py. |
370 |
added = __add_one_and_parent(tree, inv, None, FastPath(dirname(path.raw_path)), 'directory', action) |
1757.2.2
by Robert Collins
Merge basename call eliminate for smart_add. |
371 |
parent_id = inv.path2id(dirname(path.raw_path)) |
1757.2.16
by Robert Collins
Review comments. |
372 |
parent_ie = inv[parent_id] |
1757.2.7
by Robert Collins
Split out the recursive add and add of a single entry in add.py. |
373 |
__add_one(tree, inv, parent_ie, path, kind, action) |
374 |
return added + [path.raw_path] |
|
375 |
||
376 |
||
377 |
def __add_one(tree, inv, parent_ie, path, kind, action): |
|
378 |
"""Add a new entry to the inventory.
|
|
379 |
||
380 |
:param inv: Inventory which will receive the new entry.
|
|
381 |
:param parent_ie: Parent inventory entry.
|
|
382 |
:param kind: Kind of new entry (file, directory, etc)
|
|
1911.3.1
by John Arbash Meinel
Updated smart_add so that the AddAction can return a custom id. |
383 |
:param action: callback(inv, parent_ie, path, kind); return a file_id
|
384 |
or None to generate a new file id
|
|
1757.2.7
by Robert Collins
Split out the recursive add and add of a single entry in add.py. |
385 |
:returns: None
|
386 |
"""
|
|
1911.3.1
by John Arbash Meinel
Updated smart_add so that the AddAction can return a custom id. |
387 |
file_id = action(inv, parent_ie, path, kind) |
388 |
entry = bzrlib.inventory.make_entry(kind, path.base_path, parent_ie.file_id, |
|
389 |
file_id=file_id) |
|
1757.2.6
by Robert Collins
Steps towards a nicer smart add - unwind the conditional add logic - having parents not in the inventory was overly complicating the rest of the code. |
390 |
inv.add(entry) |