Closed Bug 1278921 Opened 8 years ago Closed 8 years ago

Autophone - BuildLocation regexp should prevent prefix matches for builds

Categories

(Testing Graveyard :: Autophone, defect)

Product:
Testing Graveyard
Component:
Autophone
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(Not tracked)

Status:
RESOLVED FIXED

People

(Reporter: bc, Assigned: bc)

References

Details

(Keywords: regression)

Attachments

(1 file)

bug-1278921-v1.patch
1007 bytes, patch
jmaher
: review+
Details | Diff | Splinter Review 
Sometime in the relatively recent past, additional files have been added to the dist output directory for Fennec builds. Each apk file now has a corresponding file with the extention .permissions added. For example,

fennec-50.0a1.en-US.android-arm.apk 
fennec-50.0a1.en-US.android-arm.apk.permissions

This causes problems with running Autophone against local builds since the current regexps do not anchor the match and inadvertently pick up the permissions file instead of the apk file.

        Attachment #8761245 -
        Flags: review?(jmaher)

    
    

    
  
Comment on attachment 8761245 [details] [diff] [review]
bug-1278921-v1.patch

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

I like small patches.

        Attachment #8761245 -
        Flags: review?(jmaher) → review+

    
    

    
  
deployed on autophone-2,3 2016-06-08 16:20. autophone-1 is dead, i'll deploy when it is resurrected.
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED

    
  
Product: Testing → Testing Graveyard

          You need to log in
          before you can comment on or make changes to this bug.
        






  

    
  







  

    

      
Attachment

      

      
      
      
      
    

    

      

        

          

            
General

            

              Creator: 



            

            
Created: 

            
Updated: 

            
Size: