On 28/04/16 14:45, Jan Cholasta wrote:

I have pushed my thin client WIP branch to GitHub:

All commits up to "ipalib: use relative imports for cross-plugin
imports" should be good for review. The rest is subject to change
(WARNING: I will force push into this branch).


I have fought my way through the patch set up to "frontend: allow commands to have an argument named `name`" (including). The code looks good to me but I haven't dive into parts that was pure Cut&Paste. I assume that when it worked before it will still work.

Some problems pop out in tests:

user_plugin: http://paste.fedoraproject.org/367502/34888591/
This can be fixed with simple patch: http://paste.fedoraproject.org/367503/14634889/

With patch applied there are still some errors: http://paste.fedoraproject.org/367519/49007414/

David Kupka

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

Reply via email to