<!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 text="#000000" bgcolor="#ffffff"> On 03/28/2011 04:56 PM, Adam Young wrote: <blockquote cite="mid:4D90F604.2070609@redhat.com" type="cite">To give a little more context: we are llong to split out the logic used to define the views of the entities from the reusable portion of the toolkit. This patch introduces a builder object which contains the temporary state of the entity build process. In the course of writing it, I realized a few things: 1. HBAC and SUDO have two small entities and a single large one. Thus, it makes sense to group them both into a single file per entity. Both hbac.js and sudo.js should shrink more in the future as the custom code gets better refactored and split into reusable components and configuration data. 2. policy.js was a catch all file. Automount will grow significantly this release, and so should have its own file. DNS is complicated enough that it deserves its own top level js file. policy is now reduced to two small entities, both that are very clearly policy. <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> Self NACK: jsl and unit test errors need to be fixed first. Still worth reviewing as is, as fixing that will not change the behavior or structure of the end patch. </body> </html>