[Freeipa-devel] [PATCH] 842 deprecation some sudorule options

[Rob Crittenden]

Martin Kosek wrote:
> On Thu, 2011-07-28 at 18:53 -0400, Rob Crittenden wrote:
>>
>> I created a validator for each of these three to disallow management.
>> This would be a major API change to remove them.
>>
>> They are useful in the find command so I hedged my bet and didn't
>> mark
>> the docs as deprecated, just as for -find only.
>>
>> rob
>>
>>
>
> NACK.
>
> The approach looks OK, I just think that the error message is not really
> what we want:
>
> # ipa sudorule-mod srule1 --externaluser=foo
> ipa: ERROR: invalid 'externaluser': The deny type has been deprecated.
>
> Martin
>

Yikes, I had meant to go back and fix that up. Updated patch attached.

rob
-------------- next part --------------
A non-text attachment was scrubbed...
Name: freeipa-rcrit-842-2-sudorule.patch
Type: application/mbox
Size: 9620 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/freeipa-devel/attachments/20110729/08ed51e7/attachment.mbox>