Closed Bug 960837 Opened 10 years ago Closed 10 years ago

Use the basic receipt check introduced in bug 757226 for installations

Categories

(Core Graveyard :: DOM: Apps, defect)

Product:
Core Graveyard
Component:
DOM: Apps
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(Not tracked)

Status:
RESOLVED FIXED
Milestone:
mozilla29

People

(Reporter: marco, Assigned: marco)

References

Details

Attachments

(1 file, 1 obsolete file)

Patch
19.62 KB, patch
fabrice
: review+
Details | Diff | Splinter Review 
As we've introduced these checks, we may as well use them for installations.

I've only a doubt, should the installation fail when one of the receipts is invalid or when all of them are invalid?
Attached patch Tentative patch (obsolete) — — Splinter Review 
Attaching a tentative patch that takes the first approach (the installation fails if one of the receipts is invalid).
When any one of them is invalid, in my opinion.
Attached patch Patch — — Splinter Review 

        Attachment #8361414 -
        Attachment is obsolete: true

        Attachment #8361972 -
        Flags: review?(fabrice)

        Attachment #8361972 -
        Flags: review?(fabrice) → review+

    
    

    
  
https://hg.mozilla.org/mozilla-central/rev/25caedc66f90
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla29
Depends on: 964877

    
  
Product: Core → Core Graveyard

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






  

    
  







  

    

      
Attachment

      

      
      
      
      
    

    

      

        

          

            
General

            

              Creator: 



            

            
Created: 

            
Updated: 

            
Size: