[master][PATCH] Prevent tb during software selection. (#959710)

[Samantha N. Bueno]

Side note: though there are several people reporting this issue
in the bug, I've thus far been unable to reproduce it--and therefore,
unable to test out the fix (though it's so small/simple that I am pretty
certain this should fix it).

====

If environmentid doesn't exist, don't attempt to operate on it,
otherwise a tb occurs.
---
 pyanaconda/packaging/yumpayload.py | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/pyanaconda/packaging/yumpayload.py b/pyanaconda/packaging/yumpayload.py
index 2ffc5af..e7a3cdf 100644
--- a/pyanaconda/packaging/yumpayload.py
+++ b/pyanaconda/packaging/yumpayload.py
@@ -1116,6 +1116,9 @@ reposdir=%s
         if not groups:
             return []

+        if not environmentid:
+            return []
+
         with _yum_lock:
             if not groups.has_environment(environmentid):
                 raise NoSuchGroup(environmentid)

[Samantha N. Bueno]

Disregard this. Workarounds were just added to yum, so this is no longer
necessary.

If environmendid is detected as 'None' (in yum), it now returns an empty
array, which shouldn't break anaconda anymore.

On Mon, Jun 03, 2013 at 03:10:05PM -0400, Samantha N. Bueno wrote:
> Side note: though there are several people reporting this issue
> in the bug, I've thus far been unable to reproduce it--and therefore,
> unable to test out the fix (though it's so small/simple that I am pretty
> certain this should fix it).
>
> ====
>
> If environmentid doesn't exist, don't attempt to operate on it,
> otherwise a tb occurs.
>
> pyanaconda/packaging/yumpayload.py | 3 +++
>  1 file changed, 3 insertions(+)
>
> diff --git a/pyanaconda/packaging/yumpayload.py b/pyanaconda/packaging/yumpayload.py
> index 2ffc5af..e7a3cdf 100644
> --- a/pyanaconda/packaging/yumpayload.py
> +++ b/pyanaconda/packaging/yumpayload.py
> @@ -1116,6 +1116,9 @@ reposdir=%s
>          if not groups:
>              return []
>    if not environmentid:
>        return []
>    with _yum_lock:
>        if not groups.has_environment(environmentid):
>            raise NoSuchGroup(environmentid)
>
> -- 
> 1.7.11.7
>
> anaconda-patches mailing list
> [e-mail]
> https://lists.fedorahosted.org/mailman/listinfo/anaconda-patches