[newui] repo.mirrorlist may be None

Jesse Keating jkeating at redhat.com
Fri Jul 27 21:12:37 UTC 2012


On 07/27/2012 01:59 PM, Brian C. Lane wrote:
> From: "Brian C. Lane" <bcl at redhat.com>
>
> ---
>   pyanaconda/packaging/__init__.py |    4 +++-
>   1 files changed, 3 insertions(+), 1 deletions(-)
>
> diff --git a/pyanaconda/packaging/__init__.py b/pyanaconda/packaging/__init__.py
> index ca23f53..02a1a5d 100644
> --- a/pyanaconda/packaging/__init__.py
> +++ b/pyanaconda/packaging/__init__.py
> @@ -169,7 +169,9 @@ class Payload(object):
>
>       def _repoNeedsNetwork(self, repo):
>           """ Returns True if the ksdata repo requires networking. """
> -        urls = [repo.baseurl] + repo.mirrorlist
> +        urls = [repo.baseurl]
> +        if repo.mirrorlist:
> +            urls.extend(repo.mirrorlist)
>           network_protocols = ["http:", "ftp:", "nfs:", "nfsiso:"]
>           for url in urls:
>               if any([url.startswith(p) for p in network_protocols]):
>

ACK

-- 
Jesse Keating
Fedora -- Freedom² is a feature!


More information about the anaconda-patches mailing list