<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN"> <html> <head> <meta content="text/html; charset=ISO-8859-1" http-equiv="Content-Type"> </head> <body bgcolor="#ffffff" text="#000000"> On 01/12/2011 05:17 PM, Rob Crittenden wrote: <blockquote cite="mid:4D2E2868.2050808@redhat.com" type="cite">Rob Crittenden wrote: <blockquote type="cite">Simo Sorce wrote: <blockquote type="cite">On Wed, 12 Jan 2011 11:03:31 -0500 Rob Crittenden<a class="moz-txt-link-rfc2396E" href="mailto:rcritten@redhat.com"><rcritten@redhat.com></a> wrote: <blockquote type="cite">Add an API version that is enforced both when the server is built (to disallow unexpected API changes) and when clients talk to the server. See the patch for further details. ticket 584 rob </blockquote> Technical nack, API.txt is missing. Also it would be really nice if we could test the API on build. Id the utf8 plugin thing really needed in order to test the api, or could we use PYTHON_PATHY and a fake python module by the same name just to get through the import w/o getting an exception out ? Simo. </blockquote> Updated patch attached. I worked around requiring the utf8 plugin so this should be executable from a fresh git pull and will always validate the api when building. I also made a separate patch for the mozldap dependency change (675). rob </blockquote> Moved the changes to daemons/configure.ac to patch 675. rob <pre wrap=""> <fieldset class="mimeAttachmentHeader"></fieldset> _______________________________________________ Freeipa-devel mailing list <a class="moz-txt-link-abbreviated" href="mailto:Freeipa-devel@redhat.com">Freeipa-devel@redhat.com</a> <a class="moz-txt-link-freetext" href="https://www.redhat.com/mailman/listinfo/freeipa-devel">https://www.redhat.com/mailman/listinfo/freeipa-devel</a></pre> </blockquote> Am I correct in understanding that this patch is going to break the webUI? We need to put an API version into the JSON RPC as well. This would involve a minor change to install/static/ipa.js in the ipa_cmd function, but I am not quite sure what to put there. </body> </html>