summaryrefslogtreecommitdiffstats
path: root/ipa-client
diff options
context:
space:
mode:
authorKarl MacMillan <kmacmill@redhat.com>2007-12-12 11:05:47 -0500
committerKarl MacMillan <kmacmill@redhat.com>2007-12-12 11:05:47 -0500
commit2892c28f56b855bc07a1c2cdb765bd0e5d2e5f8b (patch)
treeec69616d31e7c6efe056b5deb4d26e36a79f5d96 /ipa-client
parent7561d7c42d8d62d42cb6a50dac368c786cf75b38 (diff)
downloadfreeipa-2892c28f56b855bc07a1c2cdb765bd0e5d2e5f8b.tar.gz
freeipa-2892c28f56b855bc07a1c2cdb765bd0e5d2e5f8b.tar.xz
freeipa-2892c28f56b855bc07a1c2cdb765bd0e5d2e5f8b.zip
Improve confirmation.
Diffstat (limited to 'ipa-client')
-rw-r--r--ipa-client/ipa-install/ipa-client-install15
1 files changed, 9 insertions, 6 deletions
diff --git a/ipa-client/ipa-install/ipa-client-install b/ipa-client/ipa-install/ipa-client-install
index dfaabca3e..9cc6f6f6b 100644
--- a/ipa-client/ipa-install/ipa-client-install
+++ b/ipa-client/ipa-install/ipa-client-install
@@ -51,6 +51,13 @@ def parse_options():
return options
+def ask_for_confirmation(message):
+ yesno = raw_input(message + " [y/N]: ")
+ if not yesno or yesno.lower()[0] != "y":
+ return False
+ print "\n"
+ return True
+
def logging_setup(options):
# Always log everything (i.e., DEBUG) to the log
# file.
@@ -114,10 +121,8 @@ def main():
print "\nThe failure to use DNS to find your IPA server indicates that your resolv.conf file is not properly configured\n."
print "Autodiscovery of servers for failover cannot work with this configuration.\n"
print "If you proceed with the installation, services will be configured to always access the discovered server for all operation and will not fail over to other servers in case of failure\n"
- yesno = raw_input("Do you want to proceed and configure the system with fixed values with no DNS discovery? [y/N] ")
- if yesno.lower() != "y":
+ if not ask_for_confirmation("Do you want to proceed and configure the system with fixed values with no DNS discovery?"):
return ret
- print "\n"
print "Realm: "+ds.getRealmName()
print "DNS Domain: "+ds.getDomainName()
@@ -125,10 +130,8 @@ def main():
print "BaseDN: "+ds.getBaseDN()
print "\n"
- yesno = raw_input("Continue to configure the system with these values? [y/N] ")
- if yesno.lower() != "y":
+ if not ask_for_confirmation("Continue to configure the system with these values?"):
return 1
- print "\n"
# Configure ipa.conf
ipaconf = ipaclient.ipachangeconf.IPAChangeConf("IPA Installer")