summaryrefslogtreecommitdiffstats
path: root/client
diff options
context:
space:
mode:
authorMartin Basti <mbasti@redhat.com>2016-03-17 18:40:04 +0100
committerMartin Basti <mbasti@redhat.com>2016-03-22 10:20:51 +0100
commite93e89e1ae27e4f0ef23001f6c1247c45695ae24 (patch)
treebca16c54ee36e335ccc0a83ffd15dc457b5f2d9a /client
parent491447cc5ab8c5eff2be57d609201cefb79f7053 (diff)
downloadfreeipa-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-xclient/ipa-client-install11
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)