[Yum-devel] [PATCH] Fix for --enablerepo=broken problem, exit(1) on failures and give reason.
Seth Vidal
skvidal at fedoraproject.org
Thu Jan 31 19:14:54 UTC 2013
On Thu, 31 Jan 2013, James Antill wrote:
> Also make queryError() an exit 1 condition.
ACK - both of these look right.
Though I suspect I am going to have to punt and use the yum-api directly
for the stuff I was/am using repoquery for :-/
thanks
> ---
> repoquery.py | 11 +++++++++++
> 1 files changed, 11 insertions(+), 0 deletions(-)
>
> diff --git a/repoquery.py b/repoquery.py
> index a854c23..45c6826 100755
> --- a/repoquery.py
> +++ b/repoquery.py
> @@ -425,6 +425,8 @@ class pkgQuery:
> providers = matches.keys()
>
>
> + except yum.Errors.RepoError:
> + raise
> except yum.Errors.YumBaseError, err:
> print >>sys.stderr, "No package provides %s" % req
> return []
> @@ -595,6 +597,8 @@ class pkgQuery:
> arequirers = [pkg for pkg in areqs
> if pkg.pkgtup not in skip]
>
> + except yum.Errors.RepoError:
> + raise
> except yum.Errors.YumBaseError, err:
> print >>sys.stderr, "No package provides %s" % str(prov)
> return []
> @@ -879,6 +883,8 @@ class YumBaseQuery(yum.YumBase):
> matches = yum.YumBase.searchPackageProvides(self, [str(depstring)])
> provider = matches.keys()
> # provider.extend(yum.YumBase.returnPackagesByDep(self, depstring))
> + except yum.Errors.RepoError:
> + raise
> except yum.Errors.YumBaseError, err:
> self.logger.error("No package provides %s" % depstring)
> return self.queryPkgFactory(provider)
> @@ -1501,10 +1507,15 @@ def main(args):
> except (yum.Errors.RepoError, yum.Errors.GroupsError), e:
> repoq.logger.error(e)
> sys.exit(1)
> +
> try:
> repoq.runQuery(regexs)
> + except yum.Errors.RepoError, e:
> + repoq.logger.error(e)
> + sys.exit(1)
> except queryError, e:
> repoq.logger.error(e)
> + sys.exit(1)
>
> if __name__ == "__main__":
> misc.setup_locale()
> --
> 1.7.6.5
>
> _______________________________________________
> Yum-devel mailing list
> Yum-devel at lists.baseurl.org
> http://lists.baseurl.org/mailman/listinfo/yum-devel
>
More information about the Yum-devel
mailing list