Closed Bug 452504 Opened 11 years ago Closed 11 years ago

port bug 97954, bug 394901, bug 451466, and bug 450194 changes to comm-central

Categories

(MailNews Core :: Build Config, defect)

defect
Not set

Tracking

(Not tracked)

RESOLVED FIXED
Thunderbird 3.0a3

People

(Reporter: kairo, Assigned: kairo)

References

Details

Attachments

(2 files, 1 obsolete file)

bug 97954 - autoconf build environment for spidermonkey
http://hg.mozilla.org/mozilla-central/index.cgi/rev/661c6de0b8f0

We have ported the aclocal.m4 changes but there's more in client.mk and rules.mk that we should port.

More changes we need to port are in those bugs/changesets:

bug 394901 - make nsIUserInfo implementation on mac work, and modernize it
http://hg.mozilla.org/mozilla-central/index.cgi/rev/9ad2f77e6778

bug 451466 - wget target to get en-US binary
http://hg.mozilla.org/mozilla-central/index.cgi/rev/25b909ce268c

bug 450194 - Add -Wno-invalid-offsetof to our warnings flags, because we break the rules on purpose
http://hg.mozilla.org/mozilla-central/index.cgi/rev/41d9d32ab5a7
(In reply to comment #0)
> bug 394901 - make nsIUserInfo implementation on mac work, and modernize it
> http://hg.mozilla.org/mozilla-central/index.cgi/rev/9ad2f77e6778

http://hg.mozilla.org/mozilla-central/index.cgi/diff/9ad2f77e6778/configure.in may not need to be ported actually.
Here's the patch for porting all the mentioned changes to comm-central.
Assignee: nobody → kairo
Status: NEW → ASSIGNED
Attachment #335787 - Flags: review?(bugzilla)
We probably want the full bug 451466 for SeaMonkey, I'll do that either in an additional patch or as a followup.
(In reply to comment #3)
> We probably want the full bug 451466 for SeaMonkey, I'll do that either in an
> additional patch or as a followup.

We'll need it for TB as well, but I can always cover that elsewhere.
(In reply to comment #1)
> (In reply to comment #0)
> > bug 394901 - make nsIUserInfo implementation on mac work, and modernize it
> > http://hg.mozilla.org/mozilla-central/index.cgi/rev/9ad2f77e6778
> 
> http://hg.mozilla.org/mozilla-central/index.cgi/diff/9ad2f77e6778/configure.in
> may not need to be ported actually.

If we port this, then we can cut down on our additions:

http://mxr.mozilla.org/comm-central/search?string=framework+AddressBook
Comment on attachment 335787 [details] [diff] [review]
patch to port all those changes to comm-central

This looks good and I'm happy for it to go in as-is, but I think we should also do the -framework addition as well.
Attachment #335787 - Flags: review?(bugzilla) → review+
Right, the -framework addition sounds correct, as bug 394901 comment #15 implies that they added -framework AddressBook as a platform requirement.
Here's the updated patch, porting bug 451466 all the way for both apps, and doing a few framework changes - but I can't test on Mac unfortunately, I hope I got that right.
Attachment #335787 - Attachment is obsolete: true
Attachment #336043 - Flags: review?(bugzilla)
Attachment #336043 - Flags: review?(bugzilla) → review+
Comment on attachment 336043 [details] [diff] [review]
patch, v2: full wget, more framework patches

From the two builds I've done on mac, the framework stuff seems ok, just keep an eye on the trees when you land it ;-)
landed as http://hg.mozilla.org/comm-central/rev/0b81d75bf2f3
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
> bug 450194 - Add -Wno-invalid-offsetof to our warnings flags, because we break
> the rules on purpose

I think you missed: http://hg.mozilla.org/mozilla-central/rev/ccf420d6618b
See: Bug 450194 Comment 11 and Bug 450194 Comment 12
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Product: SeaMonkey → MailNews Core
QA Contact: build-config → build-config
Port the additional fix mentioned in the last comment by Philip.
Attachment #336203 - Flags: review?(bugzilla)
Attachment #336203 - Flags: review?(bugzilla) → review+
Pushed the additional patch as http://hg.mozilla.org/comm-central/rev/455bc7d8dbdd
Status: REOPENED → RESOLVED
Closed: 11 years ago11 years ago
Resolution: --- → FIXED
(In reply to comment #10)
> landed as http://hg.mozilla.org/comm-central/rev/0b81d75bf2f3

Ftr, this included the port of:
http://hg.mozilla.org/mozilla-central/rev/8bfe86107c17
Neaten up the handling of configure script regeneration in client.mk.
Flags: in-testsuite-
Target Milestone: --- → Thunderbird 3.0a3
(In reply to comment #14)
> http://hg.mozilla.org/mozilla-central/rev/8bfe86107c17
> Neaten up the handling of configure script regeneration in client.mk.

which was part of
http://hg.mozilla.org/mozilla-central/rev/661c6de0b8f0
Merge initial patches for bug 97954: autoconf build environment for spidermonkey
Depends on: 97954, 394901, 461466, 450194
You need to log in before you can comment on or make changes to this bug.