Closed
Bug 701783
Opened 14 years ago
Closed 13 years ago
clean up maemo/scratchbox references in buildbotcustom
Categories
(Release Engineering :: General, defect, P5)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: mozilla, Unassigned)
References
Details
(Whiteboard: [maemo][cleanup])
Attachments
(1 file)
1.75 KB,
patch
|
catlee
:
review+
mozilla
:
checked-in+
|
Details | Diff | Splinter Review |
John Ford wants to keep these for a while as they may help with a linux project later.
If we end up not needing it, we should get rid of use_scratchbox, ScratchboxCommand, and RetryingScratchboxProperty.
Comment 1•14 years ago
|
||
as much as I like planning ahead, i'm also a big fan of cutting code till it bleeds :)
if it's not being used actively we should remove it as it represents a code path to be maintained that has zero use.
it's always available in the repo.
Reporter | ||
Comment 2•13 years ago
|
||
Looks like ScratchboxCommand became MockCommand, so this is all that's left for this bug.
Attachment #622531 -
Flags: review?(catlee)
Comment 3•13 years ago
|
||
Comment on attachment 622531 [details] [diff] [review]
remove final scratchbox remnants in buildbotcustom
Review of attachment 622531 [details] [diff] [review]:
-----------------------------------------------------------------
die!!!!!
Attachment #622531 -
Flags: review?(catlee) → review+
Reporter | ||
Comment 4•13 years ago
|
||
Comment on attachment 622531 [details] [diff] [review]
remove final scratchbox remnants in buildbotcustom
http://hg.mozilla.org/build/buildbotcustom/rev/5ecf7eb36c29
Attachment #622531 -
Flags: checked-in+
Comment 5•13 years ago
|
||
A change from this bug landed on the default branches of either buildbot-configs or buildbotcustom.
Such change has been merged as of ~8:30 AM PDT into the production branches and the buildbot masters have been reconfigured.
This if FYI and a note has been added to the maintenance page:
https://wiki.mozilla.org/ReleaseEngineering:Maintenance#Reconfigs_.2F_Deployments
Reporter | ||
Updated•13 years ago
|
Status: NEW → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Assignee | ||
Updated•12 years ago
|
Product: mozilla.org → Release Engineering
You need to log in
before you can comment on or make changes to this bug.
Description
•