[Freeipa-devel] [PATCH] 0060 Add warning when uninstalling active replica

Tomas Babej tbabej at redhat.com
Mon Sep 2 16:21:04 UTC 2013


On 09/02/2013 06:07 PM, Petr Viktorin wrote:
> On 08/29/2013 05:56 PM, Ana Krivokapic wrote:
>> Hello,
>>
>> This patch addresses ticket 
>> https://fedorahosted.org/freeipa/ticket/3867.
>>
>
> Patch works well.
> It's temping to restart the discussion about how to wrap text output 
> from installation tools. Wrapping at 60 characters because it looks 
> better in the code seems suboptimal.
> Does anyone remember if we established some guideline last time this 
> came up?
>
>
I'm not sure if I'm missing something, but do we need a guideline here?

I don't see any reason why not have best of the both worlds, using print 
as a function we can wrap the text inside the parenthesis
with no effect on the output whatsoever. Or use print statement, but 
enclose the text in parenthesis. Or use backslash.

-- 
Tomas Babej
Associate Software Engeneer | Red Hat | Identity Management
RHCE | Brno Site | IRC: tbabej | freeipa.org




More information about the Freeipa-devel mailing list