booty - bootloaderInfo.py changes

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Here's my first patch.

This one basically gets rid of "str" as a variable name.  Also, I took
the opportunity to make the salt construction more concise, and pointed
out a bug that I don't know how to fix.

Shandy
97c97,98
<         str = ""
---
>         newArgs = []
>         cfgFilename = "/tmp/install.cfg"
101,103c102,103
<             f = open("/tmp/install.cfg")
<             lines = f.readlines()
<             for line in lines:
---
>             f = open(cfgFilename)
>             for line in f:
110c110
<                         str = str + "dasd=" + vparm
---
>                         newArgs.append( "dasd=" + vparm )
116c116,117
<                     pass
---
>                     log("exception parsing %s: %s" % (cfgFilename, e))
>             f.close()
119c120,121
<         ourargs = ["speakup_synth=", "apic", "noapic", "apm=", "ide=nodma", "noht", "acpi=", "video=", "pci=", "nodmraid", "nompath"]
---
>         ourargs = ["speakup_synth=", "apic", "noapic", "apm=", "ide=nodma", 
>                    "noht", "acpi=", "video=", "pci=", "nodmraid", "nompath"]
127,128c129,131
<                     if str: str = str + " "
<                     str = str + arg
---
>                     newArgs.append( arg )
> 
>         self.args = " ".join( newArgs )
130d132
<         self.args = str
588c590
<             str = self.writeLilo(instRoot, fsset, bl, langs, kernelList, 
---
>             out = self.writeLilo(instRoot, fsset, bl, langs, kernelList, 
647a650
>             saltPot = string.letters + string.digits +'./'
649,650c652
<                 salt = salt + random.choice (string.letters +
<                                              string.digits + './')
---
>                 salt += random.choice( saltPot )
1197c1199
<         str = self.writeGrub(instRoot, fsset, bl, langs, kernelList, 
---
>         out = self.writeGrub(instRoot, fsset, bl, langs, kernelList, 
1393c1395
<         str = self.writeZipl(instRoot, fsset, bl, langs, kernelList, 
---
>         out = self.writeZipl(instRoot, fsset, bl, langs, kernelList, 
1396c1398
<         str = self.writeChandevConf(bl, instRoot)
---
>         out = self.writeChandevConf(bl, instRoot)
1739c1741
<             str = self.writeYaboot(instRoot, fsset, bl, langs, kernelList, 
---
>             out = self.writeYaboot(instRoot, fsset, bl, langs, kernelList, 
1991a1994,1995
>         # TODO!!!  FIXME!!!  XXX!!!
>         # butil is not defined!!!  - assume this is in rhpl now?
d97 1
a97 2
        newArgs = []
        cfgFilename = "/tmp/install.cfg"
d101 3
a103 2
            f = open(cfgFilename)
            for line in f:
d110 1
a110 1
                        newArgs.append( "dasd=" + vparm )
d116 1
a116 2
                    log("exception parsing %s: %s" % (cfgFilename, e))
            f.close()
d119 1
a119 2
        ourargs = ["speakup_synth=", "apic", "noapic", "apm=", "ide=nodma", 
                   "noht", "acpi=", "video=", "pci=", "nodmraid", "nompath"]
d127 2
a128 3
                    newArgs.append( arg )

        self.args = " ".join( newArgs )
d130 1
d588 1
a588 1
            out = self.writeLilo(instRoot, fsset, bl, langs, kernelList, 
a647 1
            saltPot = string.letters + string.digits +'./'
d649 2
a650 1
                salt += random.choice( saltPot )
d1197 1
a1197 1
        out = self.writeGrub(instRoot, fsset, bl, langs, kernelList, 
d1393 1
a1393 1
        out = self.writeZipl(instRoot, fsset, bl, langs, kernelList, 
d1396 1
a1396 1
        out = self.writeChandevConf(bl, instRoot)
d1739 1
a1739 1
            out = self.writeYaboot(instRoot, fsset, bl, langs, kernelList, 
a1991 2
        # TODO!!!  FIXME!!!  XXX!!!
        # butil is not defined!!!  - assume this is in rhpl now?

[Index of Archives]     [Kickstart]     [Fedora Users]     [Fedora Legacy List]     [Fedora Maintainers]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [Yosemite Photos]     [KDE Users]     [Fedora Tools]
  Powered by Linux