On 11/21/2013 05:14 PM, Martin Basti wrote:
On Tue, 2013-11-19 at 10:32 +0100, Petr Viktorin wrote:
On 10/21/2013 10:29 AM, Martin Basti wrote:
On Mon, 2013-10-21 at 09:29 +0200, Martin Kosek wrote:
On 10/18/2013 05:00 PM, Martin Basti wrote:
I did not test the patch, just looked at the code and I have few comments:
1) Please put the ipalib/cli.py change to a sepparate change, we may want to
backport it separately some day and this will make it easier.
Patch 0014-2 -- CLI
Nitpick: if you switched them around it would be easier to read*:
*(at least for me, I need way too much concentration to process boolean
Patch 0016 -- migration
We have a utility, ipautil.write_tmp_file, that should do what you need
With this the created file is removed some time later when there are no
more references to the file object, so no need for the try: block.
(btw, if the try block was necessary, it should have also covered the
Also, since you changed the API, please run ./makeapi and bump the API
version in the VERSION file.
Thank you for review
Updated patches attached
Thanks! ACK, pushed to master: efffcfdbc24ad81d48c7b65443a75928b156cc49
Freeipa-devel mailing list