commit e-mails were dying with this bug:
http://www.selenic.com/mercurial/bts/issue865

Should we switch to a more current hg?

On Sat, Jun 14, 2008 at 1:55 PM, Nathan Binkert <[EMAIL PROTECTED]> wrote:
> changeset 145348cb7c42 in /z/repo/hooks
> details: http://repo.m5sim.org/hooks?cmd=changeset;node=145348cb7c42
> summary: don't raise an exception if diffstat fails
>
> diffstat:
>
> 1 file changed, 1 insertion(+), 5 deletions(-)
> notify.py |    6 +-----
>
> diffs (23 lines):
>
> diff -r e5ea96b060ce -r 145348cb7c42 notify.py
> --- a/notify.py Sat Jun 14 16:21:26 2008 -0400
> +++ b/notify.py Sat Jun 14 16:55:21 2008 -0400
> @@ -242,11 +242,14 @@ class notifier(object):
>         self.ui.pushbuffer()
>         patch.diff(self.repo, prev, ref)
>         difflines = self.ui.popbuffer().splitlines(1)
> -        if self.ui.configbool('notify', 'diffstat', True):
> -            s = patch.diffstat(difflines)
> -            # s may be nil, don't include the header if it is
> -            if s:
> -                self.ui.write('\ndiffstat:\n\n%s' % s)
> +        try:
> +            if self.ui.configbool('notify', 'diffstat', True):
> +                s = patch.diffstat(difflines)
> +                # s may be nil, don't include the header if it is
> +                if s:
> +                    self.ui.write('\ndiffstat:\n\n%s' % s)
> +        except ValueError:
> +            pass
>         if maxdiff > 0 and len(difflines) > maxdiff:
>             self.ui.write(_('\ndiffs (truncated from %d to %d lines):\n\n') %
>                           (len(difflines), maxdiff))
> _______________________________________________
> m5-dev mailing list
> m5-dev@m5sim.org
> http://m5sim.org/mailman/listinfo/m5-dev
>
>
_______________________________________________
m5-dev mailing list
m5-dev@m5sim.org
http://m5sim.org/mailman/listinfo/m5-dev

Reply via email to