diff options
author | Ales Ledvinka <aledvink@redhat.com> | 2014-04-14 11:43:01 +0200 |
---|---|---|
committer | Ales Ledvinka <aledvink@redhat.com> | 2014-04-14 11:43:01 +0200 |
commit | 373176a80f56fc4cb21d797f8369900fa922a921 (patch) | |
tree | a2ea942f9df48c10a5adabfc07ed6c57a6f645a5 /src/software/lmi | |
parent | fd34bd01f207a1bf61755597b4d43c5bc49d76e5 (diff) | |
download | openlmi-providers-373176a80f56fc4cb21d797f8369900fa922a921.tar.gz openlmi-providers-373176a80f56fc4cb21d797f8369900fa922a921.tar.xz openlmi-providers-373176a80f56fc4cb21d797f8369900fa922a921.zip |
python2.6 review one omission
Diffstat (limited to 'src/software/lmi')
-rw-r--r-- | src/software/lmi/software/core/InstallationService.py | 6 |
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}" % |