Closed
Bug 13732
Opened 26 years ago
Closed 25 years ago
Remove brprof stuff before beta
Categories
(Core Graveyard :: RDF, defect, P2)
Core Graveyard
RDF
Tracking
(Not tracked)
VERIFIED
FIXED
M14
People
(Reporter: mozilla, Assigned: dp)
Details
(Whiteboard: [PDT+])
We should probably remove the brprof stuff before Beta 1, right?
Reporter | ||
Updated•26 years ago
|
Priority: P3 → P2
Target Milestone: M12
Updated•26 years ago
|
Status: NEW → ASSIGNED
Updated•26 years ago
|
Target Milestone: M12 → M15
Due to Beta indication in Summary, putting beta1 into keyword field.
Keywords: beta1
This seems like a reasonable request; marking PDT+.
Whiteboard: [PDT+]
Updated•25 years ago
|
Target Milestone: M15 → M14
Comment 3•25 years ago
|
||
dp, i'm gonna kick this up to you. it's a 20 minute exercise in build whackage.
Assignee: waterson → dp
Status: ASSIGNED → NEW
Assignee | ||
Updated•25 years ago
|
Status: NEW → ASSIGNED
Assignee | ||
Updated•25 years ago
|
Whiteboard: [PDT+] → [PDT+] 2/14
Comment 4•25 years ago
|
||
brprof.dll was the "browsing profile" stuff that Guha wanted to do right before
he ended up leaving.
The easiest way to do this would be just to cut this out of the build. Modify:
mozilla/rdf/Makefile.in
mozilla/rdf/makefile.win
mozilla/build/mac/NGLayoutBuildList.pm
to not descend into brprof
mozilla/allmakefiles.sh
to not generate Makefile from Makefile.in
There should be absolutely no need for this stuff in the code (at least that's
what LXR says), but it'd be worth doing a clobber build just to make sure ;-).
We can leave this in the tree if you want, but if you're feeling really full of
bravado, go ahead and "cvs remove" it.
Thanks a ton, dp!
Assignee | ||
Comment 5•25 years ago
|
||
Unhooking brprof from the build is what needs to be done. Will do. Might need
scc's help on the mac.
Assignee | ||
Comment 6•25 years ago
|
||
Removed it from all builds. I didn't cvs remove the directory. After
verification monday, we can cvs remove brprof directory.
Waterson needs to give the final goahead on the cvs removal.
Comment 7•25 years ago
|
||
Fire at will, dp.
Not sure WHY this went off PDT+, beta1. Putting backon to verify fix. Is this
fixed now?
Keywords: beta1
Whiteboard: [PDT+]
Assignee | ||
Comment 10•25 years ago
|
||
Yeah. I have one last approved change to remove it from the packager.
Assignee | ||
Comment 11•25 years ago
|
||
Removed from build and package files.
Status: ASSIGNED → RESOLVED
Closed: 25 years ago
Resolution: --- → FIXED
Comment 12•25 years ago
|
||
brprof not included in package
marking verified:
windows NT 2000022708
Linux 2000022708
Mac 2000022908
Status: RESOLVED → VERIFIED
Updated•7 years ago
|
Product: Core → Core Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•