On 03/28/2011 04:56 PM, Adam Young wrote:
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.
_______________________________________________
Freeipa-devel mailing list
Freeipa-devel@redhat.com
https://www.redhat.com/mailman/listinfo/freeipa-devel
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.
_______________________________________________
Freeipa-devel mailing list
Freeipa-devel@redhat.com
https://www.redhat.com/mailman/listinfo/freeipa-devel