On 1. 3. 2013 at 09:35:32, James Antill wrote:
> On Fri, 2013-03-01 at 09:33 +0100, Jan Zeleny wrote:
> > ---
> >
> >  cli.py          | 7 ++++++-
> >  yum/__init__.py | 8 ++++++--
> >  2 files changed, 12 insertions(+), 3 deletions(-)
> >
> > diff --git a/cli.py b/cli.py
> > index 63a9236..12817fc 100755
> > --- a/cli.py
> > +++ b/cli.py
> >
> > @@ -615,6 +615,7 @@ class YumBaseCli(yum.YumBase, output.YumOutput):
> >          self.verbose_logger.log(yum.logginglevels.INFO_2,
> >
> >               _('Running transaction check'))
> >
> >          msgs = self._run_rpm_check()
> >
> > +        depsolve = True
> >
> >          if msgs:
> >              rpmlib_only = True
> >
> >              for msg in msgs:
> > @@ -625,13 +626,17 @@ class YumBaseCli(yum.YumBase, output.YumOutput):
> >                  print _("ERROR You need to update rpm to handle:")
> >
> >              else:
> >                  print _('ERROR with transaction check vs depsolve:')
> >
> > +                depsolve = False
> >
> >              for msg in msgs:
> >                  print to_utf8(msg)
> >
> >              if rpmlib_only:
> >                  return 1, [_('RPM needs to be updated')]
> >
> > -            return 1, [_('Please report this error in %s') %
> > self.conf.bugtracker_url] +            if depsolve:
> > +                return 1, []
> > +            else:
> > +                return 1, [_('Please report this error in %s') %
> > self.conf.bugtracker_url]
>  This is fine.
>
> > diff --git a/yum/__init__.py b/yum/__init__.py
> > index fd54208..6641454 100644
> > --- a/yum/__init__.py
> > +++ b/yum/__init__.py
> > @@ -6286,8 +6286,12 @@ much more problems).
> >
> >                  raise Errors.YumRPMCheckError, retmsgs
> >
> >              retmsgs = [_('ERROR with transaction check vs depsolve:')]
> >              retmsgs.extend(msgs)
> >
> > -            retmsgs.append(_('Please report this error at %s')
> > -                                         % self.conf.bugtracker_url)
> > +            retmsgs.append(_('This usually indicates pre-existing
> > dependency'\ +                             'issues in rpmdb. Please run
> > rpm -Va --nofiles to '\ +                             'see possible
> > issues.'))
>
>  This is being output just before we'd automatically run "yum check",
> which is going to output most of the real problems the rpm -Va will show
> (and if it outputs nothing tells the user to try running
> "rpm -Va --nofiles --nodigest").

I'm curious, could you please guide me through the code path how this piece of
code is reached? I looked at it three times before sending the patch, once
more now and I still can't find it.

Thanks
Jan

Attachment: signature.asc
Description: This is a digitally signed message part.

_______________________________________________
Yum-devel mailing list
[email protected]
http://lists.baseurl.org/mailman/listinfo/yum-devel

Reply via email to