Hi all

Can I please get a code review of CR 7166120.

7166120 dhcpd.conf(5) domain-search option incorrectly generated for multiple search domains

Since there is only 1 line change please find the pdiffs output.
(please let me know if you need webrev, I will create and upload one)

bash-4.1$ hg pdiffs
diff -r ff1310effa02 usr/src/cmd/installadm/dhcp.py
--- a/usr/src/cmd/installadm/dhcp.py    Mon May 07 17:32:38 2012 -0700
+++ b/usr/src/cmd/installadm/dhcp.py    Tue May 08 10:24:50 2012 -0600
@@ -1476,7 +1476,7 @@

# return output as a quoted string "example.com","sales.example.com"

-        search_domains = p.stdout.strip().split()
+        search_domains = p.stdout.replace('\\','').split()
         return ','.join(['"' + x + '"' for x in search_domains])
     except CalledProcessError:
         logging.debug("Property not set: %s", cmd)


Testing :
Case mentioned in CR :

root@solaris:~# svcprop -p config/search dns/client
in.oracle.com oracle.com\  uk.oracle.com \ sun.com\  \ uk.sun.com\

For the above configuration, dhcpd4.conf entry :
option domain-search "in.oracle.com","oracle.com","uk.oracle.com","sun.com","uk.sun.com";

Thanks
Nirmal
_______________________________________________
caiman-discuss mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/caiman-discuss

Reply via email to