Closed Bug 1270036 Opened 8 years ago Closed 8 years ago

Include modules/zlib in the tarballs generated by make_source_package.sh

Categories

(Core :: JavaScript Engine, defect)

Product:
Core
Component:
JavaScript Engine
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED FIXED
Milestone:
mozilla49
Tracking Flags:
Tracking Status
firefox49 --- fixed
firefox-esr45 --- fixed

People

(Reporter: Ms2ger, Assigned: nox)

References

(Blocks 1 open bug)

Details

Attachments

(1 file, 2 obsolete files)

0001-Fix-the-zlib-module-in-SM-tarballs.patch
716 bytes, patch
sfink
: review+
Details | Diff | Splinter Review 
      No description provided.
Attached file zlib src build file (obsolete) — 

    
    

    
  
I have extracted the archive found here: https://developer.mozilla.org/en-US/docs/Mozilla/Projects/SpiderMonkey/Releases/45 (referenced from https://people.mozilla.org/~sfink/mozjs-45.0.2.tar.bz2).

I have moved /modules/src to /modules/zlib/src.

I have added a moz.build file to /modules/zlib (see attached https://bugzilla.mozilla.org/attachment.cgi?id=8748547).

Configuration completes successfully under mozilla-build 2.2.0.

    
    

    
  


  
Assignee: nobody → nox

        Attachment #8748547 -
        Attachment is obsolete: true
Status: NEW → ASSIGNED

        Attachment #8748590 -
        Flags: review?(till)

    
    

    
  


  
Removed diffstat from patch.

        Attachment #8748590 -
        Attachment is obsolete: true

        Attachment #8748590 -
        Flags: review?(till)

        Attachment #8748595 -
        Flags: review?(till)

        Attachment #8748595 -
        Flags: review?(till) → review+

    
  
Keywords: checkin-needed

    
    

    
  
This can probably be uplifted a=npotb to esr45 once it merges to m-c.

    
    

    
  
https://hg.mozilla.org/mozilla-central/rev/cecbf2b03267
Status: ASSIGNED → RESOLVED
Closed: 8 years ago

          status-firefox49:
          --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla49

    
    

    
  
It merged to m-c; can you deal with esr45 or redirect the needinfo?
Flags: needinfo?(ryanvm)
Flags: needinfo?(ryanvm)
Whiteboard: [checkin-needed-esr45][a=npotb]
Whiteboard: [checkin-needed-esr45][a=npotb] → [a=npotb]
Whiteboard: [a=npotb]

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






  

    
  







  

    

      
Attachment

      

      
      
      
      
    

    

      

        

          

            
General

            

              Creator: 



            

            
Created: 

            
Updated: 

            
Size: