Closed
Bug 191882
Opened 23 years ago
Closed 23 years ago
libxpcom.bundle is unused yet contributes 1 meg to footprint
Categories
(Core :: XPCOM, defect)
Tracking
()
RESOLVED
FIXED
People
(Reporter: ccarlen, Assigned: ccarlen)
Details
Attachments
(1 file)
|
1.03 KB,
patch
|
netscape
:
review+
dougt
:
superreview+
asa
:
approval1.3b+
|
Details | Diff | Splinter Review |
The building of libxpcom.bundle was checked in from discussion on bug 186599.
That lib is unused because using it for GRE didn't work, yet it's included with
every nightly build. 1 megabyte of junk.
Comment 1•23 years ago
|
||
-> xpcom
Assignee: seawood → dougt
Component: Build Config → XPCOM
QA Contact: granrose → scc
| Assignee | ||
Comment 2•23 years ago
|
||
| Assignee | ||
Updated•23 years ago
|
Attachment #113548 -
Flags: review?(seawood)
Comment 3•23 years ago
|
||
Comment on attachment 113548 [details] [diff] [review]
patch
The makefile changes are fine. What are we doing for the GRE instead of using
the .bundle?
Attachment #113548 -
Flags: superreview?(dougt)
Attachment #113548 -
Flags: review?(seawood)
Attachment #113548 -
Flags: review+
| Assignee | ||
Comment 4•23 years ago
|
||
See bug 191270.
Comment 5•23 years ago
|
||
Comment on attachment 113548 [details] [diff] [review]
patch
sr=dougt
cls - thanks for quicking adding, then weeks later, quickly removing, this
support.
Attachment #113548 -
Flags: superreview?(dougt) → superreview+
| Assignee | ||
Comment 6•23 years ago
|
||
Comment on attachment 113548 [details] [diff] [review]
patch
for a= request: libxpcom.bundle is included in the Mach-O package but it's not
used and is 1 meg in size.
Attachment #113548 -
Flags: approval1.3b?
Comment 7•23 years ago
|
||
Comment on attachment 113548 [details] [diff] [review]
patch
a=asa (in behalf of drivers) for checkin to 1.3beta.
Attachment #113548 -
Flags: approval1.3b? → approval1.3b+
| Assignee | ||
Comment 9•23 years ago
|
||
Fix checked in. Hopefully, the machine which builds nightlies deletes dist or
clobbers.
Status: NEW → RESOLVED
Closed: 23 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•