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 #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 ;-)
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 ago → 11 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.
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
You need to log in before you can comment on or make changes to this bug.