/brz/remove-bazaar

To get this branch, use:
bzr branch http://gegoxaren.bato24.eu/bzr/brz/remove-bazaar

« back to all changes in this revision

Viewing changes to bzrlib/version.py

  • Committer: Vincent Ladeuil
  • Date: 2012-01-05 14:26:58 UTC
  • mto: This revision was merged to the branch mainline in revision 6468.
  • 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)

Show diffs side-by-side

added added

removed removed

Lines of Context:
bzrlib/version.py
16
16
 
17
17
"""Report on version of bzrlib"""
18
18
 
 
19
from __future__ import absolute_import
 
20
 
19
21
import os
20
22
import sys
21
23
 
22
24
import bzrlib
23
25
from bzrlib import (
24
 
    bzrdir,
25
26
    config,
 
27
    controldir,
26
28
    errors,
27
29
    osutils,
28
30
    trace,
73
75
    else:
74
76
        to_file.write(bzrlib.__path__[0] + '\n')
75
77
    if show_config:
76
 
        config_dir = os.path.normpath(config.config_dir())  # use native slashes
 
78
        config_dir = osutils.normpath(config.config_dir())  # use native slashes
77
79
        if not isinstance(config_dir, unicode):
78
80
            config_dir = config_dir.decode(osutils.get_user_encoding())
79
81
        to_file.write("  Bazaar configuration: %s\n" % (config_dir,))
98
100
    If bzr is not being run from its working tree, returns None.
99
101
    """
100
102
    try:
101
 
        control = bzrdir.BzrDir.open_containing(__file__)[0]
 
103
        control = controldir.ControlDir.open_containing(__file__)[0]
102
104
        return control.open_workingtree(recommend_upgrade=False)
103
105
    except (errors.NotBranchError, errors.UnknownFormatError,
104
106
            errors.NoWorkingTree):