On 28/04/16 14:45, Jan Cholasta wrote:
Hi,
I have pushed my thin client WIP branch to GitHub:
<https://github.com/jcholast/freeipa/tree/trac-4739>.
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).
Honza
Hi,
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:
https://www.redhat.com/mailman/listinfo/freeipa-devel
Contribute to FreeIPA: http://www.freeipa.org/page/Contribute/Code