1137110 - inspector rev in client.py is using the wrong rev in comm-beta

Status: RESOLVED FIXED

(SeaMonkey :: Build Config, defect)

Description

[Edmund Wong (:ewong)]

In comm-beta, client.py has:

    27     # URL of the default hg repository to clone for DOM Inspector.
    28     'INSPECTOR_REPO': 'https://hg.mozilla.org/dom-inspector/',
    29     # The stable revision to use
    30       'INSPECTOR_REV':  'DOMI_LATEST_RELEASE',

This is not right since DOMI_LATEST_RELEASE has been updated to fix
bug 1128291.  Instead, c-b should (I believe) use  'SEA2_26_RELBRANCH'
which doesn't have that change.

Comment 1

[Edmund Wong (:ewong)]

Attachment: proposed patch (v1)

Comment 2

[Edmund Wong (:ewong)]

Comment on attachment 8569712 [details] [diff] [review]
proposed patch (v1)

[Approval Request Comment]
Regression caused by (bug #): bug 1128291
User impact if declined: bustage
Testing completed (on m-c, etc.): 
Risk to taking this patch (and alternatives if risky):
String changes made by this patch:

Comment 3

[Ian Neal]

I think we will need to create a new branch. Probably from the DOMI_2_0_15 branch and based on http://hg.mozilla.org/dom-inspector/rev/6abb72a17277 as you probably need the compat vers changes.

Comment 4

[Edmund Wong (:ewong)]

(In reply to Ian Neal from comment #3)
> I think we will need to create a new branch. Probably from the DOMI_2_0_15
> branch and based on http://hg.mozilla.org/dom-inspector/rev/6abb72a17277 as
> you probably need the compat vers changes.

Ratty suggested we use DOMI_2_0_15_1 as the tag.  

Should I go ahead and create that tag?

Comment 5

[Ian Neal]

(In reply to Edmund Wong (:ewong) from comment #4)
> (In reply to Ian Neal from comment #3)
> > I think we will need to create a new branch. Probably from the DOMI_2_0_15
> > branch and based on http://hg.mozilla.org/dom-inspector/rev/6abb72a17277 as
> > you probably need the compat vers changes.
> 
> Ratty suggested we use DOMI_2_0_15_1 as the tag.  
> 
> Should I go ahead and create that tag?

That is fine by me, I was thinking SEA2_34_RELBRANCH but DOMI_2_0_15_1 seems okay too.

Comment 6

[Edmund Wong (:ewong)]

Attachment: proposed patch (v2)

Using DOMi cset 60242bd0c895 (as tagged as DOMI_2_0_15_1)

Comment 7

[Edmund Wong (:ewong)]

Comment on attachment 8570344 [details] [diff] [review]
proposed patch (v2)

[Approval Request Comment]
Regression caused by (bug #): 1128291
User impact if declined: bustage during osx64 build.
Testing completed (on m-c, etc.): 
Risk to taking this patch (and alternatives if risky):
String changes made by this patch:

Comment 8

[Robert Kaiser]

Comment on attachment 8570344 [details] [diff] [review]
proposed patch (v2)

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

Ah, sure, this is for beta, so I should have set this a+ right away :)

Comment 9

[Edmund Wong (:ewong)]

Comment on attachment 8570344 [details] [diff] [review]
proposed patch (v2)

Pushed to comm-beta:
https://hg.mozilla.org/releases/comm-beta/rev/8fc40ff84c71