Closed Bug 1181342 Opened 7 years ago Closed 7 years ago

Add clang 3.6 builds for TSan

Categories

(Release Engineering :: General, defect)

defect
Not set
normal

Tracking

(firefox42 fixed)

RESOLVED FIXED
Tracking Status
firefox42 --- fixed

People

(Reporter: chmanchester, Assigned: chmanchester)

References

Details

Attachments

(2 files)

I got as far as figuring out I don't have the appropriate permissions to upload to tooltool and wondering whether I should be able to run build-clang.py locally.
rail, can you help me figure out how to do this? Thanks!
Flags: needinfo?(rail)
Sure!

We have a wiki page on how to build clang here: https://wiki.mozilla.org/ReleaseEngineering/How_To/Clang_update

Tooltool is described here: https://wiki.mozilla.org/ReleaseEngineering/Applications/Tooltool

Let me know if the docs are missing something.
Flags: needinfo?(rail)
Bug 1181342 - tooltool manifests and build-clang config for clang 3.6 r=rail
Attachment #8632178 - Flags: review?(rail)
I got a working build out of https://treeherder.mozilla.org/#/jobs?repo=try&revision=0266fc9845f6
Assignee: nobody → cmanchester
(In reply to Rail Aliiev [:rail] from comment #2)
> Sure!
> 
> We have a wiki page on how to build clang here:
> https://wiki.mozilla.org/ReleaseEngineering/How_To/Clang_update
> 
> Tooltool is described here:
> https://wiki.mozilla.org/ReleaseEngineering/Applications/Tooltool
> 
> Let me know if the docs are missing something.

The only hiccup was the VM was set up with gcc 4.7.2, but the build-clang script mentioned gcc 4.7.3.
Comment on attachment 8632178 [details]
MozReview Request: Bug 1181342 - tooltool manifests and build-clang config for clang 3.6 r=rail

https://reviewboard.mozilla.org/r/13005/#review11591

Ship It!
Attachment #8632178 - Flags: review?(rail) → review+
https://reviewboard.mozilla.org/r/13005/#review11925

::: browser/config/tooltool-manifests/linux64/tsan.manifest:9
(Diff revision 1)
> +"filename": "setup.sh"

Just when I removed all setup.sh from tooltool manifests in bug 1182407 :(
Comment on attachment 8633840 [details] [diff] [review]
Follow up to use the unpack feature of tooltool instead of setup.sh

Review of attachment 8633840 [details] [diff] [review]:
-----------------------------------------------------------------

Thanks
Attachment #8633840 - Flags: review?(mh+mozilla) → review+
Component: General Automation → General
You need to log in before you can comment on or make changes to this bug.