Closed
Bug 190150
Opened 22 years ago
Closed 22 years ago
filenames too long for Mac CFM build
Categories
(Other Applications :: ChatZilla, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: myk, Assigned: rginda)
Details
(Keywords: smoketest)
Attachments
(1 file)
|
267.12 KB,
patch
|
Details | Diff | Splinter Review |
peterv says: That leaves the chatzilla bustage. A quick hacky fix would be to check in the four files mozilla/extensions/irc/xul/content/pref-irc-appearance-previewCSS.xul mozilla/extensions/irc/xul/content/pref-irc-appearance-previewCSS.js mozilla/extensions/irc/xul/content/pref-irc-appearance-previewCSS.html mozilla/extensions/irc/xul/locale/en-US/pref-irc-appearance-previewCSS.dtd with shorter names (pref-irc-appearance-prev.*) and correct the jar.mn like attached. ccarlen says: Giving them shorter names isn't a quick and hacky fix - I think it's the only way to go. Mac CVS Pro will shorten any file name that it checks out to 31 chars, chopping out the middle and inserting an asterisk. So, any file that may be referenced by a Mac CFM build cannot have a name > 31 chars.
| Reporter | ||
Comment 1•22 years ago
|
||
This was checked in last night by rginda. Rob- Can you take a look at this?
| Assignee | ||
Comment 2•22 years ago
|
||
Yes, I'm looking at this now. I have a patch almost ready, which will move all pref files into a prefpanel/ subdirectory, and hack off pref-irc- from all of the filenames.
Status: NEW → ASSIGNED
| Assignee | ||
Comment 3•22 years ago
|
||
move content/pref-irc* into content/prefpanel/, and strip the pref-irc- prefix. also consolidate the pref-irc-*.dtd files from locale/en-US into a single pref-irc.dtd.
| Assignee | ||
Updated•22 years ago
|
Attachment #112320 -
Flags: review?(peterv)
| Assignee | ||
Comment 4•22 years ago
|
||
checked in
Status: ASSIGNED → RESOLVED
Closed: 22 years ago
Resolution: --- → FIXED
Updated•22 years ago
|
Attachment #112320 -
Flags: review?(peterv)
| Reporter | ||
Updated•20 years ago
|
Product: Core → Other Applications
You need to log in
before you can comment on or make changes to this bug.
Description
•