[Freeipa-devel] [PATCH 0045] Add message for skipping NTP configuration during client install

[Martin Basti]

On 30/03/15 15:25, Gabe Alford wrote:
> Hello,
>
> With the merging of ticket 4842 
> <https://fedorahosted.org/freeipa/ticket/4842>, I believe that half of 
> ticket 3092 <https://fedorahosted.org/freeipa/ticket/3092> has been 
> done. This patch just adds a message that says that NTP configuration 
> was skipped which I believe should finish 3092 
> <https://fedorahosted.org/freeipa/ticket/3092>.
>
> Thanks,
>
> Gabe
>
>
Hello, thank you for the patch.

1)
IMO there should be:
if *not* options.conf_ntp

2)
wouldnt be better to use just else?

Martin

-- 
Martin Basti

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://listman.redhat.com/archives/freeipa-devel/attachments/20150402/68917a95/attachment.htm>