Closed
Bug 825576
Opened 11 years ago
Closed 11 years ago
Update clang (manifest) for OS X (use clang 3.2 final version, port Bug 823906)
Categories
(SeaMonkey :: Build Config, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: mcsmurf, Assigned: Callek)
Details
Attachments
(3 files, 2 obsolete files)
2.34 KB,
patch
|
mcsmurf
:
review+
|
Details | Diff | Splinter Review |
2.34 KB,
patch
|
mcsmurf
:
review+
philip.chee
:
approval-comm-aurora+
|
Details | Diff | Splinter Review |
2.34 KB,
patch
|
mcsmurf
:
review+
philip.chee
:
approval-comm-beta+
|
Details | Diff | Splinter Review |
I'm not sure if we need to do this, too: Firefox and Thunderbird updated their clang manifests (suite/config/tooltool-manifests/macosx* in our case) to use a newer clang 3.2 branch version. I've noticed that the "make check" test of the OS X 64 tinderbox on trunk fails: cppunittests TEST-UNEXPECTED-FAIL | transport_unittests | test crashed The clang update revision comment on TB says something about fixing make check failures with this newer clang version, see http://hg.mozilla.org/comm-central/rev/ec9f4a50e3b4
Reporter | ||
Comment 1•11 years ago
|
||
Justin: I'm not sure how exactly clang is updated. Is it just a matter of copying over those clang binaries from the MoCo server to the webserver that can be accessed by the SeaMonkey build slaves (and pushing this patch)?
Attachment #696696 -
Flags: review?(bugspam.Callek)
Reporter | ||
Comment 2•11 years ago
|
||
Comment on attachment 696696 [details] [diff] [review] Patch Actually, this is not the latest clang version, FF has a newer one.
Attachment #696696 -
Flags: review?(bugspam.Callek)
Reporter | ||
Comment 3•11 years ago
|
||
This one is the latest version, see http://hg.mozilla.org/mozilla-central/filelog/0d771761b9b3/browser/config/tooltool-manifests/macosx64/releng.manifest
Attachment #696696 -
Attachment is obsolete: true
Attachment #696697 -
Flags: review?(bugspam.Callek)
Reporter | ||
Updated•11 years ago
|
Summary: Update clang (manifest) for OS X? → Update clang (manifest) for OS X (use clang 3.2 final version, port Bug 823906)
Assignee | ||
Comment 4•11 years ago
|
||
Assignee: nobody → bugspam.Callek
Status: NEW → ASSIGNED
Attachment #696699 -
Flags: review?(bugzilla)
Assignee | ||
Comment 5•11 years ago
|
||
Comment on attachment 696697 [details] [diff] [review] Patch I'm r-'ing for three reasons (a) we midaired, and the ver I added is what I for-sure added to the tooltool we have access to (b) mac32 is unused now for our [successful] builds. (c) even though its unused by default, my patch changes the linux versions too.
Attachment #696697 -
Flags: review?(bugspam.Callek) → review-
Assignee | ||
Comment 6•11 years ago
|
||
(In reply to Frank Wein [:mcsmurf] from comment #1) > Created attachment 696696 [details] [diff] [review] > Patch > > Justin: I'm not sure how exactly clang is updated. Is it just a matter of > copying over those clang binaries from the MoCo server to the webserver that > can be accessed by the SeaMonkey build slaves (and pushing this patch)? Indeed, and when you filed this I was already thinking that we should update, so I *just* copied over all current clang binaries to our public server.
Assignee | ||
Comment 7•11 years ago
|
||
Do it for aurora too [Approval Request Comment] Regression caused by (bug #): none Testing completed (on m-c, etc.): This matches Firefox compiling Risk to taking this patch (and alternatives if risky): very very low String changes made by this patch: None
Attachment #696697 -
Attachment is obsolete: true
Attachment #696700 -
Flags: review?(bugzilla)
Attachment #696700 -
Flags: approval-comm-aurora?
Assignee | ||
Comment 8•11 years ago
|
||
[Approval Request Comment] Regression caused by (bug #): none Testing completed (on m-c, etc.): This matches Firefox compiling Risk to taking this patch (and alternatives if risky): very very low String changes made by this patch: None
Attachment #696701 -
Flags: review?(bugzilla)
Attachment #696701 -
Flags: approval-comm-beta?
![]() |
||
Updated•11 years ago
|
Attachment #696700 -
Flags: approval-comm-aurora? → approval-comm-aurora+
![]() |
||
Comment 9•11 years ago
|
||
Comment on attachment 696701 [details] [diff] [review] [beta] ..and finally beta rs=me
Attachment #696701 -
Flags: approval-comm-beta? → approval-comm-beta+
Reporter | ||
Updated•11 years ago
|
Attachment #696699 -
Flags: review?(bugzilla) → review+
Reporter | ||
Updated•11 years ago
|
Attachment #696700 -
Flags: review?(bugzilla) → review+
Reporter | ||
Comment 10•11 years ago
|
||
Comment on attachment 696701 [details] [diff] [review] [beta] ..and finally beta mozilla-central still uses r161152. So I'd rather not update here as usually regressions due to compiler change are discovered earlier on mozilla-central/Firefox.
Reporter | ||
Comment 11•11 years ago
|
||
Comment on attachment 696701 [details] [diff] [review] [beta] ..and finally beta Nevermind, looks like comm-beta MXR is out-of-date (mozilla-beta MXR is ok).
Attachment #696701 -
Flags: review?(bugzilla) → review+
Assignee | ||
Comment 12•11 years ago
|
||
https://hg.mozilla.org/comm-central/rev/cf44b65f2bf6 https://hg.mozilla.org/releases/comm-aurora/rev/0f5a7fd7a87c https://hg.mozilla.org/releases/comm-beta/rev/4816e242a4b6 Accidentally marked mcsmurf as approver in checkin message
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•