524682 - comm-central should pull Chatzilla from hg rather than CVS

Status: RESOLVED FIXED

(MailNews Core :: Build Config, defect)

Description

[Serge Gautherie (:sgautherie)]

+++ This bug was initially created as a clone of Bug #482686 +++

Comment 1

[Serge Gautherie (:sgautherie)]

Attachment: (Av1) Just like Venkman [Checkin: Comment 5]

Comment 2

[Robert Kaiser]

Sorry, this does not depend on bug 524654 as just having an empty existing repo doesn't help us much, and it doesn't block bug 475011 because ChatZilla can move its development to hg without depending on us using that repo - it's the other way round, we depend on their source being there to switch over.
I'll wait with review until bug 475011 is fixed and ChatZilla code is actually there.

Comment 3

[Serge Gautherie (:sgautherie)]

(In reply to comment #2)
> I'll wait with review until bug 475011 is fixed and ChatZilla code is actually
> there.

Tested fine with now loaded CZ repo.

Comment 4

[Robert Kaiser]

Comment on attachment 408725 [details] [diff] [review]
(Av1) Just like Venkman
[Checkin: Comment 5]

Looks good to me, but I'd like a second look at it from someone who's better at python - gozer, could you do that?

Comment 5

[Serge Gautherie (:sgautherie)]

Comment on attachment 408725 [details] [diff] [review]
(Av1) Just like Venkman
[Checkin: Comment 5]


http://hg.mozilla.org/comm-central/rev/e245eb08c4ba


"approval-seamonkey2.0.1=?":
No risk, Follow new CZ trunk repo.

Comment 6

[Robert Kaiser]

Comment on attachment 408725 [details] [diff] [review]
(Av1) Just like Venkman
[Checkin: Comment 5]

This unfortunately is somewhat more critical on the comm-1.9.1 branch, as it requires changes to the build automation for the releases machinery.

Comment 7

[Robert Kaiser]

I've talked to gozer, and we agreed that we should hold landing this on 1.9.1 off until Thunderbird 3.0 has been done. In practice, this probably means that we'll do it once I'm back from vacation, when I can also look into getting our buildbot factories fixed for it.

Comment 8

[Robert Kaiser]

Comment on attachment 408725 [details] [diff] [review]
(Av1) Just like Venkman
[Checkin: Comment 5]

From what I see, this patch doesn't apply as-is to the 1.9.1 branch (and 2.0.1 has been cut already), so minusing this particular patch.

Could you please attach a version that applies to the 1.9.1 branch? I'd like to have this in, esp. in conjuction with bug 534422.

Comment 9

[Serge Gautherie (:sgautherie)]

Attachment: (Bv1-191) Just like Venkman [Checkin: Comment 12]

Av1, with comment 8 suggestion(s).

Comment 10

[Robert Kaiser]

Comment on attachment 417353 [details] [diff] [review]
(Bv1-191) Just like Venkman
[Checkin: Comment 12]

Looks good, just want to have bug 534422 ready so build machines don't need more special hacks

Comment 11

[Robert Kaiser]

Comment on attachment 417353 [details] [diff] [review]
(Bv1-191) Just like Venkman
[Checkin: Comment 12]

a=me but please don't land, I'd like to have this in a single commit with bug 534422 so that no build machines (whatever they are pulling right now) might pull a newer chatzilla just to go back later.

Comment 12

[Robert Kaiser]

http://hg.mozilla.org/releases/comm-1.9.1/rev/ee5e6c3b8fdb