[PATCH 2/2] Revert "Hook up the "Remove Packages" button on the dep solving error screen." (#905899).

Brian C. Lane bcl at redhat.com
Thu Feb 28 01:13:52 UTC 2013


On Wed, Feb 27, 2013 at 10:15:56AM -0500, Chris Lumens wrote:
> This reverts commit f4ec3d682ffd93dfc7105eaa09acdd7fd672a3e8.
> 
> This patch also removes the button entirely.  All I've ever seen it do is reduce
> the installable package set to something like 30 packages and result in really
> cryptic bugs about commands that should be there failing to run.  It doesn't
> seem to help in just removing the one or two packages that have a problem,
> since it looks like it's frequently a core package with the problem.
> ---
>  pyanaconda/packaging/__init__.py     |  4 ----
>  pyanaconda/packaging/yumpayload.py   |  5 -----
>  pyanaconda/ui/gui/spokes/software.py | 31 ++++++++++++-------------------
>  3 files changed, 12 insertions(+), 28 deletions(-)

> +        from pyanaconda.threads import threadMgr

This looks fine except for all these imports. We already import
threadMgr at the top (and I'm no fan of imports inside methods)

-- 
Brian C. Lane | Anaconda Team | IRC: bcl #anaconda | Port Orchard, WA (PST8PDT)
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 482 bytes
Desc: not available
URL: <https://lists.fedorahosted.org/pipermail/anaconda-patches/attachments/20130227/0d42ee5d/attachment.sig>


More information about the anaconda-patches mailing list