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

Manage your subscription for the Freeipa-devel mailing list:
Contribute to FreeIPA: http://www.freeipa.org/page/Contribute/Code

Reply via email to