Re: [PATCH] Catch IOError when enabling repos (#500439).

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Looks fine -- although what's the exception as we should really get it
caught down in the yum level instead...

Jeremy

On Wednesday, May 13 2009, Chris Lumens said:

> ---
>  iw/task_gui.py |    4 ++--
>  1 files changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/iw/task_gui.py b/iw/task_gui.py
> index 5f688f5..a91209a 100644
> --- a/iw/task_gui.py
> +++ b/iw/task_gui.py
> @@ -59,12 +59,12 @@ def setupRepo(anaconda, repo):
>              else:
>                  repo.groups_added = True
>                  log.info("added group information for repository %s" %(repo.name))
> -    except yum.Errors.RepoError, e:
> +    except (IOError, yum.Errors.RepoError) as e:
>          anaconda.intf.messageWindow(_("Error"),
>                _("Unable to read package metadata from repository.  "
>                  "This may be due to a missing repodata directory.  "
>                  "Please ensure that your repository has been "
> -                "correctly generated.\n\n%s" %(e,)),
> +                "correctly generated.\n\n%s" % str(e)),
>                                  type="ok", custom_icon="error")
>          anaconda.backend.ayum.repos.delete(repo.id)
>          return False
> -- 
> 1.6.1.3
> 
> _______________________________________________
> Anaconda-devel-list mailing list
> Anaconda-devel-list@xxxxxxxxxx
> https://www.redhat.com/mailman/listinfo/anaconda-devel-list

_______________________________________________
Anaconda-devel-list mailing list
Anaconda-devel-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/anaconda-devel-list

[Index of Archives]     [Kickstart]     [Fedora Users]     [Fedora Legacy List]     [Fedora Maintainers]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [Yosemite Photos]     [KDE Users]     [Fedora Tools]
  Powered by Linux