Check .autoreg in the GRE directory, too

RESOLVED FIXED in mozilla1.9.1b2

Status

()

Core
XPCOM
RESOLVED FIXED
10 years ago
9 years ago

People

(Reporter: glandium, Assigned: glandium)

Tracking

Trunk
mozilla1.9.1b2
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

10 years ago
Created attachment 321999 [details] [diff] [review]
patch

When using xulrunner applications, xpcom current process directory and GRE directory are different, and may contain things that are not updated synchronously, and one may need to trigger autoregistration of components on update of one or the other.

The attached patch implements checking in both GRE_DIR and XPCOM_CURRENT_PROCESS_DIR (skipping second check when it is the same directory).
Attachment #321999 - Flags: review?(benjamin)

Comment 1

10 years ago
What failure case are you trying to solve? XR apps should be blowing away all component registration whenever the buildid of the toolkit or the application changes, see http://mxr.mozilla.org/mozilla/source/toolkit/xre/nsAppRunner.cpp#2135
(Assignee)

Comment 2

10 years ago
In debian, and maybe in other linux distributions, we separate some components and put them in different packages (such as pyxpcom or mozgnome) to avoid spurious dependencies. When installing these extra components (that can't really be put as extensions, because not all applications enable the extensions manager, but we want all applications to have these components available, and for pyxpcom, it's actually mandatory), we trigger components autoregistration by touching .autoreg. The buildid doesn't change since we install new components on the same build.

Updated

10 years ago
Attachment #321999 - Flags: review?(benjamin) → review+

Updated

10 years ago
Component: XPCOM Registry → XPCOM
Assignee: nobody → mh+mozilla
Keywords: checkin-needed
http://hg.mozilla.org/mozilla-central/rev/49b6ae6afcc4
Status: NEW → RESOLVED
Last Resolved: 9 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → mozilla1.9.1b2
You need to log in before you can comment on or make changes to this bug.