Closed Bug 653524 Opened 13 years ago Closed 13 years ago

Download script doesn't add platform to local filename

Categories

(Mozilla QA Graveyard :: Mozmill Automation, defect)

Product:
Mozilla QA Graveyard
Component:
Mozmill Automation
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(Not tracked)

Status:
RESOLVED FIXED

People

(Reporter: whimboo, Assigned: whimboo)

References

Details

Attachments

(2 files)

Patch v1
2.02 KB, patch
jgriffin
: review+
Details | Diff | Splinter Review
Follow-up
1.06 KB, patch
jgriffin
: review+
Details | Diff | Splinter Review 
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
Attached patch Patch v1 — — Splinter Review 
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
Closed: 13 years ago
Resolution: --- → FIXED

    
    

    
  
There is a typo for release builds I have to fix. Sorry.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---

    
    

    
  

      
      
      
      

        Attached patch
          
          
        Follow-up
        — 
          — Splinter Review
      

    


  

        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
Closed: 13 years ago → 13 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.
        






  

    
  







  

    

      
Attachment

      

      
      
      
      
    

    

      

        

          

            
General

            

              Creator: 



            

            
Created: 

            
Updated: 

            
Size: