Re: [Yum-devel] [PATCH] No implicit use of TextMultiFileMeter

2012-08-20 Thread James Antill
On Mon, 2012-08-20 at 16:46 +0200, Zdeněk Pavlas wrote: > --- > urlgrabber/grabber.py | 82 > - > 1 files changed, 54 insertions(+), 28 deletions(-) > > diff --git a/urlgrabber/grabber.py b/urlgrabber/grabber.py > index b3071ce..b16b9a9 100644 >

Re: [Yum-devel] [PATCH] Use TextMultiFileMeter as multi_obj

2012-08-20 Thread James Antill
On Mon, 2012-08-20 at 16:46 +0200, Zdeněk Pavlas wrote: > Extend the progress callback API, so we can set multi_objs in urlgrabber. > Subclass the TextMultiFileMeter and use it as multi-file progress object. This seems fine, although when I tried putting multifile logic just inside yum all I seem

[Yum-devel] [PATCH] Use TextMultiFileMeter as multi_obj

2012-08-20 Thread Zdeněk Pavlas
Extend the progress callback API, so we can set multi_objs in urlgrabber. Subclass the TextMultiFileMeter and use it as multi-file progress object. --- output.py | 12 ++-- yum/__init__.py |3 ++- yum/repos.py|4 ++-- yum/yumRepo.py |7 +-- 4 files changed, 19

[Yum-devel] [PATCH] No implicit use of TextMultiFileMeter

2012-08-20 Thread Zdeněk Pavlas
- Dropped the 'meter' argument of parallel_wait(), added the multi_obj option instead, defaulting to None. - progress_obj and not multi_obj case: progress_obj is used in compatibility mode, legacy users should not break. - dropped po.failure() call. #1 progress_obj does not implement this,