Re: [Freeipa-devel] [PATCH] Provide a way to display CLI-LDAP attribute relation

2011-02-10 Thread Rob Crittenden
Jan Zeleny wrote: Rob Crittenden wrote: Just a really minor nit. Can you define a label for the argument? Otherwise if you run: `ipa show-mappings` it will prompt for . rob Done, sending in attachment. Jan I made one minor change to the patch before pushing. I wrapper 'Command name' in _

Re: [Freeipa-devel] [PATCH] Provide a way to display CLI-LDAP attribute relation

2011-02-10 Thread Jan Zeleny
Rob Crittenden wrote: > Just a really minor nit. Can you define a label for the argument? > Otherwise if you run: `ipa show-mappings` it will prompt for > . > > rob Done, sending in attachment. Jan From fece796ab7894a591ef4e2fb4bb39f097c687cc1 Mon Sep 17 00:00:00 2001 From: Jan Zeleny Date: We

Re: [Freeipa-devel] [PATCH] Provide a way to display CLI-LDAP attribute relation

2011-02-10 Thread Rob Crittenden
Jan Zelený wrote: Ok, I'm sending updated patch in attachment Should I change it in class help then? That's where I copied this from. I think so. Ok, I'll send another patch, so me don't mix it together with this patch. I'll do a review of the code in cli.py, maybe the same issue is elsewhe

Re: [Freeipa-devel] [PATCH] Provide a way to display CLI-LDAP attribute relation

2011-02-10 Thread Jan Zeleny
Jan Zelený wrote: > Ok, I'm sending updated patch in attachment > > > > Should I change it in class help then? That's where I copied this from. > > > > I think so. > > Ok, I'll send another patch, so me don't mix it together with this patch. > I'll do a review of the code in cli.py, maybe the s

Re: [Freeipa-devel] [PATCH] Provide a way to display CLI-LDAP attribute relation

2011-02-01 Thread Jan Zelený
Ok, I'm sending updated patch in attachment > > Should I change it in class help then? That's where I copied this from. > > I think so. Ok, I'll send another patch, so me don't mix it together with this patch. I'll do a review of the code in cli.py, maybe the same issue is elsewhere as well. >

Re: [Freeipa-devel] [PATCH] Provide a way to display CLI-LDAP attribute relation

2011-02-01 Thread Rob Crittenden
Jan Zelený wrote: Rob Crittenden wrote: Jan Zelený wrote: Since some LDAP attributes have their cli_name value defined, so they can be more user friendly, it can be difficult for user to find out which attributes do the parameteres given to CLI really represent. This patch provides new command

Re: [Freeipa-devel] [PATCH] Provide a way to display CLI-LDAP attribute relation

2011-01-31 Thread Jan Zelený
Rob Crittenden wrote: > Jan Zelený wrote: > > Since some LDAP attributes have their cli_name value defined, > > so they can be more user friendly, it can be difficult for user to find > > out which attributes do the parameteres given to CLI really represent. > > This patch provides new command, wh

Re: [Freeipa-devel] [PATCH] Provide a way to display CLI-LDAP attribute relation

2011-01-28 Thread Rob Crittenden
Jan Zelený wrote: Since some LDAP attributes have their cli_name value defined, so they can be more user friendly, it can be difficult for user to find out which attributes do the parameteres given to CLI really represent. This patch provides new command, which will take another IPA command as an

[Freeipa-devel] [PATCH] Provide a way to display CLI-LDAP attribute relation

2011-01-26 Thread Jan Zelený
Since some LDAP attributes have their cli_name value defined, so they can be more user friendly, it can be difficult for user to find out which attributes do the parameteres given to CLI really represent. This patch provides new command, which will take another IPA command as and argument and displ