758564 - Port |Bug 562431 - Rewrite WeaveCrypto as a .jsm| to SeaMonkey

Status: RESOLVED FIXED

(SeaMonkey :: Build Config, defect, P4)

Comment 1

[Serge Gautherie (:sgautherie)]

Attachment: (Av1) Update packaging after |Bug 562431 - Rewrite WeaveCrypto as a .jsm| [Checked in: Comment 5 & 8]

Comment 2

[Justin Wood (:Callek)]

Comment on attachment 627167 [details] [diff] [review]
(Av1) Update packaging after |Bug 562431 - Rewrite WeaveCrypto as a .jsm|
[Checked in: Comment 5 & 8]

Hrm, I don't see this corresponding removed-files change for any other product, are we *sure* this is correct? [clearing approval req's until that is answered]

Comment 3

[Serge Gautherie (:sgautherie)]

(In reply to Justin Wood (:Callek) from comment #2)
> Hrm, I don't see this corresponding removed-files change for any other
> product, are we *sure* this is correct?

Known: "Firefox" (at least) often just doesn't care about its removed-files.in :-/
I'm very sure: just check that this file is actually packaged in omni.ja.

Comment 4

[Justin Wood (:Callek)]

There is a list of bugs I just approved for beta (though we're on our last beta now, with the release spinning friday or sat) which touch this same file (removed-files.in)

This is one of those bugs where I could not find a way to apply it to beta, even after mixing and matching the order here repeatedly.

Please either provide one monolithic patch which makes someones life easier, convince someone to untangle, tell us the order, or do it yourself. Otherwise I am comfortable with these slipping from this release.

I won't even attempt to land these patches on aurora for the same reasons without one of the above solutions being done.

Comment 5

[Serge Gautherie (:sgautherie)]

Comment on attachment 627167 [details] [diff] [review]
(Av1) Update packaging after |Bug 562431 - Rewrite WeaveCrypto as a .jsm|
[Checked in: Comment 5 & 8]

http://hg.mozilla.org/comm-central/rev/23060289eeb3

Comment 6

[Serge Gautherie (:sgautherie)]

(In reply to Justin Wood (:Callek) from comment #4)
> tell us the order

(In reply to Serge Gautherie (:sgautherie) from comment #5)
> http://hg.mozilla.org/comm-central/rev/23060289eeb3

Please c-n the 6 changesets in
http://hg.mozilla.org/comm-central/pushloghtml?changeset=23060289eeb3

Comment 7

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/releases/comm-aurora/rev/74f15aee612b
https://hg.mozilla.org/releases/comm-aurora/rev/497e77ecf50e
https://hg.mozilla.org/releases/comm-aurora/rev/6df4049779a9
https://hg.mozilla.org/releases/comm-aurora/rev/6156d7445a57
https://hg.mozilla.org/releases/comm-aurora/rev/e9e7cedc2e54
https://hg.mozilla.org/releases/comm-aurora/rev/463f32742814

https://hg.mozilla.org/releases/comm-beta/rev/8d7e4be7518d
https://hg.mozilla.org/releases/comm-beta/rev/4adcbda60564
https://hg.mozilla.org/releases/comm-beta/rev/4e8f318e67c9
https://hg.mozilla.org/releases/comm-beta/rev/36540f3de761
https://hg.mozilla.org/releases/comm-beta/rev/1bc1100143b2
https://hg.mozilla.org/releases/comm-beta/rev/706268432971

Comment 8

[Serge Gautherie (:sgautherie)]

Comment on attachment 627167 [details] [diff] [review]
(Av1) Update packaging after |Bug 562431 - Rewrite WeaveCrypto as a .jsm|
[Checked in: Comment 5 & 8]

(In reply to Ryan VanderMeulen from comment #7)
> https://hg.mozilla.org/releases/comm-aurora/rev/74f15aee612b
> 
> https://hg.mozilla.org/releases/comm-beta/rev/8d7e4be7518d

These are the changesets for this bug.