[Freeipa-devel] [PATCHES] 0483-0485 Move ipalib.text to ipapython

Petr Viktorin pviktori at redhat.com
Fri Feb 28 16:49:01 UTC 2014


Hello!

This moves ipalib.text to ipapython.
Why do we want this?

Firstly, it's a step towards breaking the ipapython dependency on 
ipalib, which is something we vaguely want to do in the long run for the 
sake of clean code and potential reuse.

But there's another reason: The DNS plugin defines some validators that 
are used elsewhere. I'd like to eventually move them to ipapython, so 
the DNS plugin can be made optional 
(https://fedorahosted.org/freeipa/ticket/4058).
The validators use on ipalib.text, so that has to be moved to ipapython 
(where I think it belongs).

The gettext wrappers in turn depend on the context, which I'd rather see 
in ipalib, but it's literally one line of code so it's not a big burden 
to have in ipapython. In the future we can think about somehow 
extracting it and moving it back, if needed.


(And the first patch is just some general cleanup.)

-- 
Petr³
-------------- next part --------------
A non-text attachment was scrubbed...
Name: freeipa-pviktori-0483-ipalib.util-Remove-unused-function-validate_rdn_para.patch
Type: text/x-patch
Size: 941 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/freeipa-devel/attachments/20140228/499ab8bd/attachment.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: freeipa-pviktori-0484-Move-request.context-to-ipapython.patch
Type: text/x-patch
Size: 2331 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/freeipa-devel/attachments/20140228/499ab8bd/attachment-0001.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: freeipa-pviktori-0485-Move-ipalib.text-to-ipapython.text.patch
Type: text/x-patch
Size: 79555 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/freeipa-devel/attachments/20140228/499ab8bd/attachment-0002.bin>


More information about the Freeipa-devel mailing list