Closed
Bug 646462
Opened 13 years ago
Closed 13 years ago
Mozmill 1.5.x doesn't remove temporary add-ons folders
Categories
(Testing :: Mozbase, defect)
Testing
Mozbase
Tracking
(Not tracked)
VERIFIED
FIXED
People
(Reporter: whimboo, Assigned: whimboo)
Details
(Whiteboard: [mozmill-1.5.3+])
Attachments
(1 file)
834 bytes,
patch
|
k0scist
:
review+
|
Details | Diff | Splinter Review |
If a Mozmill test has to be run with an add-on you can use the --addon command line option to get this add-on installed. For determining the ID Mozmill unpacks the XPI to a temporary location but fails in deleting it in the 1.5.x branch. Now that we promote our testruns with add-ons we have to make sure to not clutter the users system with those temporary files.
Assignee | ||
Comment 1•13 years ago
|
||
Patch with nearly no impact to Mozmill but a big jump to keep the users system clean.
Attachment #523001 -
Flags: review?(jhammel)
Assignee | ||
Comment 2•13 years ago
|
||
Just to note, this is a backport from Mozmill 2.
Comment 3•13 years ago
|
||
Comment on attachment 523001 [details] [diff] [review] Patch v1 wfm
Attachment #523001 -
Flags: review?(jhammel) → review+
Assignee | ||
Comment 4•13 years ago
|
||
Landed on hotfix-1.5 as: https://github.com/mozautomation/mozmill/commit/bb5de4912698b3d46c424068088bd642440d237e
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Assignee | ||
Updated•13 years ago
|
Whiteboard: [mozmill-1.5.x+] → [mozmill-1.5.3+]
You need to log in
before you can comment on or make changes to this bug.
Description
•