fixed most of pylint errors

also modified mof, so it's better parseable

greatly reduced pylint errors and warnings

LMI_SoftwareInstalledPackage now returns reference to
Linux_ComputerSystem instead of CIM_ComputerSystem

1 files changed, 22 insertions, 25 deletions

diff --git a/mof/LMI_Software.mof b/mof/LMI_Software.mof
index 7847915..fd24150 100644
--- a/mof/LMI_Software.mof
+++ b/mof/LMI_Software.mof
@@ -37,7 +37,7 @@ class LMI_SoftwarePackage : CIM_SoftwareElement {
         " number should be changed to reflect the new software version,"
         " and the release number should be reset to 1.")
     ]
-  String Release;
+  string Release;
 
     [ Description (
         "It is important to be careful with the version of post-release"
@@ -51,7 +51,7 @@ class LMI_SoftwarePackage : CIM_SoftwareElement {
         "Architecture name, that package is compiled for. In case of no"
         "architecture dependency, this will contain \"noarch\".")
     ]
-  String Architecture;
+  string Architecture;
 
     [ Description (
         "A software license is a legal instrument (usually by way of contract"
@@ -66,12 +66,12 @@ class LMI_SoftwarePackage : CIM_SoftwareElement {
         " Free Software Foundation and OSI. The string is obtained from RPM"
         " package." )
     ]
-  String License;
+  string License;
 
     [ Description (
         "Denotes the package group, in which the package belongs.")
     ]
-  String Group;
+  string Group;
 
     [ Description (
         "Size of RPM package in Bytes" ) ]
@@ -86,8 +86,7 @@ class LMI_SoftwarePackage : CIM_SoftwareElement {
           "The reference to newly installed package, if installation was"
           " successful. Reference to current package if it is already"
           " installed and Null otherwise.") ]
-    LMI_SoftwareInstalledPackage REF Installed
-  );
+    LMI_SoftwareInstalledPackage REF Installed);
 
     [ Description (
         "Will uninstall installed package."),
@@ -107,26 +106,26 @@ class LMI_SoftwareFileCheck : CIM_FileSpecification {
 
     [ Key, Description (
         "Absolute path of file being checked.") ]
-  String Name;
+  string Name;
 
     [ Description (
         "Composition of RPM's name and absolute file path."
         " It's format is \"rpm_name#path\"") ]
-  String CheckID;
+  string CheckID;
 
     [ Override("SoftwareElementID"), Description (
         "RPM's nevra. This means: "
         "\"name-[epoch:]version-release.architecture\"."
         " If epoch part is missing, a \"0:\" is assumed." ) ]
-  String SoftwareElementID;
+  string SoftwareElementID;
 
     [ Override("Version"), Description (
         "Version of packaged software as listed by RPM package.") ]
-  String Version;
+  string Version;
         
     [ Description (
         "True, if file is present in file system.")]
-  Boolean FileExists;
+  boolean FileExists;
 
     [ Override("FileSize"),
       Description("Size of installed file in Bytes.") ]
@@ -171,13 +170,13 @@ class LMI_SoftwareFileCheck : CIM_FileSpecification {
 
     [ Override("MD5Checksum"),
       Description("MD5 checksum of installed file.") ]
-  String MD5Checksum;
+  string MD5Checksum;
     [ Description("Checksum of installed file."
       " Hash algorithm used can be obtained with ChecksumType property.") ]
-  String FileChecksum;
+  string FileChecksum;
     [ Description("Checksum of file from RPM database."
       " Hash algorithm used can be obtained with ChecksumType property.") ]
-  String ExpectedFileChecksum;
+  string ExpectedFileChecksum;
 
     [ Description("Number of hash algorithm according to RFC4880."
       " This algorithm is used for making checksums of RPM files and"
@@ -208,11 +207,11 @@ class LMI_SoftwareFileCheck : CIM_FileSpecification {
 
     [ Description("Target destination of symbolic link as returned by"
       " readline. If file is not a symbolic link, NULL is returned.") ]
-  String LinkTarget;
+  string LinkTarget;
     [ Description("Target destination of symbolic link from RPM database"
       " as returned by readline. If file is not a symbolic link,"
       " NULL is returned.") ]
-  String ExpectedLinkTarget;
+  string ExpectedLinkTarget;
 
     [ Description("File type."),
       ValueMap { "0", "1", "2", "3"
@@ -235,12 +234,12 @@ class LMI_SoftwareFileCheck : CIM_FileSpecification {
         " performed)."
         " PassedFlagsDescriptions returns a description for each field in"
         " this array saying, what it means.") ]
-  Boolean PassedFlags[];
+  boolean PassedFlags[];
 
     [ Description (
         "Returns array of descriptions of checks corresponding to PassedFlags"
         " array.")]
-  String PassedFlagsDescriptions[];
+  string PassedFlagsDescriptions[];
 
 };
 
@@ -273,20 +272,19 @@ class LMI_SoftwareInstalledPackage : CIM_InstalledSoftwareElement {
       [ IN(true), Description (
           "Specify particular epoch of package to update to."
           " Update to newest, when empty.") ]
-    String Epoch,
+    string Epoch,
       [ IN(true), Description (
           "Specify particular version of package to update to."
           " Update to newest, when empty") ]
-    String Version,
+    string Version,
       [ IN(true), Description (
           "Specify particular release of package to update to."
           " Update to newest, when empty.") ]
-    String Release,
+    string Release,
       [ IN(false), OUT, Description (
           "The reference to newly installed package, if installation was"
           " successful. Otherwise reference to current package.") ]
-    LMI_SoftwareInstalledPackage REF Installed
-  );
+    LMI_SoftwareInstalledPackage REF Installed);
 
     [ Description (
         "Verify existence and attributes of files installed from"
@@ -300,8 +298,7 @@ class LMI_SoftwareInstalledPackage : CIM_InstalledSoftwareElement {
       [ IN(false), OUT, Description (
           "Array of references to LMI_SoftwareFileCheck, that did not pass"
           " verification.") ]
-    LMI_SoftwareFileCheck REF Failed[]
-  );
+    LMI_SoftwareFileCheck REF Failed[]);
 
 };