Re: [Freeipa-devel] [PATCH] 744 use Sudo rather than SUDO

2011-03-01 Thread Rob Crittenden

Endi Sukma Dewata wrote:

On 2/28/2011 10:47 AM, Rob Crittenden wrote:

Use Sudo instead of SUDO in labels, descriptions, etc.

ticket 1005

rob


This patch is ACKed. The capitalization is now consistent in the CLI.
However, the UI capitalizes the labels in the action panel and the title
of association facets, so we still see a mix of Sudo and SUDO in the UI.

There are still some SUDO leftover in the UI test data, but that can be
fixed in a separate patch.



pushed to master

___
Freeipa-devel mailing list
Freeipa-devel@redhat.com
https://www.redhat.com/mailman/listinfo/freeipa-devel


Re: [Freeipa-devel] [PATCH] 744 use Sudo rather than SUDO

2011-03-01 Thread Rob Crittenden

Adam Young wrote:

  On 02/28/2011 03:28 PM, Endi Sukma Dewata wrote:

On 2/28/2011 12:51 PM, Endi Sukma Dewata wrote:

On 2/28/2011 10:47 AM, Rob Crittenden wrote:

Use Sudo instead of SUDO in labels, descriptions, etc.

ticket 1005

rob


This patch is ACKed. The capitalization is now consistent in the CLI.
However, the UI capitalizes the labels in the action panel and the title
of association facets, so we still see a mix of Sudo and SUDO in the UI.

There are still some SUDO leftover in the UI test data, but that can be
fixed in a separate patch.


The attached patch fixes the UI test data.


___
Freeipa-devel mailing list
Freeipa-devel@redhat.com
https://www.redhat.com/mailman/listinfo/freeipa-devel

ACK


pushed to master

___
Freeipa-devel mailing list
Freeipa-devel@redhat.com
https://www.redhat.com/mailman/listinfo/freeipa-devel


Re: [Freeipa-devel] [PATCH] 744 use Sudo rather than SUDO

2011-02-28 Thread Adam Young

On 02/28/2011 03:28 PM, Endi Sukma Dewata wrote:

On 2/28/2011 12:51 PM, Endi Sukma Dewata wrote:

On 2/28/2011 10:47 AM, Rob Crittenden wrote:

Use Sudo instead of SUDO in labels, descriptions, etc.

ticket 1005

rob


This patch is ACKed. The capitalization is now consistent in the CLI.
However, the UI capitalizes the labels in the action panel and the title
of association facets, so we still see a mix of Sudo and SUDO in the UI.

There are still some SUDO leftover in the UI test data, but that can be
fixed in a separate patch.


The attached patch fixes the UI test data.


___
Freeipa-devel mailing list
Freeipa-devel@redhat.com
https://www.redhat.com/mailman/listinfo/freeipa-devel

ACK
___
Freeipa-devel mailing list
Freeipa-devel@redhat.com
https://www.redhat.com/mailman/listinfo/freeipa-devel

Re: [Freeipa-devel] [PATCH] 744 use Sudo rather than SUDO

2011-02-28 Thread Endi Sukma Dewata

On 2/28/2011 12:51 PM, Endi Sukma Dewata wrote:

On 2/28/2011 10:47 AM, Rob Crittenden wrote:

Use Sudo instead of SUDO in labels, descriptions, etc.

ticket 1005

rob


This patch is ACKed. The capitalization is now consistent in the CLI.
However, the UI capitalizes the labels in the action panel and the title
of association facets, so we still see a mix of Sudo and SUDO in the UI.

There are still some SUDO leftover in the UI test data, but that can be
fixed in a separate patch.


The attached patch fixes the UI test data.

--
Endi S. Dewata
From bf735f39abcd496a9ceb796a4516f340fc037b89 Mon Sep 17 00:00:00 2001
From: Endi S. Dewata 
Date: Mon, 28 Feb 2011 14:22:55 -0600
Subject: [PATCH] Replaced SUDO with Sudo in UI test data.

Ticket 1005
---
 install/ui/test/data/hbacsvc_show.json |2 +-
 install/ui/test/data/hbacsvcgroup_find.json|4 ++--
 .../ui/test/data/hbacsvcgroup_member_hbacsvc.json  |4 ++--
 install/ui/test/data/hbacsvcgroup_show.json|4 ++--
 install/ui/test/data/i18n_messages.json|6 +++---
 install/ui/test/data/ipa_init.json |   12 ++--
 install/ui/test/data/json_metadata.json|6 +++---
 install/ui/test/data/sudocmdgroup_add_member.json  |2 +-
 install/ui/test/data/sudocmdgroup_mod.json |2 +-
 .../ui/test/data/sudocmdgroup_remove_member.json   |2 +-
 10 files changed, 22 insertions(+), 22 deletions(-)

diff --git a/install/ui/test/data/hbacsvc_show.json b/install/ui/test/data/hbacsvc_show.json
index 8ebfbd7c12dad634e78ed71f757f541d5ae223ff..749d5da95b3255e2a92682268e1370430dbf341a 100644
--- a/install/ui/test/data/hbacsvc_show.json
+++ b/install/ui/test/data/hbacsvc_show.json
@@ -22,7 +22,7 @@
 "676e92a1-ec7611df-b3f78f4b-11cc007b"
 ],
 "memberof": [
-"cn=SUDO,cn=hbacservicegroups,cn=hbac,dc=dev,dc=example,dc=com"
+"cn=Sudo,cn=hbacservicegroups,cn=hbac,dc=dev,dc=example,dc=com"
 ],
 "objectclass": [
 "ipahbacservice",
diff --git a/install/ui/test/data/hbacsvcgroup_find.json b/install/ui/test/data/hbacsvcgroup_find.json
index 797530e37e18142ebf4370f2526e9991f1002a1d..503bfa3792187826281cbba79cfa8ad1a9cff14a 100644
--- a/install/ui/test/data/hbacsvcgroup_find.json
+++ b/install/ui/test/data/hbacsvcgroup_find.json
@@ -6,10 +6,10 @@
 "result": [
 {
 "cn": [
-"SUDO"
+"Sudo"
 ],
 "description": [
-"Default group of SUDO related services"
+"Default group of Sudo related services"
 ],
 "dn": "cn=sudo,cn=hbacservicegroups,cn=hbac,dc=dev,dc=example,dc=com",
 "member_hbacsvc": [
diff --git a/install/ui/test/data/hbacsvcgroup_member_hbacsvc.json b/install/ui/test/data/hbacsvcgroup_member_hbacsvc.json
index a407a622f7249a1eddefad61055dccc45fc54113..b2332f5a9324e1c66a626a10b13df2c9095c389b 100644
--- a/install/ui/test/data/hbacsvcgroup_member_hbacsvc.json
+++ b/install/ui/test/data/hbacsvcgroup_member_hbacsvc.json
@@ -18,7 +18,7 @@
 "42927a86-f46d-11df-8cc1-00163e72f2d9"
 ],
 "memberof": [
-"cn=SUDO,cn=hbacservicegroups,cn=hbac,dc=dev,dc=example,dc=com"
+"cn=Sudo,cn=hbacservicegroups,cn=hbac,dc=dev,dc=example,dc=com"
 ],
 "objectclass": [
 "ipahbacservice",
@@ -42,7 +42,7 @@
 "42970a6a-f46d-11df-8cc1-00163e72f2d9"
 ],
 "memberof": [
-"cn=SUDO,cn=hbacservicegroups,cn=hbac,dc=dev,dc=example,dc=com"
+"cn=Sudo,cn=hbacservicegroups,cn=hbac,dc=dev,dc=example,dc=com"
 ],
 "objectclass": [
 "ipahbacservice",
diff --git a/install/ui/test/data/hbacsvcgroup_show.json b/install/ui/test/data/hbacsvcgroup_show.json
index ad3c4253bb115a2b57798ef7a0b1e0d96b13e3ac..05ba55ba5de5c65937e9d744a3b99fa24e9e04c3 100644
--- a/install/ui/test/data/hbacsvcgroup_show.json
+++ b/install/ui/test/data/hbacsvcgroup_show.json
@@ -19,10 +19,10 @@
 "seealso": "rscwo"
 },
 "cn": [
-"SUDO"
+"Sudo"
 ],
 "description": [
-"Default group of SUDO related services"
+"Default group of Sudo related services"
 ],
 "dn": "cn=sudo,cn=hbacservicegroups,cn=hbac,dc=dev,dc=example,dc=com",
 "ipauniqueid": [
diff --git a/install/ui/test/data/i18n_messages.json b/install/ui/test/data/i18n_messages.json
index f5aa841ec1483618a3851871362477e2a6756241..11b02c846cf3d2fc76b580555db764639e92dfeb 100644
--- a/install/ui/test/data/i18n_message

Re: [Freeipa-devel] [PATCH] 744 use Sudo rather than SUDO

2011-02-28 Thread Endi Sukma Dewata

On 2/28/2011 10:47 AM, Rob Crittenden wrote:

Use Sudo instead of SUDO in labels, descriptions, etc.

ticket 1005

rob


This patch is ACKed. The capitalization is now consistent in the CLI. 
However, the UI capitalizes the labels in the action panel and the title 
of association facets, so we still see a mix of Sudo and SUDO in the UI.


There are still some SUDO leftover in the UI test data, but that can be 
fixed in a separate patch.


--
Endi S. Dewata

___
Freeipa-devel mailing list
Freeipa-devel@redhat.com
https://www.redhat.com/mailman/listinfo/freeipa-devel