[newui] repo.mirrorlist may be None

[Brian C. Lane]

From: "Brian C. Lane"

---
 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]):

[Jesse Keating]

On 07/27/2012 01:59 PM, Brian C. Lane wrote:
> From: "Brian C. Lane"
>
> 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