[Freeipa-devel] [PATCH] 0008 Use certmonger D-Bus API instead of messing with its files.

Martin Kosek mkosek at redhat.com
Tue Aug 19 16:35:55 UTC 2014


On 08/19/2014 05:44 PM, Rob Crittenden wrote:
> David Kupka wrote:
...
>>> b) In this case, do not use "raise e", but just "raise" to keep the
>>> exception
>>> stack trace intact for better debugging.
>> Every day there's something new to learn about python or FreeIPA.
>>>
>>> Both a) and b) should be fixed in other occasions and places.
>> I found only one occurence of a) issue. Is there some hidden or are you
>> talking about the whole FreeIPA project?

Ah, I see - there really is just one a). You can just fix the refactored
plugin, let us not save the whole world/freeipa yet.

Martin




More information about the Freeipa-devel mailing list