diff options
author | Martin Basti <mbasti@redhat.com> | 2016-09-22 14:39:16 +0200 |
---|---|---|
committer | Martin Basti <mbasti@redhat.com> | 2016-09-23 09:23:41 +0200 |
commit | 9b68d2a1f858854bc3cf2d6024f3fd3d79c2f696 (patch) | |
tree | 05b54fce4f950dc16051e2bf081f33698cd2a203 /ipaserver/install/sysupgrade.py | |
parent | 4d7d53c9664c9e68d7c6cda1a65cae7551423df7 (diff) | |
download | freeipa-9b68d2a1f858854bc3cf2d6024f3fd3d79c2f696.tar.gz freeipa-9b68d2a1f858854bc3cf2d6024f3fd3d79c2f696.tar.xz freeipa-9b68d2a1f858854bc3cf2d6024f3fd3d79c2f696.zip |
Pylint: enable global-variable-not-assigned check
the global keyword should be used only when variable from outside is
assigned inside, otherwise it has no effect and just confuses developers
Reviewed-By: Tomas Krizek <tkrizek@redhat.com>
Diffstat (limited to 'ipaserver/install/sysupgrade.py')
-rw-r--r-- | ipaserver/install/sysupgrade.py | 3 |
1 files changed, 0 insertions, 3 deletions
diff --git a/ipaserver/install/sysupgrade.py b/ipaserver/install/sysupgrade.py index 1eba38cd0..e9192ac2f 100644 --- a/ipaserver/install/sysupgrade.py +++ b/ipaserver/install/sysupgrade.py @@ -35,17 +35,14 @@ def _load_sstore(): def get_upgrade_state(module, state): _load_sstore() - global _sstore return _sstore.get_state(module, state) def set_upgrade_state(module, state, value): _load_sstore() - global _sstore _sstore.backup_state(module, state, value) def remove_upgrade_state(module, state): _load_sstore() - global _sstore _sstore.delete_state(module, state) def remove_upgrade_file(): |