Download script doesn't add platform to local filename

RESOLVED FIXED

Status

Mozilla QA Graveyard
Mozmill Automation
RESOLVED FIXED
7 years ago
4 years ago

People

(Reporter: whimboo, Assigned: whimboo)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(2 attachments)

The current version does not include the 64bit suffix into the auto-generated local filename. That means an already downloaded 32bit build will be overwritten.
Summary: Download script doesn't add 64bit suffix to chosen local filename → Download script doesn't add platform to local filename
Created attachment 528938 [details] [diff] [review]
Patch v1
Assignee: nobody → hskupin
Status: NEW → ASSIGNED
Attachment #528938 - Flags: review?(jgriffin)
Comment on attachment 528938 [details] [diff] [review]
Patch v1

Looks good.
Attachment #528938 - Flags: review?(jgriffin) → review+
Landed as:
http://hg.mozilla.org/qa/mozmill-automation/rev/cb90beeefc67
Status: ASSIGNED → RESOLVED
Last Resolved: 7 years ago
Resolution: --- → FIXED
There is a typo for release builds I have to fix. Sorry.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Created attachment 528944 [details] [diff] [review]
Follow-up
Attachment #528944 - Flags: review?(jgriffin)
Comment on attachment 528944 [details] [diff] [review]
Follow-up

Sorry I missed this the first time around :)
Attachment #528944 - Flags: review?(jgriffin) → review+
Follow-up landed as:
http://hg.mozilla.org/qa/mozmill-automation/rev/841d6ccf0e70
Status: REOPENED → RESOLVED
Last Resolved: 7 years ago7 years ago
Resolution: --- → FIXED
Product: Mozilla QA → Mozilla QA Graveyard
You need to log in before you can comment on or make changes to this bug.