Closed
Bug 3554
Opened 26 years ago
Closed 26 years ago
Unix: MOZILLA_HOME needs to get yanked out of nspr
Categories
(NSPR :: NSPR, defect, P1)
Tracking
(Not tracked)
People
(Reporter: mcafee, Assigned: mcafee)
Details
MOZILLA_HOME is conflicting with Netscape 4.5.
Dp & ramiro suggest moving to a new variable,
MOZILLA_FIVE_HOME.
Updated•26 years ago
|
Status: NEW → ASSIGNED
Comment 2•26 years ago
|
||
I was thinking of using MOZILLA5_HOME
Ramiro, Mcafee, Do you both prefer MOZILLA_FIVE_HOME
As I typed MOZILLA5 i noticed it looks a lot like MOZILLAS (S instead of 5)
MOZILLA_FIVE_HOME is very obvious and i think will work better. I think the
extra chars are worth the fact is obvious we are talking about MOZILLA FIVE.
dp, are you ready to make this change ? Go for it if you are and ill post an
announcement if you want.
-re
Assignee | ||
Comment 4•26 years ago
|
||
MOZILLA_5_HOME ? I agree with ramiro about the misreading of MOZILLA5
Comment 5•26 years ago
|
||
I will do component loading from MOZILLA_FIVE_HOME and then if that failed
./components
Then ramiro will do some more for other parts of the app. I will reassign to
ramiro once I am done
Updated•26 years ago
|
Status: ASSIGNED → RESOLVED
Closed: 26 years ago
Resolution: --- → FIXED
Comment 6•26 years ago
|
||
Using MOZILLA_FIVE_HOME now.
Assignee | ||
Updated•26 years ago
|
Status: RESOLVED → REOPENED
Assignee | ||
Comment 8•26 years ago
|
||
Reopening, there is still one occurance of MOZILLA_HOME in nspr,
http://lxr.mozilla.org/seamonkey/source/nsprpub/pr/src/linking/prlink.c#331
Wan-Teh, can we include a test for MOZILLA_FIVE_HOME in front of
MOZILLA_HOME here?
Updated•26 years ago
|
Assignee: dp → mcafee
Status: REOPENED → NEW
Comment 9•26 years ago
|
||
My part of components directory using MOZILLA_FIVE_HOME is done. hence I am
forwarding bug to err... mcafee
Assignee | ||
Updated•26 years ago
|
Assignee: mcafee → wtc
Assignee | ||
Comment 10•26 years ago
|
||
Wan-Teh, I'm assigning this to you so you can take a look at this.
We've recently switched from MOZILLA_HOME to MOZILLA_FIVE_HOME,
for all Unix versions for 5.0. I believe we need to check for
this variable ahead of MOZILLA_HOME in prlink.c.
Assignee | ||
Updated•26 years ago
|
Component: Apprunner → NSPR
Updated•26 years ago
|
Assignee: wtc → srinivas
Comment 11•26 years ago
|
||
(NSPR bugs should be assigned to
the module owner Srinivas.)
I think we should not make the
change to prlink.c that you
suggested. NSPR should not check
MOZILLA_HOME or MOZILLA_FIVE_HOME.
These are environment variables
used only by specific products.
I would even remove the existing
check for MOZILLA_HOME from prlink.c.
Instead, an NSPR client should use
PR_GetLibraryPath() and PR_SetLibraryPath()
to add the directories it's interested in
to the dynamic library search path.
Updated•26 years ago
|
QA Contact: 4078
Assignee | ||
Updated•26 years ago
|
Resolution: FIXED → ---
Comment 12•26 years ago
|
||
NSPR is a general purpose runtime library and shouldn't process product-specific
environment variables. The current reference to MOZILLA_HOME in
nspr should be removed. This can be done when the relevant changes (i.e.
explicitly set the library path) are made in the mozilla code.
I am reassigning this bug mcafee.
Assignee | ||
Comment 13•26 years ago
|
||
Are any non-mozilla products using MOZILLA_HOME? Server?
Assignee | ||
Comment 14•26 years ago
|
||
Are any non-mozilla products using MOZILLA_HOME? Server?
Comment 15•26 years ago
|
||
No, servers don't use MOZILLA_HOME.
Comment 16•26 years ago
|
||
I agree with wtc and srinivas.
I suggest you comment out this code in NSPR. And when the client breaks, we
will fix it.
I still dont know what in 4.x needed that code in NSPR. Anyone know ?
Comment 17•26 years ago
|
||
Is this really a showstopper for M3? If it can wait til m4, please re-target.
Updated•26 years ago
|
Target Milestone: M3 → M4
Comment 18•26 years ago
|
||
its not. marking m4.
Assignee | ||
Updated•26 years ago
|
Summary: Unix: MOZILLA_HOME is conflicting with Netscape 4.5 → Unix: MOZILLA_HOME needs to get yanked out of nspr
Updated•26 years ago
|
Target Milestone: M4 → M5
Comment 19•26 years ago
|
||
moving to m5
Assignee | ||
Updated•26 years ago
|
Status: NEW → RESOLVED
Closed: 26 years ago → 26 years ago
Resolution: --- → DUPLICATE
Assignee | ||
Comment 20•26 years ago
|
||
I think this is a dup of 4450:
http://bugzilla.mozilla.org/show_bug.cgi?id=4450
*** This bug has been marked as a duplicate of 4450 ***
Comment 21•26 years ago
|
||
verified.
Comment 22•26 years ago
|
||
NSPR now has its own Bugzilla product. Moving this bug to the NSPR product.
Updated•25 years ago
|
Target Milestone: M5 → ---
You need to log in
before you can comment on or make changes to this bug.
Description
•