Commit does not allow missing email address but whoami says this is allowed

Martin Pool mbp at canonical.com
Thu Nov 2 08:23:34 GMT 2006


On  1 Nov 2006, John Arbash Meinel <john at arbash-meinel.com> wrote:
> I think the whole thing should be updated to something more like:
> 
> unique_chunk = osutils.rand_chars(16)
> try:
>   user = self._config.user_email()
> except errors.NoEmailInUsername:
>   user = sanitize(self._config.username())
> s = '-'.join(unique_chunk, compact_date(self._timestamp), user)

+1

We could even save some space by just using the username part of the
address in this context; that's probably generally enough to give some
clue to what's in it.

-- 
Martin




More information about the bazaar mailing list