Closed
Bug 49240
Opened 24 years ago
Closed 24 years ago
getenv support for Mac needed in release builds
Categories
(SeaMonkey :: General, defect, P4)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: jband_mozilla, Assigned: beard)
Details
(Whiteboard: [nsbeta3+][Check in per PDT mtg])
Attachments
(1 file)
306 bytes,
patch
|
Details | Diff | Splinter Review |
the xpti typelib loader is going to use environments vars to turn on writing
stats to disk. This is useful for web and plugin developers in debug and release
builds. All platforms except Mac can read these vars in release builds.
Assignee | ||
Comment 1•24 years ago
|
||
This is really easy to enable. I am not sure of the potential risks. I will
enclose a patch that enables this.
Assignee | ||
Comment 2•24 years ago
|
||
Assignee | ||
Comment 3•24 years ago
|
||
This is a trivial fix. But I need a policy decision on this. Is it safe for us to
have code paths in the optimized product that won't get tested unless certain
environment variables are set in an ENVIRONMENT file?
Status: NEW → ASSIGNED
Keywords: nsbeta3
Comment 4•24 years ago
|
||
We've used ENV variables to control logging on 4.x, at least on Windows and Unix,
so as long as it's for diagnostic purposes and not feature control I don't see a
problem.
Assignee | ||
Comment 5•24 years ago
|
||
Will go ahead and check this patch in.
Priority: P3 → P4
Whiteboard: [nsbeta3+]
beard, please check in or minus this P4 bug...thanks!
Whiteboard: [nsbeta3+] → [nsbeta3+][Check in per PDT mtg]
Assignee | ||
Comment 7•24 years ago
|
||
Done.
Status: ASSIGNED → RESOLVED
Closed: 24 years ago
Resolution: --- → FIXED
Comment 8•24 years ago
|
||
beard, can you verify this or update the QA Contact. thanks.
Updated•20 years ago
|
Product: Browser → Seamonkey
You need to log in
before you can comment on or make changes to this bug.
Description
•