To get this branch, use:
bzr branch
http://gegoxaren.bato24.eu/bzr/brz/remove-bazaar
- Committer: Vincent Ladeuil
- Date: 2012-01-05 14:26:58 UTC
- Revision ID: v.ladeuil+lp@free.fr-20120105142658-vek3v6pzlxb751s2
Tests passing for a first rough version of a cached branch config store. The changes here are too invasive and several parallel proposals have been made.
@only_raises is evil and gave a hard time since any exception during
save_changes() was swallowed.
Possible improvements:
- add some needs_write_lock decorators to crucial
methods (_set_config_location ?) but keep locking the branch at higher levels
- decorate branch.unlock to call stack.save if last_lock() it True
outside of @only_raises scope (evil decorator)
- add @needs_write_lock to stack.set and stack.remove (will probably get
rid of most testing issues) we probably need a specialized decorator
that can relay to the store and from there to the branch or whatever is
needed. This will also helps bzr config to get it right. The
get_mutable_section trick should not be needed anymore either.
- decorate branch.unlock to call stack.save if last_lock() it True outside
of @only_raises scope (evil decorator)
@only_raises is evil and gave a hard time since any exception during
save_changes() was swallowed.
Possible improvements:
- add some needs_write_lock decorators to crucial
methods (_set_config_location ?) but keep locking the branch at higher levels
- decorate branch.unlock to call stack.save if last_lock() it True
outside of @only_raises scope (evil decorator)
- add @needs_write_lock to stack.set and stack.remove (will probably get
rid of most testing issues) we probably need a specialized decorator
that can relay to the store and from there to the branch or whatever is
needed. This will also helps bzr config to get it right. The
get_mutable_section trick should not be needed anymore either.
- decorate branch.unlock to call stack.save if last_lock() it True outside
of @only_raises scope (evil decorator)
| Filename | Latest Rev | Last Changed | Committer | Comment | Size | ||
|---|---|---|---|---|---|---|---|
 ..
|
|||||||
__init__.py |
6379.6.1 | 13 years ago | Jelmer Vernooij | Import absolute_import in a few places. | 20.8 KB |
|
|
|
account.py |
6379.6.7 | 13 years ago | Jelmer Vernooij | Move importing from future until after doc string, | 3.4 KB |
|
|
|
lp_api.py |
6379.6.7 | 13 years ago | Jelmer Vernooij | Move importing from future until after doc string, | 11.4 KB |
|
|
|
lp_api_lite.py |
6379.6.7 | 13 years ago | Jelmer Vernooij | Move importing from future until after doc string, | 10.5 KB |
|
|
|
lp_directory.py |
6379.6.7 | 13 years ago | Jelmer Vernooij | Move importing from future until after doc string, | 8 KB |
|
|
|
lp_propose.py |
6379.6.3 | 13 years ago | Jelmer Vernooij | Use absolute_import. | 8.3 KB |
|
|
|
lp_registration.py |
6379.6.5 | 13 years ago | Jelmer Vernooij | Merge bzr.dev. | 13 KB |
|
|
|
test_account.py |
5394.1.1 | 15 years ago | Vincent Ladeuil | Config files in bazaar home now use a lock | 5.2 KB |
|
|
|
test_lp_api.py |
5967.12.5 | 14 years ago | Martin Pool | Update launchpad plugin for features under tests.f | 3.7 KB |
|
|
|
test_lp_api_lite.py |
6089.1.1 | 14 years ago | Vincent Ladeuil | Merge 2.4 into trunk (including fix for #614713, # | 23 KB |
|
|
|
test_lp_directory.py |
6191.2.1 | 14 years ago | Martin Pool | Avoid NameError when given an invalid ubuntu: laun | 25.3 KB |
|
|
|
test_lp_login.py |
4505.1.4 | 16 years ago | Jonathan Lange | Use 'foo', not 'jml' | 2.5 KB |
|
|
|
test_lp_open.py |
6404.6.1 | 13 years ago | Vincent Ladeuil | Tests passing for a first rough version of a cache | 4.4 KB |
|
|
|
test_lp_service.py |
5557.1.7 | 14 years ago | John Arbash Meinel | Merge in the bzr.dev 5582 | 7.2 KB |
|
|
|
test_register.py |
6404.6.1 | 13 years ago | Vincent Ladeuil | Tests passing for a first rough version of a cache | 14.4 KB |
|
|