/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/breakin.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/breakin.py
1
 
# Copyright (C) 2006, 2007, 2009 Canonical Ltd
 
1
# Copyright (C) 2007, 2009, 2010 Canonical Ltd
2
2
#
3
3
# This program is free software; you can redistribute it and/or modify
4
4
# it under the terms of the GNU General Public License as published by
14
14
# along with this program; if not, write to the Free Software
15
15
# Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA
16
16
 
 
17
from __future__ import absolute_import
 
18
 
17
19
import os
18
20
import signal
19
21
 
41
43
        signal.signal(_breakin_signal_number, _debug)
42
44
 
43
45
 
44
 
def hook_sigquit():
45
 
    # We import this late because breakin.py is loaded as part of the main
46
 
    # 'bzr' script, so we want it to load as little as possible until things
47
 
    # are up and running
48
 
    from bzrlib import symbol_versioning, trace
49
 
    trace.mutter_callsite(2, 'Deprecated function called')
50
 
    symbol_versioning.warn(symbol_versioning.deprecation_string(
51
 
        hook_sigquit, symbol_versioning.deprecated_in((1, 18, 0))),
52
 
        DeprecationWarning, stacklevel=2)
53
 
 
54
 
    return hook_debugger_to_signal()
55
 
 
56
 
 
57
46
def determine_signal():
58
47
    global _breakin_signal_number
59
48
    global _breakin_signal_name
78
67
def hook_debugger_to_signal():
79
68
    """Add a signal handler so we drop into the debugger.
80
69
 
81
 
    On Linux and Mac, this is hooked into SIGQUIT (C-\\) on Windows, this is
 
70
    On Unix, this is hooked into SIGQUIT (C-\\), and on Windows, this is
82
71
    hooked into SIGBREAK (C-Pause).
83
72
    """
84
73