/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

Only chown() the .bzr.log when creating it, fixing NEWS entry to put it under 2.2b3

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
17
from cStringIO import StringIO
 
18
 
 
19
from bzrlib.lazy_import import lazy_import
 
20
lazy_import(globals(), """
18
21
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
 
22
import warnings
 
23
 
 
24
from bzrlib import (
 
25
    counted_lock,
 
26
    errors,
 
27
    lock,
 
28
    osutils,
 
29
    transactions,
 
30
    urlutils,
 
31
    )
 
32
""")
 
33
 
 
34
from bzrlib.decorators import (
 
35
    only_raises,
 
36
    )
 
37
from bzrlib.symbol_versioning import (
 
38
    deprecated_in,
 
39
    deprecated_method,
 
40
    )
32
41
 
33
42
 
34
43
# XXX: The tracking here of lock counts and whether the lock is held is
35
44
# somewhat redundant with what's done in LockDir; the main difference is that
36
45
# LockableFiles permits reentrancy.
37
46
 
 
47
class _LockWarner(object):
 
48
    """Hold a counter for a lock and warn if GCed while the count is >= 1.
 
49
 
 
50
    This is separate from LockableFiles because putting a __del__ on
 
51
    LockableFiles can result in uncollectable cycles.
 
52
    """
 
53
 
 
54
    def __init__(self, repr):
 
55
        self.lock_count = 0
 
56
        self.repr = repr
 
57
 
 
58
    def __del__(self):
 
59
        if self.lock_count >= 1:
 
60
            # There should have been a try/finally to unlock this.
 
61
            warnings.warn("%r was gc'd while locked" % self.repr)
 
62
 
 
63
 
38
64
class LockableFiles(object):
39
65
    """Object representing a set of related files locked within the same scope.
40
66
 
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.
 
67
    This coordinates access to the lock along with providing a transaction.
46
68
 
47
69
    LockableFiles manage a lock count and can be locked repeatedly by
48
70
    a single caller.  (The underlying lock implementation generally does not
49
71
    support this.)
50
72
 
51
73
    Instances of this class are often called control_files.
 
74
 
 
75
    This class is now deprecated; code should move to using the Transport
 
76
    directly for file operations and using the lock or CountedLock for
 
77
    locking.
52
78
    
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.
 
79
    :ivar _lock: The real underlying lock (e.g. a LockDir)
 
80
    :ivar _counted_lock: A lock decorated with a semaphore, so that it 
 
81
        can be re-entered.
59
82
    """
60
83
 
61
84
    # _lock_mode: None, or 'r' or 'w'
62
85
 
63
86
    # _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
 
87
    # times the lock has been taken *by this process*.
70
88
 
71
89
    def __init__(self, transport, lock_name, lock_class):
72
90
        """Create a LockableFiles group
73
91
 
74
 
        :param transport: Transport pointing to the directory holding the 
 
92
        :param transport: Transport pointing to the directory holding the
75
93
            control files and lock.
76
94
        :param lock_name: Name of the lock guarding these files.
77
95
        :param lock_class: Class of lock strategy to use: typically
81
99
        self.lock_name = lock_name
82
100
        self._transaction = None
83
101
        self._lock_mode = None
84
 
        self._lock_count = 0
 
102
        self._lock_warner = _LockWarner(repr(self))
85
103
        self._find_modes()
86
104
        esc_name = self._escape(lock_name)
87
105
        self._lock = lock_class(transport, esc_name,
88
106
                                file_modebits=self._file_mode,
89
107
                                dir_modebits=self._dir_mode)
 
108
        self._counted_lock = counted_lock.CountedLock(self._lock)
90
109
 
91
110
    def create_lock(self):
92
111
        """Create the lock.
102
121
    def __str__(self):
103
122
        return 'LockableFiles(%s, %s)' % (self.lock_name, self._transport.base)
104
123
 
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
124
    def break_lock(self):
114
125
        """Break the lock of this lockable files group if it is held.
115
126
 
118
129
        self._lock.break_lock()
119
130
 
120
131
    def _escape(self, file_or_path):
 
132
        """DEPRECATED: Do not use outside this class"""
121
133
        if not isinstance(file_or_path, basestring):
122
134
            file_or_path = '/'.join(file_or_path)
123
135
        if file_or_path == '':
124
136
            return u''
125
 
        return urlutils.escape(safe_unicode(file_or_path))
 
137
        return urlutils.escape(osutils.safe_unicode(file_or_path))
126
138
 
127
139
    def _find_modes(self):
128
 
        """Determine the appropriate modes for files and directories."""
 
140
        """Determine the appropriate modes for files and directories.
 
141
 
 
142
        :deprecated: Replaced by BzrDir._find_creation_modes.
 
143
        """
 
144
        # XXX: The properties created by this can be removed or deprecated
 
145
        # once all the _get_text_store methods etc no longer use them.
 
146
        # -- mbp 20080512
129
147
        try:
130
148
            st = self._transport.stat('.')
131
149
        except errors.TransportNotPossible:
132
150
            self._dir_mode = 0755
133
151
            self._file_mode = 0644
134
152
        else:
135
 
            self._dir_mode = st.st_mode & 07777
 
153
            # Check the directory mode, but also make sure the created
 
154
            # directories and files are read-write for this user. This is
 
155
            # mostly a workaround for filesystems which lie about being able to
 
156
            # write to a directory (cygwin & win32)
 
157
            self._dir_mode = (st.st_mode & 07777) | 00700
136
158
            # Remove the sticky and execute bits for files
137
159
            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_utf8(self, path, a_string):
200
 
        """Write a string, encoding as utf-8.
201
 
 
202
 
        :param path: The path to put the string, relative to the transport root.
203
 
        :param string: A file-like or string object whose contents should be copied.
204
 
        """
205
 
        # IterableFile would not be needed if Transport.put took iterables
206
 
        # instead of files.  ADHB 2005-12-25
207
 
        # RBC 20060103 surely its not needed anyway, with codecs transcode
208
 
        # file support ?
209
 
        # JAM 20060103 We definitely don't want encode(..., 'replace')
210
 
        # these are valuable files which should have exact contents.
211
 
        if not isinstance(a_string, basestring):
212
 
            raise errors.BzrBadParameterNotString(a_string)
213
 
        self.put(path, StringIO(a_string.encode('utf-8')))
214
 
 
215
 
    def lock_write(self):
216
 
        # mutter("lock write: %s (%s)", self, self._lock_count)
 
160
 
 
161
    def leave_in_place(self):
 
162
        """Set this LockableFiles to not clear the physical lock on unlock."""
 
163
        self._lock.leave_in_place()
 
164
 
 
165
    def dont_leave_in_place(self):
 
166
        """Set this LockableFiles to clear the physical lock on unlock."""
 
167
        self._lock.dont_leave_in_place()
 
168
 
 
169
    def lock_write(self, token=None):
 
170
        """Lock this group of files for writing.
 
171
 
 
172
        :param token: if this is already locked, then lock_write will fail
 
173
            unless the token matches the existing lock.
 
174
        :returns: a token if this instance supports tokens, otherwise None.
 
175
        :raises TokenLockingNotSupported: when a token is given but this
 
176
            instance doesn't support using token locks.
 
177
        :raises MismatchedToken: if the specified token doesn't match the token
 
178
            of the existing lock.
 
179
 
 
180
        A token should be passed in if you know that you have locked the object
 
181
        some other way, and need to synchronise this object's state with that
 
182
        fact.
 
183
        """
217
184
        # TODO: Upgrade locking to support using a Transport,
218
185
        # and potentially a remote locking protocol
219
186
        if self._lock_mode:
220
187
            if self._lock_mode != 'w' or not self.get_transaction().writeable():
221
188
                raise errors.ReadOnlyError(self)
222
 
            self._lock_count += 1
 
189
            self._lock.validate_token(token)
 
190
            self._lock_warner.lock_count += 1
 
191
            return self._token_from_lock
223
192
        else:
224
 
            self._lock.lock_write()
225
 
            #note('write locking %s', self)
 
193
            token_from_lock = self._lock.lock_write(token=token)
226
194
            #traceback.print_stack()
227
195
            self._lock_mode = 'w'
228
 
            self._lock_count = 1
229
 
            self._set_transaction(transactions.WriteTransaction())
 
196
            self._lock_warner.lock_count = 1
 
197
            self._set_write_transaction()
 
198
            self._token_from_lock = token_from_lock
 
199
            return token_from_lock
230
200
 
231
201
    def lock_read(self):
232
 
        # mutter("lock read: %s (%s)", self, self._lock_count)
233
202
        if self._lock_mode:
234
 
            assert self._lock_mode in ('r', 'w'), \
235
 
                   "invalid lock mode %r" % self._lock_mode
236
 
            self._lock_count += 1
 
203
            if self._lock_mode not in ('r', 'w'):
 
204
                raise ValueError("invalid lock mode %r" % (self._lock_mode,))
 
205
            self._lock_warner.lock_count += 1
237
206
        else:
238
207
            self._lock.lock_read()
239
 
            #note('read locking %s', self)
240
208
            #traceback.print_stack()
241
209
            self._lock_mode = 'r'
242
 
            self._lock_count = 1
243
 
            self._set_transaction(transactions.ReadOnlyTransaction())
244
 
            # 5K may be excessive, but hey, its a knob.
245
 
            self.get_transaction().set_cache_size(5000)
246
 
                        
 
210
            self._lock_warner.lock_count = 1
 
211
            self._set_read_transaction()
 
212
 
 
213
    def _set_read_transaction(self):
 
214
        """Setup a read transaction."""
 
215
        self._set_transaction(transactions.ReadOnlyTransaction())
 
216
        # 5K may be excessive, but hey, its a knob.
 
217
        self.get_transaction().set_cache_size(5000)
 
218
 
 
219
    def _set_write_transaction(self):
 
220
        """Setup a write transaction."""
 
221
        self._set_transaction(transactions.WriteTransaction())
 
222
 
 
223
    @only_raises(errors.LockNotHeld, errors.LockBroken)
247
224
    def unlock(self):
248
 
        # mutter("unlock: %s (%s)", self, self._lock_count)
249
225
        if not self._lock_mode:
250
 
            raise errors.LockNotHeld(self)
251
 
        if self._lock_count > 1:
252
 
            self._lock_count -= 1
 
226
            return lock.cant_unlock_not_held(self)
 
227
        if self._lock_warner.lock_count > 1:
 
228
            self._lock_warner.lock_count -= 1
253
229
        else:
254
 
            #note('unlocking %s', self)
255
230
            #traceback.print_stack()
256
231
            self._finish_transaction()
257
232
            try:
258
233
                self._lock.unlock()
259
234
            finally:
260
 
                self._lock_mode = self._lock_count = None
 
235
                self._lock_mode = self._lock_warner.lock_count = None
 
236
 
 
237
    @property
 
238
    def _lock_count(self):
 
239
        return self._lock_warner.lock_count
261
240
 
262
241
    def is_locked(self):
263
242
        """Return true if this LockableFiles group is locked"""
264
 
        return self._lock_count >= 1
 
243
        return self._lock_warner.lock_count >= 1
265
244
 
266
245
    def get_physical_lock_status(self):
267
246
        """Return physical lock status.
268
 
        
 
247
 
269
248
        Returns true if a lock is held on the transport. If no lock is held, or
270
249
        the underlying locking mechanism does not support querying lock
271
250
        status, false is returned.
323
302
    def break_lock(self):
324
303
        raise NotImplementedError(self.break_lock)
325
304
 
326
 
    def lock_write(self):
 
305
    def leave_in_place(self):
 
306
        raise NotImplementedError(self.leave_in_place)
 
307
 
 
308
    def dont_leave_in_place(self):
 
309
        raise NotImplementedError(self.dont_leave_in_place)
 
310
 
 
311
    def lock_write(self, token=None):
 
312
        if token is not None:
 
313
            raise errors.TokenLockingNotSupported(self)
327
314
        self._lock = self._transport.lock_write(self._escaped_name)
328
315
 
329
316
    def lock_read(self):
341
328
        # for old-style locks, create the file now
342
329
        self._transport.put_bytes(self._escaped_name, '',
343
330
                            mode=self._file_modebits)
 
331
 
 
332
    def validate_token(self, token):
 
333
        if token is not None:
 
334
            raise errors.TokenLockingNotSupported(self)
 
335