Closed
Bug 385759
Opened 18 years ago
Closed 18 years ago
upload breakpad symbols from fx-linux-tbox and fxnewref-linux-tbox
Categories
(Webtools Graveyard :: Tinderbox, defect)
Webtools Graveyard
Tinderbox
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: ted, Assigned: ted)
Details
Attachments
(2 files)
1008 bytes,
patch
|
preed
:
review+
|
Details | Diff | Splinter Review |
1.14 KB,
patch
|
preed
:
review+
|
Details | Diff | Splinter Review |
Now that we have Breakpad on Linux, we need to get symbols from the Linux builds.
Updated•18 years ago
|
Assignee: ted.mielczarek → ccooper
Comment 2•18 years ago
|
||
Keys are installed in the standard location.
Assignee: ccooper → ted.mielczarek
Status: ASSIGNED → NEW
Assignee | ||
Comment 3•18 years ago
|
||
We might as well get fxnewref-linux-tbox prepped so it's ready when preed flips the switch to make it official.
Summary: upload breakpad symbols from fx-linux-tbox → upload breakpad symbols from fx-linux-tbox and fxnewref-linux-tbox
Assignee | ||
Comment 4•18 years ago
|
||
Attachment #269692 -
Flags: review?
Assignee | ||
Updated•18 years ago
|
Attachment #269692 -
Flags: review? → review?(preed)
Comment 5•18 years ago
|
||
Comment on attachment 269692 [details] [diff] [review]
upload breakpad symbols from fx-linux-tbox
This looks fine; you just checking it in on trunk, or on feature_newref, too?
Attachment #269692 -
Flags: review?(preed) → review+
Assignee | ||
Comment 6•18 years ago
|
||
I forgot the branchname for newref, but if you're cool with that, I will. I will have to put in an EXTRA_BUILDID env var like we did for the textframe builds, to avoid build id collisions.
Assignee | ||
Comment 7•18 years ago
|
||
Comment on attachment 269692 [details] [diff] [review]
upload breakpad symbols from fx-linux-tbox
Checked in on trunk, I'll get branch later.
Assignee | ||
Comment 8•18 years ago
|
||
coop, does fxnewref-linux-tbox have the ffxbld key? If not, can you put it there?
This patch does pretty much the exact same thing, but it also sets MOZ_SYMBOLS_EXTRA_BUILDID=newref to avoid collisions. That can be removed when this becomes the official nightly machine.
Attachment #271108 -
Flags: review?(preed)
Comment 9•18 years ago
|
||
Comment on attachment 271108 [details] [diff] [review]
upload symbols from fxnewref-linux-tbox
You can check this in to turn it on if we ever update the ref platform again (and thus these configs are used), but I'm about to turn fxnewref off entirely, since bug 384035 is about to land (later this afternoon).
Attachment #271108 -
Flags: review?(preed) → review+
Assignee | ||
Comment 10•18 years ago
|
||
Comment on attachment 271108 [details] [diff] [review]
upload symbols from fxnewref-linux-tbox
Checked in.
Assignee | ||
Updated•18 years ago
|
Status: NEW → RESOLVED
Closed: 18 years ago
Resolution: --- → FIXED
Component: Tinderbox Configuration → Tinderbox
Product: mozilla.org → Webtools
Updated•11 years ago
|
Product: Webtools → Webtools Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•