Closed Bug 177573 Opened 22 years ago Closed 22 years ago

chimera.js is interpreted before all.js

Categories

(Camino Graveyard :: Preferences, defect)

PowerPC
macOS
defect
Not set
normal

Tracking

(Not tracked)

VERIFIED FIXED

People

(Reporter: ccarlen, Assigned: sfraser_bugs)

Details

I think that a product-specific prefs file like chimera.js should be able to override prefs in all.js. Currently, because of the way the interpreting of pref default files is ordered, we get (from some prints inserted): openPrefFile: opening /Users/conrad/Development/CHIMERA/mozilla/chimera/build/Navigator.app/Contents/MacOS/defaults/pref/initpref.js openPrefFile: opening /Users/conrad/Development/CHIMERA/mozilla/chimera/build/Navigator.app/Contents/MacOS/defaults/pref/security-prefs.js openPrefFile: opening /Users/conrad/Development/CHIMERA/mozilla/chimera/build/Navigator.app/Contents/MacOS/defaults/pref/chimera.js openPrefFile: opening /Users/conrad/Development/CHIMERA/mozilla/chimera/build/Navigator.app/Contents/MacOS/defaults/pref/all.js openPrefFile: opening /Users/conrad/Development/CHIMERA/mozilla/chimera/build/Navigator.app/Contents/MacOS/defaults/pref/debug-developer.js There is a way around this: See http://lxr.mozilla.org/seamonkey/source/modules/libpref/src/nsPrefService.cpp#495 I think we should rename chimera.js -> chimera-all.js
Fixing summary.
Summary: chimera.js is interpreted after all.js → chimera.js is interpreted before all.js
I'm trying to understand how "all.js" can sort after "chimera.js"
Look at the code in the lxr link - It's just an inverted sort.
They reverse sort just to fix the '.' vs '-' problem? That's insane.
I thought they meant that "-" meant "modifier" and so foo-bar.js should come after foo.js. Still, inverting the whole sort to accomplish that is wacky. But, we need some reasonable way of ordering these so that we can override things in all.js.
I have to fix this for favicons. I'll rename it to all-chimera.js.
Assignee: ccarlen → sfraser
Done. chimera.js was renamed to all-chimera.js. Stupid prefs code.
Status: NEW → RESOLVED
Closed: 22 years ago
Resolution: --- → FIXED
yep. vrfy'd with 2002.11.14.04.
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.