diff options
| author | Martin Basti <mbasti@redhat.com> | 2016-09-26 14:08:17 +0200 |
|---|---|---|
| committer | Martin Basti <mbasti@redhat.com> | 2016-09-27 13:35:58 +0200 |
| commit | 0f88f8fe889ae4801fc8d5ece1ad51c5246718ac (patch) | |
| tree | 4f38fb6210abaa9f354353df2abda96d1b8fb612 /install/tools | |
| parent | 452b08754d02b89c0e3117b83d9156e6110943c9 (diff) | |
| download | freeipa-0f88f8fe889ae4801fc8d5ece1ad51c5246718ac.tar.gz freeipa-0f88f8fe889ae4801fc8d5ece1ad51c5246718ac.tar.xz freeipa-0f88f8fe889ae4801fc8d5ece1ad51c5246718ac.zip | |
Remove unused variables in the code
This commit removes unused variables or rename variables as "expected to
be unused" by using "_" prefix.
This covers only cases where fix was easy or only one unused variable
was in a module
Reviewed-By: Florence Blanc-Renaud <frenaud@redhat.com>
Reviewed-By: Stanislav Laznicka <slaznick@redhat.com>
Diffstat (limited to 'install/tools')
| -rwxr-xr-x | install/tools/ipa-csreplica-manage | 4 | ||||
| -rwxr-xr-x | install/tools/ipa-dns-install | 2 | ||||
| -rwxr-xr-x | install/tools/ipa-replica-conncheck | 2 |
3 files changed, 2 insertions, 6 deletions
diff --git a/install/tools/ipa-csreplica-manage b/install/tools/ipa-csreplica-manage index 2a2f2ae8a..87c1c2a95 100755 --- a/install/tools/ipa-csreplica-manage +++ b/install/tools/ipa-csreplica-manage @@ -232,9 +232,6 @@ def del_link(realm, replica1, replica2, dirman_passwd, force=False): print("Deleted replication agreement from '%s' to '%s'" % (replica1, replica2)) def del_master(realm, hostname, options): - - force_del = False - delrepl = None # 1. Connect to the local dogtag DS server @@ -258,7 +255,6 @@ def del_master(realm, hostname, options): sys.exit(1) else: print("Unable to connect to replica %s, forcing removal" % hostname) - force_del = True # 4. Get list of agreements. if delrepl is None: diff --git a/install/tools/ipa-dns-install b/install/tools/ipa-dns-install index 413862235..05bd31556 100755 --- a/install/tools/ipa-dns-install +++ b/install/tools/ipa-dns-install @@ -89,7 +89,7 @@ def parse_options(): parser.add_option("--force", dest="force", action="store_true", help="Force install") - options, args = parser.parse_args() + options, _args = parser.parse_args() safe_options = parser.get_safe_opts(options) if options.dnssec_master and options.disable_dnssec_master: diff --git a/install/tools/ipa-replica-conncheck b/install/tools/ipa-replica-conncheck index 0c583cfc1..067afb7b0 100755 --- a/install/tools/ipa-replica-conncheck +++ b/install/tools/ipa-replica-conncheck @@ -185,7 +185,7 @@ def parse_options(): parser.add_option("--no-log", dest="log_to_file", action="store_false", default=True, help="Do not log into file") - options, args = parser.parse_args() + options, _args = parser.parse_args() safe_options = parser.get_safe_opts(options) if options.master and options.replica: |