Closed
Bug 334877
Opened 19 years ago
Closed 18 years ago
move non-prefwindow xpfe/components chrome to suite/
Categories
(SeaMonkey :: Build Config, defect)
SeaMonkey
Build Config
Tracking
(Not tracked)
RESOLVED
FIXED
seamonkey2.0a1
People
(Reporter: kairo, Assigned: kairo)
References
Details
Attachments
(2 files, 3 obsolete files)
(deleted),
patch
|
neil
:
review+
neil
:
superreview+
|
Details | Diff | Splinter Review |
(deleted),
text/plain
|
Details |
After prefwindow chrome is moved to suite/ by bug 333383, I'll do the move of the rest of xpfe/components chrome here
Assignee | ||
Comment 1•19 years ago
|
||
Here's the jar.mn part of this, all remaining communicator chrome of xpofe/components gets moved to suite/ - with the exception of the old alerts and update services, which are obsoleted by suiterunner anyways. Even though the moved files are used by both old xpfe-based suite and suiterunner, it probably makes little sense to move files that are obviously killed soon anyways.
The patch is against files already changed by bug bug 333383 already, so that only differences to that are shown.
The cvsmoves file is coming up soon.
Attachment #219282 -
Flags: superreview?(neil)
Attachment #219282 -
Flags: review?(neil)
Assignee | ||
Comment 2•19 years ago
|
||
Those are the files that need to be moved/copied - again, I'll open a cvs moves bug once I got reviews so I know the new locations of those files are correct.
Attachment #219284 -
Flags: superreview?(neil)
Attachment #219284 -
Flags: review?(neil)
Assignee | ||
Comment 3•19 years ago
|
||
While generating the next cvsmoves file, I realized I used locales/chrome here instead of locales/en-US/chrome - sorry!
Attachment #219284 -
Attachment is obsolete: true
Attachment #219327 -
Flags: superreview?(neil)
Attachment #219327 -
Flags: review?(neil)
Attachment #219284 -
Flags: superreview?(neil)
Attachment #219284 -
Flags: review?(neil)
Assignee | ||
Comment 4•19 years ago
|
||
This has suffered bitrot due to various other checkins (mainly my own ones), here's an un-bitrotted version (and real cvs diff now, as the prefwindow move is already in).
Attachment #219282 -
Attachment is obsolete: true
Attachment #222401 -
Flags: superreview?(neil)
Attachment #222401 -
Flags: review?(neil)
Attachment #219282 -
Flags: superreview?(neil)
Attachment #219282 -
Flags: review?(neil)
Updated•19 years ago
|
Attachment #219327 -
Flags: superreview?(neil)
Attachment #219327 -
Flags: superreview+
Attachment #219327 -
Flags: review?(neil)
Attachment #219327 -
Flags: review+
Updated•19 years ago
|
Attachment #222401 -
Flags: superreview?(neil)
Attachment #222401 -
Flags: superreview+
Attachment #222401 -
Flags: review?(neil)
Attachment #222401 -
Flags: review+
Assignee | ||
Comment 5•18 years ago
|
||
Comment on attachment 219327 [details] [diff] [review]
cvsmoves for communicator components chrome
This file has a small glitch in the directory.dtd line, I'll attach a corrected version to the cvs copy bug.
Attachment #219327 -
Attachment is obsolete: true
Assignee | ||
Comment 6•18 years ago
|
||
Comment on attachment 222401 [details] [diff] [review]
jar.mn changes for moving relevant components chrome, unbitrotted
Checked in.
Leaving this open until I'm sure it works and I have cvs removed those files that are unused after that step.
Assignee | ||
Comment 7•18 years ago
|
||
This is the list of files that were obsoleted by this bug, and were cvs removed a moment ago.
Assignee | ||
Comment 8•18 years ago
|
||
FIXED, finally ;-)
Status: NEW → RESOLVED
Closed: 18 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•