On Mon, 2010-04-19 at 14:59 +0200, Martin Nagy wrote:
> Hi,
> as per comments from David, here is the patch. David, do you think this
> is acceptable? Let me know if you have any more suggestions.

Oops, forgot the patch, sorry.
Martin
>From ded3f5ceacca839131698aa995ebacd5f948b58d Mon Sep 17 00:00:00 2001
From: Martin Nagy <[email protected]>
Date: Tue, 6 Apr 2010 19:47:21 +0200
Subject: [PATCH] Some more changes for DNS forwarders prompt

---
 ipaserver/install/installutils.py |    6 +++---
 1 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/ipaserver/install/installutils.py b/ipaserver/install/installutils.py
index df3c891..54d8df3 100644
--- a/ipaserver/install/installutils.py
+++ b/ipaserver/install/installutils.py
@@ -151,9 +151,8 @@ def read_ip_address(host_name, fstore):
 
 def read_dns_forwarders():
     addrs = []
-    if ipautil.user_input("Do you wish to configure DNS forwarders?", False):
-        print "Please enter the IP addresses of DNS forwarders that you want to use."
-        print "After you are done, enter a blank line to stop."
+    if ipautil.user_input("Do you want to configure DNS forwarders?", True):
+        print "Enter the IP address of DNS forwarder to use, or press Enter to finish."
 
         while True:
             ip = ipautil.user_input("Enter IP address for a DNS forwarder",
@@ -164,6 +163,7 @@ def read_dns_forwarders():
                 print "You cannot use localhost as a DNS forwarder"
                 continue
             if not verify_ip_address(ip):
+                print "DNS forwarder %s not added" % ip
                 continue
 
             print "DNS forwarder %s added" % ip
-- 
1.6.2.5

_______________________________________________
Freeipa-devel mailing list
[email protected]
https://www.redhat.com/mailman/listinfo/freeipa-devel

Reply via email to