Closed Bug 758680 Opened 12 years ago Closed 12 years ago

Remove usetinderbox=1 support

Categories

(Tree Management Graveyard :: TBPL, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: emorley, Assigned: emorley)

References

Details

Attachments

(1 file, 1 obsolete file)

https://tbpl.mozilla.org/?usetinderbox=1 no longer shows anything useful (only pending/running builds; I thought there were a few things still using it, like Valgrind, but I guess not) - so support for it can hopefully now be removed. (This just leaves bug 756027 and then we can ditch tinderbox once and for all :-)).
Attached patch Patch v1 (obsolete) — Splinter Review
Assignee: nobody → bmo
Status: NEW → ASSIGNED
Attachment #627267 - Flags: review?(arpad.borsos)
Attached patch Patch v2Splinter Review
Spotted getTinderboxSummary.php was now unused, so this version takes it out too.
Attachment #627267 - Attachment is obsolete: true
Attachment #627267 - Flags: review?(arpad.borsos)
Attachment #627276 - Flags: review?(arpad.borsos)
Oh noes... what's gonna be next?  kill tinderbox?  :(
Comment on attachment 627276 [details] [diff] [review]
Patch v2

Review of attachment 627276 [details] [diff] [review]:
-----------------------------------------------------------------

Awesome!
Attachment #627276 - Flags: review?(arpad.borsos) → review+
http://hg.mozilla.org/users/mstange_themasta.com/tinderboxpushlog/rev/3e1393cc6519
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Blocks: 759293
Blocks: 759313
Blocks: 759319
Depends on: 760321
Product: Webtools → Tree Management
Product: Tree Management → Tree Management Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: