bzr branch
http://gegoxaren.bato24.eu/bzr/brz/remove-bazaar
4988.10.3
by John Arbash Meinel
Merge bzr.dev 5007, resolve conflict, update NEWS |
1 |
# Copyright (C) 2005-2010 Canonical Ltd
|
1442.1.1
by Robert Collins
move config_dir into bzrlib.config |
2 |
# Authors: Robert Collins <robert.collins@canonical.com>
|
2323.6.2
by Martin Pool
Move responsibility for suggesting upgrades to ui object |
3 |
# and others
|
1442.1.1
by Robert Collins
move config_dir into bzrlib.config |
4 |
#
|
5 |
# This program is free software; you can redistribute it and/or modify
|
|
6 |
# it under the terms of the GNU General Public License as published by
|
|
7 |
# the Free Software Foundation; either version 2 of the License, or
|
|
8 |
# (at your option) any later version.
|
|
9 |
#
|
|
10 |
# This program is distributed in the hope that it will be useful,
|
|
11 |
# but WITHOUT ANY WARRANTY; without even the implied warranty of
|
|
12 |
# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
|
|
13 |
# GNU General Public License for more details.
|
|
14 |
#
|
|
15 |
# You should have received a copy of the GNU General Public License
|
|
16 |
# along with this program; if not, write to the Free Software
|
|
4183.7.1
by Sabin Iacob
update FSF mailing address |
17 |
# Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA
|
1442.1.1
by Robert Collins
move config_dir into bzrlib.config |
18 |
|
1442.1.20
by Robert Collins
add some documentation on options |
19 |
"""Configuration that affects the behaviour of Bazaar.
|
20 |
||
21 |
Currently this configuration resides in ~/.bazaar/bazaar.conf
|
|
1770.2.2
by Aaron Bentley
Rename branches.conf to locations.conf |
22 |
and ~/.bazaar/locations.conf, which is written to by bzr.
|
1442.1.20
by Robert Collins
add some documentation on options |
23 |
|
1461
by Robert Collins
Typo in config.py (Thanks Fabbione) |
24 |
In bazaar.conf the following options may be set:
|
1442.1.20
by Robert Collins
add some documentation on options |
25 |
[DEFAULT]
|
26 |
editor=name-of-program
|
|
27 |
email=Your Name <your@email.address>
|
|
28 |
check_signatures=require|ignore|check-available(default)
|
|
29 |
create_signatures=always|never|when-required(default)
|
|
1442.1.56
by Robert Collins
gpg_signing_command configuration item |
30 |
gpg_signing_command=name-of-program
|
1553.2.9
by Erik Bågfors
log_formatter => log_format for "named" formatters |
31 |
log_format=name-of-format
|
1442.1.20
by Robert Collins
add some documentation on options |
32 |
|
1770.2.2
by Aaron Bentley
Rename branches.conf to locations.conf |
33 |
in locations.conf, you specify the url of a branch and options for it.
|
1442.1.20
by Robert Collins
add some documentation on options |
34 |
Wildcards may be used - * and ? as normal in shell completion. Options
|
1770.2.2
by Aaron Bentley
Rename branches.conf to locations.conf |
35 |
set in both bazaar.conf and locations.conf are overridden by the locations.conf
|
1442.1.20
by Robert Collins
add some documentation on options |
36 |
setting.
|
37 |
[/home/robertc/source]
|
|
38 |
recurse=False|True(default)
|
|
39 |
email= as above
|
|
3943.8.1
by Marius Kruger
remove all trailing whitespace from bzr source |
40 |
check_signatures= as above
|
1442.1.20
by Robert Collins
add some documentation on options |
41 |
create_signatures= as above.
|
42 |
||
43 |
explanation of options
|
|
44 |
----------------------
|
|
45 |
editor - this option sets the pop up editor to use during commits.
|
|
46 |
email - this option sets the user id bzr will use when committing.
|
|
47 |
check_signatures - this option controls whether bzr will require good gpg
|
|
3943.8.1
by Marius Kruger
remove all trailing whitespace from bzr source |
48 |
signatures, ignore them, or check them if they are
|
1442.1.20
by Robert Collins
add some documentation on options |
49 |
present.
|
3943.8.1
by Marius Kruger
remove all trailing whitespace from bzr source |
50 |
create_signatures - this option controls whether bzr will always create
|
1442.1.20
by Robert Collins
add some documentation on options |
51 |
gpg signatures, never create them, or create them if the
|
52 |
branch is configured to require them.
|
|
1887.2.1
by Adeodato Simó
Fix some typos and grammar issues. |
53 |
log_format - this option sets the default log format. Possible values are
|
54 |
long, short, line, or a plugin can register new formats.
|
|
1553.6.2
by Erik Bågfors
documentation and NEWS |
55 |
|
56 |
In bazaar.conf you can also define aliases in the ALIASES sections, example
|
|
57 |
||
58 |
[ALIASES]
|
|
59 |
lastlog=log --line -r-10..-1
|
|
60 |
ll=log --line -r-10..-1
|
|
61 |
h=help
|
|
62 |
up=pull
|
|
1442.1.20
by Robert Collins
add some documentation on options |
63 |
"""
|
1442.1.1
by Robert Collins
move config_dir into bzrlib.config |
64 |
|
1996.3.31
by John Arbash Meinel
Make bzrlib.config use lazy importing |
65 |
import os |
66 |
import sys |
|
1474
by Robert Collins
Merge from Aaron Bentley. |
67 |
|
5345.5.4
by Vincent Ladeuil
Start implementing config files locking. |
68 |
from bzrlib.decorators import needs_write_lock |
1996.3.31
by John Arbash Meinel
Make bzrlib.config use lazy importing |
69 |
from bzrlib.lazy_import import lazy_import |
70 |
lazy_import(globals(), """ |
|
1474
by Robert Collins
Merge from Aaron Bentley. |
71 |
import errno
|
1770.2.1
by Aaron Bentley
Use create_signature for signing policy, deprecate check_signatures for this |
72 |
from fnmatch import fnmatch
|
73 |
import re
|
|
2900.2.22
by Vincent Ladeuil
Polishing. |
74 |
from cStringIO import StringIO
|
1442.1.2
by Robert Collins
create a config module - there is enough config logic to make this worthwhile, and start testing config processing. |
75 |
|
76 |
import bzrlib
|
|
1996.3.31
by John Arbash Meinel
Make bzrlib.config use lazy importing |
77 |
from bzrlib import (
|
4797.59.2
by Vincent Ladeuil
Use AtomicFile and avoid all unicode/encoding issues around transport (thanks jam). |
78 |
atomicfile,
|
2900.2.10
by Vincent Ladeuil
Add -Dauth handling. |
79 |
debug,
|
1996.3.31
by John Arbash Meinel
Make bzrlib.config use lazy importing |
80 |
errors,
|
5345.5.4
by Vincent Ladeuil
Start implementing config files locking. |
81 |
lockdir,
|
2681.1.8
by Aaron Bentley
Add Thunderbird support to bzr send |
82 |
mail_client,
|
1996.3.31
by John Arbash Meinel
Make bzrlib.config use lazy importing |
83 |
osutils,
|
3757.3.1
by Vincent Ladeuil
Add credential stores plugging. |
84 |
registry,
|
2120.6.9
by James Henstridge
Fixes for issues brought up in John's review |
85 |
symbol_versioning,
|
1551.15.35
by Aaron Bentley
Warn when setting config values that will be masked (#122286) |
86 |
trace,
|
5345.5.4
by Vincent Ladeuil
Start implementing config files locking. |
87 |
transport,
|
2900.2.12
by Vincent Ladeuil
Since all schemes query AuthenticationConfig then prompt user, make that |
88 |
ui,
|
1996.3.31
by John Arbash Meinel
Make bzrlib.config use lazy importing |
89 |
urlutils,
|
2245.4.3
by Alexander Belchenko
config.py: changing _auto_user_id() and config_dir() to use functions from win32utils |
90 |
win32utils,
|
1996.3.31
by John Arbash Meinel
Make bzrlib.config use lazy importing |
91 |
)
|
2991.2.4
by Vincent Ladeuil
Various fixes following local testing environment rebuild. |
92 |
from bzrlib.util.configobj import configobj
|
1996.3.31
by John Arbash Meinel
Make bzrlib.config use lazy importing |
93 |
""") |
94 |
||
1442.1.6
by Robert Collins
first stage major overhaul of configs, giving use BranchConfigs, LocationConfigs and GlobalConfigs |
95 |
|
1442.1.14
by Robert Collins
Create a default signature checking policy of CHECK_IF_POSSIBLE |
96 |
CHECK_IF_POSSIBLE=0 |
97 |
CHECK_ALWAYS=1 |
|
98 |
CHECK_NEVER=2 |
|
99 |
||
100 |
||
1770.2.1
by Aaron Bentley
Use create_signature for signing policy, deprecate check_signatures for this |
101 |
SIGN_WHEN_REQUIRED=0 |
102 |
SIGN_ALWAYS=1 |
|
103 |
SIGN_NEVER=2 |
|
104 |
||
105 |
||
2120.6.1
by James Henstridge
add support for norecurse and appendpath policies when reading configuration files |
106 |
POLICY_NONE = 0 |
107 |
POLICY_NORECURSE = 1 |
|
108 |
POLICY_APPENDPATH = 2 |
|
109 |
||
2120.6.8
by James Henstridge
Change syntax for setting config option policies. Rather than |
110 |
_policy_name = { |
111 |
POLICY_NONE: None, |
|
112 |
POLICY_NORECURSE: 'norecurse', |
|
113 |
POLICY_APPENDPATH: 'appendpath', |
|
114 |
}
|
|
115 |
_policy_value = { |
|
116 |
None: POLICY_NONE, |
|
117 |
'none': POLICY_NONE, |
|
118 |
'norecurse': POLICY_NORECURSE, |
|
119 |
'appendpath': POLICY_APPENDPATH, |
|
120 |
}
|
|
2120.6.4
by James Henstridge
add support for specifying policy when storing options |
121 |
|
122 |
||
123 |
STORE_LOCATION = POLICY_NONE |
|
124 |
STORE_LOCATION_NORECURSE = POLICY_NORECURSE |
|
125 |
STORE_LOCATION_APPENDPATH = POLICY_APPENDPATH |
|
126 |
STORE_BRANCH = 3 |
|
127 |
STORE_GLOBAL = 4 |
|
128 |
||
3224.5.10
by Andrew Bennetts
Replace some duplication with a different form of hackery. |
129 |
_ConfigObj = None |
130 |
def ConfigObj(*args, **kwargs): |
|
131 |
global _ConfigObj |
|
132 |
if _ConfigObj is None: |
|
133 |
class ConfigObj(configobj.ConfigObj): |
|
134 |
||
135 |
def get_bool(self, section, key): |
|
136 |
return self[section].as_bool(key) |
|
137 |
||
138 |
def get_value(self, section, name): |
|
139 |
# Try [] for the old DEFAULT section.
|
|
140 |
if section == "DEFAULT": |
|
141 |
try: |
|
142 |
return self[name] |
|
143 |
except KeyError: |
|
144 |
pass
|
|
145 |
return self[section][name] |
|
146 |
_ConfigObj = ConfigObj |
|
147 |
return _ConfigObj(*args, **kwargs) |
|
1474
by Robert Collins
Merge from Aaron Bentley. |
148 |
|
149 |
||
1442.1.6
by Robert Collins
first stage major overhaul of configs, giving use BranchConfigs, LocationConfigs and GlobalConfigs |
150 |
class Config(object): |
151 |
"""A configuration policy - what username, editor, gpg needs etc.""" |
|
152 |
||
4503.2.2
by Vincent Ladeuil
Get a bool or none from a config file. |
153 |
def __init__(self): |
154 |
super(Config, self).__init__() |
|
155 |
||
1442.1.6
by Robert Collins
first stage major overhaul of configs, giving use BranchConfigs, LocationConfigs and GlobalConfigs |
156 |
def get_editor(self): |
157 |
"""Get the users pop up editor.""" |
|
158 |
raise NotImplementedError |
|
159 |
||
4603.1.10
by Aaron Bentley
Provide change editor via config. |
160 |
def get_change_editor(self, old_tree, new_tree): |
161 |
from bzrlib import diff |
|
162 |
cmd = self._get_change_editor() |
|
163 |
if cmd is None: |
|
164 |
return None |
|
165 |
return diff.DiffFromTool.from_string(cmd, old_tree, new_tree, |
|
166 |
sys.stdout) |
|
167 |
||
168 |
||
2681.1.8
by Aaron Bentley
Add Thunderbird support to bzr send |
169 |
def get_mail_client(self): |
170 |
"""Get a mail client to use""" |
|
171 |
selected_client = self.get_user_option('mail_client') |
|
3638.2.4
by Neil Martinsen-Burrell
Address JAMs review. Dont use register_lazy and dont lazy_import anything other than modules |
172 |
_registry = mail_client.mail_client_registry |
2681.1.10
by Aaron Bentley
Clean up handling of unknown mail clients |
173 |
try: |
3638.2.4
by Neil Martinsen-Burrell
Address JAMs review. Dont use register_lazy and dont lazy_import anything other than modules |
174 |
mail_client_class = _registry.get(selected_client) |
2681.1.10
by Aaron Bentley
Clean up handling of unknown mail clients |
175 |
except KeyError: |
176 |
raise errors.UnknownMailClient(selected_client) |
|
177 |
return mail_client_class(self) |
|
2681.1.8
by Aaron Bentley
Add Thunderbird support to bzr send |
178 |
|
1442.1.15
by Robert Collins
make getting the signature checking policy a template method |
179 |
def _get_signature_checking(self): |
180 |
"""Template method to override signature checking policy.""" |
|
181 |
||
1770.2.1
by Aaron Bentley
Use create_signature for signing policy, deprecate check_signatures for this |
182 |
def _get_signing_policy(self): |
183 |
"""Template method to override signature creation policy.""" |
|
184 |
||
1993.3.6
by James Henstridge
get rid of the recurse argument to get_user_option() |
185 |
def _get_user_option(self, option_name): |
1442.1.69
by Robert Collins
config.Config has a 'get_user_option' call that accepts an option name. |
186 |
"""Template method to provide a user option.""" |
187 |
return None |
|
188 |
||
1993.3.6
by James Henstridge
get rid of the recurse argument to get_user_option() |
189 |
def get_user_option(self, option_name): |
1442.1.69
by Robert Collins
config.Config has a 'get_user_option' call that accepts an option name. |
190 |
"""Get a generic option - no special process, no default.""" |
1993.3.6
by James Henstridge
get rid of the recurse argument to get_user_option() |
191 |
return self._get_user_option(option_name) |
1442.1.69
by Robert Collins
config.Config has a 'get_user_option' call that accepts an option name. |
192 |
|
4503.2.2
by Vincent Ladeuil
Get a bool or none from a config file. |
193 |
def get_user_option_as_bool(self, option_name): |
194 |
"""Get a generic option as a boolean - no special process, no default. |
|
195 |
||
196 |
:return None if the option doesn't exist or its value can't be
|
|
4840.2.4
by Vincent Ladeuil
Implement config.get_user_option_as_list. |
197 |
interpreted as a boolean. Returns True or False otherwise.
|
4503.2.2
by Vincent Ladeuil
Get a bool or none from a config file. |
198 |
"""
|
199 |
s = self._get_user_option(option_name) |
|
4989.2.12
by Vincent Ladeuil
Display a warning if an option value is not boolean. |
200 |
if s is None: |
201 |
# The option doesn't exist
|
|
202 |
return None |
|
4989.2.15
by Vincent Ladeuil
Fixed as per Andrew's review. |
203 |
val = ui.bool_from_string(s) |
4989.2.12
by Vincent Ladeuil
Display a warning if an option value is not boolean. |
204 |
if val is None: |
205 |
# The value can't be interpreted as a boolean
|
|
206 |
trace.warning('Value "%s" is not a boolean for "%s"', |
|
207 |
s, option_name) |
|
208 |
return val |
|
4503.2.2
by Vincent Ladeuil
Get a bool or none from a config file. |
209 |
|
4840.2.4
by Vincent Ladeuil
Implement config.get_user_option_as_list. |
210 |
def get_user_option_as_list(self, option_name): |
211 |
"""Get a generic option as a list - no special process, no default. |
|
212 |
||
213 |
:return None if the option doesn't exist. Returns the value as a list
|
|
214 |
otherwise.
|
|
215 |
"""
|
|
216 |
l = self._get_user_option(option_name) |
|
217 |
if isinstance(l, (str, unicode)): |
|
218 |
# A single value, most probably the user forgot the final ','
|
|
219 |
l = [l] |
|
220 |
return l |
|
221 |
||
1442.1.56
by Robert Collins
gpg_signing_command configuration item |
222 |
def gpg_signing_command(self): |
223 |
"""What program should be used to sign signatures?""" |
|
1442.1.59
by Robert Collins
Add re-sign command to generate a digital signature on a single revision. |
224 |
result = self._gpg_signing_command() |
225 |
if result is None: |
|
226 |
result = "gpg" |
|
227 |
return result |
|
228 |
||
229 |
def _gpg_signing_command(self): |
|
230 |
"""See gpg_signing_command().""" |
|
231 |
return None |
|
1442.1.56
by Robert Collins
gpg_signing_command configuration item |
232 |
|
1553.2.9
by Erik Bågfors
log_formatter => log_format for "named" formatters |
233 |
def log_format(self): |
234 |
"""What log format should be used""" |
|
235 |
result = self._log_format() |
|
1553.2.4
by Erik Bågfors
Support for setting the default log format at a configuration option |
236 |
if result is None: |
237 |
result = "long" |
|
238 |
return result |
|
239 |
||
1553.2.9
by Erik Bågfors
log_formatter => log_format for "named" formatters |
240 |
def _log_format(self): |
241 |
"""See log_format().""" |
|
1553.2.4
by Erik Bågfors
Support for setting the default log format at a configuration option |
242 |
return None |
243 |
||
1472
by Robert Collins
post commit hook, first pass implementation |
244 |
def post_commit(self): |
245 |
"""An ordered list of python functions to call. |
|
246 |
||
247 |
Each function takes branch, rev_id as parameters.
|
|
248 |
"""
|
|
249 |
return self._post_commit() |
|
250 |
||
251 |
def _post_commit(self): |
|
252 |
"""See Config.post_commit.""" |
|
253 |
return None |
|
254 |
||
1442.1.6
by Robert Collins
first stage major overhaul of configs, giving use BranchConfigs, LocationConfigs and GlobalConfigs |
255 |
def user_email(self): |
256 |
"""Return just the email component of a username.""" |
|
1185.33.31
by Martin Pool
Make annotate cope better with revisions committed without a valid |
257 |
return extract_email_address(self.username()) |
1442.1.6
by Robert Collins
first stage major overhaul of configs, giving use BranchConfigs, LocationConfigs and GlobalConfigs |
258 |
|
259 |
def username(self): |
|
260 |
"""Return email-style username. |
|
3943.8.1
by Marius Kruger
remove all trailing whitespace from bzr source |
261 |
|
1442.1.6
by Robert Collins
first stage major overhaul of configs, giving use BranchConfigs, LocationConfigs and GlobalConfigs |
262 |
Something similar to 'Martin Pool <mbp@sourcefrog.net>'
|
3943.8.1
by Marius Kruger
remove all trailing whitespace from bzr source |
263 |
|
5187.2.1
by Parth Malwankar
removed comment about deprecated BZREMAIL. |
264 |
$BZR_EMAIL can be set to override this, then
|
1442.1.6
by Robert Collins
first stage major overhaul of configs, giving use BranchConfigs, LocationConfigs and GlobalConfigs |
265 |
the concrete policy type is checked, and finally
|
1185.37.2
by Jamie Wilkinson
Fix a typo and grammar in Config.username() docstring. |
266 |
$EMAIL is examined.
|
5187.2.12
by Parth Malwankar
trivial clarification in docstring. |
267 |
If no username can be found, errors.NoWhoami exception is raised.
|
3943.8.1
by Marius Kruger
remove all trailing whitespace from bzr source |
268 |
|
1442.1.6
by Robert Collins
first stage major overhaul of configs, giving use BranchConfigs, LocationConfigs and GlobalConfigs |
269 |
TODO: Check it's reasonably well-formed.
|
270 |
"""
|
|
1861.4.1
by Matthieu Moy
BZREMAIL renamed to BZR_EMAIL. |
271 |
v = os.environ.get('BZR_EMAIL') |
272 |
if v: |
|
3224.5.4
by Andrew Bennetts
Fix test suite, mainly weeding out uses of bzrlib.user_encoding. |
273 |
return v.decode(osutils.get_user_encoding()) |
2900.3.1
by Tim Penhey
Removed some annoying trailing whitespace. |
274 |
|
1442.1.6
by Robert Collins
first stage major overhaul of configs, giving use BranchConfigs, LocationConfigs and GlobalConfigs |
275 |
v = self._get_user_id() |
276 |
if v: |
|
277 |
return v |
|
2900.3.1
by Tim Penhey
Removed some annoying trailing whitespace. |
278 |
|
1442.1.6
by Robert Collins
first stage major overhaul of configs, giving use BranchConfigs, LocationConfigs and GlobalConfigs |
279 |
v = os.environ.get('EMAIL') |
280 |
if v: |
|
3224.5.4
by Andrew Bennetts
Fix test suite, mainly weeding out uses of bzrlib.user_encoding. |
281 |
return v.decode(osutils.get_user_encoding()) |
1442.1.6
by Robert Collins
first stage major overhaul of configs, giving use BranchConfigs, LocationConfigs and GlobalConfigs |
282 |
|
5187.2.6
by Parth Malwankar
lockdir no long mandates whoami but uses unicode version of getuser |
283 |
raise errors.NoWhoami() |
5187.2.3
by Parth Malwankar
init and init-repo now fail before creating dir if username is not set. |
284 |
|
285 |
def ensure_username(self): |
|
5187.2.11
by Parth Malwankar
documentation updates |
286 |
"""Raise errors.NoWhoami if username is not set. |
5187.2.3
by Parth Malwankar
init and init-repo now fail before creating dir if username is not set. |
287 |
|
288 |
This method relies on the username() function raising the error.
|
|
289 |
"""
|
|
290 |
self.username() |
|
1442.1.6
by Robert Collins
first stage major overhaul of configs, giving use BranchConfigs, LocationConfigs and GlobalConfigs |
291 |
|
1442.1.14
by Robert Collins
Create a default signature checking policy of CHECK_IF_POSSIBLE |
292 |
def signature_checking(self): |
293 |
"""What is the current policy for signature checking?.""" |
|
1442.1.15
by Robert Collins
make getting the signature checking policy a template method |
294 |
policy = self._get_signature_checking() |
295 |
if policy is not None: |
|
296 |
return policy |
|
1442.1.14
by Robert Collins
Create a default signature checking policy of CHECK_IF_POSSIBLE |
297 |
return CHECK_IF_POSSIBLE |
1442.1.6
by Robert Collins
first stage major overhaul of configs, giving use BranchConfigs, LocationConfigs and GlobalConfigs |
298 |
|
1770.2.1
by Aaron Bentley
Use create_signature for signing policy, deprecate check_signatures for this |
299 |
def signing_policy(self): |
300 |
"""What is the current policy for signature checking?.""" |
|
301 |
policy = self._get_signing_policy() |
|
302 |
if policy is not None: |
|
303 |
return policy |
|
304 |
return SIGN_WHEN_REQUIRED |
|
305 |
||
1442.1.21
by Robert Collins
create signature_needed() call for commit to trigger creating signatures |
306 |
def signature_needed(self): |
307 |
"""Is a signature needed when committing ?.""" |
|
1770.2.1
by Aaron Bentley
Use create_signature for signing policy, deprecate check_signatures for this |
308 |
policy = self._get_signing_policy() |
309 |
if policy is None: |
|
310 |
policy = self._get_signature_checking() |
|
311 |
if policy is not None: |
|
2900.2.10
by Vincent Ladeuil
Add -Dauth handling. |
312 |
trace.warning("Please use create_signatures," |
313 |
" not check_signatures to set signing policy.") |
|
1770.2.1
by Aaron Bentley
Use create_signature for signing policy, deprecate check_signatures for this |
314 |
if policy == CHECK_ALWAYS: |
315 |
return True |
|
316 |
elif policy == SIGN_ALWAYS: |
|
1442.1.21
by Robert Collins
create signature_needed() call for commit to trigger creating signatures |
317 |
return True |
318 |
return False |
|
319 |
||
1553.6.12
by Erik Bågfors
remove AliasConfig, based on input from abentley |
320 |
def get_alias(self, value): |
321 |
return self._get_alias(value) |
|
322 |
||
323 |
def _get_alias(self, value): |
|
324 |
pass
|
|
325 |
||
1770.2.7
by Aaron Bentley
Set/get nickname using BranchConfig |
326 |
def get_nickname(self): |
327 |
return self._get_nickname() |
|
328 |
||
329 |
def _get_nickname(self): |
|
330 |
return None |
|
331 |
||
1551.18.17
by Aaron Bentley
Introduce bzr_remote_path configuration variable |
332 |
def get_bzr_remote_path(self): |
333 |
try: |
|
334 |
return os.environ['BZR_REMOTE_PATH'] |
|
335 |
except KeyError: |
|
336 |
path = self.get_user_option("bzr_remote_path") |
|
337 |
if path is None: |
|
338 |
path = 'bzr' |
|
339 |
return path |
|
340 |
||
4840.2.6
by Vincent Ladeuil
Implement config.suppress_warning. |
341 |
def suppress_warning(self, warning): |
342 |
"""Should the warning be suppressed or emitted. |
|
343 |
||
344 |
:param warning: The name of the warning being tested.
|
|
345 |
||
346 |
:returns: True if the warning should be suppressed, False otherwise.
|
|
347 |
"""
|
|
348 |
warnings = self.get_user_option_as_list('suppress_warnings') |
|
349 |
if warnings is None or warning not in warnings: |
|
350 |
return False |
|
351 |
else: |
|
352 |
return True |
|
353 |
||
1442.1.15
by Robert Collins
make getting the signature checking policy a template method |
354 |
|
1442.1.18
by Robert Collins
permit per branch location overriding of signature checking policy |
355 |
class IniBasedConfig(Config): |
356 |
"""A configuration policy that draws from ini files.""" |
|
1442.1.6
by Robert Collins
first stage major overhaul of configs, giving use BranchConfigs, LocationConfigs and GlobalConfigs |
357 |
|
5345.1.1
by Vincent Ladeuil
Deprecate the get_filename parameter in IniBasedConfig. |
358 |
def __init__(self, get_filename=symbol_versioning.DEPRECATED_PARAMETER, |
5345.2.8
by Vincent Ladeuil
Introduce a 'from_bytes' constructor for config objects. |
359 |
file_name=None): |
5345.2.5
by Vincent Ladeuil
Add docstring. |
360 |
"""Base class for configuration files using an ini-like syntax. |
361 |
||
362 |
:param file_name: The configuration file path.
|
|
363 |
"""
|
|
4503.2.2
by Vincent Ladeuil
Get a bool or none from a config file. |
364 |
super(IniBasedConfig, self).__init__() |
5345.1.8
by Vincent Ladeuil
Make the test_listen_to_the_last_speaker pass and fix fallouts. |
365 |
self.file_name = file_name |
5345.1.1
by Vincent Ladeuil
Deprecate the get_filename parameter in IniBasedConfig. |
366 |
if symbol_versioning.deprecated_passed(get_filename): |
367 |
symbol_versioning.warn( |
|
368 |
'IniBasedConfig.__init__(get_filename) was deprecated in 2.3.'
|
|
369 |
' Use file_name instead.', |
|
370 |
DeprecationWarning, |
|
371 |
stacklevel=2) |
|
5345.1.8
by Vincent Ladeuil
Make the test_listen_to_the_last_speaker pass and fix fallouts. |
372 |
if get_filename is not None: |
5345.1.1
by Vincent Ladeuil
Deprecate the get_filename parameter in IniBasedConfig. |
373 |
self.file_name = get_filename() |
374 |
else: |
|
375 |
self.file_name = file_name |
|
5345.2.8
by Vincent Ladeuil
Introduce a 'from_bytes' constructor for config objects. |
376 |
self._content = None |
4503.2.2
by Vincent Ladeuil
Get a bool or none from a config file. |
377 |
self._parser = None |
378 |
||
5345.2.8
by Vincent Ladeuil
Introduce a 'from_bytes' constructor for config objects. |
379 |
@classmethod
|
5345.1.26
by Vincent Ladeuil
Merge lockable-config-files into remove-gratuitous-ensure-config-dir-exist-calls resolving conflicts |
380 |
def from_string(cls, str_or_unicode, file_name=None, save=False): |
5345.5.13
by Vincent Ladeuil
Merge simplify-test-config-building into lockable-config-files resolving conflicts |
381 |
"""Create a config object from a string. |
5345.2.8
by Vincent Ladeuil
Introduce a 'from_bytes' constructor for config objects. |
382 |
|
5345.2.9
by Vincent Ladeuil
Rename IniBaseConfig.from_bytes to from_string. |
383 |
:param str_or_unicode: A string representing the file content. This will
|
5345.2.8
by Vincent Ladeuil
Introduce a 'from_bytes' constructor for config objects. |
384 |
be utf-8 encoded.
|
5345.1.25
by Vincent Ladeuil
Move the '_save' parameter from '__init__' to 'from_bytes', fix fallouts. |
385 |
|
386 |
:param file_name: The configuration file path.
|
|
387 |
||
388 |
:param _save: Whether the file should be saved upon creation.
|
|
5345.2.8
by Vincent Ladeuil
Introduce a 'from_bytes' constructor for config objects. |
389 |
"""
|
5345.1.25
by Vincent Ladeuil
Move the '_save' parameter from '__init__' to 'from_bytes', fix fallouts. |
390 |
conf = cls(file_name=file_name) |
5345.1.26
by Vincent Ladeuil
Merge lockable-config-files into remove-gratuitous-ensure-config-dir-exist-calls resolving conflicts |
391 |
conf._create_from_string(str_or_unicode, save) |
5345.2.8
by Vincent Ladeuil
Introduce a 'from_bytes' constructor for config objects. |
392 |
return conf |
393 |
||
5345.1.26
by Vincent Ladeuil
Merge lockable-config-files into remove-gratuitous-ensure-config-dir-exist-calls resolving conflicts |
394 |
def _create_from_string(self, str_or_unicode, save): |
5345.5.13
by Vincent Ladeuil
Merge simplify-test-config-building into lockable-config-files resolving conflicts |
395 |
self._content = StringIO(str_or_unicode.encode('utf-8')) |
5345.1.16
by Vincent Ladeuil
Allows tests to save the config file at build time. |
396 |
# Some tests use in-memory configs, some other always need the config
|
397 |
# file to exist on disk.
|
|
5345.1.25
by Vincent Ladeuil
Move the '_save' parameter from '__init__' to 'from_bytes', fix fallouts. |
398 |
if save: |
5345.1.16
by Vincent Ladeuil
Allows tests to save the config file at build time. |
399 |
self._write_config_file() |
5345.5.12
by Vincent Ladeuil
Fix fallouts from replacing '_content' by 'from_bytes' for config files. |
400 |
|
5345.1.4
by Vincent Ladeuil
Deprecate the ``file`` parameter of the ``config._get_parser()`` method. |
401 |
def _get_parser(self, file=symbol_versioning.DEPRECATED_PARAMETER): |
1185.12.51
by Aaron Bentley
Allowed second call of _get_parser() to not require a file |
402 |
if self._parser is not None: |
403 |
return self._parser |
|
5345.1.4
by Vincent Ladeuil
Deprecate the ``file`` parameter of the ``config._get_parser()`` method. |
404 |
if symbol_versioning.deprecated_passed(file): |
405 |
symbol_versioning.warn( |
|
406 |
'IniBasedConfig._get_parser(file=xxx) was deprecated in 2.3.'
|
|
5345.1.5
by Vincent Ladeuil
Fix fallouts by slightly editing the tests. More refactoring avoided to keep the review light. |
407 |
' Use IniBasedConfig(_content=xxx) instead.', |
5345.1.4
by Vincent Ladeuil
Deprecate the ``file`` parameter of the ``config._get_parser()`` method. |
408 |
DeprecationWarning, |
409 |
stacklevel=2) |
|
410 |
if self._content is not None: |
|
411 |
co_input = self._content |
|
412 |
elif self.file_name is None: |
|
413 |
raise AssertionError('We have no content to create the config') |
|
1442.1.6
by Robert Collins
first stage major overhaul of configs, giving use BranchConfigs, LocationConfigs and GlobalConfigs |
414 |
else: |
5345.1.4
by Vincent Ladeuil
Deprecate the ``file`` parameter of the ``config._get_parser()`` method. |
415 |
co_input = self.file_name |
1185.12.51
by Aaron Bentley
Allowed second call of _get_parser() to not require a file |
416 |
try: |
5345.1.4
by Vincent Ladeuil
Deprecate the ``file`` parameter of the ``config._get_parser()`` method. |
417 |
self._parser = ConfigObj(co_input, encoding='utf-8') |
1474
by Robert Collins
Merge from Aaron Bentley. |
418 |
except configobj.ConfigObjError, e: |
1185.12.51
by Aaron Bentley
Allowed second call of _get_parser() to not require a file |
419 |
raise errors.ParseConfigError(e.errors, e.config.filename) |
5345.5.1
by Vincent Ladeuil
Implement config.reload and make sure we have a file name when using it. |
420 |
# Make sure self.reload() will use the right file name
|
5345.1.8
by Vincent Ladeuil
Make the test_listen_to_the_last_speaker pass and fix fallouts. |
421 |
self._parser.filename = self.file_name |
1185.12.49
by Aaron Bentley
Switched to ConfigObj |
422 |
return self._parser |
1442.1.6
by Robert Collins
first stage major overhaul of configs, giving use BranchConfigs, LocationConfigs and GlobalConfigs |
423 |
|
5345.5.1
by Vincent Ladeuil
Implement config.reload and make sure we have a file name when using it. |
424 |
def reload(self): |
425 |
"""Reload the config file from disk.""" |
|
426 |
if self.file_name is None: |
|
427 |
raise AssertionError('We need a file name to reload the config') |
|
428 |
if self._parser is not None: |
|
429 |
self._parser.reload() |
|
430 |
||
1993.3.3
by James Henstridge
make _get_matching_sections() return (section, extra_path) tuples, and adjust other code to match |
431 |
def _get_matching_sections(self): |
432 |
"""Return an ordered list of (section_name, extra_path) pairs. |
|
433 |
||
434 |
If the section contains inherited configuration, extra_path is
|
|
435 |
a string containing the additional path components.
|
|
436 |
"""
|
|
437 |
section = self._get_section() |
|
438 |
if section is not None: |
|
439 |
return [(section, '')] |
|
440 |
else: |
|
441 |
return [] |
|
442 |
||
1442.1.18
by Robert Collins
permit per branch location overriding of signature checking policy |
443 |
def _get_section(self): |
444 |
"""Override this to define the section used by the config.""" |
|
445 |
return "DEFAULT" |
|
446 |
||
2120.6.1
by James Henstridge
add support for norecurse and appendpath policies when reading configuration files |
447 |
def _get_option_policy(self, section, option_name): |
448 |
"""Return the policy for the given (section, option_name) pair.""" |
|
449 |
return POLICY_NONE |
|
450 |
||
4603.1.10
by Aaron Bentley
Provide change editor via config. |
451 |
def _get_change_editor(self): |
452 |
return self.get_user_option('change_editor') |
|
453 |
||
1442.1.16
by Robert Collins
allow global overriding of signature policy to never check |
454 |
def _get_signature_checking(self): |
455 |
"""See Config._get_signature_checking.""" |
|
1474
by Robert Collins
Merge from Aaron Bentley. |
456 |
policy = self._get_user_option('check_signatures') |
457 |
if policy: |
|
458 |
return self._string_to_signature_policy(policy) |
|
1442.1.18
by Robert Collins
permit per branch location overriding of signature checking policy |
459 |
|
1770.2.1
by Aaron Bentley
Use create_signature for signing policy, deprecate check_signatures for this |
460 |
def _get_signing_policy(self): |
1773.4.3
by Martin Pool
[merge] bzr.dev |
461 |
"""See Config._get_signing_policy""" |
1770.2.1
by Aaron Bentley
Use create_signature for signing policy, deprecate check_signatures for this |
462 |
policy = self._get_user_option('create_signatures') |
463 |
if policy: |
|
464 |
return self._string_to_signing_policy(policy) |
|
465 |
||
1442.1.18
by Robert Collins
permit per branch location overriding of signature checking policy |
466 |
def _get_user_id(self): |
467 |
"""Get the user id from the 'email' key in the current section.""" |
|
1474
by Robert Collins
Merge from Aaron Bentley. |
468 |
return self._get_user_option('email') |
1442.1.18
by Robert Collins
permit per branch location overriding of signature checking policy |
469 |
|
1993.3.6
by James Henstridge
get rid of the recurse argument to get_user_option() |
470 |
def _get_user_option(self, option_name): |
1442.1.69
by Robert Collins
config.Config has a 'get_user_option' call that accepts an option name. |
471 |
"""See Config._get_user_option.""" |
1993.3.3
by James Henstridge
make _get_matching_sections() return (section, extra_path) tuples, and adjust other code to match |
472 |
for (section, extra_path) in self._get_matching_sections(): |
473 |
try: |
|
2120.6.1
by James Henstridge
add support for norecurse and appendpath policies when reading configuration files |
474 |
value = self._get_parser().get_value(section, option_name) |
1993.3.3
by James Henstridge
make _get_matching_sections() return (section, extra_path) tuples, and adjust other code to match |
475 |
except KeyError: |
2120.6.1
by James Henstridge
add support for norecurse and appendpath policies when reading configuration files |
476 |
continue
|
477 |
policy = self._get_option_policy(section, option_name) |
|
478 |
if policy == POLICY_NONE: |
|
479 |
return value |
|
480 |
elif policy == POLICY_NORECURSE: |
|
481 |
# norecurse items only apply to the exact path
|
|
482 |
if extra_path: |
|
483 |
continue
|
|
484 |
else: |
|
485 |
return value |
|
486 |
elif policy == POLICY_APPENDPATH: |
|
2120.6.3
by James Henstridge
add some more tests for getting policy options, and behaviour of get_user_option in the presence of config policies |
487 |
if extra_path: |
488 |
value = urlutils.join(value, extra_path) |
|
489 |
return value |
|
2120.6.6
by James Henstridge
fix test_set_push_location test |
490 |
else: |
491 |
raise AssertionError('Unexpected config policy %r' % policy) |
|
1993.3.3
by James Henstridge
make _get_matching_sections() return (section, extra_path) tuples, and adjust other code to match |
492 |
else: |
1993.3.1
by James Henstridge
first go at making location config lookup recursive |
493 |
return None |
1442.1.69
by Robert Collins
config.Config has a 'get_user_option' call that accepts an option name. |
494 |
|
1442.1.59
by Robert Collins
Add re-sign command to generate a digital signature on a single revision. |
495 |
def _gpg_signing_command(self): |
1442.1.56
by Robert Collins
gpg_signing_command configuration item |
496 |
"""See Config.gpg_signing_command.""" |
1472
by Robert Collins
post commit hook, first pass implementation |
497 |
return self._get_user_option('gpg_signing_command') |
1442.1.56
by Robert Collins
gpg_signing_command configuration item |
498 |
|
1553.2.9
by Erik Bågfors
log_formatter => log_format for "named" formatters |
499 |
def _log_format(self): |
500 |
"""See Config.log_format.""" |
|
501 |
return self._get_user_option('log_format') |
|
1553.2.4
by Erik Bågfors
Support for setting the default log format at a configuration option |
502 |
|
1472
by Robert Collins
post commit hook, first pass implementation |
503 |
def _post_commit(self): |
504 |
"""See Config.post_commit.""" |
|
505 |
return self._get_user_option('post_commit') |
|
1442.1.6
by Robert Collins
first stage major overhaul of configs, giving use BranchConfigs, LocationConfigs and GlobalConfigs |
506 |
|
1442.1.16
by Robert Collins
allow global overriding of signature policy to never check |
507 |
def _string_to_signature_policy(self, signature_string): |
508 |
"""Convert a string to a signing policy.""" |
|
1442.1.17
by Robert Collins
allow global overriding of signature policy to force checking, or (pointless but allowed) to set auto checking |
509 |
if signature_string.lower() == 'check-available': |
510 |
return CHECK_IF_POSSIBLE |
|
1442.1.16
by Robert Collins
allow global overriding of signature policy to never check |
511 |
if signature_string.lower() == 'ignore': |
512 |
return CHECK_NEVER |
|
1442.1.17
by Robert Collins
allow global overriding of signature policy to force checking, or (pointless but allowed) to set auto checking |
513 |
if signature_string.lower() == 'require': |
514 |
return CHECK_ALWAYS |
|
1442.1.16
by Robert Collins
allow global overriding of signature policy to never check |
515 |
raise errors.BzrError("Invalid signatures policy '%s'" |
516 |
% signature_string) |
|
1442.1.6
by Robert Collins
first stage major overhaul of configs, giving use BranchConfigs, LocationConfigs and GlobalConfigs |
517 |
|
1770.2.1
by Aaron Bentley
Use create_signature for signing policy, deprecate check_signatures for this |
518 |
def _string_to_signing_policy(self, signature_string): |
519 |
"""Convert a string to a signing policy.""" |
|
520 |
if signature_string.lower() == 'when-required': |
|
521 |
return SIGN_WHEN_REQUIRED |
|
522 |
if signature_string.lower() == 'never': |
|
523 |
return SIGN_NEVER |
|
524 |
if signature_string.lower() == 'always': |
|
525 |
return SIGN_ALWAYS |
|
526 |
raise errors.BzrError("Invalid signing policy '%s'" |
|
527 |
% signature_string) |
|
528 |
||
1553.6.12
by Erik Bågfors
remove AliasConfig, based on input from abentley |
529 |
def _get_alias(self, value): |
530 |
try: |
|
3943.8.1
by Marius Kruger
remove all trailing whitespace from bzr source |
531 |
return self._get_parser().get_value("ALIASES", |
1553.6.12
by Erik Bågfors
remove AliasConfig, based on input from abentley |
532 |
value) |
533 |
except KeyError: |
|
534 |
pass
|
|
535 |
||
1770.2.7
by Aaron Bentley
Set/get nickname using BranchConfig |
536 |
def _get_nickname(self): |
537 |
return self.get_user_option('nickname') |
|
538 |
||
4708.2.1
by Martin
Ensure all files opened by bazaar proper are explicitly closed |
539 |
def _write_config_file(self): |
5345.1.1
by Vincent Ladeuil
Deprecate the get_filename parameter in IniBasedConfig. |
540 |
if self.file_name is None: |
541 |
raise AssertionError('We cannot save, self.file_name is None') |
|
5345.1.9
by Vincent Ladeuil
Refactor config dir check. |
542 |
conf_dir = os.path.dirname(self.file_name) |
543 |
ensure_config_dir_exists(conf_dir) |
|
5345.1.1
by Vincent Ladeuil
Deprecate the get_filename parameter in IniBasedConfig. |
544 |
atomic_file = atomicfile.AtomicFile(self.file_name) |
5050.6.1
by Vincent Ladeuil
Merge 2.1 into 2.2 including fixes for bug #525571 and bug #494221 |
545 |
self._get_parser().write(atomic_file) |
546 |
atomic_file.commit() |
|
547 |
atomic_file.close() |
|
5345.3.3
by Vincent Ladeuil
Merge bzr.dev into deprecate-get-filename resolving conflicts |
548 |
osutils.copy_ownership_from_path(self.file_name) |
4708.2.1
by Martin
Ensure all files opened by bazaar proper are explicitly closed |
549 |
|
1442.1.18
by Robert Collins
permit per branch location overriding of signature checking policy |
550 |
|
5345.5.4
by Vincent Ladeuil
Start implementing config files locking. |
551 |
class LockableConfig(IniBasedConfig): |
552 |
"""A configuration needing explicit locking for access. |
|
553 |
||
554 |
If several processes try to write the config file, the accesses need to be
|
|
555 |
serialized.
|
|
5345.5.8
by Vincent Ladeuil
More doc and ensure that the config is locked when _write_config_file is called. |
556 |
|
557 |
Daughter classes should decorate all methods that update a config with the
|
|
558 |
``@needs_write_lock`` decorator (they call, directly or indirectly, the
|
|
559 |
``_write_config_file()`` method. These methods (typically ``set_option()``
|
|
560 |
and variants must reload the config file from disk before calling
|
|
561 |
``_write_config_file()``), this can be achieved by calling the
|
|
562 |
``self.reload()`` method. Note that the lock scope should cover both the
|
|
563 |
reading and the writing of the config file which is why the decorator can't
|
|
564 |
be applied to ``_write_config_file()`` only.
|
|
565 |
||
566 |
This should be enough to implement the following logic:
|
|
567 |
- lock for exclusive write access,
|
|
568 |
- reload the config file from disk,
|
|
569 |
- set the new value
|
|
570 |
- unlock
|
|
571 |
||
572 |
This logic guarantees that a writer can update a value without erasing an
|
|
573 |
update made by another writer.
|
|
5345.5.4
by Vincent Ladeuil
Start implementing config files locking. |
574 |
"""
|
575 |
||
5345.5.5
by Vincent Ladeuil
Make bb.test_version.TestVersionUnicodeOutput.test_unicode_bzr_home pass. |
576 |
lock_name = 'lock' |
577 |
||
5345.1.25
by Vincent Ladeuil
Move the '_save' parameter from '__init__' to 'from_bytes', fix fallouts. |
578 |
def __init__(self, file_name): |
579 |
super(LockableConfig, self).__init__(file_name=file_name) |
|
5345.5.5
by Vincent Ladeuil
Make bb.test_version.TestVersionUnicodeOutput.test_unicode_bzr_home pass. |
580 |
self.dir = osutils.dirname(osutils.safe_unicode(self.file_name)) |
581 |
self.transport = transport.get_transport(self.dir) |
|
582 |
self._lock = lockdir.LockDir(self.transport, 'lock') |
|
5345.5.4
by Vincent Ladeuil
Start implementing config files locking. |
583 |
|
5345.1.26
by Vincent Ladeuil
Merge lockable-config-files into remove-gratuitous-ensure-config-dir-exist-calls resolving conflicts |
584 |
def _create_from_string(self, unicode_bytes, save): |
585 |
super(LockableConfig, self)._create_from_string(unicode_bytes, False) |
|
5345.1.25
by Vincent Ladeuil
Move the '_save' parameter from '__init__' to 'from_bytes', fix fallouts. |
586 |
if save: |
5345.1.24
by Vincent Ladeuil
Implement _save for LockableConfig too. |
587 |
# We need to handle the saving here (as opposed to IniBasedConfig)
|
588 |
# to be able to lock
|
|
589 |
self.lock_write() |
|
590 |
self._write_config_file() |
|
591 |
self.unlock() |
|
5345.5.4
by Vincent Ladeuil
Start implementing config files locking. |
592 |
|
593 |
def lock_write(self, token=None): |
|
5345.5.8
by Vincent Ladeuil
More doc and ensure that the config is locked when _write_config_file is called. |
594 |
"""Takes a write lock in the directory containing the config file. |
595 |
||
596 |
If the directory doesn't exist it is created.
|
|
597 |
"""
|
|
5345.5.5
by Vincent Ladeuil
Make bb.test_version.TestVersionUnicodeOutput.test_unicode_bzr_home pass. |
598 |
ensure_config_dir_exists(self.dir) |
5345.5.4
by Vincent Ladeuil
Start implementing config files locking. |
599 |
return self._lock.lock_write(token) |
600 |
||
601 |
def unlock(self): |
|
602 |
self._lock.unlock() |
|
603 |
||
5345.5.9
by Vincent Ladeuil
Implements 'bzr lock --config <file>'. |
604 |
def break_lock(self): |
605 |
self._lock.break_lock() |
|
606 |
||
5345.5.8
by Vincent Ladeuil
More doc and ensure that the config is locked when _write_config_file is called. |
607 |
def _write_config_file(self): |
608 |
if self._lock is None or not self._lock.is_held: |
|
609 |
# NB: if the following exception is raised it probably means a
|
|
610 |
# missing @needs_write_lock decorator on one of the callers.
|
|
611 |
raise errors.ObjectNotLocked(self) |
|
612 |
super(LockableConfig, self)._write_config_file() |
|
613 |
||
5345.5.4
by Vincent Ladeuil
Start implementing config files locking. |
614 |
|
615 |
class GlobalConfig(LockableConfig): |
|
1442.1.18
by Robert Collins
permit per branch location overriding of signature checking policy |
616 |
"""The configuration that should be used for a specific location.""" |
617 |
||
5345.2.8
by Vincent Ladeuil
Introduce a 'from_bytes' constructor for config objects. |
618 |
def __init__(self): |
619 |
super(GlobalConfig, self).__init__(file_name=config_filename()) |
|
5345.1.1
by Vincent Ladeuil
Deprecate the get_filename parameter in IniBasedConfig. |
620 |
|
5345.1.25
by Vincent Ladeuil
Move the '_save' parameter from '__init__' to 'from_bytes', fix fallouts. |
621 |
@classmethod
|
5345.1.26
by Vincent Ladeuil
Merge lockable-config-files into remove-gratuitous-ensure-config-dir-exist-calls resolving conflicts |
622 |
def from_string(cls, str_or_unicode, save=False): |
5345.5.13
by Vincent Ladeuil
Merge simplify-test-config-building into lockable-config-files resolving conflicts |
623 |
"""Create a config object from a string. |
5345.1.25
by Vincent Ladeuil
Move the '_save' parameter from '__init__' to 'from_bytes', fix fallouts. |
624 |
|
5345.5.13
by Vincent Ladeuil
Merge simplify-test-config-building into lockable-config-files resolving conflicts |
625 |
:param str_or_unicode: A string representing the file content. This
|
626 |
will be utf-8 encoded.
|
|
5345.1.25
by Vincent Ladeuil
Move the '_save' parameter from '__init__' to 'from_bytes', fix fallouts. |
627 |
|
628 |
:param save: Whether the file should be saved upon creation.
|
|
629 |
"""
|
|
630 |
conf = cls() |
|
5345.1.26
by Vincent Ladeuil
Merge lockable-config-files into remove-gratuitous-ensure-config-dir-exist-calls resolving conflicts |
631 |
conf._create_from_string(str_or_unicode, save) |
5345.1.25
by Vincent Ladeuil
Move the '_save' parameter from '__init__' to 'from_bytes', fix fallouts. |
632 |
return conf |
5345.5.12
by Vincent Ladeuil
Fix fallouts from replacing '_content' by 'from_bytes' for config files. |
633 |
|
1442.1.18
by Robert Collins
permit per branch location overriding of signature checking policy |
634 |
def get_editor(self): |
1474
by Robert Collins
Merge from Aaron Bentley. |
635 |
return self._get_user_option('editor') |
1442.1.18
by Robert Collins
permit per branch location overriding of signature checking policy |
636 |
|
5345.5.4
by Vincent Ladeuil
Start implementing config files locking. |
637 |
@needs_write_lock
|
1816.2.1
by Robey Pointer
add set_user_option to GlobalConfig, and make /etc/passwd username lookup try harder with encodings |
638 |
def set_user_option(self, option, value): |
639 |
"""Save option and its value in the configuration.""" |
|
2900.3.2
by Tim Penhey
A working alias command. |
640 |
self._set_option(option, value, 'DEFAULT') |
641 |
||
642 |
def get_aliases(self): |
|
643 |
"""Return the aliases section.""" |
|
644 |
if 'ALIASES' in self._get_parser(): |
|
645 |
return self._get_parser()['ALIASES'] |
|
646 |
else: |
|
647 |
return {} |
|
648 |
||
5345.5.8
by Vincent Ladeuil
More doc and ensure that the config is locked when _write_config_file is called. |
649 |
@needs_write_lock
|
2900.3.2
by Tim Penhey
A working alias command. |
650 |
def set_alias(self, alias_name, alias_command): |
651 |
"""Save the alias in the configuration.""" |
|
652 |
self._set_option(alias_name, alias_command, 'ALIASES') |
|
653 |
||
5345.5.8
by Vincent Ladeuil
More doc and ensure that the config is locked when _write_config_file is called. |
654 |
@needs_write_lock
|
2900.3.2
by Tim Penhey
A working alias command. |
655 |
def unset_alias(self, alias_name): |
656 |
"""Unset an existing alias.""" |
|
5345.5.10
by Vincent Ladeuil
Add a missing config.reload(). |
657 |
self.reload() |
2900.3.2
by Tim Penhey
A working alias command. |
658 |
aliases = self._get_parser().get('ALIASES') |
2900.3.7
by Tim Penhey
Updates from Aaron's review. |
659 |
if not aliases or alias_name not in aliases: |
660 |
raise errors.NoSuchAlias(alias_name) |
|
2900.3.2
by Tim Penhey
A working alias command. |
661 |
del aliases[alias_name] |
2900.3.12
by Tim Penhey
Final review comments. |
662 |
self._write_config_file() |
2900.3.2
by Tim Penhey
A working alias command. |
663 |
|
664 |
def _set_option(self, option, value, section): |
|
5345.5.1
by Vincent Ladeuil
Implement config.reload and make sure we have a file name when using it. |
665 |
self.reload() |
2900.3.7
by Tim Penhey
Updates from Aaron's review. |
666 |
self._get_parser().setdefault(section, {})[option] = value |
2900.3.12
by Tim Penhey
Final review comments. |
667 |
self._write_config_file() |
2900.3.2
by Tim Penhey
A working alias command. |
668 |
|
1442.1.18
by Robert Collins
permit per branch location overriding of signature checking policy |
669 |
|
5345.5.7
by Vincent Ladeuil
Make LocationConfig use a lock too. |
670 |
class LocationConfig(LockableConfig): |
1442.1.6
by Robert Collins
first stage major overhaul of configs, giving use BranchConfigs, LocationConfigs and GlobalConfigs |
671 |
"""A configuration object that gives the policy for a location.""" |
672 |
||
5345.2.8
by Vincent Ladeuil
Introduce a 'from_bytes' constructor for config objects. |
673 |
def __init__(self, location): |
5345.1.2
by Vincent Ladeuil
Get rid of 'branches.conf' references. |
674 |
super(LocationConfig, self).__init__( |
5345.2.8
by Vincent Ladeuil
Introduce a 'from_bytes' constructor for config objects. |
675 |
file_name=locations_config_filename()) |
1878.1.1
by John Arbash Meinel
Entries in locations.conf should prefer local paths if available (bug #53653) |
676 |
# local file locations are looked up by local path, rather than
|
677 |
# by file url. This is because the config file is a user
|
|
678 |
# file, and we would rather not expose the user to file urls.
|
|
679 |
if location.startswith('file://'): |
|
680 |
location = urlutils.local_path_from_url(location) |
|
1442.1.6
by Robert Collins
first stage major overhaul of configs, giving use BranchConfigs, LocationConfigs and GlobalConfigs |
681 |
self.location = location |
682 |
||
5345.2.8
by Vincent Ladeuil
Introduce a 'from_bytes' constructor for config objects. |
683 |
@classmethod
|
5345.1.26
by Vincent Ladeuil
Merge lockable-config-files into remove-gratuitous-ensure-config-dir-exist-calls resolving conflicts |
684 |
def from_string(cls, str_or_unicode, location, save=False): |
685 |
"""Create a config object from a string. |
|
5345.2.8
by Vincent Ladeuil
Introduce a 'from_bytes' constructor for config objects. |
686 |
|
5345.2.9
by Vincent Ladeuil
Rename IniBaseConfig.from_bytes to from_string. |
687 |
:param str_or_unicode: A string representing the file content. This will
|
5345.2.8
by Vincent Ladeuil
Introduce a 'from_bytes' constructor for config objects. |
688 |
be utf-8 encoded.
|
689 |
||
690 |
:param location: The location url to filter the configuration.
|
|
5345.1.25
by Vincent Ladeuil
Move the '_save' parameter from '__init__' to 'from_bytes', fix fallouts. |
691 |
|
692 |
:param save: Whether the file should be saved upon creation.
|
|
5345.2.8
by Vincent Ladeuil
Introduce a 'from_bytes' constructor for config objects. |
693 |
"""
|
694 |
conf = cls(location) |
|
5345.1.26
by Vincent Ladeuil
Merge lockable-config-files into remove-gratuitous-ensure-config-dir-exist-calls resolving conflicts |
695 |
conf._create_from_string(str_or_unicode, save) |
5345.2.8
by Vincent Ladeuil
Introduce a 'from_bytes' constructor for config objects. |
696 |
return conf |
697 |
||
1993.3.1
by James Henstridge
first go at making location config lookup recursive |
698 |
def _get_matching_sections(self): |
699 |
"""Return an ordered list of section names matching this location.""" |
|
1185.12.49
by Aaron Bentley
Switched to ConfigObj |
700 |
sections = self._get_parser() |
1442.1.12
by Robert Collins
LocationConfig section retrieval falls into my lap |
701 |
location_names = self.location.split('/') |
1442.1.18
by Robert Collins
permit per branch location overriding of signature checking policy |
702 |
if self.location.endswith('/'): |
703 |
del location_names[-1] |
|
1442.1.12
by Robert Collins
LocationConfig section retrieval falls into my lap |
704 |
matches=[] |
1442.1.10
by Robert Collins
explicit over glob test passes |
705 |
for section in sections: |
1878.1.1
by John Arbash Meinel
Entries in locations.conf should prefer local paths if available (bug #53653) |
706 |
# location is a local path if possible, so we need
|
707 |
# to convert 'file://' urls to local paths if necessary.
|
|
708 |
# This also avoids having file:///path be a more exact
|
|
709 |
# match than '/path'.
|
|
710 |
if section.startswith('file://'): |
|
711 |
section_path = urlutils.local_path_from_url(section) |
|
712 |
else: |
|
713 |
section_path = section |
|
714 |
section_names = section_path.split('/') |
|
1442.1.12
by Robert Collins
LocationConfig section retrieval falls into my lap |
715 |
if section.endswith('/'): |
716 |
del section_names[-1] |
|
717 |
names = zip(location_names, section_names) |
|
718 |
matched = True |
|
719 |
for name in names: |
|
720 |
if not fnmatch(name[0], name[1]): |
|
721 |
matched = False |
|
722 |
break
|
|
723 |
if not matched: |
|
724 |
continue
|
|
725 |
# so, for the common prefix they matched.
|
|
726 |
# if section is longer, no match.
|
|
727 |
if len(section_names) > len(location_names): |
|
728 |
continue
|
|
1993.3.3
by James Henstridge
make _get_matching_sections() return (section, extra_path) tuples, and adjust other code to match |
729 |
matches.append((len(section_names), section, |
730 |
'/'.join(location_names[len(section_names):]))) |
|
1442.1.12
by Robert Collins
LocationConfig section retrieval falls into my lap |
731 |
matches.sort(reverse=True) |
1993.3.3
by James Henstridge
make _get_matching_sections() return (section, extra_path) tuples, and adjust other code to match |
732 |
sections = [] |
733 |
for (length, section, extra_path) in matches: |
|
734 |
sections.append((section, extra_path)) |
|
735 |
# should we stop looking for parent configs here?
|
|
1993.3.1
by James Henstridge
first go at making location config lookup recursive |
736 |
try: |
1993.3.3
by James Henstridge
make _get_matching_sections() return (section, extra_path) tuples, and adjust other code to match |
737 |
if self._get_parser()[section].as_bool('ignore_parents'): |
738 |
break
|
|
1993.3.1
by James Henstridge
first go at making location config lookup recursive |
739 |
except KeyError: |
740 |
pass
|
|
1993.3.3
by James Henstridge
make _get_matching_sections() return (section, extra_path) tuples, and adjust other code to match |
741 |
return sections |
1442.1.9
by Robert Collins
exact section test passes |
742 |
|
2120.6.1
by James Henstridge
add support for norecurse and appendpath policies when reading configuration files |
743 |
def _get_option_policy(self, section, option_name): |
744 |
"""Return the policy for the given (section, option_name) pair.""" |
|
745 |
# check for the old 'recurse=False' flag
|
|
746 |
try: |
|
747 |
recurse = self._get_parser()[section].as_bool('recurse') |
|
748 |
except KeyError: |
|
749 |
recurse = True |
|
750 |
if not recurse: |
|
751 |
return POLICY_NORECURSE |
|
752 |
||
2120.6.10
by James Henstridge
Catch another deprecation warning, and more cleanup |
753 |
policy_key = option_name + ':policy' |
2120.6.8
by James Henstridge
Change syntax for setting config option policies. Rather than |
754 |
try: |
755 |
policy_name = self._get_parser()[section][policy_key] |
|
756 |
except KeyError: |
|
757 |
policy_name = None |
|
2120.6.1
by James Henstridge
add support for norecurse and appendpath policies when reading configuration files |
758 |
|
2120.6.8
by James Henstridge
Change syntax for setting config option policies. Rather than |
759 |
return _policy_value[policy_name] |
2120.6.1
by James Henstridge
add support for norecurse and appendpath policies when reading configuration files |
760 |
|
2120.6.4
by James Henstridge
add support for specifying policy when storing options |
761 |
def _set_option_policy(self, section, option_name, option_policy): |
762 |
"""Set the policy for the given option name in the given section.""" |
|
763 |
# The old recurse=False option affects all options in the
|
|
764 |
# section. To handle multiple policies in the section, we
|
|
765 |
# need to convert it to a policy_norecurse key.
|
|
766 |
try: |
|
767 |
recurse = self._get_parser()[section].as_bool('recurse') |
|
768 |
except KeyError: |
|
769 |
pass
|
|
770 |
else: |
|
2120.6.9
by James Henstridge
Fixes for issues brought up in John's review |
771 |
symbol_versioning.warn( |
2120.6.11
by James Henstridge
s/0.13/0.14/ in deprecation warning |
772 |
'The recurse option is deprecated as of 0.14. '
|
2120.6.9
by James Henstridge
Fixes for issues brought up in John's review |
773 |
'The section "%s" has been converted to use policies.' |
774 |
% section, |
|
775 |
DeprecationWarning) |
|
2120.6.4
by James Henstridge
add support for specifying policy when storing options |
776 |
del self._get_parser()[section]['recurse'] |
2120.6.9
by James Henstridge
Fixes for issues brought up in John's review |
777 |
if not recurse: |
778 |
for key in self._get_parser()[section].keys(): |
|
779 |
if not key.endswith(':policy'): |
|
780 |
self._get_parser()[section][key + |
|
781 |
':policy'] = 'norecurse' |
|
2120.6.4
by James Henstridge
add support for specifying policy when storing options |
782 |
|
2120.6.9
by James Henstridge
Fixes for issues brought up in John's review |
783 |
policy_key = option_name + ':policy' |
2120.6.8
by James Henstridge
Change syntax for setting config option policies. Rather than |
784 |
policy_name = _policy_name[option_policy] |
785 |
if policy_name is not None: |
|
786 |
self._get_parser()[section][policy_key] = policy_name |
|
787 |
else: |
|
788 |
if policy_key in self._get_parser()[section]: |
|
789 |
del self._get_parser()[section][policy_key] |
|
2120.6.4
by James Henstridge
add support for specifying policy when storing options |
790 |
|
5345.5.7
by Vincent Ladeuil
Make LocationConfig use a lock too. |
791 |
@needs_write_lock
|
2120.6.4
by James Henstridge
add support for specifying policy when storing options |
792 |
def set_user_option(self, option, value, store=STORE_LOCATION): |
1490
by Robert Collins
Implement a 'bzr push' command, with saved locations; update diff to return 1. |
793 |
"""Save option and its value in the configuration.""" |
3376.2.4
by Martin Pool
Remove every assert statement from bzrlib! |
794 |
if store not in [STORE_LOCATION, |
2120.6.4
by James Henstridge
add support for specifying policy when storing options |
795 |
STORE_LOCATION_NORECURSE, |
3376.2.4
by Martin Pool
Remove every assert statement from bzrlib! |
796 |
STORE_LOCATION_APPENDPATH]: |
797 |
raise ValueError('bad storage policy %r for %r' % |
|
798 |
(store, option)) |
|
5345.5.1
by Vincent Ladeuil
Implement config.reload and make sure we have a file name when using it. |
799 |
self.reload() |
1490
by Robert Collins
Implement a 'bzr push' command, with saved locations; update diff to return 1. |
800 |
location = self.location |
801 |
if location.endswith('/'): |
|
802 |
location = location[:-1] |
|
5345.1.24
by Vincent Ladeuil
Implement _save for LockableConfig too. |
803 |
parser = self._get_parser() |
5345.1.21
by Vincent Ladeuil
Slight rewrite to make the method more readable. |
804 |
if not location in parser and not location + '/' in parser: |
805 |
parser[location] = {} |
|
806 |
elif location + '/' in parser: |
|
1490
by Robert Collins
Implement a 'bzr push' command, with saved locations; update diff to return 1. |
807 |
location = location + '/' |
5345.1.21
by Vincent Ladeuil
Slight rewrite to make the method more readable. |
808 |
parser[location][option]=value |
2120.6.4
by James Henstridge
add support for specifying policy when storing options |
809 |
# the allowed values of store match the config policies
|
810 |
self._set_option_policy(location, option, store) |
|
4708.2.1
by Martin
Ensure all files opened by bazaar proper are explicitly closed |
811 |
self._write_config_file() |
1490
by Robert Collins
Implement a 'bzr push' command, with saved locations; update diff to return 1. |
812 |
|
1442.1.6
by Robert Collins
first stage major overhaul of configs, giving use BranchConfigs, LocationConfigs and GlobalConfigs |
813 |
|
814 |
class BranchConfig(Config): |
|
815 |
"""A configuration object giving the policy for a branch.""" |
|
816 |
||
5345.1.3
by Vincent Ladeuil
Make __init__ the first method in the BranchConfig class. |
817 |
def __init__(self, branch): |
818 |
super(BranchConfig, self).__init__() |
|
819 |
self._location_config = None |
|
820 |
self._branch_data_config = None |
|
821 |
self._global_config = None |
|
822 |
self.branch = branch |
|
823 |
self.option_sources = (self._get_location_config, |
|
824 |
self._get_branch_data_config, |
|
825 |
self._get_global_config) |
|
826 |
||
1770.2.5
by Aaron Bentley
Integrate branch.conf into BranchConfig |
827 |
def _get_branch_data_config(self): |
828 |
if self._branch_data_config is None: |
|
829 |
self._branch_data_config = TreeConfig(self.branch) |
|
830 |
return self._branch_data_config |
|
831 |
||
1442.1.6
by Robert Collins
first stage major overhaul of configs, giving use BranchConfigs, LocationConfigs and GlobalConfigs |
832 |
def _get_location_config(self): |
833 |
if self._location_config is None: |
|
834 |
self._location_config = LocationConfig(self.branch.base) |
|
835 |
return self._location_config |
|
836 |
||
1770.2.5
by Aaron Bentley
Integrate branch.conf into BranchConfig |
837 |
def _get_global_config(self): |
838 |
if self._global_config is None: |
|
839 |
self._global_config = GlobalConfig() |
|
840 |
return self._global_config |
|
841 |
||
842 |
def _get_best_value(self, option_name): |
|
843 |
"""This returns a user option from local, tree or global config. |
|
844 |
||
845 |
They are tried in that order. Use get_safe_value if trusted values
|
|
846 |
are necessary.
|
|
847 |
"""
|
|
848 |
for source in self.option_sources: |
|
849 |
value = getattr(source(), option_name)() |
|
850 |
if value is not None: |
|
851 |
return value |
|
852 |
return None |
|
853 |
||
854 |
def _get_safe_value(self, option_name): |
|
855 |
"""This variant of get_best_value never returns untrusted values. |
|
3943.8.1
by Marius Kruger
remove all trailing whitespace from bzr source |
856 |
|
1770.2.5
by Aaron Bentley
Integrate branch.conf into BranchConfig |
857 |
It does not return values from the branch data, because the branch may
|
858 |
not be controlled by the user.
|
|
859 |
||
860 |
We may wish to allow locations.conf to control whether branches are
|
|
861 |
trusted in the future.
|
|
862 |
"""
|
|
863 |
for source in (self._get_location_config, self._get_global_config): |
|
864 |
value = getattr(source(), option_name)() |
|
865 |
if value is not None: |
|
866 |
return value |
|
867 |
return None |
|
868 |
||
1442.1.6
by Robert Collins
first stage major overhaul of configs, giving use BranchConfigs, LocationConfigs and GlobalConfigs |
869 |
def _get_user_id(self): |
870 |
"""Return the full user id for the branch. |
|
3943.8.1
by Marius Kruger
remove all trailing whitespace from bzr source |
871 |
|
3407.2.14
by Martin Pool
Remove more cases of getting transport via control_files |
872 |
e.g. "John Hacker <jhacker@example.com>"
|
1442.1.6
by Robert Collins
first stage major overhaul of configs, giving use BranchConfigs, LocationConfigs and GlobalConfigs |
873 |
This is looked up in the email controlfile for the branch.
|
874 |
"""
|
|
875 |
try: |
|
3407.2.16
by Martin Pool
Remove RemoteBranch reliance on control_files._transport |
876 |
return (self.branch._transport.get_bytes("email") |
3224.5.4
by Andrew Bennetts
Fix test suite, mainly weeding out uses of bzrlib.user_encoding. |
877 |
.decode(osutils.get_user_encoding()) |
1442.1.6
by Robert Collins
first stage major overhaul of configs, giving use BranchConfigs, LocationConfigs and GlobalConfigs |
878 |
.rstrip("\r\n")) |
879 |
except errors.NoSuchFile, e: |
|
880 |
pass
|
|
3943.8.1
by Marius Kruger
remove all trailing whitespace from bzr source |
881 |
|
1770.2.5
by Aaron Bentley
Integrate branch.conf into BranchConfig |
882 |
return self._get_best_value('_get_user_id') |
1442.1.6
by Robert Collins
first stage major overhaul of configs, giving use BranchConfigs, LocationConfigs and GlobalConfigs |
883 |
|
4603.1.10
by Aaron Bentley
Provide change editor via config. |
884 |
def _get_change_editor(self): |
885 |
return self._get_best_value('_get_change_editor') |
|
886 |
||
1442.1.19
by Robert Collins
BranchConfigs inherit signature_checking policy from their LocationConfig. |
887 |
def _get_signature_checking(self): |
888 |
"""See Config._get_signature_checking.""" |
|
1770.2.5
by Aaron Bentley
Integrate branch.conf into BranchConfig |
889 |
return self._get_best_value('_get_signature_checking') |
1442.1.19
by Robert Collins
BranchConfigs inherit signature_checking policy from their LocationConfig. |
890 |
|
1770.2.1
by Aaron Bentley
Use create_signature for signing policy, deprecate check_signatures for this |
891 |
def _get_signing_policy(self): |
892 |
"""See Config._get_signing_policy.""" |
|
1770.2.5
by Aaron Bentley
Integrate branch.conf into BranchConfig |
893 |
return self._get_best_value('_get_signing_policy') |
1770.2.1
by Aaron Bentley
Use create_signature for signing policy, deprecate check_signatures for this |
894 |
|
1993.3.6
by James Henstridge
get rid of the recurse argument to get_user_option() |
895 |
def _get_user_option(self, option_name): |
1442.1.69
by Robert Collins
config.Config has a 'get_user_option' call that accepts an option name. |
896 |
"""See Config._get_user_option.""" |
1770.2.5
by Aaron Bentley
Integrate branch.conf into BranchConfig |
897 |
for source in self.option_sources: |
1993.3.6
by James Henstridge
get rid of the recurse argument to get_user_option() |
898 |
value = source()._get_user_option(option_name) |
1770.2.5
by Aaron Bentley
Integrate branch.conf into BranchConfig |
899 |
if value is not None: |
900 |
return value |
|
901 |
return None |
|
902 |
||
1551.15.35
by Aaron Bentley
Warn when setting config values that will be masked (#122286) |
903 |
def set_user_option(self, name, value, store=STORE_BRANCH, |
904 |
warn_masked=False): |
|
2120.6.4
by James Henstridge
add support for specifying policy when storing options |
905 |
if store == STORE_BRANCH: |
1770.2.6
by Aaron Bentley
Ensure branch.conf works properly |
906 |
self._get_branch_data_config().set_option(value, name) |
2120.6.4
by James Henstridge
add support for specifying policy when storing options |
907 |
elif store == STORE_GLOBAL: |
2120.6.7
by James Henstridge
Fix GlobalConfig.set_user_option() call |
908 |
self._get_global_config().set_user_option(name, value) |
2120.6.4
by James Henstridge
add support for specifying policy when storing options |
909 |
else: |
910 |
self._get_location_config().set_user_option(name, value, store) |
|
1551.15.35
by Aaron Bentley
Warn when setting config values that will be masked (#122286) |
911 |
if not warn_masked: |
912 |
return
|
|
913 |
if store in (STORE_GLOBAL, STORE_BRANCH): |
|
914 |
mask_value = self._get_location_config().get_user_option(name) |
|
915 |
if mask_value is not None: |
|
916 |
trace.warning('Value "%s" is masked by "%s" from' |
|
917 |
' locations.conf', value, mask_value) |
|
918 |
else: |
|
919 |
if store == STORE_GLOBAL: |
|
920 |
branch_config = self._get_branch_data_config() |
|
921 |
mask_value = branch_config.get_user_option(name) |
|
922 |
if mask_value is not None: |
|
923 |
trace.warning('Value "%s" is masked by "%s" from' |
|
1551.15.37
by Aaron Bentley
Don't treat a format string as a normal string |
924 |
' branch.conf', value, mask_value) |
1551.15.35
by Aaron Bentley
Warn when setting config values that will be masked (#122286) |
925 |
|
1442.1.59
by Robert Collins
Add re-sign command to generate a digital signature on a single revision. |
926 |
def _gpg_signing_command(self): |
1442.1.56
by Robert Collins
gpg_signing_command configuration item |
927 |
"""See Config.gpg_signing_command.""" |
1770.2.5
by Aaron Bentley
Integrate branch.conf into BranchConfig |
928 |
return self._get_safe_value('_gpg_signing_command') |
3943.8.1
by Marius Kruger
remove all trailing whitespace from bzr source |
929 |
|
1472
by Robert Collins
post commit hook, first pass implementation |
930 |
def _post_commit(self): |
931 |
"""See Config.post_commit.""" |
|
1770.2.5
by Aaron Bentley
Integrate branch.conf into BranchConfig |
932 |
return self._get_safe_value('_post_commit') |
1472
by Robert Collins
post commit hook, first pass implementation |
933 |
|
1770.2.7
by Aaron Bentley
Set/get nickname using BranchConfig |
934 |
def _get_nickname(self): |
1824.1.1
by Robert Collins
Add BranchConfig.has_explicit_nickname call. |
935 |
value = self._get_explicit_nickname() |
1770.2.7
by Aaron Bentley
Set/get nickname using BranchConfig |
936 |
if value is not None: |
937 |
return value |
|
2120.5.2
by Alexander Belchenko
(jam) Fix for bug #66857 |
938 |
return urlutils.unescape(self.branch.base.split('/')[-2]) |
1770.2.7
by Aaron Bentley
Set/get nickname using BranchConfig |
939 |
|
1824.1.1
by Robert Collins
Add BranchConfig.has_explicit_nickname call. |
940 |
def has_explicit_nickname(self): |
941 |
"""Return true if a nickname has been explicitly assigned.""" |
|
942 |
return self._get_explicit_nickname() is not None |
|
943 |
||
944 |
def _get_explicit_nickname(self): |
|
945 |
return self._get_best_value('_get_nickname') |
|
946 |
||
1553.2.9
by Erik Bågfors
log_formatter => log_format for "named" formatters |
947 |
def _log_format(self): |
948 |
"""See Config.log_format.""" |
|
1770.2.5
by Aaron Bentley
Integrate branch.conf into BranchConfig |
949 |
return self._get_best_value('_log_format') |
1442.1.1
by Robert Collins
move config_dir into bzrlib.config |
950 |
|
1553.6.12
by Erik Bågfors
remove AliasConfig, based on input from abentley |
951 |
|
1185.31.43
by John Arbash Meinel
Reintroduced ensure_config_dir_exists() for sftp |
952 |
def ensure_config_dir_exists(path=None): |
953 |
"""Make sure a configuration directory exists. |
|
954 |
This makes sure that the directory exists.
|
|
955 |
On windows, since configuration directories are 2 levels deep,
|
|
956 |
it makes sure both the directory and the parent directory exists.
|
|
957 |
"""
|
|
958 |
if path is None: |
|
959 |
path = config_dir() |
|
960 |
if not os.path.isdir(path): |
|
961 |
if sys.platform == 'win32': |
|
962 |
parent_dir = os.path.dirname(path) |
|
963 |
if not os.path.isdir(parent_dir): |
|
2900.2.10
by Vincent Ladeuil
Add -Dauth handling. |
964 |
trace.mutter('creating config parent directory: %r', parent_dir) |
1185.31.43
by John Arbash Meinel
Reintroduced ensure_config_dir_exists() for sftp |
965 |
os.mkdir(parent_dir) |
2900.2.10
by Vincent Ladeuil
Add -Dauth handling. |
966 |
trace.mutter('creating config directory: %r', path) |
5116.2.4
by Parth Malwankar
removed mkdir_with_ownership as its probably cleaner to just use copy_ownership |
967 |
os.mkdir(path) |
5116.2.6
by Parth Malwankar
renamed copy_ownership to copy_ownership_from_path. |
968 |
osutils.copy_ownership_from_path(path) |
1185.31.43
by John Arbash Meinel
Reintroduced ensure_config_dir_exists() for sftp |
969 |
|
1532
by Robert Collins
Merge in John Meinels integration branch. |
970 |
|
1442.1.1
by Robert Collins
move config_dir into bzrlib.config |
971 |
def config_dir(): |
972 |
"""Return per-user configuration directory. |
|
973 |
||
1442.1.2
by Robert Collins
create a config module - there is enough config logic to make this worthwhile, and start testing config processing. |
974 |
By default this is ~/.bazaar/
|
3943.8.1
by Marius Kruger
remove all trailing whitespace from bzr source |
975 |
|
1442.1.1
by Robert Collins
move config_dir into bzrlib.config |
976 |
TODO: Global option --config-dir to override this.
|
977 |
"""
|
|
1185.38.1
by John Arbash Meinel
Adding my win32 patch for moving the home directory. |
978 |
base = os.environ.get('BZR_HOME', None) |
979 |
if sys.platform == 'win32': |
|
980 |
if base is None: |
|
2245.4.3
by Alexander Belchenko
config.py: changing _auto_user_id() and config_dir() to use functions from win32utils |
981 |
base = win32utils.get_appdata_location_unicode() |
1185.38.1
by John Arbash Meinel
Adding my win32 patch for moving the home directory. |
982 |
if base is None: |
983 |
base = os.environ.get('HOME', None) |
|
984 |
if base is None: |
|
2991.2.2
by Vincent Ladeuil
No tests worth adding after upgrading to configobj-4.4.0. |
985 |
raise errors.BzrError('You must have one of BZR_HOME, APPDATA,' |
986 |
' or HOME set') |
|
1996.3.31
by John Arbash Meinel
Make bzrlib.config use lazy importing |
987 |
return osutils.pathjoin(base, 'bazaar', '2.0') |
1185.38.1
by John Arbash Meinel
Adding my win32 patch for moving the home directory. |
988 |
else: |
989 |
if base is None: |
|
990 |
base = os.path.expanduser("~") |
|
1996.3.31
by John Arbash Meinel
Make bzrlib.config use lazy importing |
991 |
return osutils.pathjoin(base, ".bazaar") |
1185.31.32
by John Arbash Meinel
Updated the bzr sourcecode to use bzrlib.osutils.pathjoin rather than os.path.join to enforce internal use of / instead of \ |
992 |
|
993 |
||
1442.1.2
by Robert Collins
create a config module - there is enough config logic to make this worthwhile, and start testing config processing. |
994 |
def config_filename(): |
995 |
"""Return per-user configuration ini file filename.""" |
|
1996.3.31
by John Arbash Meinel
Make bzrlib.config use lazy importing |
996 |
return osutils.pathjoin(config_dir(), 'bazaar.conf') |
1442.1.2
by Robert Collins
create a config module - there is enough config logic to make this worthwhile, and start testing config processing. |
997 |
|
998 |
||
1770.2.2
by Aaron Bentley
Rename branches.conf to locations.conf |
999 |
def locations_config_filename(): |
1000 |
"""Return per-user configuration ini file filename.""" |
|
1996.3.31
by John Arbash Meinel
Make bzrlib.config use lazy importing |
1001 |
return osutils.pathjoin(config_dir(), 'locations.conf') |
1770.2.2
by Aaron Bentley
Rename branches.conf to locations.conf |
1002 |
|
1442.1.2
by Robert Collins
create a config module - there is enough config logic to make this worthwhile, and start testing config processing. |
1003 |
|
2900.2.3
by Vincent Ladeuil
Credentials matching implementation. |
1004 |
def authentication_config_filename(): |
1005 |
"""Return per-user authentication ini file filename.""" |
|
1006 |
return osutils.pathjoin(config_dir(), 'authentication.conf') |
|
1007 |
||
1008 |
||
1836.1.6
by John Arbash Meinel
Creating a helper function for getting the user ignore filename |
1009 |
def user_ignore_config_filename(): |
1010 |
"""Return the user default ignore filename""" |
|
1996.3.31
by John Arbash Meinel
Make bzrlib.config use lazy importing |
1011 |
return osutils.pathjoin(config_dir(), 'ignore') |
1836.1.6
by John Arbash Meinel
Creating a helper function for getting the user ignore filename |
1012 |
|
1013 |
||
4584.3.4
by Martin Pool
Add crash_dir and xdg_cache_dir functions |
1014 |
def crash_dir(): |
1015 |
"""Return the directory name to store crash files. |
|
1016 |
||
1017 |
This doesn't implicitly create it.
|
|
1018 |
||
4634.128.2
by Martin Pool
Write crash files into /var/crash where apport can see them. |
1019 |
On Windows it's in the config directory; elsewhere it's /var/crash
|
4634.128.18
by Martin Pool
Update apport crash tests |
1020 |
which may be monitored by apport. It can be overridden by
|
1021 |
$APPORT_CRASH_DIR.
|
|
4584.3.4
by Martin Pool
Add crash_dir and xdg_cache_dir functions |
1022 |
"""
|
1023 |
if sys.platform == 'win32': |
|
1024 |
return osutils.pathjoin(config_dir(), 'Crash') |
|
1025 |
else: |
|
4634.128.2
by Martin Pool
Write crash files into /var/crash where apport can see them. |
1026 |
# XXX: hardcoded in apport_python_hook.py; therefore here too -- mbp
|
1027 |
# 2010-01-31
|
|
4634.128.18
by Martin Pool
Update apport crash tests |
1028 |
return os.environ.get('APPORT_CRASH_DIR', '/var/crash') |
4584.3.4
by Martin Pool
Add crash_dir and xdg_cache_dir functions |
1029 |
|
1030 |
||
1031 |
def xdg_cache_dir(): |
|
4584.3.23
by Martin Pool
Correction to xdg_cache_dir and add a simple test |
1032 |
# See http://standards.freedesktop.org/basedir-spec/latest/ar01s03.html
|
1033 |
# Possibly this should be different on Windows?
|
|
1034 |
e = os.environ.get('XDG_CACHE_DIR', None) |
|
1035 |
if e: |
|
1036 |
return e |
|
1037 |
else: |
|
1038 |
return os.path.expanduser('~/.cache') |
|
4584.3.4
by Martin Pool
Add crash_dir and xdg_cache_dir functions |
1039 |
|
1040 |
||
3063.3.2
by Lukáš Lalinský
Move the name and e-mail address extraction logic to config.parse_username. |
1041 |
def parse_username(username): |
1042 |
"""Parse e-mail username and return a (name, address) tuple.""" |
|
1043 |
match = re.match(r'(.*?)\s*<?([\w+.-]+@[\w+.-]+)>?', username) |
|
1044 |
if match is None: |
|
1045 |
return (username, '') |
|
1046 |
else: |
|
1047 |
return (match.group(1), match.group(2)) |
|
1048 |
||
1049 |
||
1185.16.52
by Martin Pool
- add extract_email_address |
1050 |
def extract_email_address(e): |
1051 |
"""Return just the address part of an email string. |
|
3063.3.2
by Lukáš Lalinský
Move the name and e-mail address extraction logic to config.parse_username. |
1052 |
|
3943.8.1
by Marius Kruger
remove all trailing whitespace from bzr source |
1053 |
That is just the user@domain part, nothing else.
|
1185.16.52
by Martin Pool
- add extract_email_address |
1054 |
This part is required to contain only ascii characters.
|
1055 |
If it can't be extracted, raises an error.
|
|
3063.3.2
by Lukáš Lalinský
Move the name and e-mail address extraction logic to config.parse_username. |
1056 |
|
1185.16.52
by Martin Pool
- add extract_email_address |
1057 |
>>> extract_email_address('Jane Tester <jane@test.com>')
|
1058 |
"jane@test.com"
|
|
1059 |
"""
|
|
3063.3.2
by Lukáš Lalinský
Move the name and e-mail address extraction logic to config.parse_username. |
1060 |
name, email = parse_username(e) |
1061 |
if not email: |
|
2055.2.2
by John Arbash Meinel
Switch extract_email_address() to use a more specific exception |
1062 |
raise errors.NoEmailInUsername(e) |
3063.3.2
by Lukáš Lalinský
Move the name and e-mail address extraction logic to config.parse_username. |
1063 |
return email |
1185.35.11
by Aaron Bentley
Added support for branch nicks |
1064 |
|
1185.85.30
by John Arbash Meinel
Fixing 'bzr push' exposed that IniBasedConfig didn't handle unicode. |
1065 |
|
1770.2.5
by Aaron Bentley
Integrate branch.conf into BranchConfig |
1066 |
class TreeConfig(IniBasedConfig): |
1185.35.11
by Aaron Bentley
Added support for branch nicks |
1067 |
"""Branch configuration data associated with its contents, not location""" |
2900.2.3
by Vincent Ladeuil
Credentials matching implementation. |
1068 |
|
3408.3.1
by Martin Pool
Remove erroneous handling of branch.conf for RemoteBranch |
1069 |
# XXX: Really needs a better name, as this is not part of the tree! -- mbp 20080507
|
1070 |
||
1185.35.11
by Aaron Bentley
Added support for branch nicks |
1071 |
def __init__(self, branch): |
4226.1.5
by Robert Collins
Reinstate the use of the Branch.get_config_file verb. |
1072 |
self._config = branch._get_config() |
1185.35.11
by Aaron Bentley
Added support for branch nicks |
1073 |
self.branch = branch |
1074 |
||
1770.2.5
by Aaron Bentley
Integrate branch.conf into BranchConfig |
1075 |
def _get_parser(self, file=None): |
1076 |
if file is not None: |
|
1077 |
return IniBasedConfig._get_parser(file) |
|
3242.1.2
by Aaron Bentley
Turn BzrDirConfig into TransportConfig, reduce code duplication |
1078 |
return self._config._get_configobj() |
1185.35.11
by Aaron Bentley
Added support for branch nicks |
1079 |
|
1080 |
def get_option(self, name, section=None, default=None): |
|
1081 |
self.branch.lock_read() |
|
1082 |
try: |
|
3242.1.2
by Aaron Bentley
Turn BzrDirConfig into TransportConfig, reduce code duplication |
1083 |
return self._config.get_option(name, section, default) |
1185.35.11
by Aaron Bentley
Added support for branch nicks |
1084 |
finally: |
1085 |
self.branch.unlock() |
|
1086 |
||
1087 |
def set_option(self, value, name, section=None): |
|
1088 |
"""Set a per-branch configuration option""" |
|
1089 |
self.branch.lock_write() |
|
1090 |
try: |
|
3242.1.2
by Aaron Bentley
Turn BzrDirConfig into TransportConfig, reduce code duplication |
1091 |
self._config.set_option(value, name, section) |
1185.35.11
by Aaron Bentley
Added support for branch nicks |
1092 |
finally: |
1093 |
self.branch.unlock() |
|
2900.2.3
by Vincent Ladeuil
Credentials matching implementation. |
1094 |
|
1095 |
||
1096 |
class AuthenticationConfig(object): |
|
1097 |
"""The authentication configuration file based on a ini file. |
|
1098 |
||
1099 |
Implements the authentication.conf file described in
|
|
1100 |
doc/developers/authentication-ring.txt.
|
|
1101 |
"""
|
|
1102 |
||
1103 |
def __init__(self, _file=None): |
|
2900.2.4
by Vincent Ladeuil
Cosmetic changes. |
1104 |
self._config = None # The ConfigObj |
2900.2.3
by Vincent Ladeuil
Credentials matching implementation. |
1105 |
if _file is None: |
2900.2.24
by Vincent Ladeuil
Review feedback. |
1106 |
self._filename = authentication_config_filename() |
1107 |
self._input = self._filename = authentication_config_filename() |
|
2900.2.3
by Vincent Ladeuil
Credentials matching implementation. |
1108 |
else: |
2900.2.24
by Vincent Ladeuil
Review feedback. |
1109 |
# Tests can provide a string as _file
|
1110 |
self._filename = None |
|
2900.2.3
by Vincent Ladeuil
Credentials matching implementation. |
1111 |
self._input = _file |
1112 |
||
1113 |
def _get_config(self): |
|
1114 |
if self._config is not None: |
|
1115 |
return self._config |
|
1116 |
try: |
|
2900.2.22
by Vincent Ladeuil
Polishing. |
1117 |
# FIXME: Should we validate something here ? Includes: empty
|
1118 |
# sections are useless, at least one of
|
|
1119 |
# user/password/password_encoding should be defined, etc.
|
|
2900.2.4
by Vincent Ladeuil
Cosmetic changes. |
1120 |
|
1121 |
# Note: the encoding below declares that the file itself is utf-8
|
|
1122 |
# encoded, but the values in the ConfigObj are always Unicode.
|
|
2900.2.3
by Vincent Ladeuil
Credentials matching implementation. |
1123 |
self._config = ConfigObj(self._input, encoding='utf-8') |
1124 |
except configobj.ConfigObjError, e: |
|
1125 |
raise errors.ParseConfigError(e.errors, e.config.filename) |
|
1126 |
return self._config |
|
1127 |
||
2900.2.5
by Vincent Ladeuil
ake ftp aware of authentication config. |
1128 |
def _save(self): |
1129 |
"""Save the config file, only tests should use it for now.""" |
|
2900.2.26
by Vincent Ladeuil
Fix forgotten reference to _get_filename and duplicated code. |
1130 |
conf_dir = os.path.dirname(self._filename) |
2900.2.5
by Vincent Ladeuil
ake ftp aware of authentication config. |
1131 |
ensure_config_dir_exists(conf_dir) |
4708.2.2
by Martin
Workingtree changes sitting around since November, more explict closing of files in bzrlib |
1132 |
f = file(self._filename, 'wb') |
1133 |
try: |
|
1134 |
self._get_config().write(f) |
|
1135 |
finally: |
|
1136 |
f.close() |
|
2900.2.5
by Vincent Ladeuil
ake ftp aware of authentication config. |
1137 |
|
1138 |
def _set_option(self, section_name, option_name, value): |
|
1139 |
"""Set an authentication configuration option""" |
|
1140 |
conf = self._get_config() |
|
1141 |
section = conf.get(section_name) |
|
1142 |
if section is None: |
|
1143 |
conf[section] = {} |
|
1144 |
section = conf[section] |
|
1145 |
section[option_name] = value |
|
1146 |
self._save() |
|
1147 |
||
4081.1.1
by Jean-Francois Roy
A 'realm' optional argument was added to the get_credentials and set_credentials |
1148 |
def get_credentials(self, scheme, host, port=None, user=None, path=None, |
1149 |
realm=None): |
|
2900.2.4
by Vincent Ladeuil
Cosmetic changes. |
1150 |
"""Returns the matching credentials from authentication.conf file. |
1151 |
||
1152 |
:param scheme: protocol
|
|
1153 |
||
1154 |
:param host: the server address
|
|
1155 |
||
1156 |
:param port: the associated port (optional)
|
|
1157 |
||
1158 |
:param user: login (optional)
|
|
1159 |
||
1160 |
:param path: the absolute path on the server (optional)
|
|
4081.1.1
by Jean-Francois Roy
A 'realm' optional argument was added to the get_credentials and set_credentials |
1161 |
|
1162 |
:param realm: the http authentication realm (optional)
|
|
2900.2.4
by Vincent Ladeuil
Cosmetic changes. |
1163 |
|
2900.2.15
by Vincent Ladeuil
AuthenticationConfig can be queried for logins too (first step). |
1164 |
:return: A dict containing the matching credentials or None.
|
2900.2.4
by Vincent Ladeuil
Cosmetic changes. |
1165 |
This includes:
|
1166 |
- name: the section name of the credentials in the
|
|
1167 |
authentication.conf file,
|
|
4081.1.1
by Jean-Francois Roy
A 'realm' optional argument was added to the get_credentials and set_credentials |
1168 |
- user: can't be different from the provided user if any,
|
4107.1.7
by Jean-Francois Roy
No longer deleting the extra credentials keys in get_credentials. |
1169 |
- scheme: the server protocol,
|
1170 |
- host: the server address,
|
|
1171 |
- port: the server port (can be None),
|
|
1172 |
- path: the absolute server path (can be None),
|
|
1173 |
- realm: the http specific authentication realm (can be None),
|
|
2900.2.15
by Vincent Ladeuil
AuthenticationConfig can be queried for logins too (first step). |
1174 |
- password: the decoded password, could be None if the credential
|
1175 |
defines only the user
|
|
2900.2.4
by Vincent Ladeuil
Cosmetic changes. |
1176 |
- verify_certificates: https specific, True if the server
|
1177 |
certificate should be verified, False otherwise.
|
|
1178 |
"""
|
|
2900.2.3
by Vincent Ladeuil
Credentials matching implementation. |
1179 |
credentials = None |
1180 |
for auth_def_name, auth_def in self._get_config().items(): |
|
3418.2.1
by Vincent Ladeuil
Fix #217650 by catching declarations outside sections. |
1181 |
if type(auth_def) is not configobj.Section: |
1182 |
raise ValueError("%s defined outside a section" % auth_def_name) |
|
1183 |
||
2900.2.5
by Vincent Ladeuil
ake ftp aware of authentication config. |
1184 |
a_scheme, a_host, a_user, a_path = map( |
1185 |
auth_def.get, ['scheme', 'host', 'user', 'path']) |
|
2900.2.4
by Vincent Ladeuil
Cosmetic changes. |
1186 |
|
2900.2.5
by Vincent Ladeuil
ake ftp aware of authentication config. |
1187 |
try: |
1188 |
a_port = auth_def.as_int('port') |
|
1189 |
except KeyError: |
|
1190 |
a_port = None |
|
2900.2.22
by Vincent Ladeuil
Polishing. |
1191 |
except ValueError: |
1192 |
raise ValueError("'port' not numeric in %s" % auth_def_name) |
|
2900.2.4
by Vincent Ladeuil
Cosmetic changes. |
1193 |
try: |
1194 |
a_verify_certificates = auth_def.as_bool('verify_certificates') |
|
1195 |
except KeyError: |
|
1196 |
a_verify_certificates = True |
|
2900.2.22
by Vincent Ladeuil
Polishing. |
1197 |
except ValueError: |
1198 |
raise ValueError( |
|
1199 |
"'verify_certificates' not boolean in %s" % auth_def_name) |
|
2900.2.4
by Vincent Ladeuil
Cosmetic changes. |
1200 |
|
1201 |
# Attempt matching
|
|
2900.2.3
by Vincent Ladeuil
Credentials matching implementation. |
1202 |
if a_scheme is not None and scheme != a_scheme: |
1203 |
continue
|
|
1204 |
if a_host is not None: |
|
1205 |
if not (host == a_host |
|
1206 |
or (a_host.startswith('.') and host.endswith(a_host))): |
|
1207 |
continue
|
|
2900.2.4
by Vincent Ladeuil
Cosmetic changes. |
1208 |
if a_port is not None and port != a_port: |
2900.2.3
by Vincent Ladeuil
Credentials matching implementation. |
1209 |
continue
|
1210 |
if (a_path is not None and path is not None |
|
1211 |
and not path.startswith(a_path)): |
|
1212 |
continue
|
|
1213 |
if (a_user is not None and user is not None |
|
1214 |
and a_user != user): |
|
2900.2.10
by Vincent Ladeuil
Add -Dauth handling. |
1215 |
# Never contradict the caller about the user to be used
|
2900.2.3
by Vincent Ladeuil
Credentials matching implementation. |
1216 |
continue
|
2900.2.15
by Vincent Ladeuil
AuthenticationConfig can be queried for logins too (first step). |
1217 |
if a_user is None: |
2900.2.3
by Vincent Ladeuil
Credentials matching implementation. |
1218 |
# Can't find a user
|
1219 |
continue
|
|
4081.1.1
by Jean-Francois Roy
A 'realm' optional argument was added to the get_credentials and set_credentials |
1220 |
# Prepare a credentials dictionary with additional keys
|
1221 |
# for the credential providers
|
|
2900.2.24
by Vincent Ladeuil
Review feedback. |
1222 |
credentials = dict(name=auth_def_name, |
3418.4.2
by Vincent Ladeuil
Fix bug #199440 by taking into account that a section may not |
1223 |
user=a_user, |
4081.1.1
by Jean-Francois Roy
A 'realm' optional argument was added to the get_credentials and set_credentials |
1224 |
scheme=a_scheme, |
1225 |
host=host, |
|
1226 |
port=port, |
|
1227 |
path=path, |
|
1228 |
realm=realm, |
|
3418.4.2
by Vincent Ladeuil
Fix bug #199440 by taking into account that a section may not |
1229 |
password=auth_def.get('password', None), |
2900.2.24
by Vincent Ladeuil
Review feedback. |
1230 |
verify_certificates=a_verify_certificates) |
4081.1.1
by Jean-Francois Roy
A 'realm' optional argument was added to the get_credentials and set_credentials |
1231 |
# Decode the password in the credentials (or get one)
|
2900.2.22
by Vincent Ladeuil
Polishing. |
1232 |
self.decode_password(credentials, |
1233 |
auth_def.get('password_encoding', None)) |
|
2900.2.10
by Vincent Ladeuil
Add -Dauth handling. |
1234 |
if 'auth' in debug.debug_flags: |
1235 |
trace.mutter("Using authentication section: %r", auth_def_name) |
|
2900.2.3
by Vincent Ladeuil
Credentials matching implementation. |
1236 |
break
|
1237 |
||
4283.1.1
by Jelmer Vernooij
Support fallback credential stores. |
1238 |
if credentials is None: |
1239 |
# No credentials were found in authentication.conf, try the fallback
|
|
1240 |
# credentials stores.
|
|
1241 |
credentials = credential_store_registry.get_fallback_credentials( |
|
1242 |
scheme, host, port, user, path, realm) |
|
1243 |
||
2900.2.3
by Vincent Ladeuil
Credentials matching implementation. |
1244 |
return credentials |
1245 |
||
3777.3.2
by Aaron Bentley
Reverse order of scheme and password |
1246 |
def set_credentials(self, name, host, user, scheme=None, password=None, |
4081.1.1
by Jean-Francois Roy
A 'realm' optional argument was added to the get_credentials and set_credentials |
1247 |
port=None, path=None, verify_certificates=None, |
1248 |
realm=None): |
|
3777.3.1
by Aaron Bentley
Update docs |
1249 |
"""Set authentication credentials for a host. |
1250 |
||
1251 |
Any existing credentials with matching scheme, host, port and path
|
|
1252 |
will be deleted, regardless of name.
|
|
1253 |
||
1254 |
:param name: An arbitrary name to describe this set of credentials.
|
|
1255 |
:param host: Name of the host that accepts these credentials.
|
|
1256 |
:param user: The username portion of these credentials.
|
|
1257 |
:param scheme: The URL scheme (e.g. ssh, http) the credentials apply
|
|
1258 |
to.
|
|
3777.3.2
by Aaron Bentley
Reverse order of scheme and password |
1259 |
:param password: Password portion of these credentials.
|
3777.3.1
by Aaron Bentley
Update docs |
1260 |
:param port: The IP port on the host that these credentials apply to.
|
1261 |
:param path: A filesystem path on the host that these credentials
|
|
1262 |
apply to.
|
|
1263 |
:param verify_certificates: On https, verify server certificates if
|
|
1264 |
True.
|
|
4081.1.1
by Jean-Francois Roy
A 'realm' optional argument was added to the get_credentials and set_credentials |
1265 |
:param realm: The http authentication realm (optional).
|
3777.3.1
by Aaron Bentley
Update docs |
1266 |
"""
|
3777.1.8
by Aaron Bentley
Commit work-in-progress |
1267 |
values = {'host': host, 'user': user} |
1268 |
if password is not None: |
|
1269 |
values['password'] = password |
|
1270 |
if scheme is not None: |
|
1271 |
values['scheme'] = scheme |
|
1272 |
if port is not None: |
|
1273 |
values['port'] = '%d' % port |
|
1274 |
if path is not None: |
|
1275 |
values['path'] = path |
|
3777.1.10
by Aaron Bentley
Ensure credentials are stored |
1276 |
if verify_certificates is not None: |
1277 |
values['verify_certificates'] = str(verify_certificates) |
|
4081.1.1
by Jean-Francois Roy
A 'realm' optional argument was added to the get_credentials and set_credentials |
1278 |
if realm is not None: |
1279 |
values['realm'] = realm |
|
3777.1.11
by Aaron Bentley
Ensure changed-name updates clear old values |
1280 |
config = self._get_config() |
1281 |
for_deletion = [] |
|
1282 |
for section, existing_values in config.items(): |
|
4081.1.1
by Jean-Francois Roy
A 'realm' optional argument was added to the get_credentials and set_credentials |
1283 |
for key in ('scheme', 'host', 'port', 'path', 'realm'): |
3777.1.11
by Aaron Bentley
Ensure changed-name updates clear old values |
1284 |
if existing_values.get(key) != values.get(key): |
1285 |
break
|
|
1286 |
else: |
|
1287 |
del config[section] |
|
1288 |
config.update({name: values}) |
|
3777.1.10
by Aaron Bentley
Ensure credentials are stored |
1289 |
self._save() |
3777.1.8
by Aaron Bentley
Commit work-in-progress |
1290 |
|
4304.2.1
by Vincent Ladeuil
Fix bug #367726 by reverting some default user handling introduced |
1291 |
def get_user(self, scheme, host, port=None, realm=None, path=None, |
4222.3.10
by Jelmer Vernooij
Avoid using the default username in the case of SMTP. |
1292 |
prompt=None, ask=False, default=None): |
2900.2.15
by Vincent Ladeuil
AuthenticationConfig can be queried for logins too (first step). |
1293 |
"""Get a user from authentication file. |
1294 |
||
1295 |
:param scheme: protocol
|
|
1296 |
||
1297 |
:param host: the server address
|
|
1298 |
||
1299 |
:param port: the associated port (optional)
|
|
1300 |
||
1301 |
:param realm: the realm sent by the server (optional)
|
|
1302 |
||
1303 |
:param path: the absolute path on the server (optional)
|
|
1304 |
||
4222.3.4
by Jelmer Vernooij
Default to getpass.getuser() in AuthenticationConfig.get_user(), but allow |
1305 |
:param ask: Ask the user if there is no explicitly configured username
|
1306 |
(optional)
|
|
1307 |
||
4304.2.1
by Vincent Ladeuil
Fix bug #367726 by reverting some default user handling introduced |
1308 |
:param default: The username returned if none is defined (optional).
|
1309 |
||
2900.2.15
by Vincent Ladeuil
AuthenticationConfig can be queried for logins too (first step). |
1310 |
:return: The found user.
|
1311 |
"""
|
|
2900.2.16
by Vincent Ladeuil
Make hhtp proxy aware of AuthenticationConfig (for password). |
1312 |
credentials = self.get_credentials(scheme, host, port, user=None, |
4081.1.1
by Jean-Francois Roy
A 'realm' optional argument was added to the get_credentials and set_credentials |
1313 |
path=path, realm=realm) |
2900.2.15
by Vincent Ladeuil
AuthenticationConfig can be queried for logins too (first step). |
1314 |
if credentials is not None: |
1315 |
user = credentials['user'] |
|
1316 |
else: |
|
1317 |
user = None |
|
4222.3.2
by Jelmer Vernooij
Prompt for user names if they are not in the configuration. |
1318 |
if user is None: |
4222.3.4
by Jelmer Vernooij
Default to getpass.getuser() in AuthenticationConfig.get_user(), but allow |
1319 |
if ask: |
1320 |
if prompt is None: |
|
1321 |
# Create a default prompt suitable for most cases
|
|
1322 |
prompt = scheme.upper() + ' %(host)s username' |
|
1323 |
# Special handling for optional fields in the prompt
|
|
1324 |
if port is not None: |
|
1325 |
prompt_host = '%s:%d' % (host, port) |
|
1326 |
else: |
|
1327 |
prompt_host = host |
|
1328 |
user = ui.ui_factory.get_username(prompt, host=prompt_host) |
|
4222.3.2
by Jelmer Vernooij
Prompt for user names if they are not in the configuration. |
1329 |
else: |
4222.3.10
by Jelmer Vernooij
Avoid using the default username in the case of SMTP. |
1330 |
user = default |
2900.2.15
by Vincent Ladeuil
AuthenticationConfig can be queried for logins too (first step). |
1331 |
return user |
1332 |
||
2900.2.12
by Vincent Ladeuil
Since all schemes query AuthenticationConfig then prompt user, make that |
1333 |
def get_password(self, scheme, host, user, port=None, |
1334 |
realm=None, path=None, prompt=None): |
|
1335 |
"""Get a password from authentication file or prompt the user for one. |
|
1336 |
||
1337 |
:param scheme: protocol
|
|
1338 |
||
1339 |
:param host: the server address
|
|
1340 |
||
1341 |
:param port: the associated port (optional)
|
|
1342 |
||
1343 |
:param user: login
|
|
1344 |
||
1345 |
:param realm: the realm sent by the server (optional)
|
|
1346 |
||
1347 |
:param path: the absolute path on the server (optional)
|
|
1348 |
||
1349 |
:return: The found password or the one entered by the user.
|
|
1350 |
"""
|
|
4081.1.1
by Jean-Francois Roy
A 'realm' optional argument was added to the get_credentials and set_credentials |
1351 |
credentials = self.get_credentials(scheme, host, port, user, path, |
1352 |
realm) |
|
2900.2.12
by Vincent Ladeuil
Since all schemes query AuthenticationConfig then prompt user, make that |
1353 |
if credentials is not None: |
1354 |
password = credentials['password'] |
|
3420.1.3
by Vincent Ladeuil
John's review feedback. |
1355 |
if password is not None and scheme is 'ssh': |
3420.1.2
by Vincent Ladeuil
Fix bug #203186 by ignoring passwords for ssh and warning user. |
1356 |
trace.warning('password ignored in section [%s],' |
1357 |
' use an ssh agent instead'
|
|
1358 |
% credentials['name']) |
|
1359 |
password = None |
|
2900.2.16
by Vincent Ladeuil
Make hhtp proxy aware of AuthenticationConfig (for password). |
1360 |
else: |
1361 |
password = None |
|
2900.2.19
by Vincent Ladeuil
Mention proxy and https in the password prompts, with tests. |
1362 |
# Prompt user only if we could't find a password
|
2900.2.15
by Vincent Ladeuil
AuthenticationConfig can be queried for logins too (first step). |
1363 |
if password is None: |
2900.2.12
by Vincent Ladeuil
Since all schemes query AuthenticationConfig then prompt user, make that |
1364 |
if prompt is None: |
3420.1.2
by Vincent Ladeuil
Fix bug #203186 by ignoring passwords for ssh and warning user. |
1365 |
# Create a default prompt suitable for most cases
|
2900.2.19
by Vincent Ladeuil
Mention proxy and https in the password prompts, with tests. |
1366 |
prompt = '%s' % scheme.upper() + ' %(user)s@%(host)s password' |
2900.2.12
by Vincent Ladeuil
Since all schemes query AuthenticationConfig then prompt user, make that |
1367 |
# Special handling for optional fields in the prompt
|
1368 |
if port is not None: |
|
1369 |
prompt_host = '%s:%d' % (host, port) |
|
1370 |
else: |
|
1371 |
prompt_host = host |
|
2900.2.19
by Vincent Ladeuil
Mention proxy and https in the password prompts, with tests. |
1372 |
password = ui.ui_factory.get_password(prompt, |
1373 |
host=prompt_host, user=user) |
|
2900.2.12
by Vincent Ladeuil
Since all schemes query AuthenticationConfig then prompt user, make that |
1374 |
return password |
1375 |
||
2900.2.22
by Vincent Ladeuil
Polishing. |
1376 |
def decode_password(self, credentials, encoding): |
3757.3.1
by Vincent Ladeuil
Add credential stores plugging. |
1377 |
try: |
1378 |
cs = credential_store_registry.get_credential_store(encoding) |
|
1379 |
except KeyError: |
|
1380 |
raise ValueError('%r is not a known password_encoding' % encoding) |
|
1381 |
credentials['password'] = cs.decode_password(credentials) |
|
2900.2.22
by Vincent Ladeuil
Polishing. |
1382 |
return credentials |
3242.1.1
by Aaron Bentley
Implement BzrDir configuration |
1383 |
|
3242.3.17
by Aaron Bentley
Whitespace cleanup |
1384 |
|
3757.3.1
by Vincent Ladeuil
Add credential stores plugging. |
1385 |
class CredentialStoreRegistry(registry.Registry): |
1386 |
"""A class that registers credential stores. |
|
1387 |
||
1388 |
A credential store provides access to credentials via the password_encoding
|
|
1389 |
field in authentication.conf sections.
|
|
1390 |
||
4283.1.1
by Jelmer Vernooij
Support fallback credential stores. |
1391 |
Except for stores provided by bzr itself, most stores are expected to be
|
3757.3.1
by Vincent Ladeuil
Add credential stores plugging. |
1392 |
provided by plugins that will therefore use
|
1393 |
register_lazy(password_encoding, module_name, member_name, help=help,
|
|
4283.1.1
by Jelmer Vernooij
Support fallback credential stores. |
1394 |
fallback=fallback) to install themselves.
|
4283.2.1
by Vincent Ladeuil
Add a test and cleanup some PEP8 issues. |
1395 |
|
1396 |
A fallback credential store is one that is queried if no credentials can be
|
|
1397 |
found via authentication.conf.
|
|
3757.3.1
by Vincent Ladeuil
Add credential stores plugging. |
1398 |
"""
|
1399 |
||
1400 |
def get_credential_store(self, encoding=None): |
|
1401 |
cs = self.get(encoding) |
|
1402 |
if callable(cs): |
|
1403 |
cs = cs() |
|
1404 |
return cs |
|
1405 |
||
4283.1.2
by Jelmer Vernooij
Add tests, NEWS item. |
1406 |
def is_fallback(self, name): |
4283.2.1
by Vincent Ladeuil
Add a test and cleanup some PEP8 issues. |
1407 |
"""Check if the named credentials store should be used as fallback.""" |
4283.1.2
by Jelmer Vernooij
Add tests, NEWS item. |
1408 |
return self.get_info(name) |
1409 |
||
4283.1.1
by Jelmer Vernooij
Support fallback credential stores. |
1410 |
def get_fallback_credentials(self, scheme, host, port=None, user=None, |
4283.1.2
by Jelmer Vernooij
Add tests, NEWS item. |
1411 |
path=None, realm=None): |
1412 |
"""Request credentials from all fallback credentials stores. |
|
4283.2.1
by Vincent Ladeuil
Add a test and cleanup some PEP8 issues. |
1413 |
|
1414 |
The first credentials store that can provide credentials wins.
|
|
4283.1.2
by Jelmer Vernooij
Add tests, NEWS item. |
1415 |
"""
|
4283.2.1
by Vincent Ladeuil
Add a test and cleanup some PEP8 issues. |
1416 |
credentials = None |
4283.1.1
by Jelmer Vernooij
Support fallback credential stores. |
1417 |
for name in self.keys(): |
4283.1.2
by Jelmer Vernooij
Add tests, NEWS item. |
1418 |
if not self.is_fallback(name): |
4283.1.1
by Jelmer Vernooij
Support fallback credential stores. |
1419 |
continue
|
1420 |
cs = self.get_credential_store(name) |
|
4283.2.1
by Vincent Ladeuil
Add a test and cleanup some PEP8 issues. |
1421 |
credentials = cs.get_credentials(scheme, host, port, user, |
1422 |
path, realm) |
|
4283.1.1
by Jelmer Vernooij
Support fallback credential stores. |
1423 |
if credentials is not None: |
4283.2.1
by Vincent Ladeuil
Add a test and cleanup some PEP8 issues. |
1424 |
# We found some credentials
|
1425 |
break
|
|
1426 |
return credentials |
|
4283.1.1
by Jelmer Vernooij
Support fallback credential stores. |
1427 |
|
1428 |
def register(self, key, obj, help=None, override_existing=False, |
|
4283.2.1
by Vincent Ladeuil
Add a test and cleanup some PEP8 issues. |
1429 |
fallback=False): |
4283.1.1
by Jelmer Vernooij
Support fallback credential stores. |
1430 |
"""Register a new object to a name. |
1431 |
||
1432 |
:param key: This is the key to use to request the object later.
|
|
1433 |
:param obj: The object to register.
|
|
1434 |
:param help: Help text for this entry. This may be a string or
|
|
1435 |
a callable. If it is a callable, it should take two
|
|
1436 |
parameters (registry, key): this registry and the key that
|
|
1437 |
the help was registered under.
|
|
1438 |
:param override_existing: Raise KeyErorr if False and something has
|
|
1439 |
already been registered for that key. If True, ignore if there
|
|
1440 |
is an existing key (always register the new value).
|
|
1441 |
:param fallback: Whether this credential store should be
|
|
1442 |
used as fallback.
|
|
1443 |
"""
|
|
4283.2.1
by Vincent Ladeuil
Add a test and cleanup some PEP8 issues. |
1444 |
return super(CredentialStoreRegistry, |
1445 |
self).register(key, obj, help, info=fallback, |
|
1446 |
override_existing=override_existing) |
|
4283.1.1
by Jelmer Vernooij
Support fallback credential stores. |
1447 |
|
1448 |
def register_lazy(self, key, module_name, member_name, |
|
4283.2.1
by Vincent Ladeuil
Add a test and cleanup some PEP8 issues. |
1449 |
help=None, override_existing=False, |
4283.1.1
by Jelmer Vernooij
Support fallback credential stores. |
1450 |
fallback=False): |
1451 |
"""Register a new credential store to be loaded on request. |
|
1452 |
||
1453 |
:param module_name: The python path to the module. Such as 'os.path'.
|
|
1454 |
:param member_name: The member of the module to return. If empty or
|
|
1455 |
None, get() will return the module itself.
|
|
1456 |
:param help: Help text for this entry. This may be a string or
|
|
1457 |
a callable.
|
|
1458 |
:param override_existing: If True, replace the existing object
|
|
1459 |
with the new one. If False, if there is already something
|
|
1460 |
registered with the same key, raise a KeyError
|
|
1461 |
:param fallback: Whether this credential store should be
|
|
1462 |
used as fallback.
|
|
1463 |
"""
|
|
1464 |
return super(CredentialStoreRegistry, self).register_lazy( |
|
1465 |
key, module_name, member_name, help, |
|
1466 |
info=fallback, override_existing=override_existing) |
|
1467 |
||
3757.3.1
by Vincent Ladeuil
Add credential stores plugging. |
1468 |
|
1469 |
credential_store_registry = CredentialStoreRegistry() |
|
1470 |
||
1471 |
||
1472 |
class CredentialStore(object): |
|
1473 |
"""An abstract class to implement storage for credentials""" |
|
1474 |
||
1475 |
def decode_password(self, credentials): |
|
4283.2.1
by Vincent Ladeuil
Add a test and cleanup some PEP8 issues. |
1476 |
"""Returns a clear text password for the provided credentials.""" |
3757.3.1
by Vincent Ladeuil
Add credential stores plugging. |
1477 |
raise NotImplementedError(self.decode_password) |
1478 |
||
4283.2.1
by Vincent Ladeuil
Add a test and cleanup some PEP8 issues. |
1479 |
def get_credentials(self, scheme, host, port=None, user=None, path=None, |
4283.1.1
by Jelmer Vernooij
Support fallback credential stores. |
1480 |
realm=None): |
1481 |
"""Return the matching credentials from this credential store. |
|
1482 |
||
1483 |
This method is only called on fallback credential stores.
|
|
1484 |
"""
|
|
1485 |
raise NotImplementedError(self.get_credentials) |
|
1486 |
||
1487 |
||
3757.3.1
by Vincent Ladeuil
Add credential stores plugging. |
1488 |
|
1489 |
class PlainTextCredentialStore(CredentialStore): |
|
5131.2.1
by Martin
Permit bzrlib to run under python -OO by explictly assigning to __doc__ for user-visible docstrings |
1490 |
__doc__ = """Plain text credential store for the authentication.conf file""" |
3757.3.1
by Vincent Ladeuil
Add credential stores plugging. |
1491 |
|
1492 |
def decode_password(self, credentials): |
|
1493 |
"""See CredentialStore.decode_password.""" |
|
1494 |
return credentials['password'] |
|
1495 |
||
1496 |
||
1497 |
credential_store_registry.register('plain', PlainTextCredentialStore, |
|
1498 |
help=PlainTextCredentialStore.__doc__) |
|
1499 |
credential_store_registry.default_key = 'plain' |
|
1500 |
||
1501 |
||
3242.3.14
by Aaron Bentley
Make BzrDirConfig use TransportConfig |
1502 |
class BzrDirConfig(object): |
1503 |
||
4288.1.1
by Robert Collins
Add support for a RemoteBzrDirConfig to support optimising push operations which need to look for default stacking locations. |
1504 |
def __init__(self, bzrdir): |
1505 |
self._bzrdir = bzrdir |
|
1506 |
self._config = bzrdir._get_config() |
|
3242.1.1
by Aaron Bentley
Implement BzrDir configuration |
1507 |
|
3242.3.11
by Aaron Bentley
Clean up BzrDirConfig usage |
1508 |
def set_default_stack_on(self, value): |
1509 |
"""Set the default stacking location. |
|
1510 |
||
1511 |
It may be set to a location, or None.
|
|
1512 |
||
1513 |
This policy affects all branches contained by this bzrdir, except for
|
|
1514 |
those under repositories.
|
|
1515 |
"""
|
|
4288.1.1
by Robert Collins
Add support for a RemoteBzrDirConfig to support optimising push operations which need to look for default stacking locations. |
1516 |
if self._config is None: |
1517 |
raise errors.BzrError("Cannot set configuration in %s" % self._bzrdir) |
|
3242.3.11
by Aaron Bentley
Clean up BzrDirConfig usage |
1518 |
if value is None: |
3242.3.14
by Aaron Bentley
Make BzrDirConfig use TransportConfig |
1519 |
self._config.set_option('', 'default_stack_on') |
3242.3.11
by Aaron Bentley
Clean up BzrDirConfig usage |
1520 |
else: |
3242.3.14
by Aaron Bentley
Make BzrDirConfig use TransportConfig |
1521 |
self._config.set_option(value, 'default_stack_on') |
3242.3.11
by Aaron Bentley
Clean up BzrDirConfig usage |
1522 |
|
1523 |
def get_default_stack_on(self): |
|
1524 |
"""Return the default stacking location. |
|
1525 |
||
1526 |
This will either be a location, or None.
|
|
1527 |
||
1528 |
This policy affects all branches contained by this bzrdir, except for
|
|
1529 |
those under repositories.
|
|
1530 |
"""
|
|
4288.1.1
by Robert Collins
Add support for a RemoteBzrDirConfig to support optimising push operations which need to look for default stacking locations. |
1531 |
if self._config is None: |
1532 |
return None |
|
3242.3.14
by Aaron Bentley
Make BzrDirConfig use TransportConfig |
1533 |
value = self._config.get_option('default_stack_on') |
3242.3.11
by Aaron Bentley
Clean up BzrDirConfig usage |
1534 |
if value == '': |
1535 |
value = None |
|
1536 |
return value |
|
1537 |
||
3242.3.14
by Aaron Bentley
Make BzrDirConfig use TransportConfig |
1538 |
|
1539 |
class TransportConfig(object): |
|
3242.1.5
by Aaron Bentley
Update per review comments |
1540 |
"""A Config that reads/writes a config file on a Transport. |
3242.1.4
by Aaron Bentley
Clean-up |
1541 |
|
1542 |
It is a low-level object that considers config data to be name/value pairs
|
|
1543 |
that may be associated with a section. Assigning meaning to the these
|
|
1544 |
values is done at higher levels like TreeConfig.
|
|
1545 |
"""
|
|
3242.3.14
by Aaron Bentley
Make BzrDirConfig use TransportConfig |
1546 |
|
1547 |
def __init__(self, transport, filename): |
|
1548 |
self._transport = transport |
|
1549 |
self._filename = filename |
|
1550 |
||
3242.1.1
by Aaron Bentley
Implement BzrDir configuration |
1551 |
def get_option(self, name, section=None, default=None): |
1552 |
"""Return the value associated with a named option. |
|
1553 |
||
1554 |
:param name: The name of the value
|
|
1555 |
:param section: The section the option is in (if any)
|
|
1556 |
:param default: The value to return if the value is not set
|
|
1557 |
:return: The value or default value
|
|
1558 |
"""
|
|
1559 |
configobj = self._get_configobj() |
|
1560 |
if section is None: |
|
1561 |
section_obj = configobj |
|
1562 |
else: |
|
1563 |
try: |
|
1564 |
section_obj = configobj[section] |
|
1565 |
except KeyError: |
|
1566 |
return default |
|
1567 |
return section_obj.get(name, default) |
|
1568 |
||
1569 |
def set_option(self, value, name, section=None): |
|
1570 |
"""Set the value associated with a named option. |
|
1571 |
||
1572 |
:param value: The value to set
|
|
1573 |
:param name: The name of the value to set
|
|
1574 |
:param section: The section the option is in (if any)
|
|
1575 |
"""
|
|
1576 |
configobj = self._get_configobj() |
|
1577 |
if section is None: |
|
1578 |
configobj[name] = value |
|
1579 |
else: |
|
1580 |
configobj.setdefault(section, {})[name] = value |
|
1581 |
self._set_configobj(configobj) |
|
1582 |
||
4288.1.2
by Robert Collins
Create a server verb for doing BzrDir.get_config() |
1583 |
def _get_config_file(self): |
1584 |
try: |
|
4852.1.10
by John Arbash Meinel
Use a StringIO instead, otherwise we get failures with smart server requests. |
1585 |
return StringIO(self._transport.get_bytes(self._filename)) |
4288.1.2
by Robert Collins
Create a server verb for doing BzrDir.get_config() |
1586 |
except errors.NoSuchFile: |
1587 |
return StringIO() |
|
1588 |
||
3242.1.1
by Aaron Bentley
Implement BzrDir configuration |
1589 |
def _get_configobj(self): |
4708.2.1
by Martin
Ensure all files opened by bazaar proper are explicitly closed |
1590 |
f = self._get_config_file() |
1591 |
try: |
|
1592 |
return ConfigObj(f, encoding='utf-8') |
|
1593 |
finally: |
|
1594 |
f.close() |
|
3242.1.1
by Aaron Bentley
Implement BzrDir configuration |
1595 |
|
1596 |
def _set_configobj(self, configobj): |
|
1597 |
out_file = StringIO() |
|
1598 |
configobj.write(out_file) |
|
1599 |
out_file.seek(0) |
|
1600 |
self._transport.put_file(self._filename, out_file) |