Closed Bug 49240 Opened 24 years ago Closed 24 years ago

getenv support for Mac needed in release builds

Categories

(SeaMonkey :: General, defect, P4)

PowerPC
Mac System 8.0
defect

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: jband_mozilla, Assigned: beard)

Details

(Whiteboard: [nsbeta3+][Check in per PDT mtg])

Attachments

(1 file)

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.
This is really easy to enable. I am not sure of the potential risks. I will enclose a patch that enables this.
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
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.
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]
Done.
Status: ASSIGNED → RESOLVED
Closed: 24 years ago
Resolution: --- → FIXED
beard, can you verify this or update the QA Contact. thanks.
Product: Browser → Seamonkey
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: