summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorAles Ledvinka <aledvink@redhat.com>2014-04-14 11:43:01 +0200
committerAles Ledvinka <aledvink@redhat.com>2014-04-14 11:43:01 +0200
commit373176a80f56fc4cb21d797f8369900fa922a921 (patch)
treea2ea942f9df48c10a5adabfc07ed6c57a6f645a5
parentfd34bd01f207a1bf61755597b4d43c5bc49d76e5 (diff)
downloadopenlmi-providers-373176a80f56fc4cb21d797f8369900fa922a921.tar.gz
openlmi-providers-373176a80f56fc4cb21d797f8369900fa922a921.tar.xz
openlmi-providers-373176a80f56fc4cb21d797f8369900fa922a921.zip
python2.6 review one omission
-rw-r--r--src/software/lmi/software/core/InstallationService.py6
1 files changed, 3 insertions, 3 deletions
diff --git a/src/software/lmi/software/core/InstallationService.py b/src/software/lmi/software/core/InstallationService.py
index 6cc079c..4aeca30 100644
--- a/src/software/lmi/software/core/InstallationService.py
+++ b/src/software/lmi/software/core/InstallationService.py
@@ -681,7 +681,7 @@ def _install_or_remove_check_params(
elif not isinstance(install_options, list):
raise InstallationError(values.Unspecified_Error,
"InstallOptions must be a list of uint16 values.")
- options = set((p for p in install_options))
+ options = set(p for p in install_options)
if options - supported_options:
raise InstallationError(values.Unspecified_Error,
@@ -700,10 +700,10 @@ def _install_or_remove_check_params(
"install option \"%s\" can not have any"
" associated value: %s" % (opt, val))
_check_target_and_collection(env, method, target, collection)
- exclusive = [opt for opt in options if opt in set((
+ exclusive = [opt for opt in options if opt in (
values.InstallOptions.Install,
values.InstallOptions.Update,
- values.InstallOptions.Uninstall ))]
+ values.InstallOptions.Uninstall )]
if len(exclusive) > 1:
raise InstallationError(values.Unspecified_Error,
"specified more than one mutually exclusive option at once: {%s}" %