/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/lockable_files.py

  • Committer: Robert Collins
  • Date: 2010-05-06 11:08:10 UTC
  • mto: This revision was merged to the branch mainline in revision 5223.
  • Revision ID: robertc@robertcollins.net-20100506110810-h3j07fh5gmw54s25
Cleaner matcher matching revised unlocking protocol.

Show diffs side-by-side

added added

removed removed

Lines of Context:
1
 
# Copyright (C) 2005, 2006, 2008 Canonical Ltd
 
1
# Copyright (C) 2005, 2006, 2008, 2009 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
12
12
#
13
13
# You should have received a copy of the GNU General Public License
14
14
# along with this program; if not, write to the Free Software
15
 
# Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307  USA
 
15
# Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA
16
16
 
17
 
from cStringIO import StringIO
 
17
from bzrlib.lazy_import import lazy_import
 
18
lazy_import(globals(), """
18
19
import codecs
19
 
#import traceback
20
 
from warnings import warn
21
 
 
22
 
import bzrlib
23
 
from bzrlib.decorators import (needs_read_lock,
24
 
        needs_write_lock)
25
 
import bzrlib.errors as errors
26
 
from bzrlib.errors import BzrError
27
 
from bzrlib.osutils import file_iterator, safe_unicode
28
 
from bzrlib.symbol_versioning import (deprecated_method,
29
 
        )
30
 
from bzrlib.trace import mutter, note
31
 
import bzrlib.transactions as transactions
32
 
import bzrlib.urlutils as urlutils
 
20
import warnings
 
21
 
 
22
from bzrlib import (
 
23
    counted_lock,
 
24
    errors,
 
25
    lock,
 
26
    osutils,
 
27
    transactions,
 
28
    urlutils,
 
29
    )
 
30
""")
 
31
 
 
32
from bzrlib.decorators import (
 
33
    only_raises,
 
34
    )
33
35
 
34
36
 
35
37
# XXX: The tracking here of lock counts and whether the lock is held is
36
38
# somewhat redundant with what's done in LockDir; the main difference is that
37
39
# LockableFiles permits reentrancy.
38
40
 
 
41
class _LockWarner(object):
 
42
    """Hold a counter for a lock and warn if GCed while the count is >= 1.
 
43
 
 
44
    This is separate from LockableFiles because putting a __del__ on
 
45
    LockableFiles can result in uncollectable cycles.
 
46
    """
 
47
 
 
48
    def __init__(self, repr):
 
49
        self.lock_count = 0
 
50
        self.repr = repr
 
51
 
 
52
    def __del__(self):
 
53
        if self.lock_count >= 1:
 
54
            # There should have been a try/finally to unlock this.
 
55
            warnings.warn("%r was gc'd while locked" % self.repr)
 
56
 
 
57
 
39
58
class LockableFiles(object):
40
59
    """Object representing a set of related files locked within the same scope.
41
60
 
42
 
    These files are used by a WorkingTree, Repository or Branch, and should
43
 
    generally only be touched by that object.
44
 
 
45
 
    LockableFiles also provides some policy on top of Transport for encoding
46
 
    control files as utf-8.
 
61
    This coordinates access to the lock along with providing a transaction.
47
62
 
48
63
    LockableFiles manage a lock count and can be locked repeatedly by
49
64
    a single caller.  (The underlying lock implementation generally does not
50
65
    support this.)
51
66
 
52
67
    Instances of this class are often called control_files.
 
68
 
 
69
    This class is now deprecated; code should move to using the Transport
 
70
    directly for file operations and using the lock or CountedLock for
 
71
    locking.
53
72
    
54
 
    This object builds on top of a Transport, which is used to actually write
55
 
    the files to disk, and an OSLock or LockDir, which controls how access to
56
 
    the files is controlled.  The particular type of locking used is set when
57
 
    the object is constructed.  In older formats OSLocks are used everywhere.
58
 
    in newer formats a LockDir is used for Repositories and Branches, and 
59
 
    OSLocks for the local filesystem.
 
73
    :ivar _lock: The real underlying lock (e.g. a LockDir)
 
74
    :ivar _counted_lock: A lock decorated with a semaphore, so that it 
 
75
        can be re-entered.
60
76
    """
61
77
 
62
78
    # _lock_mode: None, or 'r' or 'w'
63
79
 
64
80
    # _lock_count: If _lock_mode is true, a positive count of the number of
65
 
    # times the lock has been taken *by this process*.   
66
 
    
67
 
    # If set to False (by a plugin, etc) BzrBranch will not set the
68
 
    # mode on created files or directories
69
 
    _set_file_mode = True
70
 
    _set_dir_mode = True
 
81
    # times the lock has been taken *by this process*.
71
82
 
72
83
    def __init__(self, transport, lock_name, lock_class):
73
84
        """Create a LockableFiles group
74
85
 
75
 
        :param transport: Transport pointing to the directory holding the 
 
86
        :param transport: Transport pointing to the directory holding the
76
87
            control files and lock.
77
88
        :param lock_name: Name of the lock guarding these files.
78
89
        :param lock_class: Class of lock strategy to use: typically
82
93
        self.lock_name = lock_name
83
94
        self._transaction = None
84
95
        self._lock_mode = None
85
 
        self._lock_count = 0
 
96
        self._lock_warner = _LockWarner(repr(self))
86
97
        self._find_modes()
87
98
        esc_name = self._escape(lock_name)
88
99
        self._lock = lock_class(transport, esc_name,
89
100
                                file_modebits=self._file_mode,
90
101
                                dir_modebits=self._dir_mode)
 
102
        self._counted_lock = counted_lock.CountedLock(self._lock)
91
103
 
92
104
    def create_lock(self):
93
105
        """Create the lock.
103
115
    def __str__(self):
104
116
        return 'LockableFiles(%s, %s)' % (self.lock_name, self._transport.base)
105
117
 
106
 
    def __del__(self):
107
 
        if self.is_locked():
108
 
            # do not automatically unlock; there should have been a
109
 
            # try/finally to unlock this.
110
 
            warn("%r was gc'd while locked" % self)
111
 
 
112
118
    def break_lock(self):
113
119
        """Break the lock of this lockable files group if it is held.
114
120
 
117
123
        self._lock.break_lock()
118
124
 
119
125
    def _escape(self, file_or_path):
 
126
        """DEPRECATED: Do not use outside this class"""
120
127
        if not isinstance(file_or_path, basestring):
121
128
            file_or_path = '/'.join(file_or_path)
122
129
        if file_or_path == '':
123
130
            return u''
124
 
        return urlutils.escape(safe_unicode(file_or_path))
 
131
        return urlutils.escape(osutils.safe_unicode(file_or_path))
125
132
 
126
133
    def _find_modes(self):
127
 
        """Determine the appropriate modes for files and directories."""
 
134
        """Determine the appropriate modes for files and directories.
 
135
 
 
136
        :deprecated: Replaced by BzrDir._find_creation_modes.
 
137
        """
 
138
        # XXX: The properties created by this can be removed or deprecated
 
139
        # once all the _get_text_store methods etc no longer use them.
 
140
        # -- mbp 20080512
128
141
        try:
129
142
            st = self._transport.stat('.')
130
143
        except errors.TransportNotPossible:
138
151
            self._dir_mode = (st.st_mode & 07777) | 00700
139
152
            # Remove the sticky and execute bits for files
140
153
            self._file_mode = self._dir_mode & ~07111
141
 
        if not self._set_dir_mode:
142
 
            self._dir_mode = None
143
 
        if not self._set_file_mode:
144
 
            self._file_mode = None
145
 
 
146
 
    def controlfilename(self, file_or_path):
147
 
        """Return location relative to branch."""
148
 
        return self._transport.abspath(self._escape(file_or_path))
149
 
 
150
 
    @needs_read_lock
151
 
    def get(self, relpath):
152
 
        """Get a file as a bytestream."""
153
 
        relpath = self._escape(relpath)
154
 
        return self._transport.get(relpath)
155
 
 
156
 
    @needs_read_lock
157
 
    def get_utf8(self, relpath):
158
 
        """Get a file as a unicode stream."""
159
 
        relpath = self._escape(relpath)
160
 
        # DO NOT introduce an errors=replace here.
161
 
        return codecs.getreader('utf-8')(self._transport.get(relpath))
162
 
 
163
 
    @needs_write_lock
164
 
    def put(self, path, file):
165
 
        """Write a file.
166
 
        
167
 
        :param path: The path to put the file, relative to the .bzr control
168
 
                     directory
169
 
        :param f: A file-like or string object whose contents should be copied.
170
 
        """
171
 
        self._transport.put_file(self._escape(path), file, mode=self._file_mode)
172
 
 
173
 
    @needs_write_lock
174
 
    def put_bytes(self, path, a_string):
175
 
        """Write a string of bytes.
176
 
 
177
 
        :param path: The path to put the bytes, relative to the transport root.
178
 
        :param string: A string object, whose exact bytes are to be copied.
179
 
        """
180
 
        self._transport.put_bytes(self._escape(path), a_string,
181
 
                                  mode=self._file_mode)
182
 
 
183
 
    @needs_write_lock
184
 
    def put_utf8(self, path, a_string):
185
 
        """Write a string, encoding as utf-8.
186
 
 
187
 
        :param path: The path to put the string, relative to the transport root.
188
 
        :param string: A string or unicode object whose contents should be copied.
189
 
        """
190
 
        # IterableFile would not be needed if Transport.put took iterables
191
 
        # instead of files.  ADHB 2005-12-25
192
 
        # RBC 20060103 surely its not needed anyway, with codecs transcode
193
 
        # file support ?
194
 
        # JAM 20060103 We definitely don't want encode(..., 'replace')
195
 
        # these are valuable files which should have exact contents.
196
 
        if not isinstance(a_string, basestring):
197
 
            raise errors.BzrBadParameterNotString(a_string)
198
 
        self.put_bytes(path, a_string.encode('utf-8'))
199
154
 
200
155
    def leave_in_place(self):
201
156
        """Set this LockableFiles to not clear the physical lock on unlock."""
207
162
 
208
163
    def lock_write(self, token=None):
209
164
        """Lock this group of files for writing.
210
 
        
 
165
 
211
166
        :param token: if this is already locked, then lock_write will fail
212
167
            unless the token matches the existing lock.
213
168
        :returns: a token if this instance supports tokens, otherwise None.
220
175
        some other way, and need to synchronise this object's state with that
221
176
        fact.
222
177
        """
223
 
        # mutter("lock write: %s (%s)", self, self._lock_count)
224
178
        # TODO: Upgrade locking to support using a Transport,
225
179
        # and potentially a remote locking protocol
226
180
        if self._lock_mode:
227
181
            if self._lock_mode != 'w' or not self.get_transaction().writeable():
228
182
                raise errors.ReadOnlyError(self)
229
183
            self._lock.validate_token(token)
230
 
            self._lock_count += 1
 
184
            self._lock_warner.lock_count += 1
231
185
            return self._token_from_lock
232
186
        else:
233
187
            token_from_lock = self._lock.lock_write(token=token)
234
 
            #note('write locking %s', self)
235
188
            #traceback.print_stack()
236
189
            self._lock_mode = 'w'
237
 
            self._lock_count = 1
238
 
            self._set_transaction(transactions.WriteTransaction())
 
190
            self._lock_warner.lock_count = 1
 
191
            self._set_write_transaction()
239
192
            self._token_from_lock = token_from_lock
240
193
            return token_from_lock
241
194
 
242
195
    def lock_read(self):
243
 
        # mutter("lock read: %s (%s)", self, self._lock_count)
244
196
        if self._lock_mode:
245
 
            assert self._lock_mode in ('r', 'w'), \
246
 
                   "invalid lock mode %r" % self._lock_mode
247
 
            self._lock_count += 1
 
197
            if self._lock_mode not in ('r', 'w'):
 
198
                raise ValueError("invalid lock mode %r" % (self._lock_mode,))
 
199
            self._lock_warner.lock_count += 1
248
200
        else:
249
201
            self._lock.lock_read()
250
 
            #note('read locking %s', self)
251
202
            #traceback.print_stack()
252
203
            self._lock_mode = 'r'
253
 
            self._lock_count = 1
254
 
            self._set_transaction(transactions.ReadOnlyTransaction())
255
 
            # 5K may be excessive, but hey, its a knob.
256
 
            self.get_transaction().set_cache_size(5000)
257
 
                        
 
204
            self._lock_warner.lock_count = 1
 
205
            self._set_read_transaction()
 
206
 
 
207
    def _set_read_transaction(self):
 
208
        """Setup a read transaction."""
 
209
        self._set_transaction(transactions.ReadOnlyTransaction())
 
210
        # 5K may be excessive, but hey, its a knob.
 
211
        self.get_transaction().set_cache_size(5000)
 
212
 
 
213
    def _set_write_transaction(self):
 
214
        """Setup a write transaction."""
 
215
        self._set_transaction(transactions.WriteTransaction())
 
216
 
 
217
    @only_raises(errors.LockNotHeld, errors.LockBroken)
258
218
    def unlock(self):
259
 
        # mutter("unlock: %s (%s)", self, self._lock_count)
260
219
        if not self._lock_mode:
261
 
            raise errors.LockNotHeld(self)
262
 
        if self._lock_count > 1:
263
 
            self._lock_count -= 1
 
220
            return lock.cant_unlock_not_held(self)
 
221
        if self._lock_warner.lock_count > 1:
 
222
            self._lock_warner.lock_count -= 1
264
223
        else:
265
 
            #note('unlocking %s', self)
266
224
            #traceback.print_stack()
267
225
            self._finish_transaction()
268
226
            try:
269
227
                self._lock.unlock()
270
228
            finally:
271
 
                self._lock_mode = self._lock_count = None
 
229
                self._lock_mode = self._lock_warner.lock_count = None
 
230
 
 
231
    @property
 
232
    def _lock_count(self):
 
233
        return self._lock_warner.lock_count
272
234
 
273
235
    def is_locked(self):
274
236
        """Return true if this LockableFiles group is locked"""
275
 
        return self._lock_count >= 1
 
237
        return self._lock_warner.lock_count >= 1
276
238
 
277
239
    def get_physical_lock_status(self):
278
240
        """Return physical lock status.
279
 
        
 
241
 
280
242
        Returns true if a lock is held on the transport. If no lock is held, or
281
243
        the underlying locking mechanism does not support querying lock
282
244
        status, false is returned.
364
326
    def validate_token(self, token):
365
327
        if token is not None:
366
328
            raise errors.TokenLockingNotSupported(self)
367
 
        
 
329