Thinking a little more on the issue 0001 is not really needed as 0002
makes the "if mediaDevice" True.
Regards.
On Mon, Aug 10, 2009 at 06:18:24PM +0200, Joel Granados wrote:
> Hey list:
>
> Wouldn't the attached patch be better? We have the added value of doing
> all the stuff we do for the media when we have it. Additionally, since
> we are checking that self.ayum.tree exist and is mounted, I dont think
> it should be in an elif.
>
> Regards.
> On Mon, Aug 10, 2009 at 11:45:10AM -0400, Chris Lumens wrote:
> > ---
> > yuminstall.py | 2 +-
> > 1 files changed, 1 insertions(+), 1 deletions(-)
> >
> > diff --git a/yuminstall.py b/yuminstall.py
> > index 84445b0..a56fd71 100644
.
.
.
> yuminstall.py | 3 ++-
> 1 files changed, 2 insertions(+), 1 deletions(-)
>
> diff --git a/yuminstall.py b/yuminstall.py
> index 84445b0..3dbc92e 100644
> --- a/yuminstall.py
> +++ b/yuminstall.py
> @@ -1036,7 +1036,8 @@ reposdir=/etc/anaconda.repos.d,/tmp/updates/anaconda.repos.d,/tmp/product/anacon
> "%s/var/cache/yum/%s" %(anaconda.rootPath, repo))
> except Exception, e:
> log.debug("Error setting up media repository: %s" %(e,))
> - else:
> +
> + if os.path.exists(self.ayum.tree) and os.path.ismount(self.ayum.tree):
> isys.umount(self.ayum.tree)
>
> anaconda.backend.removeInstallImage()
> --
> 1.6.0.6
>
> _______________________________________________
> Anaconda-devel-list mailing list
> Anaconda-devel-list@xxxxxxxxxx
> https://www.redhat.com/mailman/listinfo/anaconda-devel-list
--
Joel Andres Granados
Brno, Czech Republic, Red Hat.
Attachment:
pgpuVfXcnTuwC.pgp
Description: PGP signature
_______________________________________________ Anaconda-devel-list mailing list Anaconda-devel-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/anaconda-devel-list
