ACK.

--
Endi S. Dewata

On 2/18/2013 10:29 AM, Petr Vobornik wrote:
First patch adds just a devel tool usable in browser. Second is
foundation for specifying which dependency errors can be ignored in Web
UI build. Third is an usage of second.

These patches were develop as part of ongoing refactoring work but they
are independent and beneficial for all UI work so I'm sending them now.

1) [PATCH] Develop.js extended

Develop.js contains code useful only for debugging. It is not part of
FreeIPA release.

Is loaded by typing require(['freeipa/develop']); in browser JS console.
It adds IPA global variable and provide easier way of loading AMD
modules into window.ipadev[providedNameOrModuleName] variable.


2) [PATCH] Allow to specify modules for which builder doesn't raise
  dependency error

One can specify module ids provided by other means (already built layer
file) in providedMids array of build profile file's package section.
Builder then ignores dependency errors for specified modules. This
allows to build layers without source codes of their dependencies, with
no expected errors raised.

Example:

packages:[
     {
         name: "freeipa",
         location: "freeipa",
         providedMids: [
             'dojo/_base/declare',
             'dojo/_base/lang',
             'dojo/_base/array',
             'dojo/Stateful'
             //etc
         ]
     }
],

[PATCH] Web UI build profile updated

freeipa.profile was updated accordingly to contain all modules in dojo
layer.

This change removes expected errors during the build and therefore it
won't confuse others during rpm build. It also helps during development
because developer will notice real dependency errors (those not
specified this way).

_______________________________________________
Freeipa-devel mailing list
Freeipa-devel@redhat.com
https://www.redhat.com/mailman/listinfo/freeipa-devel

Reply via email to