Comment on this script. Possible error in communication with list arg between functions

[Phoe6]

Hi all,
Part of my script is to check for pre-requisite rpms to be
installed.
If its installed, I just display the rpm version as in rpm database,
otherwise I output a message saying:
' rpm is not installed' and collect the rpm name in a list
(notInstalled).
At the end if the len(notInstalle d) is greater than 0 then I display
them all asking it to be installed and exit the program.
My Script is below:
-------
#!/usr/bin/env python
import os
import sys

notInstalled = []

def checkForRpm(rpm name):
''' Check for the presence of the RPM. '''
cin,cout,cerr = os.popen3('rpm -q ' + rpmname)
output = cout.read()
global notInstalled
if len(output) <= 0:
print rpmname + ' not installed.'
notInstalled.ap pend(rpmname)
else:
print output

def preReqCheckRpms ():
''' Check for the required RPMS '''
listOfRpms =
['firefox','sent hil','binutils' ,'gcc','cpp','g libc-devel','glibc-headers','glibc-kernheaders','c ompat-db','compat-gcc','compat-gcc-c++','compat-libstdc++','com pat-libstdc++-devel','gnome-libs','openmoti f21','setarch']

for eachRpm in listOfRpms:
checkForRpm(eac hRpm)
global notInstalled
if len(notInstalle d) 0:
print 'The following RPMS are not installed:'
for eachRpm in notInstalled:
print eachRpm
print 'Please install them for the installation to
continue.'
sys.exit(-1)


def main():
preReqCheckRpms ()

if __name__ == '__main__':
main()
----

* This is NOT Working.
* notInstalled.ap pend(rpmname) is not taking effect in
checkForRpm(rpm name)
* I dont know how to communicate the notInstalled list to
preReqCheckRpms .

--
Senthil

[Juho Schultz]

Phoe6 wrote:

Hi all,
Part of my script is to check for pre-requisite rpms to be
installed.
If its installed, I just display the rpm version as in rpm database,
otherwise I output a message saying:
' rpm is not installed' and collect the rpm name in a list
(notInstalled).
At the end if the len(notInstalle d) is greater than 0 then I display
them all asking it to be installed and exit the program.
My Script is below:
-------
def checkForRpm(rpm name):
''' Check for the presence of the RPM. '''
cin,cout,cerr = os.popen3('rpm -q ' + rpmname)
output = cout.read()
global notInstalled
if len(output) <= 0:
print rpmname + ' not installed.'
notInstalled.ap pend(rpmname)
else:
print output

def preReqCheckRpms ():
''' Check for the required RPMS '''
listOfRpms = ['firefox','sent hil',]
for eachRpm in listOfRpms:
checkForRpm(eac hRpm)
global notInstalled
if len(notInstalle d) 0:
print 'The following RPMS are not installed:'
for eachRpm in notInstalled:
print eachRpm
print 'Please install them for the installation to
continue.'
sys.exit(-1)

* This is NOT Working.
* notInstalled.ap pend(rpmname) is not taking effect in
checkForRpm(rpm name)
* I dont know how to communicate the notInstalled list to
preReqCheckRpms .

--
Senthil

I think return values should be used for communication between
functions. Maybe something like this could work for you (not tested).

def checkForRpm(rpm name):
# <cut start of function>
# Strings with 0 lenght are False
if output:
print output
else:
print rpmname + ' is not installed'
return output

def checkPreReqs():
missingRpms = []
for requiredRpm in listOfRpms:
if not checkForRpm(req uiredRpm):
missingRpms.app end(requiredRpm )
# you could also do this by a list comprehension
missingRpms = [reqRpm for reqRpm in listOfRpms if not
checkForRpm(req Rpm)]
# or you could use the builtin function filter() - see
filter.__doc__ for that

# now, list of lenght 0 is False.
if missingRpms:
# print error messages, exit.

--
Juho Schultz

[Phoe6]

Juho Schultz wrote:

I think return values should be used for communication between
functions. Maybe something like this could work for you (not tested).

def checkForRpm(rpm name):
# <cut start of function>
# Strings with 0 lenght are False
if output:
print output
else:
print rpmname + ' is not installed'
return output

def checkPreReqs():
missingRpms = []
for requiredRpm in listOfRpms:
if not checkForRpm(req uiredRpm):
missingRpms.app end(requiredRpm )
# you could also do this by a list comprehension
missingRpms = [reqRpm for reqRpm in listOfRpms if not
checkForRpm(req Rpm)]
# or you could use the builtin function filter() - see
filter.__doc__ for that

Thanks Juho. I got the logic to workout the problem and I was able to
solve it. There was bit alteration of my code was required. Its working
now. Thanks.
---
def checkForRpm(rpm name):
''' Check for the presence of the RPM. '''
cin,cout,cerr = os.popen3('rpm -q ' + rpmname)
rpmStatus = cout.read()
print rpmStatus,
return rpmStatus

def preReqCheckRpms ():
''' Check for the required RPMS '''
listOfRpms =
['firefox','sent hil','binutils' ,'gcc','cpp','g libc-devel','glibc-headers','glibc-kernheaders','c ompat-db','compat-gcc','compat-gcc-c++','compat-libstdc++','com pat-libstdc++-devel','gnome-libs','openmoti f21','setarch']
missingRpms = []
for requiredRpm in listOfRpms:
if checkForRpm(req uiredRpm).find( 'is not installed') >
0:
missingRpms.app end(requiredRpm )
if missingRpms:
print 'The following RPMS are not installed:'
for eachMissingRpm in missingRpms:
print eachMissingRpm
print 'Please install them for the installation to
continue.'
sys.exit(-1)

---