Closed Bug 769808 Opened 11 years ago Closed 11 years ago

Update mozbase on m-c to include fix for bug 769789

Categories

(Testing :: Mozbase, defect)

Product:
Testing
Component:
Mozbase
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(Not tracked)

Status:
RESOLVED FIXED
Milestone:
mozilla16

People

(Reporter: rain1, Assigned: rain1)

References

Details

Attachments

(1 file, 1 obsolete file)

rollup of changes, without mozdevice
98.95 KB, patch
k0scist
: review+
Details | Diff | Splinter Review 
      No description provided.
mozrunner should be bumped to 5.7, mozprocess to 0.3 .

at least mozrunner, mozprocess, and mozprofile should be updated on m-c; mozinfo and ManifestDestiny may also need updating.  Or it may be easier just do update the whole she-bang
Depends on: 769789
See Also: → 769789
I've updated the versions, tagged, and pushed to pypi: https://github.com/mozilla/mozbase/commit/4e4c769ea30403847adce2ead7483278c14131e2

Still need to roll the m-c patch
:ahal, I noticed you authored https://wiki.mozilla.org/Auto-tools/HowTo/MirrorRepo linked to from https://wiki.mozilla.org/Auto-tools/Projects/MozBase#Mirroring . How standard is this methodology?  Any chance you want to roll a patch and/or make sure i don't mess it up?
Attached patch rollup of changes (obsolete) — — Splinter Review 

        Attachment #638420 -
        Flags: review?(jhammel)

    
    

    
  


  

        Attachment #638420 -
        Attachment is obsolete: true

        Attachment #638420 -
        Flags: review?(jhammel)

        Attachment #638421 -
        Flags: review?(jhammel)

    
    

    
  
Pushed to try, though tbpl's having some issues I think: https://tbpl.mozilla.org/?tree=Try&rev=3cba50d2ea95

    
    

    
  
I've noted several nits from looking at the diff (attachment 638421 [details] [diff] [review]) that should be fixed in upstream mozbase: bug 770293

I'm fine pushing to m-c before taking the changes assuming the missing license blocks are tolerable

    
    

    
  
Comment on attachment 638421 [details] [diff] [review]
rollup of changes, without mozdevice

Looks good.

        Attachment #638421 -
        Flags: review?(jhammel) → review+

    
    

    
  
http://hg.mozilla.org/integration/mozilla-inbound/rev/a9fda5347abd
Assignee: nobody → sagarwal
Status: NEW → ASSIGNED

    
    

    
  
and tag created.  that should be it
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED

    
    

    
  
Oops, should be closed when it hits m-c
Status: RESOLVED → REOPENED
Resolution: FIXED → ---

    
    

    
  
https://hg.mozilla.org/mozilla-central/rev/a9fda5347abd
Status: REOPENED → RESOLVED
Closed: 11 years ago → 11 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla16

    
    

    
  
I botched the update a little :/

Pushed a followup: https://hg.mozilla.org/mozilla-central/rev/947af3f855e1

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