I took care of all tabs in those patches. So what here is against PEP8? Well I see one place where the indentation is 2 not 4 spaces, but thats all..
Martin
----- "David Cantrell" <dcantrell@xxxxxxxxxx> wrote:
> -----BEGIN PGP SIGNED MESSAGE-----
> Hash: SHA1
>
> General comment for Python coding style (even more important than
> C)... let's
> all follow PEP 8 -or- at least adhere to the existing coding style in
> the file
> we are modifying.
>
> On Wed, 2 Dec 2009, Martin Sivak wrote:
>
> > - stage2 support for DD repos
> > - get all modules which were loaded from DD
> > - get the RPMs using filedependencies
> > - install everything into the target system
>
> Stronger comment explaining what's happening.
>
> > ---
> > anaconda | 4 ++++
> > backend.py | 17 ++++++++++++-----
> > constants.py | 5 +++++
> > flags.py | 1 +
> > yuminstall.py | 39 +++++++++++++++++++++++++++++----------
> > 5 files changed, 51 insertions(+), 15 deletions(-)
> >
> > diff --git a/anaconda b/anaconda
> > index 55eaf8d..3e614b9 100755
> > --- a/anaconda
> > +++ b/anaconda
> > @@ -240,6 +240,7 @@ def parseOptions():
> > op.add_option("--nomount", dest="rescue_nomount",
> action="store_true", default=False)
> > op.add_option("--updates", dest="updateSrc", action="store",
> type="string")
> > op.add_option("--dogtail", dest="dogtail", action="store",
> type="string")
> > + op.add_option("--dlabel", action="store_true", default=False)
> >
> > return op.parse_args()
> >
> > @@ -666,6 +667,9 @@ if __name__ == "__main__":
> > # Default is to prompt to mount the installed system.
> > anaconda.rescue_mount = not opts.rescue_nomount
> >
> > + if opts.dlabel: #autodetected driverdisc in use
> > + flags.dlabel = True
> > +
> > if opts.noipv4:
> > flags.useIPv4 = False
> >
> > diff --git a/backend.py b/backend.py
> > index fcd3571..4e425cf 100644
> > --- a/backend.py
> > +++ b/backend.py
> > @@ -73,8 +73,8 @@ class AnacondaBackend:
> >
> > def copyFirmware(self, anaconda):
> > # Multiple driver disks may be loaded, so we need to glob
> for all
> > - # the firmware files in all the driver disk directories.
> > - for f in glob.glob("/tmp/DD-*/firmware/*"):
> > + # the firmware files in the common DD firmware directory
> > + for f in glob.glob(DD_EXTRACTED+"/lib/firmware/*"):
> > try:
> > shutil.copyfile(f, "%s/lib/firmware/" %
> anaconda.rootPath)
> > except IOError, e:
> > @@ -94,16 +94,23 @@ class AnacondaBackend:
> > has_iscsi_disk = True
> > break
> >
> > - if anaconda.id.extraModules:
> > - self.copyFirmware(anaconda)
> > + #always copy the firmware files from DD
> > + self.copyFirmware(anaconda)
> >
> > if anaconda.id.extraModules or has_iscsi_disk:
> > for (n, arch, tag) in
> self.kernelVersionList(anaconda.rootPath):
> > packages.recreateInitrd(n, anaconda.rootPath)
> >
> > - for d in glob.glob("/tmp/DD-*"):
> > + #copy RPMS
> > + for d in glob.glob(DD_RPMS):
> > shutil.copytree(d, "/root/" + os.path.basename(d))
> >
> > + #copy modules and firmware
> > + try:
> > + shutil.copytree(DD_EXTRACTED, "/root/DD")
> > + except IOError, e:
> > + pass
> > +
> > storage.writeEscrowPackets(anaconda)
> >
> > sys.stdout.flush()
> > diff --git a/constants.py b/constants.py
> > index 9c4fbb1..fb81c90 100644
> > --- a/constants.py
> > +++ b/constants.py
> > @@ -82,3 +82,8 @@ else:
> > # this string will be combined with "An unhandled exception"...
> > # the leading space is not a typo.
> > exceptionText += _(" against anaconda at %s") %(bugzillaUrl,)
> > +
> > +# DriverDisc Paths
> > +DD_EXTRACTED = "/tmp/DD"
> > +DD_RPMS = "/tmp/DD-*"
> > +
> > diff --git a/flags.py b/flags.py
> > index 93472e3..e0d7329 100644
> > --- a/flags.py
> > +++ b/flags.py
> > @@ -70,6 +70,7 @@ class Flags:
> > self.__dict__['flags']['test'] = 0
> > self.__dict__['flags']['rootpath'] = 0
> > self.__dict__['flags']['livecdInstall'] = 0
> > + self.__dict__['flags']['dlabel'] = 0
> > self.__dict__['flags']['ibft'] = 1
> > self.__dict__['flags']['iscsi'] = 0
> > self.__dict__['flags']['serial'] = 0
> > diff --git a/yuminstall.py b/yuminstall.py
> > index accf452..4eac19c 100644
> > --- a/yuminstall.py
> > +++ b/yuminstall.py
> > @@ -649,16 +649,18 @@ class AnacondaYum(YumSorter):
> >
> > extraRepos = []
> >
> > - if self.anaconda.id.extraModules:
> > - for d in glob.glob("/tmp/DD-*/rpms"):
> > - dirname = os.path.basename(os.path.dirname(d))
> > - rid = "anaconda-%s" % dirname
> > -
> > - repo = AnacondaYumRepo(rid)
> > - repo.baseurl = [ "file:///%s" % d ]
> > - repo.name = "Driver Disk %s" %
> dirname.split("-")[1]
> > - repo.enable()
> > - extraRepos.append(repo)
> > + (_, _, _, _, ddArch) = os.uname()
>
> 'ddArch = os.uname()[4]' instead?
>
> > +
> > + #Add the Driver disc repos to Yum
> > + for d in glob.glob(DD_RPMS):
> > + dirname = os.path.basename(d)
> > + rid = "anaconda-%s" % dirname
> > +
> > + repo = AnacondaYumRepo(rid)
> > + repo.baseurl = [ "file:///%s" % d ]
> > + repo.name = "Driver Disk %s" % dirname.split("-")[1]
> > + repo.enable()
> > + extraRepos.append(repo)
> >
> > if self.anaconda.isKickstart:
> > # This is the same pattern as from
> loader/urls.c:splitProxyParam.
> > @@ -1292,6 +1294,23 @@
> reposdir=/etc/anaconda.repos.d,/tmp/updates/anaconda.repos.d,/tmp/product/anacon
> > log.info("selecting package %s for module %s" %
> (pkg.name, name))
> > self.ayum.install(po=pkg)
> >
> > + #We need to install the packages which contain modules from
> DriverDiscs
> > + for modPath in isys.modulesWithPaths():
> > + if modPath.startswith(DD_EXTRACTED):
> > + moduleProvides = modPath[len(DD_EXTRACTED):]
> > + else:
> > + continue
> > +
> > + pkgs = self.ayum.returnPackagesByDep(moduleProvides)
> > +
> > + if not pkgs:
> > + log.warning("Didn't find any package providing %s"
> % (modPath,))
> > +
> > + for pkg in pkgs:
> > + log.info("selecting package %s for %s" % (pkg.name,
> modPath))
> > + self.ayum.install(po=pkg)
> > +
> > +
> > def selectBestKernel(self, anaconda):
> > """Find the best kernel package which is available and
> select it."""
> >
> >
>
> - --
> David Cantrell <dcantrell@xxxxxxxxxx>
> Red Hat / Honolulu, HI
>
> -----BEGIN PGP SIGNATURE-----
> Version: GnuPG v1.4.10 (GNU/Linux)
>
> iEYEARECAAYFAksdahkACgkQ5hsjjIy1Vkl3CwCfTfV/xH7iP3rKwqJAqa8nt1IV
> +XwAn3N936OwX8IxMAX4P+ZdGEEdFGSQ
> =t7t0
> -----END PGP SIGNATURE-----
>
> _______________________________________________
> Anaconda-devel-list mailing list
> Anaconda-devel-list@xxxxxxxxxx
> https://www.redhat.com/mailman/listinfo/anaconda-devel-list
_______________________________________________
Anaconda-devel-list mailing list
Anaconda-devel-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/anaconda-devel-list
