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 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.
Attachment #88183 - Flags: review+
Created attachment 88184 [details] [diff] [review] Updated patch to enclose the complete MRE rule per Adam's comments
Attachment #88183 - Attachment is obsolete: true
Comment on attachment 88184 [details] [diff] [review] Updated patch to enclose the complete MRE rule per Adam's comments sr=darin
Attachment #88184 - Flags: superreview+
Patch checked in to the trunk. Thank you all for the help.
Status: NEW → RESOLVED
Last Resolved: 17 years ago
Resolution: --- → FIXED
Status: RESOLVED → VERIFIED
Component: Embedding: APIs → Installer: GRE
You need to log in before you can comment on or make changes to this bug.