On Mon, 2011-11-07 at 12:02 +0100, Zdeněk Pavlas wrote:
> Changes are local to the last 'po' loop. Error path is elliminated
> with the 'errors' argument to getPackage(). OK path is merged to
> checkfunc.
> ---
> yum/__init__.py | 58 ++++++++++++++++++++++++------------------------------
> 1 files changed, 26 insertions(+), 32 deletions(-)
>
> diff --git a/yum/__init__.py b/yum/__init__.py
> index b07bbb2..e29b219 100644
> --- a/yum/__init__.py
> +++ b/yum/__init__.py
> @@ -65,6 +65,7 @@ from parser import ConfigPreProcessor, varReplace
> import transactioninfo
> import urlgrabber
> from urlgrabber.grabber import URLGrabber, URLGrabError
> +from urlgrabber.grabber import parallel_wait
It's going to be useful for yum to work with the old and new
urlgrabber ... even if we then add fake requires into the spec.
One of the big things short term. is if you can get these two patches
to work with current urlgrabber, then you can just push them upstream
and I'll create a new rawhide drop ... will make testing _much_ easier
for everyone (and will help any MultiFileMeter convertion).
> + po.localpath = po.repo.getPackage(
> + po,
> + checkfunc=(checkfunc, (po,), {}),
> + text=os.path.basename(po.relativepath),
> + cache=po.repo.http_caching != 'none',
> + errors=lambda msg, po=po: adderror(po, msg),
> + )
As a style thing I don't really like this, looks too confusing ... just
creating some extra variables is pretty free.
_______________________________________________
Yum-devel mailing list
[email protected]
http://lists.baseurl.org/mailman/listinfo/yum-devel
