D960: bundle2: immediate exit for ctrl+c (issue5692)

yuja (Yuya Nishihara) phabricator at mercurial-scm.org
Fri Oct 6 11:05:56 EDT 2017


yuja added inline comments.

INLINE COMMENTS

> quark wrote in bundle2.py:380
> Actually `KeyboardInterrupt` is tested here. So there is something else going on. Maybe we should add `error.SignalInterrupt` to the list?

SignalInterrupt is a subclass of KeyboardInterrupt, so there might
be deeper problem.

FWIW, I think it's probably a good idea to replace this blacklist
with `isinstance(exc, Exception)`.

REPOSITORY
  rHG Mercurial

REVISION DETAIL
  https://phab.mercurial-scm.org/D960

To: durham, #hg-reviewers, quark
Cc: yuja, mercurial-devel


More information about the Mercurial-devel mailing list