Closed
Bug 706135
Opened 13 years ago
Closed 13 years ago
upgrade clang on the slaves
Categories
(Release Engineering :: General, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: espindola, Assigned: rail)
References
Details
Attachments
(3 files, 3 obsolete files)
2.01 KB,
patch
|
rail
:
review+
espindola
:
checked-in+
|
Details | Diff | Splinter Review |
551 bytes,
patch
|
rail
:
review+
espindola
:
checked-in+
|
Details | Diff | Splinter Review |
2.28 KB,
patch
|
rail
:
review+
rail
:
checked-in+
|
Details | Diff | Splinter Review |
No description provided.
Attachment #577640 -
Flags: review?(rail)
Reporter | ||
Comment 1•13 years ago
|
||
Reporter | ||
Comment 2•13 years ago
|
||
Attachment #577642 -
Flags: review?(rail)
Reporter | ||
Comment 3•13 years ago
|
||
this is similar to the previous patch, but install clang in a versioned directory, like we do with gcc.
Attachment #577640 -
Attachment is obsolete: true
Attachment #577640 -
Flags: review?(rail)
Attachment #577722 -
Flags: review?(rail)
Reporter | ||
Comment 4•13 years ago
|
||
Attachment #577641 -
Attachment is obsolete: true
Attachment #577641 -
Flags: review?(rail)
Attachment #577727 -
Flags: review?(rail)
Reporter | ||
Comment 5•13 years ago
|
||
Attachment #577642 -
Attachment is obsolete: true
Attachment #577642 -
Flags: review?(rail)
Attachment #577731 -
Flags: review?(rail)
Assignee | ||
Updated•13 years ago
|
Attachment #577722 -
Flags: review?(rail) → review+
Assignee | ||
Updated•13 years ago
|
Attachment #577727 -
Flags: review?(rail) → review+
Assignee | ||
Updated•13 years ago
|
Attachment #577731 -
Flags: review?(rail) → review+
Reporter | ||
Updated•13 years ago
|
Attachment #577722 -
Flags: checked-in+
Reporter | ||
Updated•13 years ago
|
Attachment #577727 -
Flags: checked-in+
Assignee | ||
Comment 7•13 years ago
|
||
Comment on attachment 577731 [details] [diff] [review] update the puppet manifest http://hg.mozilla.org/build/puppet-manifests/rev/fe750c14a706
Attachment #577731 -
Flags: checked-in+
Assignee | ||
Comment 8•13 years ago
|
||
Deployed this weekend.
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Reporter | ||
Comment 9•13 years ago
|
||
Thanks a lot. A try of the new clang is at https://tbpl.mozilla.org/?tree=Try&rev=5bf05c228e27
Reporter | ||
Comment 10•13 years ago
|
||
I forgot we need -fgnu89-inline on centos5.A new try push is at https://tbpl.mozilla.org/?tree=Try&rev=a4c2bb5b1f3c
Reporter | ||
Comment 11•13 years ago
|
||
and now with 64 bit OS X too :-( https://tbpl.mozilla.org/?tree=Try&rev=c5a1d178b774
Reporter | ||
Comment 12•13 years ago
|
||
and now with bug 707648 fixed https://tbpl.mozilla.org/?tree=Try&rev=4ca3192706b0
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
•