[Freeipa-devel] [PATCH] Reword help for the user module

2011-02-16 Thread Jan Zelený
The first part of the ticket has already been solved, hence it is not a part of 
this patch.

https://fedorahosted.org/freeipa/ticket/351

Jan
From 0d649884896d67759187a605526fefc31b4ad81c Mon Sep 17 00:00:00 2001
From: Jan Zeleny 
Date: Wed, 16 Feb 2011 03:10:14 -0500
Subject: [PATCH] Reword help for the user module

https://fedorahosted.org/freeipa/ticket/351
---
 ipalib/plugins/user.py |3 +++
 1 files changed, 3 insertions(+), 0 deletions(-)

diff --git a/ipalib/plugins/user.py b/ipalib/plugins/user.py
index e74b8bed25979ffdd6c29d383e3420ad8c3350a8..343f6883c20bab85e0c00a9863299d50d3952edb 100644
--- a/ipalib/plugins/user.py
+++ b/ipalib/plugins/user.py
@@ -32,6 +32,9 @@ Disabling a user account prevents that user from obtaining new Kerberos
 credentials. It does not invalidate any credentials that have already
 been issued.
 
+The password management is not a part of this modules. For more information
+about this topic please see 'ipa help passwd'.
+
 EXAMPLES:
 
  Add a new user:
-- 
1.7.4

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

Re: [Freeipa-devel] [PATCH] Reword help for the user module

2011-02-16 Thread Martin Kosek
On Wed, 2011-02-16 at 09:13 +0100, Jan Zelený wrote:
> The first part of the ticket has already been solved, hence it is not a part 
> of 
> this patch.
> 
> https://fedorahosted.org/freeipa/ticket/351
> 
> Jan

NACK

Just a minor issue - s/this modules/this module/

Martin

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

Re: [Freeipa-devel] [PATCH] Reword help for the user module

2011-02-16 Thread Jan Zelený
Martin Kosek  wrote:
> On Wed, 2011-02-16 at 09:13 +0100, Jan Zelený wrote:
> > The first part of the ticket has already been solved, hence it is not a
> > part of this patch.
> > 
> > https://fedorahosted.org/freeipa/ticket/351
> > 
> > Jan
> 
> NACK
> 
> Just a minor issue - s/this modules/this module/

Thanks, the second version is in attachment.

Jan
From 0d649884896d67759187a605526fefc31b4ad81c Mon Sep 17 00:00:00 2001
From: Jan Zeleny 
Date: Wed, 16 Feb 2011 03:10:14 -0500
Subject: [PATCH] Reword help for the user module

https://fedorahosted.org/freeipa/ticket/351
---
 ipalib/plugins/user.py |3 +++
 1 files changed, 3 insertions(+), 0 deletions(-)

diff --git a/ipalib/plugins/user.py b/ipalib/plugins/user.py
index e74b8bed25979ffdd6c29d383e3420ad8c3350a8..343f6883c20bab85e0c00a9863299d50d3952edb 100644
--- a/ipalib/plugins/user.py
+++ b/ipalib/plugins/user.py
@@ -32,6 +32,9 @@ Disabling a user account prevents that user from obtaining new Kerberos
 credentials. It does not invalidate any credentials that have already
 been issued.
 
+The password management is not a part of this module. For more information
+about this topic please see 'ipa help passwd'.
+
 EXAMPLES:
 
  Add a new user:
-- 
1.7.4

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

Re: [Freeipa-devel] [PATCH] Reword help for the user module

2011-02-16 Thread Martin Kosek
On Wed, 2011-02-16 at 09:43 +0100, Jan Zelený wrote:
> Martin Kosek  wrote:
> > On Wed, 2011-02-16 at 09:13 +0100, Jan Zelený wrote:
> > > The first part of the ticket has already been solved, hence it is not a
> > > part of this patch.
> > > 
> > > https://fedorahosted.org/freeipa/ticket/351
> > > 
> > > Jan
> > 
> > NACK
> > 
> > Just a minor issue - s/this modules/this module/
> 
> Thanks, the second version is in attachment.
> 
> Jan

Now it looks good to me. Except I have a feeling that there should be a
colon before "please": ... about this topic, please see 'ipa help
passwd'.

I will let the final ack for a native speaker.

Martin

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

Re: [Freeipa-devel] [PATCH] Reword help for the user module

2011-02-16 Thread Rob Crittenden

Jan Zelený wrote:

The first part of the ticket has already been solved, hence it is not a part of
this patch.

https://fedorahosted.org/freeipa/ticket/351

Jan


ack, pushed to master

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


Re: [Freeipa-devel] [PATCH] Reword help for the user module

2011-02-16 Thread Rob Crittenden

Martin Kosek wrote:

On Wed, 2011-02-16 at 09:43 +0100, Jan Zelený wrote:

Martin Kosek  wrote:

On Wed, 2011-02-16 at 09:13 +0100, Jan Zelený wrote:

The first part of the ticket has already been solved, hence it is not a
part of this patch.

https://fedorahosted.org/freeipa/ticket/351

Jan


NACK

Just a minor issue - s/this modules/this module/


Thanks, the second version is in attachment.

Jan


Now it looks good to me. Except I have a feeling that there should be a
colon before "please": ... about this topic, please see 'ipa help
passwd'.

I will let the final ack for a native speaker.

Martin


Bah, I pushed the first patch. I've applied Jan's differences and made 
one minor tweak myself and pushed that out under the 1-liner rule.


rob

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