[Freeipa-devel] [PATCH 0274] DNS: Check if dns package is installed
Petr Spacek
pspacek at redhat.com
Thu Jul 2 06:50:11 UTC 2015
On 1.7.2015 20:29, Tomas Babej wrote:
>
>
> On 07/01/2015 04:45 PM, Petr Spacek wrote:
>> On 1.7.2015 15:32, Martin Basti wrote:
>>> https://fedorahosted.org/freeipa/ticket/4058
>>> Requires patch freeipa-pspacek-0052
>>
>> ACK
>>
>
> I must admit I don't really like wrapping a constant in the method in
> the TaskNamespace object.
>
> We're interested in the constant itself - there's no case I can imagine
> where the name of the freeipa's dns package will be dynamic.
>
> For paths we have BasePathNamespace that contains all the paths, maybe
> we should introduce something similar for the non-path platform
> dependent constants?
Generally I support this but it seems like a 4.3 material (and out of scope of
#4058). We need to finish 4.2 now.
Please ACK or NACK ASAP.
--
Petr^2 Spacek
More information about the Freeipa-devel
mailing list