diff options
author | Martin Basti <mbasti@redhat.com> | 2016-03-17 18:40:04 +0100 |
---|---|---|
committer | Martin Basti <mbasti@redhat.com> | 2016-03-22 10:20:51 +0100 |
commit | e93e89e1ae27e4f0ef23001f6c1247c45695ae24 (patch) | |
tree | bca16c54ee36e335ccc0a83ffd15dc457b5f2d9a /client | |
parent | 491447cc5ab8c5eff2be57d609201cefb79f7053 (diff) | |
download | freeipa-e93e89e1ae27e4f0ef23001f6c1247c45695ae24.tar.gz freeipa-e93e89e1ae27e4f0ef23001f6c1247c45695ae24.tar.xz freeipa-e93e89e1ae27e4f0ef23001f6c1247c45695ae24.zip |
Pylint: fix definition of global variables
Global variables should be defined in the outer space, not just marked
as global inside functions.
Removes unused global variables
Reviewed-By: Petr Spacek <pspacek@redhat.com>
Reviewed-By: Lukas Slebodnik <lslebodn@redhat.com>
Diffstat (limited to 'client')
-rwxr-xr-x | client/ipa-client-install | 11 |
1 files changed, 5 insertions, 6 deletions
diff --git a/client/ipa-client-install b/client/ipa-client-install index 884491eb0..be203586a 100755 --- a/client/ipa-client-install +++ b/client/ipa-client-install @@ -74,6 +74,11 @@ CLIENT_NOT_CONFIGURED = 2 CLIENT_ALREADY_CONFIGURED = 3 CLIENT_UNINSTALL_ERROR = 4 # error after restoring files/state + +fstore = sysrestore.FileStore(paths.IPA_CLIENT_SYSRESTORE) +statestore = sysrestore.StateFile(paths.IPA_CLIENT_SYSRESTORE) + + def parse_options(): def validate_ca_cert_file_option(option, opt, value, parser): if not os.path.exists(value): @@ -3068,12 +3073,6 @@ def main(): env={"PATH":"/bin:/sbin:/usr/kerberos/bin:/usr/kerberos/sbin:/usr/bin:/usr/sbin"} - global fstore - fstore = sysrestore.FileStore(paths.IPA_CLIENT_SYSRESTORE) - - global statestore - statestore = sysrestore.StateFile(paths.IPA_CLIENT_SYSRESTORE) - if options.uninstall: return uninstall(options, env) |