← Back to branch summary

~bzr-pqm/bzr/bzr.dev

« back to all changes in this revision

Viewing changes to bzrlib/lockdir.py

  • Committer: John Arbash Meinel
  • Date: 2007-02-09 16:41:00 UTC
  • mto: This revision was merged to the branch mainline in revision 2294.
  • Revision ID: john@arbash-meinel.com-20070209164100-t9t1760kq6y5qgwn
Add get_cached_ascii for dealing with how cElementTree handles ascii strings

Show diffs side-by-side

added added

removed removed

Lines of Context:
bzrlib/lockdir.py
1
 
# Copyright (C) 2006, 2007, 2008, 2009 Canonical Ltd
 
1
# Copyright (C) 2006 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., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA
 
15
# Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307  USA
16
16
 
17
17
"""On-disk mutex protecting a resource
18
18
 
21
21
internal locks (such as flock etc) because they can be seen across all
22
22
transports, including http.
23
23
 
24
 
Objects can be read if there is only physical read access; therefore
 
24
Objects can be read if there is only physical read access; therefore 
25
25
readers can never be required to create a lock, though they will
26
26
check whether a writer is using the lock.  Writers can't detect
27
27
whether anyone else is reading from the resource as they write.
56
56
 
57
57
The desired characteristics are:
58
58
 
59
 
* Locks are not reentrant.  (That is, a client that tries to take a
 
59
* Locks are not reentrant.  (That is, a client that tries to take a 
60
60
  lock it already holds may deadlock or fail.)
61
61
* Stale locks can be guessed at by a heuristic
62
62
* Lost locks can be broken by any client
78
78
and deadlocks will likely occur if the locks are aliased.
79
79
 
80
80
In the future we may add a "freshen" method which can be called
81
 
by a lock holder to check that their lock has not been broken, and to
 
81
by a lock holder to check that their lock has not been broken, and to 
82
82
update the timestamp within it.
83
83
 
84
84
Example usage:
88
88
>>> t = MemoryTransport()
89
89
>>> l = LockDir(t, 'sample-lock')
90
90
>>> l.create()
91
 
>>> token = l.wait_lock()
 
91
>>> l.wait_lock()
92
92
>>> # do something here
93
93
>>> l.unlock()
94
94
 
95
95
"""
96
96
 
97
 
 
98
 
# TODO: We sometimes have the problem that our attempt to rename '1234' to
99
 
# 'held' fails because the transport server moves into an existing directory,
100
 
# rather than failing the rename.  If we made the info file name the same as
101
 
# the locked directory name we would avoid this problem because moving into
102
 
# the held directory would implicitly clash.  However this would not mesh with
103
 
# the existing locking code and needs a new format of the containing object.
104
 
# -- robertc, mbp 20070628
105
 
 
106
97
import os
107
98
import time
 
99
from cStringIO import StringIO
108
100
 
109
101
from bzrlib import (
110
 
    debug,
111
102
    errors,
112
 
    lock,
113
 
    osutils,
114
103
    )
115
104
import bzrlib.config
116
 
from bzrlib.decorators import only_raises
117
105
from bzrlib.errors import (
118
106
        DirectoryNotEmpty,
119
107
        FileExists,
120
108
        LockBreakMismatch,
121
109
        LockBroken,
122
110
        LockContention,
123
 
        LockFailed,
124
111
        LockNotHeld,
125
112
        NoSuchFile,
126
113
        PathError,
127
114
        ResourceBusy,
128
 
        TransportError,
 
115
        UnlockableTransport,
129
116
        )
130
117
from bzrlib.trace import mutter, note
131
 
from bzrlib.osutils import format_delta, rand_chars, get_host_name
 
118
from bzrlib.transport import Transport
 
119
from bzrlib.osutils import rand_chars, format_delta
 
120
from bzrlib.rio import read_stanza, Stanza
132
121
import bzrlib.ui
133
122
 
134
 
from bzrlib.lazy_import import lazy_import
135
 
lazy_import(globals(), """
136
 
from bzrlib import rio
137
 
""")
138
123
 
139
124
# XXX: At the moment there is no consideration of thread safety on LockDir
140
125
# objects.  This should perhaps be updated - e.g. if two threads try to take a
155
140
_DEFAULT_POLL_SECONDS = 1.0
156
141
 
157
142
 
158
 
class LockDir(lock.Lock):
159
 
    """Write-lock guarding access to data.
160
 
    """
 
143
class LockDir(object):
 
144
    """Write-lock guarding access to data."""
161
145
 
162
146
    __INFO_NAME = '/info'
163
147
 
168
152
 
169
153
        :param transport: Transport which will contain the lock
170
154
 
171
 
        :param path: Path to the lock within the base directory of the
 
155
        :param path: Path to the lock within the base directory of the 
172
156
            transport.
173
157
        """
 
158
        assert isinstance(transport, Transport), \
 
159
            ("not a transport: %r" % transport)
174
160
        self.transport = transport
175
161
        self.path = path
176
162
        self._lock_held = False
177
 
        self._locked_via_token = False
178
163
        self._fake_read_lock = False
179
164
        self._held_dir = path + '/held'
180
165
        self._held_info_path = self._held_dir + self.__INFO_NAME
181
166
        self._file_modebits = file_modebits
182
167
        self._dir_modebits = dir_modebits
 
168
        self.nonce = rand_chars(20)
183
169
 
184
170
        self._report_function = note
185
171
 
193
179
    def create(self, mode=None):
194
180
        """Create the on-disk lock.
195
181
 
196
 
        This is typically only called when the object/directory containing the
 
182
        This is typically only called when the object/directory containing the 
197
183
        directory is first created.  The lock is not held when it's created.
198
184
        """
199
 
        self._trace("create lock directory")
200
 
        try:
201
 
            self.transport.mkdir(self.path, mode=mode)
202
 
        except (TransportError, PathError), e:
203
 
            raise LockFailed(self, e)
204
 
 
205
 
 
206
 
    def _attempt_lock(self):
207
 
        """Make the pending directory and attempt to rename into place.
208
 
 
209
 
        If the rename succeeds, we read back the info file to check that we
210
 
        really got the lock.
211
 
 
212
 
        If we fail to acquire the lock, this method is responsible for
213
 
        cleaning up the pending directory if possible.  (But it doesn't do
214
 
        that yet.)
215
 
 
216
 
        :returns: The nonce of the lock, if it was successfully acquired.
217
 
 
218
 
        :raises LockContention: If the lock is held by someone else.  The exception
219
 
            contains the info of the current holder of the lock.
 
185
        if self.transport.is_readonly():
 
186
            raise UnlockableTransport(self.transport)
 
187
        self.transport.mkdir(self.path, mode=mode)
 
188
 
 
189
    def attempt_lock(self):
 
190
        """Take the lock; fail if it's already held.
 
191
        
 
192
        If you wish to block until the lock can be obtained, call wait_lock()
 
193
        instead.
220
194
        """
221
 
        self._trace("lock_write...")
222
 
        start_time = time.time()
223
 
        try:
224
 
            tmpname = self._create_pending_dir()
225
 
        except (errors.TransportError, PathError), e:
226
 
            self._trace("... failed to create pending dir, %s", e)
227
 
            raise LockFailed(self, e)
228
 
        try:
 
195
        if self._fake_read_lock:
 
196
            raise LockContention(self)
 
197
        if self.transport.is_readonly():
 
198
            raise UnlockableTransport(self.transport)
 
199
        try:
 
200
            tmpname = '%s/pending.%s.tmp' % (self.path, rand_chars(20))
 
201
            try:
 
202
                self.transport.mkdir(tmpname)
 
203
            except NoSuchFile:
 
204
                # This may raise a FileExists exception
 
205
                # which is okay, it will be caught later and determined
 
206
                # to be a LockContention.
 
207
                self.create(mode=self._dir_modebits)
 
208
                
 
209
                # After creating the lock directory, try again
 
210
                self.transport.mkdir(tmpname)
 
211
 
 
212
            info_bytes = self._prepare_info()
 
213
            # We use put_file_non_atomic because we just created a new unique
 
214
            # directory so we don't have to worry about files existing there.
 
215
            # We'll rename the whole directory into place to get atomic
 
216
            # properties
 
217
            self.transport.put_bytes_non_atomic(tmpname + self.__INFO_NAME,
 
218
                                                info_bytes)
 
219
 
229
220
            self.transport.rename(tmpname, self._held_dir)
230
 
        except (errors.TransportError, PathError, DirectoryNotEmpty,
231
 
                FileExists, ResourceBusy), e:
232
 
            self._trace("... contention, %s", e)
233
 
            self._remove_pending_dir(tmpname)
234
 
            raise LockContention(self)
235
 
        except Exception, e:
236
 
            self._trace("... lock failed, %s", e)
237
 
            self._remove_pending_dir(tmpname)
238
 
            raise
239
 
        # We must check we really got the lock, because Launchpad's sftp
240
 
        # server at one time had a bug were the rename would successfully
241
 
        # move the new directory into the existing directory, which was
242
 
        # incorrect.  It's possible some other servers or filesystems will
243
 
        # have a similar bug allowing someone to think they got the lock
244
 
        # when it's already held.
245
 
        #
246
 
        # See <https://bugs.edge.launchpad.net/bzr/+bug/498378> for one case.
247
 
        #
248
 
        # Strictly the check is unnecessary and a waste of time for most
249
 
        # people, but probably worth trapping if something is wrong.
250
 
        info = self.peek()
251
 
        self._trace("after locking, info=%r", info)
252
 
        if info is None:
253
 
            raise LockFailed(self, "lock was renamed into place, but "
254
 
                "now is missing!")
255
 
        if info['nonce'] != self.nonce:
256
 
            self._trace("rename succeeded, "
257
 
                "but lock is still held by someone else")
258
 
            raise LockContention(self)
259
 
        self._lock_held = True
260
 
        self._trace("... lock succeeded after %dms",
261
 
                (time.time() - start_time) * 1000)
262
 
        return self.nonce
263
 
 
264
 
    def _remove_pending_dir(self, tmpname):
265
 
        """Remove the pending directory
266
 
 
267
 
        This is called if we failed to rename into place, so that the pending
268
 
        dirs don't clutter up the lockdir.
269
 
        """
270
 
        self._trace("remove %s", tmpname)
271
 
        try:
272
 
            self.transport.delete(tmpname + self.__INFO_NAME)
273
 
            self.transport.rmdir(tmpname)
274
 
        except PathError, e:
275
 
            note("error removing pending lock: %s", e)
276
 
 
277
 
    def _create_pending_dir(self):
278
 
        tmpname = '%s/%s.tmp' % (self.path, rand_chars(10))
279
 
        try:
280
 
            self.transport.mkdir(tmpname)
281
 
        except NoSuchFile:
282
 
            # This may raise a FileExists exception
283
 
            # which is okay, it will be caught later and determined
284
 
            # to be a LockContention.
285
 
            self._trace("lock directory does not exist, creating it")
286
 
            self.create(mode=self._dir_modebits)
287
 
            # After creating the lock directory, try again
288
 
            self.transport.mkdir(tmpname)
289
 
        self.nonce = rand_chars(20)
290
 
        info_bytes = self._prepare_info()
291
 
        # We use put_file_non_atomic because we just created a new unique
292
 
        # directory so we don't have to worry about files existing there.
293
 
        # We'll rename the whole directory into place to get atomic
294
 
        # properties
295
 
        self.transport.put_bytes_non_atomic(tmpname + self.__INFO_NAME,
296
 
                                            info_bytes)
297
 
        return tmpname
298
 
 
299
 
    @only_raises(LockNotHeld, LockBroken)
 
221
            self._lock_held = True
 
222
            self.confirm()
 
223
        except (PathError, DirectoryNotEmpty, FileExists, ResourceBusy), e:
 
224
            mutter("contention on %r: %s", self, e)
 
225
            raise LockContention(self)
 
226
 
300
227
    def unlock(self):
301
228
        """Release a held lock
302
229
        """
304
231
            self._fake_read_lock = False
305
232
            return
306
233
        if not self._lock_held:
307
 
            return lock.cant_unlock_not_held(self)
308
 
        if self._locked_via_token:
309
 
            self._locked_via_token = False
310
 
            self._lock_held = False
311
 
        else:
312
 
            old_nonce = self.nonce
313
 
            # rename before deleting, because we can't atomically remove the
314
 
            # whole tree
315
 
            start_time = time.time()
316
 
            self._trace("unlocking")
317
 
            tmpname = '%s/releasing.%s.tmp' % (self.path, rand_chars(20))
318
 
            # gotta own it to unlock
319
 
            self.confirm()
320
 
            self.transport.rename(self._held_dir, tmpname)
321
 
            self._lock_held = False
322
 
            self.transport.delete(tmpname + self.__INFO_NAME)
323
 
            try:
324
 
                self.transport.rmdir(tmpname)
325
 
            except DirectoryNotEmpty, e:
326
 
                # There might have been junk left over by a rename that moved
327
 
                # another locker within the 'held' directory.  do a slower
328
 
                # deletion where we list the directory and remove everything
329
 
                # within it.
330
 
                #
331
 
                # Maybe this should be broader to allow for ftp servers with
332
 
                # non-specific error messages?
333
 
                self._trace("doing recursive deletion of non-empty directory "
334
 
                        "%s", tmpname)
335
 
                self.transport.delete_tree(tmpname)
336
 
            self._trace("... unlock succeeded after %dms",
337
 
                    (time.time() - start_time) * 1000)
338
 
            result = lock.LockResult(self.transport.abspath(self.path),
339
 
                                     old_nonce)
340
 
            for hook in self.hooks['lock_released']:
341
 
                hook(result)
 
234
            raise LockNotHeld(self)
 
235
        # rename before deleting, because we can't atomically remove the whole
 
236
        # tree
 
237
        tmpname = '%s/releasing.%s.tmp' % (self.path, rand_chars(20))
 
238
        # gotta own it to unlock
 
239
        self.confirm()
 
240
        self.transport.rename(self._held_dir, tmpname)
 
241
        self._lock_held = False
 
242
        self.transport.delete(tmpname + self.__INFO_NAME)
 
243
        self.transport.rmdir(tmpname)
342
244
 
343
245
    def break_lock(self):
344
246
        """Break a lock not held by this instance of LockDir.
353
255
            lock_info = '\n'.join(self._format_lock_info(holder_info))
354
256
            if bzrlib.ui.ui_factory.get_boolean("Break %s" % lock_info):
355
257
                self.force_break(holder_info)
356
 
 
 
258
        
357
259
    def force_break(self, dead_holder_info):
358
260
        """Release a lock held by another process.
359
261
 
367
269
        LockBreakMismatch is raised.
368
270
 
369
271
        After the lock is broken it will not be held by any process.
370
 
        It is possible that another process may sneak in and take the
 
272
        It is possible that another process may sneak in and take the 
371
273
        lock before the breaking process acquires it.
372
274
        """
373
275
        if not isinstance(dead_holder_info, dict):
382
284
        tmpname = '%s/broken.%s.tmp' % (self.path, rand_chars(20))
383
285
        self.transport.rename(self._held_dir, tmpname)
384
286
        # check that we actually broke the right lock, not someone else;
385
 
        # there's a small race window between checking it and doing the
 
287
        # there's a small race window between checking it and doing the 
386
288
        # rename.
387
289
        broken_info_path = tmpname + self.__INFO_NAME
388
290
        broken_info = self._read_info_file(broken_info_path)
390
292
            raise LockBreakMismatch(self, broken_info, dead_holder_info)
391
293
        self.transport.delete(broken_info_path)
392
294
        self.transport.rmdir(tmpname)
393
 
        result = lock.LockResult(self.transport.abspath(self.path),
394
 
                                 current_info.get('nonce'))
395
 
        for hook in self.hooks['lock_broken']:
396
 
            hook(result)
397
295
 
398
296
    def _check_not_locked(self):
399
297
        """If the lock is held by this instance, raise an error."""
407
305
        or if the lock has been affected by a bug.
408
306
 
409
307
        If the lock is not thought to be held, raises LockNotHeld.  If
410
 
        the lock is thought to be held but has been broken, raises
 
308
        the lock is thought to be held but has been broken, raises 
411
309
        LockBroken.
412
310
        """
413
311
        if not self._lock_held:
419
317
        if info.get('nonce') != self.nonce:
420
318
            # there is a lock, but not ours
421
319
            raise LockBroken(self)
422
 
 
 
320
        
423
321
    def _read_info_file(self, path):
424
322
        """Read one given info file.
425
323
 
426
324
        peek() reads the info file of the lock holder, if any.
427
325
        """
428
 
        return self._parse_info(self.transport.get_bytes(path))
 
326
        return self._parse_info(self.transport.get(path))
429
327
 
430
328
    def peek(self):
431
329
        """Check if the lock is held by anyone.
432
 
 
 
330
        
433
331
        If it is held, this returns the lock info structure as a rio Stanza,
434
332
        which contains some information about the current lock holder.
435
333
        Otherwise returns None.
436
334
        """
437
335
        try:
438
336
            info = self._read_info_file(self._held_info_path)
439
 
            self._trace("peek -> held")
 
337
            assert isinstance(info, dict), \
 
338
                    "bad parse result %r" % info
440
339
            return info
441
340
        except NoSuchFile, e:
442
 
            self._trace("peek -> not held")
 
341
            return None
443
342
 
444
343
    def _prepare_info(self):
445
344
        """Write information about a pending lock to a temporary file.
446
345
        """
 
346
        import socket
447
347
        # XXX: is creating this here inefficient?
448
348
        config = bzrlib.config.GlobalConfig()
449
349
        try:
450
350
            user = config.user_email()
451
351
        except errors.NoEmailInUsername:
452
352
            user = config.username()
453
 
        s = rio.Stanza(hostname=get_host_name(),
 
353
        s = Stanza(hostname=socket.gethostname(),
454
354
                   pid=str(os.getpid()),
455
355
                   start_time=str(int(time.time())),
456
356
                   nonce=self.nonce,
458
358
                   )
459
359
        return s.to_string()
460
360
 
461
 
    def _parse_info(self, info_bytes):
462
 
        # TODO: Handle if info_bytes is empty
463
 
        return rio.read_stanza(osutils.split_lines(info_bytes)).as_dict()
464
 
 
465
 
    def attempt_lock(self):
466
 
        """Take the lock; fail if it's already held.
467
 
 
468
 
        If you wish to block until the lock can be obtained, call wait_lock()
469
 
        instead.
470
 
 
471
 
        :return: The lock token.
472
 
        :raises LockContention: if the lock is held by someone else.
473
 
        """
474
 
        if self._fake_read_lock:
475
 
            raise LockContention(self)
476
 
        result = self._attempt_lock()
477
 
        hook_result = lock.LockResult(self.transport.abspath(self.path),
478
 
                self.nonce)
479
 
        for hook in self.hooks['lock_acquired']:
480
 
            hook(hook_result)
481
 
        return result
482
 
 
483
 
    def wait_lock(self, timeout=None, poll=None, max_attempts=None):
 
361
    def _parse_info(self, info_file):
 
362
        return read_stanza(info_file.readlines()).as_dict()
 
363
 
 
364
    def wait_lock(self, timeout=None, poll=None):
484
365
        """Wait a certain period for a lock.
485
366
 
486
367
        If the lock can be acquired within the bounded time, it
488
369
        is raised.  Either way, this function should return within
489
370
        approximately `timeout` seconds.  (It may be a bit more if
490
371
        a transport operation takes a long time to complete.)
491
 
 
492
 
        :param timeout: Approximate maximum amount of time to wait for the
493
 
        lock, in seconds.
494
 
 
495
 
        :param poll: Delay in seconds between retrying the lock.
496
 
 
497
 
        :param max_attempts: Maximum number of times to try to lock.
498
 
 
499
 
        :return: The lock token.
500
372
        """
501
373
        if timeout is None:
502
374
            timeout = _DEFAULT_TIMEOUT_SECONDS
503
375
        if poll is None:
504
376
            poll = _DEFAULT_POLL_SECONDS
505
 
        # XXX: the transport interface doesn't let us guard against operations
506
 
        # there taking a long time, so the total elapsed time or poll interval
507
 
        # may be more than was requested.
 
377
 
 
378
        # XXX: the transport interface doesn't let us guard 
 
379
        # against operations there taking a long time.
508
380
        deadline = time.time() + timeout
509
381
        deadline_str = None
510
382
        last_info = None
511
 
        attempt_count = 0
512
383
        while True:
513
 
            attempt_count += 1
514
384
            try:
515
 
                return self.attempt_lock()
 
385
                self.attempt_lock()
 
386
                return
516
387
            except LockContention:
517
 
                # possibly report the blockage, then try again
518
388
                pass
519
 
            # TODO: In a few cases, we find out that there's contention by
520
 
            # reading the held info and observing that it's not ours.  In
521
 
            # those cases it's a bit redundant to read it again.  However,
522
 
            # the normal case (??) is that the rename fails and so we
523
 
            # don't know who holds the lock.  For simplicity we peek
524
 
            # always.
525
389
            new_info = self.peek()
 
390
            mutter('last_info: %s, new info: %s', last_info, new_info)
526
391
            if new_info is not None and new_info != last_info:
527
392
                if last_info is None:
528
393
                    start = 'Unable to obtain'
533
398
                if deadline_str is None:
534
399
                    deadline_str = time.strftime('%H:%M:%S',
535
400
                                                 time.localtime(deadline))
536
 
                lock_url = self.transport.abspath(self.path)
537
 
                # See <https://bugs.edge.launchpad.net/bzr/+bug/250451>
538
 
                # the URL here is sometimes not one that is useful to the
539
 
                # user, perhaps being wrapped in a lp-%d or chroot decorator,
540
 
                # especially if this error is issued from the server.
541
401
                self._report_function('%s %s\n'
542
 
                    '%s\n' # held by
543
 
                    '%s\n' # locked ... ago
544
 
                    'Will continue to try until %s, unless '
545
 
                    'you press Ctrl-C.\n'
546
 
                    'See "bzr help break-lock" for more.',
547
 
                    start,
548
 
                    formatted_info[0],
549
 
                    formatted_info[1],
550
 
                    formatted_info[2],
551
 
                    deadline_str,
552
 
                    )
 
402
                                      '%s\n' # held by
 
403
                                      '%s\n' # locked ... ago
 
404
                                      'Will continue to try until %s\n',
 
405
                                      start,
 
406
                                      formatted_info[0],
 
407
                                      formatted_info[1],
 
408
                                      formatted_info[2],
 
409
                                      deadline_str)
553
410
 
554
 
            if (max_attempts is not None) and (attempt_count >= max_attempts):
555
 
                self._trace("exceeded %d attempts")
556
 
                raise LockContention(self)
557
411
            if time.time() + poll < deadline:
558
 
                self._trace("waiting %ss", poll)
559
412
                time.sleep(poll)
560
413
            else:
561
 
                self._trace("timeout after waiting %ss", timeout)
562
414
                raise LockContention(self)
563
415
 
564
 
    def leave_in_place(self):
565
 
        self._locked_via_token = True
566
 
 
567
 
    def dont_leave_in_place(self):
568
 
        self._locked_via_token = False
569
 
 
570
 
    def lock_write(self, token=None):
571
 
        """Wait for and acquire the lock.
572
 
 
573
 
        :param token: if this is already locked, then lock_write will fail
574
 
            unless the token matches the existing lock.
575
 
        :returns: a token if this instance supports tokens, otherwise None.
576
 
        :raises TokenLockingNotSupported: when a token is given but this
577
 
            instance doesn't support using token locks.
578
 
        :raises MismatchedToken: if the specified token doesn't match the token
579
 
            of the existing lock.
580
 
 
581
 
        A token should be passed in if you know that you have locked the object
582
 
        some other way, and need to synchronise this object's state with that
583
 
        fact.
584
 
 
585
 
        XXX: docstring duplicated from LockableFiles.lock_write.
586
 
        """
587
 
        if token is not None:
588
 
            self.validate_token(token)
589
 
            self.nonce = token
590
 
            self._lock_held = True
591
 
            self._locked_via_token = True
592
 
            return token
593
 
        else:
594
 
            return self.wait_lock()
 
416
    def lock_write(self):
 
417
        """Wait for and acquire the lock."""
 
418
        self.wait_lock()
595
419
 
596
420
    def lock_read(self):
597
421
        """Compatibility-mode shared lock.
598
422
 
599
 
        LockDir doesn't support shared read-only locks, so this
 
423
        LockDir doesn't support shared read-only locks, so this 
600
424
        just pretends that the lock is taken but really does nothing.
601
425
        """
602
 
        # At the moment Branches are commonly locked for read, but
 
426
        # At the moment Branches are commonly locked for read, but 
603
427
        # we can't rely on that remotely.  Once this is cleaned up,
604
 
        # reenable this warning to prevent it coming back in
 
428
        # reenable this warning to prevent it coming back in 
605
429
        # -- mbp 20060303
606
430
        ## warn("LockDir.lock_read falls back to write lock")
607
431
        if self._lock_held or self._fake_read_lock:
608
432
            raise LockContention(self)
609
433
        self._fake_read_lock = True
610
434
 
 
435
    def wait(self, timeout=20, poll=0.5):
 
436
        """Wait a certain period for a lock to be released."""
 
437
        # XXX: the transport interface doesn't let us guard 
 
438
        # against operations there taking a long time.
 
439
        deadline = time.time() + timeout
 
440
        while True:
 
441
            if self.peek():
 
442
                return
 
443
            if time.time() + poll < deadline:
 
444
                time.sleep(poll)
 
445
            else:
 
446
                raise LockContention(self)
 
447
 
611
448
    def _format_lock_info(self, info):
612
449
        """Turn the contents of peek() into something for the user"""
613
450
        lock_url = self.transport.abspath(self.path)
618
455
            'locked %s' % (format_delta(delta),),
619
456
            ]
620
457
 
621
 
    def validate_token(self, token):
622
 
        if token is not None:
623
 
            info = self.peek()
624
 
            if info is None:
625
 
                # Lock isn't held
626
 
                lock_token = None
627
 
            else:
628
 
                lock_token = info.get('nonce')
629
 
            if token != lock_token:
630
 
                raise errors.TokenMismatch(token, lock_token)
631
 
            else:
632
 
                self._trace("revalidated by token %r", token)
633
 
 
634
 
    def _trace(self, format, *args):
635
 
        if 'lock' not in debug.debug_flags:
636
 
            return
637
 
        mutter(str(self) + ": " + (format % args))