Closed
Bug 743704
Opened 12 years ago
Closed 12 years ago
Upgrade clang to 154343
Categories
(Release Engineering :: General, defect, P2)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: espindola, Assigned: espindola)
Details
Attachments
(4 files, 3 obsolete files)
735 bytes,
patch
|
rail
:
review+
espindola
:
checked-in+
|
Details | Diff | Splinter Review |
769 bytes,
patch
|
rail
:
review+
espindola
:
checked-in+
|
Details | Diff | Splinter Review |
2.01 KB,
patch
|
rail
:
review+
rail
:
checked-in+
|
Details | Diff | Splinter Review |
3.33 KB,
patch
|
rail
:
review+
rail
:
checked-in+
|
Details | Diff | Splinter Review |
No description provided.
Assignee | ||
Comment 1•12 years ago
|
||
Assignee | ||
Comment 2•12 years ago
|
||
Also add the version to the name so that we can install multiple at same time (same thing we do with gcc).
Attachment #613320 -
Flags: review?(rail)
Assignee | ||
Updated•12 years ago
|
Summary: Upgrade clang to 154322 → Upgrade clang to 154343
Assignee | ||
Comment 3•12 years ago
|
||
Attachment #613320 -
Attachment is obsolete: true
Attachment #613320 -
Flags: review?(rail)
Attachment #613388 -
Flags: review?(rail)
Assignee | ||
Comment 4•12 years ago
|
||
Attachment #613318 -
Attachment is obsolete: true
Attachment #613318 -
Flags: review?(rail)
Attachment #613390 -
Flags: review?(rail)
Assignee | ||
Comment 5•12 years ago
|
||
Attachment #613388 -
Attachment is obsolete: true
Attachment #613388 -
Flags: review?(rail)
Attachment #613471 -
Flags: review?(rail)
Assignee | ||
Comment 6•12 years ago
|
||
Keep the old packages this time. I will send one more patch removing them once the new ones pass try.
Attachment #613490 -
Flags: review?(rail)
Assignee | ||
Comment 7•12 years ago
|
||
Forgot to add, I have been able to build firefox on linux 32/64 bits opt/dbg and on OS X universal and 32/64 dbg.
Updated•12 years ago
|
Attachment #613390 -
Flags: review?(rail) → review+
Updated•12 years ago
|
Attachment #613471 -
Flags: review?(rail) → review+
Assignee | ||
Comment 8•12 years ago
|
||
Comment on attachment 613471 [details] [diff] [review] update clang to 154343 on linux https://hg.mozilla.org/build/rpm-sources/rev/d9e7dbd0d60a
Attachment #613471 -
Flags: checked-in+
Assignee | ||
Comment 9•12 years ago
|
||
Comment on attachment 613390 [details] [diff] [review] update clang to 154343 on os x https://hg.mozilla.org/build/braindump/rev/a9854316c124
Attachment #613390 -
Flags: checked-in+
Comment 10•12 years ago
|
||
The builds are ready for deployment. Depending on the status of Firefox 12.0b4 release, I'm going to deploy them tonight or tomorrow. http://people.mozilla.org/~raliiev/clang/
Assignee: respindola → rail
Priority: -- → P2
Updated•12 years ago
|
Attachment #613490 -
Flags: review?(rail) → review+
Comment 11•12 years ago
|
||
Comment on attachment 613490 [details] [diff] [review] puppet manifest for r154343 http://hg.mozilla.org/build/puppet-manifests/rev/784dcbbb21cd
Attachment #613490 -
Flags: checked-in+
Comment 12•12 years ago
|
||
Rafael, it turned out that we're running out of space on some builders after installing another clang version in parallel. We need to do one the following ASAP: 1) remove the current one, rebuild new version with the same name so it replaces the old one 2) remove the old one
Assignee | ||
Comment 13•12 years ago
|
||
I will try to push to try right now
Assignee | ||
Comment 14•12 years ago
|
||
Assignee: rail → respindola
Attachment #614603 -
Flags: review?(rail)
Assignee | ||
Comment 15•12 years ago
|
||
https://tbpl.mozilla.org/?tree=Try&rev=c60e92dfe9fa
Assignee | ||
Comment 16•12 years ago
|
||
And now with -Werror disabled because I don't have time to fix our code at the moment https://tbpl.mozilla.org/?tree=Try&rev=3428e6faedb8
Assignee | ||
Comment 17•12 years ago
|
||
The tests are looking good so far. Rail, would you mind submitting the new patch if you find it OK? Thanks.
Assignee | ||
Comment 18•12 years ago
|
||
BTW, is the low disk problem a transient thing or we have to find a better way of testing package updates?
Updated•12 years ago
|
Attachment #614603 -
Flags: review?(rail) → review+
Comment 19•12 years ago
|
||
Comment on attachment 614603 [details] [diff] [review] remove old package (if the try run is ok) http://hg.mozilla.org/build/puppet-manifests/rev/7cdfa6d4e0a2
Attachment #614603 -
Flags: checked-in+
Updated•12 years ago
|
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Updated•11 years ago
|
Product: mozilla.org → Release Engineering
You need to log in
before you can comment on or make changes to this bug.
Description
•