Martin Basti
|
31a9ef4f8b
|
IPAChangeConf: use constant for empty line
Instead of copy&paste is better to use constant. It makes code shorter
and improves readability, saves resources.
https://fedorahosted.org/freeipa/ticket/6392
Reviewed-By: Stanislav Laznicka <slaznick@redhat.com>
|
2016-11-11 12:13:56 +01:00 |
|
Martin Basti
|
1c9267803c
|
client: import IPAChangeConf directly instead the module
We should use as specific import as possible, better for python memory
consumption and speed, and looks better in code.
https://fedorahosted.org/freeipa/ticket/6392
Reviewed-By: Stanislav Laznicka <slaznick@redhat.com>
|
2016-11-11 12:13:56 +01:00 |
|
Martin Basti
|
c30b45ab15
|
client: remove extra return from hardcode_ldap_server
https://fedorahosted.org/freeipa/ticket/6392
Reviewed-By: Stanislav Laznicka <slaznick@redhat.com>
|
2016-11-11 12:13:56 +01:00 |
|
Martin Basti
|
cc6efb9798
|
client: install function: return constant not hardcoded number
https://fedorahosted.org/freeipa/ticket/6392
Reviewed-By: Stanislav Laznicka <slaznick@redhat.com>
|
2016-11-11 12:13:56 +01:00 |
|
Martin Basti
|
49f201e2b2
|
client: remove unneded return from configure_ipa_conf
Function always returns return code 0, and this code is even not used
elsewehere.
https://fedorahosted.org/freeipa/ticket/6392
Reviewed-By: Stanislav Laznicka <slaznick@redhat.com>
|
2016-11-11 12:13:56 +01:00 |
|
Martin Basti
|
5c16608a0d
|
client: remove unneded return configure_krb5_conf
Function configure_krb5_conf always returns 0 as return state. Remove
the 'return' statement and let exceptions work
https://fedorahosted.org/freeipa/ticket/6392
Reviewed-By: Stanislav Laznicka <slaznick@redhat.com>
|
2016-11-11 12:13:56 +01:00 |
|
Martin Basti
|
f98faec478
|
ipa-client-install: move client install to module
This commit only moves the code from ipa-client-install to module
ipaclient/install/client.py and fixes PEP8.
https://fedorahosted.org/freeipa/ticket/6392
Reviewed-By: Stanislav Laznicka <slaznick@redhat.com>
|
2016-11-11 12:13:56 +01:00 |
|