~bzr-pqm/bzr/bzr.dev

« back to all changes in this revision

Viewing changes to bzrlib/lockdir.py

  • Committer: Canonical.com Patch Queue Manager
  • Date: 2010-01-20 17:19:44 UTC
  • mfrom: (4973.1.1 integration)
  • Revision ID: pqm@pqm.ubuntu.com-20100120171944-8gkub20gotjcye67
(spiv) Implement a per-file merge hook

Show diffs side-by-side

added added

removed removed

Lines of Context:
1
 
# Copyright (C) 2006-2011 Canonical Ltd
 
1
# Copyright (C) 2006, 2007, 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
92
92
>>> # do something here
93
93
>>> l.unlock()
94
94
 
95
 
Some classes of stale locks can be predicted by checking: the host name is the
96
 
same as the local host name; the user name is the same as the local user; the
97
 
process id no longer exists.  The check on user name is not strictly necessary
98
 
but helps protect against colliding host names.
99
95
"""
100
96
 
101
97
 
107
103
# the existing locking code and needs a new format of the containing object.
108
104
# -- robertc, mbp 20070628
109
105
 
110
 
import errno
111
106
import os
112
107
import time
113
108
 
114
109
from bzrlib import (
115
 
    config,
116
110
    debug,
117
111
    errors,
118
112
    lock,
119
113
    osutils,
120
 
    ui,
121
 
    urlutils,
122
114
    )
 
115
import bzrlib.config
123
116
from bzrlib.decorators import only_raises
124
117
from bzrlib.errors import (
125
118
        DirectoryNotEmpty,
127
120
        LockBreakMismatch,
128
121
        LockBroken,
129
122
        LockContention,
130
 
        LockCorrupt,
131
123
        LockFailed,
132
124
        LockNotHeld,
133
125
        NoSuchFile,
137
129
        )
138
130
from bzrlib.trace import mutter, note
139
131
from bzrlib.osutils import format_delta, rand_chars, get_host_name
 
132
import bzrlib.ui
140
133
 
141
134
from bzrlib.lazy_import import lazy_import
142
135
lazy_import(globals(), """
158
151
# files/dirs created.
159
152
 
160
153
 
161
 
_DEFAULT_TIMEOUT_SECONDS = 30
 
154
_DEFAULT_TIMEOUT_SECONDS = 300
162
155
_DEFAULT_POLL_SECONDS = 1.0
163
156
 
164
157
 
168
161
 
169
162
    __INFO_NAME = '/info'
170
163
 
171
 
    def __init__(self, transport, path, file_modebits=0644, dir_modebits=0755,
172
 
        extra_holder_info=None):
 
164
    def __init__(self, transport, path, file_modebits=0644, dir_modebits=0755):
173
165
        """Create a new LockDir object.
174
166
 
175
167
        The LockDir is initially unlocked - this just creates the object.
178
170
 
179
171
        :param path: Path to the lock within the base directory of the
180
172
            transport.
181
 
 
182
 
        :param extra_holder_info: If passed, {str:str} dict of extra or
183
 
            updated information to insert into the info file when the lock is
184
 
            taken.
185
173
        """
186
174
        self.transport = transport
187
175
        self.path = path
192
180
        self._held_info_path = self._held_dir + self.__INFO_NAME
193
181
        self._file_modebits = file_modebits
194
182
        self._dir_modebits = dir_modebits
 
183
 
195
184
        self._report_function = note
196
 
        self.extra_holder_info = extra_holder_info
197
 
        self._warned_about_lock_holder = None
198
185
 
199
186
    def __repr__(self):
200
187
        return '%s(%s%s)' % (self.__class__.__name__,
215
202
        except (TransportError, PathError), e:
216
203
            raise LockFailed(self, e)
217
204
 
 
205
 
218
206
    def _attempt_lock(self):
219
207
        """Make the pending directory and attempt to rename into place.
220
208
 
227
215
 
228
216
        :returns: The nonce of the lock, if it was successfully acquired.
229
217
 
230
 
        :raises LockContention: If the lock is held by someone else.  The
231
 
            exception contains the info of the current holder of the lock.
 
218
        :raises LockContention: If the lock is held by someone else.  The exception
 
219
            contains the info of the current holder of the lock.
232
220
        """
233
221
        self._trace("lock_write...")
234
222
        start_time = time.time()
237
225
        except (errors.TransportError, PathError), e:
238
226
            self._trace("... failed to create pending dir, %s", e)
239
227
            raise LockFailed(self, e)
240
 
        while True:
241
 
            try:
242
 
                self.transport.rename(tmpname, self._held_dir)
243
 
                break
244
 
            except (errors.TransportError, PathError, DirectoryNotEmpty,
245
 
                    FileExists, ResourceBusy), e:
246
 
                self._trace("... contention, %s", e)
247
 
                other_holder = self.peek()
248
 
                self._trace("other holder is %r" % other_holder)
249
 
                try:
250
 
                    self._handle_lock_contention(other_holder)
251
 
                except:
252
 
                    self._remove_pending_dir(tmpname)
253
 
                    raise
254
 
            except Exception, e:
255
 
                self._trace("... lock failed, %s", e)
256
 
                self._remove_pending_dir(tmpname)
257
 
                raise
 
228
        try:
 
229
            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
258
239
        # We must check we really got the lock, because Launchpad's sftp
259
240
        # server at one time had a bug were the rename would successfully
260
241
        # move the new directory into the existing directory, which was
262
243
        # have a similar bug allowing someone to think they got the lock
263
244
        # when it's already held.
264
245
        #
265
 
        # See <https://bugs.launchpad.net/bzr/+bug/498378> for one case.
 
246
        # See <https://bugs.edge.launchpad.net/bzr/+bug/498378> for one case.
266
247
        #
267
248
        # Strictly the check is unnecessary and a waste of time for most
268
249
        # people, but probably worth trapping if something is wrong.
271
252
        if info is None:
272
253
            raise LockFailed(self, "lock was renamed into place, but "
273
254
                "now is missing!")
274
 
        if info.get('nonce') != self.nonce:
 
255
        if info['nonce'] != self.nonce:
275
256
            self._trace("rename succeeded, "
276
257
                "but lock is still held by someone else")
277
258
            raise LockContention(self)
280
261
                (time.time() - start_time) * 1000)
281
262
        return self.nonce
282
263
 
283
 
    def _handle_lock_contention(self, other_holder):
284
 
        """A lock we want to take is held by someone else.
285
 
 
286
 
        This function can: tell the user about it; possibly detect that it's
287
 
        safe or appropriate to steal the lock, or just raise an exception.
288
 
 
289
 
        If this function returns (without raising an exception) the lock will
290
 
        be attempted again.
291
 
 
292
 
        :param other_holder: A LockHeldInfo for the current holder; note that
293
 
            it might be None if the lock can be seen to be held but the info
294
 
            can't be read.
295
 
        """
296
 
        if (other_holder is not None):
297
 
            if other_holder.is_lock_holder_known_dead():
298
 
                if self.get_config().get_user_option_as_bool(
299
 
                    'locks.steal_dead',
300
 
                    default=False):
301
 
                    ui.ui_factory.show_user_warning(
302
 
                        'locks_steal_dead',
303
 
                        lock_url=urlutils.join(self.transport.base, self.path),
304
 
                        other_holder_info=unicode(other_holder))
305
 
                    self.force_break(other_holder)
306
 
                    self._trace("stole lock from dead holder")
307
 
                    return
308
 
        raise LockContention(self)
309
 
 
310
264
    def _remove_pending_dir(self, tmpname):
311
265
        """Remove the pending directory
312
266
 
332
286
            self.create(mode=self._dir_modebits)
333
287
            # After creating the lock directory, try again
334
288
            self.transport.mkdir(tmpname)
335
 
        info = LockHeldInfo.for_this_process(self.extra_holder_info)
336
 
        self.nonce = info.get('nonce')
 
289
        self.nonce = rand_chars(20)
 
290
        info_bytes = self._prepare_info()
337
291
        # We use put_file_non_atomic because we just created a new unique
338
292
        # directory so we don't have to worry about files existing there.
339
293
        # We'll rename the whole directory into place to get atomic
340
294
        # properties
341
295
        self.transport.put_bytes_non_atomic(tmpname + self.__INFO_NAME,
342
 
            info.to_bytes())
 
296
                                            info_bytes)
343
297
        return tmpname
344
298
 
345
299
    @only_raises(LockNotHeld, LockBroken)
389
343
    def break_lock(self):
390
344
        """Break a lock not held by this instance of LockDir.
391
345
 
392
 
        This is a UI centric function: it uses the ui.ui_factory to
 
346
        This is a UI centric function: it uses the bzrlib.ui.ui_factory to
393
347
        prompt for input if a lock is detected and there is any doubt about
394
 
        it possibly being still active.  force_break is the non-interactive
395
 
        version.
396
 
 
397
 
        :returns: LockResult for the broken lock.
 
348
        it possibly being still active.
398
349
        """
399
350
        self._check_not_locked()
400
 
        try:
401
 
            holder_info = self.peek()
402
 
        except LockCorrupt, e:
403
 
            # The lock info is corrupt.
404
 
            if ui.ui_factory.get_boolean(u"Break (corrupt %r)" % (self,)):
405
 
                self.force_break_corrupt(e.file_data)
406
 
            return
 
351
        holder_info = self.peek()
407
352
        if holder_info is not None:
408
 
            if ui.ui_factory.confirm_action(
409
 
                u"Break %(lock_info)s",
410
 
                'bzrlib.lockdir.break',
411
 
                dict(lock_info=unicode(holder_info))):
412
 
                result = self.force_break(holder_info)
413
 
                ui.ui_factory.show_message(
414
 
                    "Broke lock %s" % result.lock_url)
 
353
            lock_info = '\n'.join(self._format_lock_info(holder_info))
 
354
            if bzrlib.ui.ui_factory.get_boolean("Break %s" % lock_info):
 
355
                self.force_break(holder_info)
415
356
 
416
357
    def force_break(self, dead_holder_info):
417
358
        """Release a lock held by another process.
420
361
        it still thinks it has the lock there will be two concurrent writers.
421
362
        In general the user's approval should be sought for lock breaks.
422
363
 
 
364
        dead_holder_info must be the result of a previous LockDir.peek() call;
 
365
        this is used to check that it's still held by the same process that
 
366
        the user decided was dead.  If this is not the current holder,
 
367
        LockBreakMismatch is raised.
 
368
 
423
369
        After the lock is broken it will not be held by any process.
424
370
        It is possible that another process may sneak in and take the
425
371
        lock before the breaking process acquires it.
426
 
 
427
 
        :param dead_holder_info:
428
 
            Must be the result of a previous LockDir.peek() call; this is used
429
 
            to check that it's still held by the same process that the user
430
 
            decided was dead.  If this is not the current holder,
431
 
            LockBreakMismatch is raised.
432
 
 
433
 
        :returns: LockResult for the broken lock.
434
372
        """
435
 
        if not isinstance(dead_holder_info, LockHeldInfo):
 
373
        if not isinstance(dead_holder_info, dict):
436
374
            raise ValueError("dead_holder_info: %r" % dead_holder_info)
437
375
        self._check_not_locked()
438
376
        current_info = self.peek()
456
394
                                 current_info.get('nonce'))
457
395
        for hook in self.hooks['lock_broken']:
458
396
            hook(result)
459
 
        return result
460
 
 
461
 
    def force_break_corrupt(self, corrupt_info_lines):
462
 
        """Release a lock that has been corrupted.
463
 
 
464
 
        This is very similar to force_break, it except it doesn't assume that
465
 
        self.peek() can work.
466
 
 
467
 
        :param corrupt_info_lines: the lines of the corrupted info file, used
468
 
            to check that the lock hasn't changed between reading the (corrupt)
469
 
            info file and calling force_break_corrupt.
470
 
        """
471
 
        # XXX: this copes with unparseable info files, but what about missing
472
 
        # info files?  Or missing lock dirs?
473
 
        self._check_not_locked()
474
 
        tmpname = '%s/broken.%s.tmp' % (self.path, rand_chars(20))
475
 
        self.transport.rename(self._held_dir, tmpname)
476
 
        # check that we actually broke the right lock, not someone else;
477
 
        # there's a small race window between checking it and doing the
478
 
        # rename.
479
 
        broken_info_path = tmpname + self.__INFO_NAME
480
 
        broken_content = self.transport.get_bytes(broken_info_path)
481
 
        broken_lines = osutils.split_lines(broken_content)
482
 
        if broken_lines != corrupt_info_lines:
483
 
            raise LockBreakMismatch(self, broken_lines, corrupt_info_lines)
484
 
        self.transport.delete(broken_info_path)
485
 
        self.transport.rmdir(tmpname)
486
 
        result = lock.LockResult(self.transport.abspath(self.path))
487
 
        for hook in self.hooks['lock_broken']:
488
 
            hook(result)
489
397
 
490
398
    def _check_not_locked(self):
491
399
        """If the lock is held by this instance, raise an error."""
517
425
 
518
426
        peek() reads the info file of the lock holder, if any.
519
427
        """
520
 
        return LockHeldInfo.from_info_file_bytes(
521
 
            self.transport.get_bytes(path))
 
428
        return self._parse_info(self.transport.get_bytes(path))
522
429
 
523
430
    def peek(self):
524
431
        """Check if the lock is held by anyone.
525
432
 
526
 
        If it is held, this returns the lock info structure as a dict
 
433
        If it is held, this returns the lock info structure as a rio Stanza,
527
434
        which contains some information about the current lock holder.
528
435
        Otherwise returns None.
529
436
        """
537
444
    def _prepare_info(self):
538
445
        """Write information about a pending lock to a temporary file.
539
446
        """
 
447
        # XXX: is creating this here inefficient?
 
448
        config = bzrlib.config.GlobalConfig()
 
449
        try:
 
450
            user = config.user_email()
 
451
        except errors.NoEmailInUsername:
 
452
            user = config.username()
 
453
        s = rio.Stanza(hostname=get_host_name(),
 
454
                   pid=str(os.getpid()),
 
455
                   start_time=str(int(time.time())),
 
456
                   nonce=self.nonce,
 
457
                   user=user,
 
458
                   )
 
459
        return s.to_string()
 
460
 
 
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()
540
464
 
541
465
    def attempt_lock(self):
542
466
        """Take the lock; fail if it's already held.
556
480
            hook(hook_result)
557
481
        return result
558
482
 
559
 
    def lock_url_for_display(self):
560
 
        """Give a nicely-printable representation of the URL of this lock."""
561
 
        # As local lock urls are correct we display them.
562
 
        # We avoid displaying remote lock urls.
563
 
        lock_url = self.transport.abspath(self.path)
564
 
        if lock_url.startswith('file://'):
565
 
            lock_url = lock_url.split('.bzr/')[0]
566
 
        else:
567
 
            lock_url = ''
568
 
        return lock_url
569
 
 
570
483
    def wait_lock(self, timeout=None, poll=None, max_attempts=None):
571
484
        """Wait a certain period for a lock.
572
485
 
596
509
        deadline_str = None
597
510
        last_info = None
598
511
        attempt_count = 0
599
 
        lock_url = self.lock_url_for_display()
600
512
        while True:
601
513
            attempt_count += 1
602
514
            try:
617
529
                else:
618
530
                    start = 'Lock owner changed for'
619
531
                last_info = new_info
620
 
                msg = u'%s lock %s %s.' % (start, lock_url, new_info)
 
532
                formatted_info = self._format_lock_info(new_info)
621
533
                if deadline_str is None:
622
534
                    deadline_str = time.strftime('%H:%M:%S',
623
 
                                                    time.localtime(deadline))
624
 
                if timeout > 0:
625
 
                    msg += ('\nWill continue to try until %s, unless '
626
 
                        'you press Ctrl-C.'
627
 
                        % deadline_str)
628
 
                msg += '\nSee "bzr help break-lock" for more.'
629
 
                self._report_function(msg)
 
535
                                                 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
                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
                    )
 
553
 
630
554
            if (max_attempts is not None) and (attempt_count >= max_attempts):
631
555
                self._trace("exceeded %d attempts")
632
556
                raise LockContention(self)
634
558
                self._trace("waiting %ss", poll)
635
559
                time.sleep(poll)
636
560
            else:
637
 
                # As timeout is always 0 for remote locks
638
 
                # this block is applicable only for local
639
 
                # lock contention
640
561
                self._trace("timeout after waiting %ss", timeout)
641
 
                raise LockContention('(local)', lock_url)
 
562
                raise LockContention(self)
642
563
 
643
564
    def leave_in_place(self):
644
565
        self._locked_via_token = True
687
608
            raise LockContention(self)
688
609
        self._fake_read_lock = True
689
610
 
 
611
    def _format_lock_info(self, info):
 
612
        """Turn the contents of peek() into something for the user"""
 
613
        lock_url = self.transport.abspath(self.path)
 
614
        delta = time.time() - int(info['start_time'])
 
615
        return [
 
616
            'lock %s' % (lock_url,),
 
617
            'held by %(user)s on host %(hostname)s [process #%(pid)s]' % info,
 
618
            'locked %s' % (format_delta(delta),),
 
619
            ]
 
620
 
690
621
    def validate_token(self, token):
691
622
        if token is not None:
692
623
            info = self.peek()
704
635
        if 'lock' not in debug.debug_flags:
705
636
            return
706
637
        mutter(str(self) + ": " + (format % args))
707
 
 
708
 
    def get_config(self):
709
 
        """Get the configuration that governs this lockdir."""
710
 
        # XXX: This really should also use the locationconfig at least, but
711
 
        # that seems a bit hard to hook up at the moment. -- mbp 20110329
712
 
        return config.GlobalConfig()
713
 
 
714
 
 
715
 
class LockHeldInfo(object):
716
 
    """The information recorded about a held lock.
717
 
 
718
 
    This information is recorded into the lock when it's taken, and it can be
719
 
    read back by any process with access to the lockdir.  It can be used, for
720
 
    example, to tell the user who holds the lock, or to try to detect whether
721
 
    the lock holder is still alive.
722
 
 
723
 
    Prior to bzr 2.4 a simple dict was used instead of an object.
724
 
    """
725
 
 
726
 
    def __init__(self, info_dict):
727
 
        self.info_dict = info_dict
728
 
 
729
 
    def __repr__(self):
730
 
        """Return a debugging representation of this object."""
731
 
        return "%s(%r)" % (self.__class__.__name__, self.info_dict)
732
 
 
733
 
    def __unicode__(self):
734
 
        """Return a user-oriented description of this object."""
735
 
        d = self.to_readable_dict()
736
 
        return (
737
 
            u'held by %(user)s on %(hostname)s (process #%(pid)s), '
738
 
            u'acquired %(time_ago)s' % d)
739
 
 
740
 
    def to_readable_dict(self):
741
 
        """Turn the holder info into a dict of human-readable attributes.
742
 
 
743
 
        For example, the start time is presented relative to the current time,
744
 
        rather than as seconds since the epoch.
745
 
 
746
 
        Returns a list of [user, hostname, pid, time_ago] all as readable
747
 
        strings.
748
 
        """
749
 
        start_time = self.info_dict.get('start_time')
750
 
        if start_time is None:
751
 
            time_ago = '(unknown)'
752
 
        else:
753
 
            time_ago = format_delta(
754
 
                time.time() - int(self.info_dict['start_time']))
755
 
        user = self.info_dict.get('user', '<unknown>')
756
 
        hostname = self.info_dict.get('hostname', '<unknown>')
757
 
        pid = self.info_dict.get('pid', '<unknown>')
758
 
        return dict(
759
 
            user=user,
760
 
            hostname=hostname,
761
 
            pid=pid,
762
 
            time_ago=time_ago)
763
 
 
764
 
    def get(self, field_name):
765
 
        """Return the contents of a field from the lock info, or None."""
766
 
        return self.info_dict.get(field_name)
767
 
 
768
 
    @classmethod
769
 
    def for_this_process(cls, extra_holder_info):
770
 
        """Return a new LockHeldInfo for a lock taken by this process.
771
 
        """
772
 
        info = dict(
773
 
            hostname=get_host_name(),
774
 
            pid=str(os.getpid()),
775
 
            nonce=rand_chars(20),
776
 
            start_time=str(int(time.time())),
777
 
            user=get_username_for_lock_info(),
778
 
            )
779
 
        if extra_holder_info is not None:
780
 
            info.update(extra_holder_info)
781
 
        return cls(info)
782
 
 
783
 
    def to_bytes(self):
784
 
        s = rio.Stanza(**self.info_dict)
785
 
        return s.to_string()
786
 
 
787
 
    @classmethod
788
 
    def from_info_file_bytes(cls, info_file_bytes):
789
 
        """Construct from the contents of the held file."""
790
 
        lines = osutils.split_lines(info_file_bytes)
791
 
        try:
792
 
            stanza = rio.read_stanza(lines)
793
 
        except ValueError, e:
794
 
            mutter('Corrupt lock info file: %r', lines)
795
 
            raise LockCorrupt("could not parse lock info file: " + str(e),
796
 
                lines)
797
 
        if stanza is None:
798
 
            # see bug 185013; we fairly often end up with the info file being
799
 
            # empty after an interruption; we could log a message here but
800
 
            # there may not be much we can say
801
 
            return cls({})
802
 
        else:
803
 
            return cls(stanza.as_dict())
804
 
 
805
 
    def __cmp__(self, other):
806
 
        """Value comparison of lock holders."""
807
 
        return (
808
 
            cmp(type(self), type(other))
809
 
            or cmp(self.info_dict, other.info_dict))
810
 
 
811
 
    def is_locked_by_this_process(self):
812
 
        """True if this process seems to be the current lock holder."""
813
 
        return (
814
 
            self.get('hostname') == get_host_name()
815
 
            and self.get('pid') == str(os.getpid())
816
 
            and self.get('user') == get_username_for_lock_info())
817
 
 
818
 
    def is_lock_holder_known_dead(self):
819
 
        """True if the lock holder process is known to be dead.
820
 
 
821
 
        False if it's either known to be still alive, or if we just can't tell.
822
 
 
823
 
        We can be fairly sure the lock holder is dead if it declared the same
824
 
        hostname and there is no process with the given pid alive.  If people
825
 
        have multiple machines with the same hostname this may cause trouble.
826
 
 
827
 
        This doesn't check whether the lock holder is in fact the same process
828
 
        calling this method.  (In that case it will return true.)
829
 
        """
830
 
        if self.get('hostname') != get_host_name():
831
 
            return False
832
 
        if self.get('hostname') == 'localhost':
833
 
            # Too ambiguous.
834
 
            return False
835
 
        if self.get('user') != get_username_for_lock_info():
836
 
            # Could well be another local process by a different user, but
837
 
            # just to be safe we won't conclude about this either.
838
 
            return False
839
 
        pid_str = self.info_dict.get('pid', None)
840
 
        if not pid_str:
841
 
            mutter("no pid recorded in %r" % (self, ))
842
 
            return False
843
 
        try:
844
 
            pid = int(pid_str)
845
 
        except ValueError:
846
 
            mutter("can't parse pid %r from %r"
847
 
                % (pid_str, self))
848
 
            return False
849
 
        return osutils.is_local_pid_dead(pid)
850
 
 
851
 
 
852
 
def get_username_for_lock_info():
853
 
    """Get a username suitable for putting into a lock.
854
 
 
855
 
    It's ok if what's written here is not a proper email address as long
856
 
    as it gives some clue who the user is.
857
 
    """
858
 
    try:
859
 
        return config.GlobalConfig().username()
860
 
    except errors.NoWhoami:
861
 
        return osutils.getuser_unicode()