Closed
Bug 452504
Opened 16 years ago
Closed 16 years ago
port bug 97954, bug 394901, bug 451466, and bug 450194 changes to comm-central
Categories
(MailNews Core :: Build Config, defect)
MailNews Core
Build Config
Tracking
(Not tracked)
RESOLVED
FIXED
Thunderbird 3.0a3
People
(Reporter: kairo, Assigned: kairo)
References
Details
Attachments
(2 files, 1 obsolete file)
12.28 KB,
patch
|
standard8
:
review+
|
Details | Diff | Splinter Review |
786 bytes,
patch
|
standard8
:
review+
|
Details | Diff | Splinter Review |
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
Assignee | ||
Comment 1•16 years ago
|
||
(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.
Assignee | ||
Comment 2•16 years ago
|
||
Here's the patch for porting all the mentioned changes to comm-central.
Assignee | ||
Comment 3•16 years ago
|
||
We probably want the full bug 451466 for SeaMonkey, I'll do that either in an additional patch or as a followup.
Comment 4•16 years ago
|
||
(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.
Comment 5•16 years ago
|
||
(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 6•16 years ago
|
||
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+
Assignee | ||
Comment 7•16 years ago
|
||
Right, the -framework addition sounds correct, as bug 394901 comment #15 implies that they added -framework AddressBook as a platform requirement.
Assignee | ||
Comment 8•16 years ago
|
||
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)
Updated•16 years ago
|
Attachment #336043 -
Flags: review?(bugzilla) → review+
Comment 9•16 years ago
|
||
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 ;-)
Assignee | ||
Comment 10•16 years ago
|
||
landed as http://hg.mozilla.org/comm-central/rev/0b81d75bf2f3
Status: ASSIGNED → RESOLVED
Closed: 16 years ago
Resolution: --- → FIXED
Comment 11•16 years ago
|
||
> 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 → ---
Assignee | ||
Updated•16 years ago
|
Product: SeaMonkey → MailNews Core
QA Contact: build-config → build-config
Assignee | ||
Comment 12•16 years ago
|
||
Port the additional fix mentioned in the last comment by Philip.
Attachment #336203 -
Flags: review?(bugzilla)
Updated•16 years ago
|
Attachment #336203 -
Flags: review?(bugzilla) → review+
Assignee | ||
Comment 13•16 years ago
|
||
Pushed the additional patch as http://hg.mozilla.org/comm-central/rev/455bc7d8dbdd
Status: REOPENED → RESOLVED
Closed: 16 years ago → 16 years ago
Resolution: --- → FIXED
Comment 14•14 years ago
|
||
(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
Comment 15•14 years ago
|
||
(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.
Description
•