Attachment 307121 Details for Bug 443987 – actually pass along the verbose flag to IPAClient

[patch] actually pass along the verbose flag to IPAClient

freeipa-32-verbose.patch (text/plain), 8.32 KB, created by Rob Crittenden on 2008-05-29 18:38:30 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Rob Crittenden

Created: 2008-05-29 18:38:30 UTC

Size: 8.32 KB

patch

obsolete

>From f60b33b7bc10cfabea88ff748a759fc0474a1c1f Mon Sep 17 00:00:00 2001
>From: Rob Crittenden <rcritten@redhat.com>
>Date: Thu, 29 May 2008 14:36:25 -0400
>Subject: [PATCH] Actually pass along the verbose option to IPAClient.
>
>443987
>---
> ipa-admintools/ipa-adddelegation  |    2 +-
> ipa-admintools/ipa-addgroup       |    2 +-
> ipa-admintools/ipa-addservice     |    2 +-
> ipa-admintools/ipa-adduser        |    2 +-
> ipa-admintools/ipa-deldelegation  |    2 +-
> ipa-admintools/ipa-delgroup       |    2 +-
> ipa-admintools/ipa-delservice     |    2 +-
> ipa-admintools/ipa-deluser        |    2 +-
> ipa-admintools/ipa-findgroup      |    2 +-
> ipa-admintools/ipa-findservice    |    2 +-
> ipa-admintools/ipa-listdelegation |    2 +-
> ipa-admintools/ipa-lockuser       |    2 +-
> ipa-admintools/ipa-moddelegation  |    2 +-
> ipa-admintools/ipa-modgroup       |    2 +-
> ipa-admintools/ipa-moduser        |    2 +-
> ipa-admintools/ipa-passwd         |    2 +-
> ipa-admintools/ipa-pwpolicy       |    2 +-
> 17 files changed, 17 insertions(+), 17 deletions(-)
>
>diff --git a/ipa-admintools/ipa-adddelegation b/ipa-admintools/ipa-adddelegation
>index 112a0d3..fa855e2 100644
>--- a/ipa-admintools/ipa-adddelegation
>+++ b/ipa-admintools/ipa-adddelegation
>@@ -76,7 +76,7 @@ def main():
>         usage()
> 
>     if options.list:
>-        client = ipaclient.IPAClient()
>+        client = ipaclient.IPAClient(verbose=options.verbose)
>         l = client.get_all_attrs()
> 
>         for x in l:
>diff --git a/ipa-admintools/ipa-addgroup b/ipa-admintools/ipa-addgroup
>index 4f55224..fea8428 100644
>--- a/ipa-admintools/ipa-addgroup
>+++ b/ipa-admintools/ipa-addgroup
>@@ -107,7 +107,7 @@ def main():
>     group.setValue('cn', cn)
>     group.setValue('description', desc)
> 
>-    client = ipaclient.IPAClient()
>+    client = ipaclient.IPAClient(verbose=options.verbose)
>     client.add_group(group)
>     print cn + " successfully added"
> 
>diff --git a/ipa-admintools/ipa-addservice b/ipa-admintools/ipa-addservice
>index 535fddf..ddf75a8 100644
>--- a/ipa-admintools/ipa-addservice
>+++ b/ipa-admintools/ipa-addservice
>@@ -74,7 +74,7 @@ def main():
>         usage()
>     princ_name = args[1]
> 
>-    client = ipaclient.IPAClient()
>+    client = ipaclient.IPAClient(verbose=options.verbose)
> 
>     client.add_service_principal(princ_name, "%d" % options.force)
> 
>diff --git a/ipa-admintools/ipa-adduser b/ipa-admintools/ipa-adduser
>index caff5a9..131c8a7 100644
>--- a/ipa-admintools/ipa-adduser
>+++ b/ipa-admintools/ipa-adduser
>@@ -221,7 +221,7 @@ def main():
>     if shell:
>         user.setValue('loginshell', shell)
> 
>-    client = ipaclient.IPAClient()
>+    client = ipaclient.IPAClient(verbose=options.verbose)
>     client.add_user(user)
> 
>     # Set the User's password
>diff --git a/ipa-admintools/ipa-deldelegation b/ipa-admintools/ipa-deldelegation
>index 624aefc..4284af8 100644
>--- a/ipa-admintools/ipa-deldelegation
>+++ b/ipa-admintools/ipa-deldelegation
>@@ -66,7 +66,7 @@ def main():
>     if options.usage or len(args) != 2:
>         usage()
> 
>-    client = ipaclient.IPAClient()
>+    client = ipaclient.IPAClient(verbose=options.verbose)
>     aci_entry = client.get_aci_entry(aci_fields)
> 
>     aci_str_list = aci_entry.getValues('aci')
>diff --git a/ipa-admintools/ipa-delgroup b/ipa-admintools/ipa-delgroup
>index 629f79f..7ae62d2 100644
>--- a/ipa-admintools/ipa-delgroup
>+++ b/ipa-admintools/ipa-delgroup
>@@ -60,7 +60,7 @@ def main():
>     if options.usage or len(args) != 2:
>         usage()
> 
>-    client = ipaclient.IPAClient()
>+    client = ipaclient.IPAClient(verbose=options.verbose)
>     groups = client.find_groups(args[1], ['cn','description','gidnumber','nsAccountLock'])
> 
>     counter = groups[0]
>diff --git a/ipa-admintools/ipa-delservice b/ipa-admintools/ipa-delservice
>index 7397c75..3d7d184 100644
>--- a/ipa-admintools/ipa-delservice
>+++ b/ipa-admintools/ipa-delservice
>@@ -66,7 +66,7 @@ def main():
>         usage()
>     princ_name = args[1]
> 
>-    client = ipaclient.IPAClient()
>+    client = ipaclient.IPAClient(verbose=options.verbose)
> 
>     hosts = client.find_service_principal(args[1], sattrs=None)
>     counter = hosts[0]
>diff --git a/ipa-admintools/ipa-deluser b/ipa-admintools/ipa-deluser
>index 8a4b4ad..cfa935e 100644
>--- a/ipa-admintools/ipa-deluser
>+++ b/ipa-admintools/ipa-deluser
>@@ -60,7 +60,7 @@ def main():
>     if options.usage or len(args) != 2:
>         usage()
> 
>-    client = ipaclient.IPAClient()
>+    client = ipaclient.IPAClient(verbose=options.verbose)
>     ret = client.delete_user(args[1])
>     print args[1] + " successfully deleted"
> 
>diff --git a/ipa-admintools/ipa-findgroup b/ipa-admintools/ipa-findgroup
>index d884977..9e514ee 100644
>--- a/ipa-admintools/ipa-findgroup
>+++ b/ipa-admintools/ipa-findgroup
>@@ -68,7 +68,7 @@ def main():
>     if options.usage or len(args) != 2:
>         usage()
> 
>-    client = ipaclient.IPAClient()
>+    client = ipaclient.IPAClient(verbose=options.verbose)
>     if options.all is None:
>         groups = client.find_groups(args[1], ['cn','description','gidnumber','nsAccountLock'])
>     else:
>diff --git a/ipa-admintools/ipa-findservice b/ipa-admintools/ipa-findservice
>index d427e72..690c22e 100644
>--- a/ipa-admintools/ipa-findservice
>+++ b/ipa-admintools/ipa-findservice
>@@ -63,7 +63,7 @@ def main():
>     if options.usage or len(args) != 2:
>         usage()
> 
>-    client = ipaclient.IPAClient()
>+    client = ipaclient.IPAClient(verbose=options.verbose)
>     hosts = client.find_service_principal(args[1], sattrs=None)
> 
>     counter = hosts[0]
>diff --git a/ipa-admintools/ipa-listdelegation b/ipa-admintools/ipa-listdelegation
>index 73c070e..50e80c6 100644
>--- a/ipa-admintools/ipa-listdelegation
>+++ b/ipa-admintools/ipa-listdelegation
>@@ -65,7 +65,7 @@ def main():
>     if options.usage:
>         usage()
> 
>-    client = ipaclient.IPAClient()
>+    client = ipaclient.IPAClient(verbose=options.verbose)
>     aci_entry = client.get_aci_entry(aci_fields)
> 
>     aci_str_list = aci_entry.getValues('aci')
>diff --git a/ipa-admintools/ipa-lockuser b/ipa-admintools/ipa-lockuser
>index 0fea7be..4940a71 100644
>--- a/ipa-admintools/ipa-lockuser
>+++ b/ipa-admintools/ipa-lockuser
>@@ -63,7 +63,7 @@ def main():
>         usage()
> 
>     msg = "inactivated"
>-    client = ipaclient.IPAClient()
>+    client = ipaclient.IPAClient(verbose=options.verbose)
>     if options.unlock:
>         try:
>             ret = client.mark_user_active(args[1])
>diff --git a/ipa-admintools/ipa-moddelegation b/ipa-admintools/ipa-moddelegation
>index 39f6b22..f1498a9 100644
>--- a/ipa-admintools/ipa-moddelegation
>+++ b/ipa-admintools/ipa-moddelegation
>@@ -75,7 +75,7 @@ def main():
>     options, args = parse_options()
> 
>     if options.list:
>-        client = ipaclient.IPAClient()
>+        client = ipaclient.IPAClient(verbose=options.verbose)
>         l = client.get_all_attrs()
> 
>         for x in l:
>diff --git a/ipa-admintools/ipa-modgroup b/ipa-admintools/ipa-modgroup
>index 11aba4a..0d09342 100644
>--- a/ipa-admintools/ipa-modgroup
>+++ b/ipa-admintools/ipa-modgroup
>@@ -111,7 +111,7 @@ def main():
>     options, args = parse_options()
> 
>     if options.list:
>-        client = ipaclient.IPAClient()
>+        client = ipaclient.IPAClient(verbose=options.verbose)
>         list = client.get_all_attrs()
> 
>         for x in list:
>diff --git a/ipa-admintools/ipa-moduser b/ipa-admintools/ipa-moduser
>index 39b0fab..db609a1 100644
>--- a/ipa-admintools/ipa-moduser
>+++ b/ipa-admintools/ipa-moduser
>@@ -102,7 +102,7 @@ def main():
>     options, args = parse_options()
> 
>     if options.list:
>-        client = ipaclient.IPAClient()
>+        client = ipaclient.IPAClient(verbose=options.verbose)
>         list = client.get_all_attrs()
> 
>         for x in list:
>diff --git a/ipa-admintools/ipa-passwd b/ipa-admintools/ipa-passwd
>index e9d04a6..2dd574e 100644
>--- a/ipa-admintools/ipa-passwd
>+++ b/ipa-admintools/ipa-passwd
>@@ -116,7 +116,7 @@ def main():
>         print "Password change cancelled"
>         return 1
> 
>-    client = ipaclient.IPAClient()
>+    client = ipaclient.IPAClient(verbose=options.verbose)
>     client.modifyPassword(principal, '', password)
> 
>     return 0
>diff --git a/ipa-admintools/ipa-pwpolicy b/ipa-admintools/ipa-pwpolicy
>index 71f824c..5621857 100644
>--- a/ipa-admintools/ipa-pwpolicy
>+++ b/ipa-admintools/ipa-pwpolicy
>@@ -116,7 +116,7 @@ def main():
>     if options.usage:
>         usage()
> 
>-    client = ipaclient.IPAClient()
>+    client = ipaclient.IPAClient(verbose=options.verbose)
> 
>     if options.show: 
>         show_policy(client)
>-- 
>1.5.4
>

Actions: View | Diff

Attachments on bug 443987: 303639 | 307121 | 308978