[Yum-devel] [PATCH] interrupt_callback=None: reraise instead of retrying

[James Antill]

On Wed, 2012-06-13 at 12:25 +0200, Zdeněk Pavlas wrote:
> When yum fetches metalink.xml with _getMetalink(), we use
> interrupt_callback=None and retry=10.  Ctrl-C then starts
> a new attempt, and user has to abort it 10 times.
> 
> An alternative is to fix this in _getMetalink() with a custom
> interrupt callback (we should not reuse the default callback
> that skips to the next mirror, as there are no mirrors),
> but this is simpler and fixes all possible callers.

 But then we have "KeyboardInterrupt" escaping from urlgrabber, no? What
does that do?
 In the case of getting metalink, doesn't urlgrabber act like there is
one mirror ... so the default callback will dtrt too?