[Freeipa-devel] [PATCH 0041] permission-add does not prompt for ipapermright option in interactive mode

Gabe Alford redhatrises at gmail.com
Wed Feb 11 16:28:41 UTC 2015


Oops. My mistake. Corrected patch attached.

On Wed, Feb 11, 2015 at 8:59 AM, Martin Basti <mbasti at redhat.com> wrote:

>  Sorry, alwaysask didnt work.
> It was asking for rights during permission-mod.
>
> I replaced alwaysask with flag "ask_create".
> Sorry for late catch.
>
> Updated patch attached.
>
> PS: your name+email is missing in commit message, is it on purpose? And
> time wasn't correct in previous patch.
>
>
>
> On 11/02/15 15:06, Gabe Alford wrote:
>
> Good point. I personally was not aware of all that the API can do. Thanks
> Martin^2! Updated patch attached.
>
> On Tue, Feb 10, 2015 at 11:42 AM, Martin Basti <mbasti at redhat.com> wrote:
>
>>   On 29/01/15 17:10, Gabe Alford wrote:
>>
>>   Hello,
>>
>>  Fix for https://fedorahosted.org/freeipa/ticket/4872
>>
>>  Thanks,
>>
>> Gabe
>>
>>
>>  _______________________________________________
>> Freeipa-devel mailing listFreeipa-devel at redhat.comhttps://www.redhat.com/mailman/listinfo/freeipa-devel
>>
>>  Thank you for your patch.
>>
>> IMO, would be better to use flag, alwaysask for ipapermright, instead of
>> creating new callback:
>>
>>          StrEnum(
>>              'ipapermright*',
>>              cli_name='right',
>>              deprecated_cli_aliases={'permissions'},
>>              label=_('Granted rights'),
>>              doc=_('Rights to grant '
>>                    '(read, search, compare, write, add, delete, all)'),
>>              values=(u'read', u'search', u'compare',
>>                      u'write', u'add', u'delete', u'all'),
>> +            alwaysask=True,
>>          ),
>>
>> This change requires to generate new API.txt
>>
>> please run ./makeapi  and increment API version in VERSION file.
>>
>> Thank you in advance :-)
>>
>> Martin^2
>>
>> --
>> Martin Basti
>>
>>
>
>
> --
> Martin Basti
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://listman.redhat.com/archives/freeipa-devel/attachments/20150211/7e0b5c93/attachment.htm>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: freeipa-rga-0041-3-permission-add-does-not-prompt-for-ipapermright-in-i.patch
Type: application/octet-stream
Size: 3185 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/freeipa-devel/attachments/20150211/7e0b5c93/attachment.obj>


More information about the Freeipa-devel mailing list