Closed
Bug 2246
Opened 26 years ago
Closed 25 years ago
[PP] Powerplant removal
Categories
(Core Graveyard :: Viewer App, defect, P2)
Tracking
(Not tracked)
M16
People
(Reporter: mozeditor, Assigned: scc-obsolete)
Details
(Keywords: platform-parity)
We need to remove powerplant from seamonkey - at least from the non-app
libraries. I (Joe Francis) will kick things off by rewriting the timer code to
not use LPeriodical.
Anyone reading this should feel free to add new description to this bug report
describing other places we need to "unplant".
Reporter | ||
Updated•26 years ago
|
Status: NEW → ASSIGNED
Comment 1•26 years ago
|
||
adding build people to the cc list...
Comment 2•26 years ago
|
||
adding build people to the cc list...
Comment 3•26 years ago
|
||
joe, what is the schedule for doing this?
added JJ Enser to list.
the change as i understand it is fairly trivial. we just stop building the
powerplant library.
Reporter | ||
Comment 5•26 years ago
|
||
The schedule is "soon". Today or tomorrow.
Reporter | ||
Updated•26 years ago
|
Status: ASSIGNED → RESOLVED
Closed: 26 years ago
Resolution: --- → FIXED
Reporter | ||
Comment 6•26 years ago
|
||
Powerplant is gone. Affected parts of the tree are base, widget, netlib, and
viewer, and also the NGLayoutBuildList.pm script.
I have posted a note to n.p.m.mac about it, along with advice for those who were
using the pieces of PowerPlant that I just removed.
Comment 7•26 years ago
|
||
I'm not quite sure how QA can verify this. ;)
----
Simon/Steve/Mike/Chris/(et al ;), would one of you be open to marking this
verified, if you can definitively validate (or just vouch for? ;) that the MW
Powerplant stuff is totally out of Seamonkey?
Thanks!
Updated•26 years ago
|
Status: RESOLVED → VERIFIED
Comment 8•26 years ago
|
||
It's out. We no longer build it. Verified.
Reporter | ||
Comment 9•26 years ago
|
||
It depends on what one means by "totally out of".
There is a dependency on Powerplant header files (but not on the powerplant lib)
in viewer. There may be header dependencies in other places - don't know for
sure. But we don't need to build a powerplant library anymore - nothing links
against it.
Comment 10•26 years ago
|
||
Inserting Milestone info.
Updated•26 years ago
|
Status: VERIFIED → REOPENED
Comment 11•26 years ago
|
||
ftang wrote:
although we don't use powerplant anymore Caps.mcp don't build unless PowerPlant
source are installed (because xpassert.h include
PP_Type.h and UException.h). Can someone clean it up ?
Updated•26 years ago
|
Assignee: jfrancis → pierre
Status: REOPENED → NEW
Target Milestone: M2 → M4
Updated•26 years ago
|
Resolution: FIXED → ---
Comment 12•26 years ago
|
||
This also keeps us from building without exceptions.
Comment 13•26 years ago
|
||
adding scc to cc list
Updated•26 years ago
|
Assignee: pierre → scc
Comment 14•26 years ago
|
||
reassigned to scc
Comment 15•26 years ago
|
||
Actually, xpassert.h should not be included. You should be using nsDebug.h
instead. Nothing starting with xp should be in the build any more, including
xp_file and xpassert.
Assignee | ||
Updated•26 years ago
|
Status: NEW → ASSIGNED
Updated•26 years ago
|
Target Milestone: M4 → M5
Comment 16•26 years ago
|
||
targetting m5, cc ftang. where is xpassert.h being included in current code? who
should get this bug?
Comment 17•26 years ago
|
||
I reassigned this bug to scc last month because it was somewhat related to the
removal of C++ exceptions and he's already on top of that.
Assignee | ||
Updated•26 years ago
|
Target Milestone: M5 → M7
Assignee | ||
Comment 18•26 years ago
|
||
moving to M7 as per discussion with choffman
Assignee | ||
Updated•26 years ago
|
Target Milestone: M7 → M10
Assignee | ||
Comment 19•26 years ago
|
||
This is not high enough priority to delay fixing my other bugs for. I'm moving
this one out to M10 ... and the bulk of the work can't happen till the next
release of the CodeWarrior tools anyway.
Comment 20•25 years ago
|
||
Moving to M11 per today's bug triage.
Assignee | ||
Updated•25 years ago
|
Target Milestone: M11 → M16
Assignee | ||
Comment 21•25 years ago
|
||
No visible effect to users. Not dogfood.
Assignee | ||
Comment 22•25 years ago
|
||
Removing the last of the PowerPlant header files is now part of the `conversion
to Pro5' bug. Therefor, marking this bug a duplicate.
*** This bug has been marked as a duplicate of 25949 ***
Status: ASSIGNED → RESOLVED
Closed: 26 years ago → 25 years ago
Resolution: --- → DUPLICATE
Updated•16 years ago
|
Product: Core → Core Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•