/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 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
 
 
21
 
import bzrlib
22
 
from bzrlib.decorators import (needs_read_lock,
23
 
        needs_write_lock)
24
 
import bzrlib.errors as errors
25
 
from bzrlib.errors import BzrError
26
 
from bzrlib.osutils import file_iterator, safe_unicode
27
 
from bzrlib.symbol_versioning import (deprecated_method, 
28
 
        zero_eight)
29
 
from bzrlib.trace import mutter, note
30
 
import bzrlib.transactions as transactions
31
 
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
    )
32
35
 
33
36
 
34
37
# XXX: The tracking here of lock counts and whether the lock is held is
35
38
# somewhat redundant with what's done in LockDir; the main difference is that
36
39
# LockableFiles permits reentrancy.
37
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
 
38
58
class LockableFiles(object):
39
59
    """Object representing a set of related files locked within the same scope.
40
60
 
41
 
    These files are used by a WorkingTree, Repository or Branch, and should
42
 
    generally only be touched by that object.
43
 
 
44
 
    LockableFiles also provides some policy on top of Transport for encoding
45
 
    control files as utf-8.
 
61
    This coordinates access to the lock along with providing a transaction.
46
62
 
47
63
    LockableFiles manage a lock count and can be locked repeatedly by
48
64
    a single caller.  (The underlying lock implementation generally does not
49
65
    support this.)
50
66
 
51
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.
52
72
    
53
 
    This object builds on top of a Transport, which is used to actually write
54
 
    the files to disk, and an OSLock or LockDir, which controls how access to
55
 
    the files is controlled.  The particular type of locking used is set when
56
 
    the object is constructed.  In older formats OSLocks are used everywhere.
57
 
    in newer formats a LockDir is used for Repositories and Branches, and 
58
 
    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.
59
76
    """
60
77
 
61
78
    # _lock_mode: None, or 'r' or 'w'
62
79
 
63
80
    # _lock_count: If _lock_mode is true, a positive count of the number of
64
 
    # times the lock has been taken *by this process*.   
65
 
    
66
 
    # If set to False (by a plugin, etc) BzrBranch will not set the
67
 
    # mode on created files or directories
68
 
    _set_file_mode = True
69
 
    _set_dir_mode = True
 
81
    # times the lock has been taken *by this process*.
70
82
 
71
83
    def __init__(self, transport, lock_name, lock_class):
72
84
        """Create a LockableFiles group
73
85
 
74
 
        :param transport: Transport pointing to the directory holding the 
 
86
        :param transport: Transport pointing to the directory holding the
75
87
            control files and lock.
76
88
        :param lock_name: Name of the lock guarding these files.
77
89
        :param lock_class: Class of lock strategy to use: typically
81
93
        self.lock_name = lock_name
82
94
        self._transaction = None
83
95
        self._lock_mode = None
84
 
        self._lock_count = 0
 
96
        self._lock_warner = _LockWarner(repr(self))
85
97
        self._find_modes()
86
98
        esc_name = self._escape(lock_name)
87
99
        self._lock = lock_class(transport, esc_name,
88
100
                                file_modebits=self._file_mode,
89
101
                                dir_modebits=self._dir_mode)
 
102
        self._counted_lock = counted_lock.CountedLock(self._lock)
90
103
 
91
104
    def create_lock(self):
92
105
        """Create the lock.
102
115
    def __str__(self):
103
116
        return 'LockableFiles(%s, %s)' % (self.lock_name, self._transport.base)
104
117
 
105
 
    def __del__(self):
106
 
        if self.is_locked():
107
 
            # XXX: This should show something every time, and be suitable for
108
 
            # headless operation and embedding
109
 
            from warnings import warn
110
 
            warn("file group %r was not explicitly unlocked" % self)
111
 
            self._lock.unlock()
112
 
 
113
118
    def break_lock(self):
114
119
        """Break the lock of this lockable files group if it is held.
115
120
 
118
123
        self._lock.break_lock()
119
124
 
120
125
    def _escape(self, file_or_path):
 
126
        """DEPRECATED: Do not use outside this class"""
121
127
        if not isinstance(file_or_path, basestring):
122
128
            file_or_path = '/'.join(file_or_path)
123
129
        if file_or_path == '':
124
130
            return u''
125
 
        return urlutils.escape(safe_unicode(file_or_path))
 
131
        return urlutils.escape(osutils.safe_unicode(file_or_path))
126
132
 
127
133
    def _find_modes(self):
128
 
        """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
129
141
        try:
130
142
            st = self._transport.stat('.')
131
143
        except errors.TransportNotPossible:
132
144
            self._dir_mode = 0755
133
145
            self._file_mode = 0644
134
146
        else:
135
 
            self._dir_mode = st.st_mode & 07777
 
147
            # Check the directory mode, but also make sure the created
 
148
            # directories and files are read-write for this user. This is
 
149
            # mostly a workaround for filesystems which lie about being able to
 
150
            # write to a directory (cygwin & win32)
 
151
            self._dir_mode = (st.st_mode & 07777) | 00700
136
152
            # Remove the sticky and execute bits for files
137
153
            self._file_mode = self._dir_mode & ~07111
138
 
        if not self._set_dir_mode:
139
 
            self._dir_mode = None
140
 
        if not self._set_file_mode:
141
 
            self._file_mode = None
142
 
 
143
 
    def controlfilename(self, file_or_path):
144
 
        """Return location relative to branch."""
145
 
        return self._transport.abspath(self._escape(file_or_path))
146
 
 
147
 
    @deprecated_method(zero_eight)
148
 
    def controlfile(self, file_or_path, mode='r'):
149
 
        """Open a control file for this branch.
150
 
 
151
 
        There are two classes of file in a lockable directory: text
152
 
        and binary.  binary files are untranslated byte streams.  Text
153
 
        control files are stored with Unix newlines and in UTF-8, even
154
 
        if the platform or locale defaults are different.
155
 
 
156
 
        Such files are not openable in write mode : they are managed via
157
 
        put and put_utf8 which atomically replace old versions using
158
 
        atomicfile.
159
 
        """
160
 
 
161
 
        relpath = self._escape(file_or_path)
162
 
        # TODO: codecs.open() buffers linewise, so it was overloaded with
163
 
        # a much larger buffer, do we need to do the same for getreader/getwriter?
164
 
        if mode == 'rb': 
165
 
            return self.get(relpath)
166
 
        elif mode == 'wb':
167
 
            raise BzrError("Branch.controlfile(mode='wb') is not supported, use put[_utf8]")
168
 
        elif mode == 'r':
169
 
            return self.get_utf8(relpath)
170
 
        elif mode == 'w':
171
 
            raise BzrError("Branch.controlfile(mode='w') is not supported, use put[_utf8]")
172
 
        else:
173
 
            raise BzrError("invalid controlfile mode %r" % mode)
174
 
 
175
 
    @needs_read_lock
176
 
    def get(self, relpath):
177
 
        """Get a file as a bytestream."""
178
 
        relpath = self._escape(relpath)
179
 
        return self._transport.get(relpath)
180
 
 
181
 
    @needs_read_lock
182
 
    def get_utf8(self, relpath):
183
 
        """Get a file as a unicode stream."""
184
 
        relpath = self._escape(relpath)
185
 
        # DO NOT introduce an errors=replace here.
186
 
        return codecs.getreader('utf-8')(self._transport.get(relpath))
187
 
 
188
 
    @needs_write_lock
189
 
    def put(self, path, file):
190
 
        """Write a file.
191
 
        
192
 
        :param path: The path to put the file, relative to the .bzr control
193
 
                     directory
194
 
        :param f: A file-like or string object whose contents should be copied.
195
 
        """
196
 
        self._transport.put_file(self._escape(path), file, mode=self._file_mode)
197
 
 
198
 
    @needs_write_lock
199
 
    def put_bytes(self, path, a_string):
200
 
        """Write a string of bytes.
201
 
 
202
 
        :param path: The path to put the bytes, relative to the transport root.
203
 
        :param string: A string object, whose exact bytes are to be copied.
204
 
        """
205
 
        self._transport.put_bytes(self._escape(path), a_string,
206
 
                                  mode=self._file_mode)
207
 
 
208
 
    @needs_write_lock
209
 
    def put_utf8(self, path, a_string):
210
 
        """Write a string, encoding as utf-8.
211
 
 
212
 
        :param path: The path to put the string, relative to the transport root.
213
 
        :param string: A string or unicode object whose contents should be copied.
214
 
        """
215
 
        # IterableFile would not be needed if Transport.put took iterables
216
 
        # instead of files.  ADHB 2005-12-25
217
 
        # RBC 20060103 surely its not needed anyway, with codecs transcode
218
 
        # file support ?
219
 
        # JAM 20060103 We definitely don't want encode(..., 'replace')
220
 
        # these are valuable files which should have exact contents.
221
 
        if not isinstance(a_string, basestring):
222
 
            raise errors.BzrBadParameterNotString(a_string)
223
 
        self.put_bytes(path, a_string.encode('utf-8'))
224
154
 
225
155
    def leave_in_place(self):
226
156
        """Set this LockableFiles to not clear the physical lock on unlock."""
232
162
 
233
163
    def lock_write(self, token=None):
234
164
        """Lock this group of files for writing.
235
 
        
 
165
 
236
166
        :param token: if this is already locked, then lock_write will fail
237
167
            unless the token matches the existing lock.
238
168
        :returns: a token if this instance supports tokens, otherwise None.
245
175
        some other way, and need to synchronise this object's state with that
246
176
        fact.
247
177
        """
248
 
        # mutter("lock write: %s (%s)", self, self._lock_count)
249
178
        # TODO: Upgrade locking to support using a Transport,
250
179
        # and potentially a remote locking protocol
251
180
        if self._lock_mode:
252
181
            if self._lock_mode != 'w' or not self.get_transaction().writeable():
253
182
                raise errors.ReadOnlyError(self)
254
183
            self._lock.validate_token(token)
255
 
            self._lock_count += 1
 
184
            self._lock_warner.lock_count += 1
256
185
            return self._token_from_lock
257
186
        else:
258
187
            token_from_lock = self._lock.lock_write(token=token)
259
 
            #note('write locking %s', self)
260
188
            #traceback.print_stack()
261
189
            self._lock_mode = 'w'
262
 
            self._lock_count = 1
263
 
            self._set_transaction(transactions.WriteTransaction())
 
190
            self._lock_warner.lock_count = 1
 
191
            self._set_write_transaction()
264
192
            self._token_from_lock = token_from_lock
265
193
            return token_from_lock
266
194
 
267
195
    def lock_read(self):
268
 
        # mutter("lock read: %s (%s)", self, self._lock_count)
269
196
        if self._lock_mode:
270
 
            assert self._lock_mode in ('r', 'w'), \
271
 
                   "invalid lock mode %r" % self._lock_mode
272
 
            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
273
200
        else:
274
201
            self._lock.lock_read()
275
 
            #note('read locking %s', self)
276
202
            #traceback.print_stack()
277
203
            self._lock_mode = 'r'
278
 
            self._lock_count = 1
279
 
            self._set_transaction(transactions.ReadOnlyTransaction())
280
 
            # 5K may be excessive, but hey, its a knob.
281
 
            self.get_transaction().set_cache_size(5000)
282
 
                        
 
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)
283
218
    def unlock(self):
284
 
        # mutter("unlock: %s (%s)", self, self._lock_count)
285
219
        if not self._lock_mode:
286
 
            raise errors.LockNotHeld(self)
287
 
        if self._lock_count > 1:
288
 
            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
289
223
        else:
290
 
            #note('unlocking %s', self)
291
224
            #traceback.print_stack()
292
225
            self._finish_transaction()
293
226
            try:
294
227
                self._lock.unlock()
295
228
            finally:
296
 
                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
297
234
 
298
235
    def is_locked(self):
299
236
        """Return true if this LockableFiles group is locked"""
300
 
        return self._lock_count >= 1
 
237
        return self._lock_warner.lock_count >= 1
301
238
 
302
239
    def get_physical_lock_status(self):
303
240
        """Return physical lock status.
304
 
        
 
241
 
305
242
        Returns true if a lock is held on the transport. If no lock is held, or
306
243
        the underlying locking mechanism does not support querying lock
307
244
        status, false is returned.
389
326
    def validate_token(self, token):
390
327
        if token is not None:
391
328
            raise errors.TokenLockingNotSupported(self)
392
 
        
 
329