151574 - Add MRE packaging make target to embedding/config

Status: VERIFIED FIXED

(Core Graveyard :: Installer: GRE, defect)

Description

[Chak Nanga]

We now have the MRE infrastructure bugs checked into the trunk. A user now can
write an embedding app to be run in an MRE based env.

Earlier this week, I discussed with Adam and Chris about adding an MRE packaging
step to the existing embedding/config/makefile. Adding this MRE packaging step
will take us closer to having true MRE installers down the road - all we may
need to do is to have an xpi wrappers around the generated MRE packages.

Here's a summary of *what* needs to happen:

When the user runs "nmake -f makefile MRE" we do the following:

   * Create a directory named "MRE" under dist which contains the base embedding
files as defined basebroser-*. So, we'll have the "components" dir with all the
files under it, a "plugins" dir with the nul plugin, and the other dlls which
get copied under "MRE" (xpcom.dll, zlib.dll, mozreg.dll, nss*.dll etc)
   * Create a directory named "MreAppSupport" (i'm open to suggestions on a
different name). This will have the default support files/directories the user
needs in order to run in an MRE based env. It should have the "chrome",
"defaults", "res" directories and an empty "components" and "plugins" dir
indicating to the user that app specific plugins and components must be places
in those directories.


Now we need decide *how* to fix this:

Since all that needs to happend here is to separate out contents of the existing
"Embed" dir, building the "MRE" target can:
   * Generate "Embed" if one is not present
   * Write a new script to separate out the "Embed" dir into "MRE" and
"MreAppSupport" directories as described above.

Comment 1

[Chak Nanga]

Attachment: Patch to build mre and mre_app_support packages

Comment 2

[Adam Lock]

Comment on attachment 88183 [details] [diff] [review]
Patch to build  mre and mre_app_support packages

r=adamlock

might be better to enclose the mre rule entirely by the ifeq WINNT section
instead of just the actions.

Comment 3

[Chak Nanga]

Attachment: Updated patch to enclose the complete MRE rule per Adam's comments

Comment 4

[Darin Fisher]

Comment on attachment 88184 [details] [diff] [review]
Updated patch to enclose the complete MRE rule per Adam's comments

sr=darin

Comment 5

[Chak Nanga]

Patch checked in to the trunk. Thank you all for the help.

Comment 6

[Charles Rosendahl]

Verified