Closed Bug 1356933 Opened 8 years ago Closed 8 years ago

Use tooltool-download.sh helper in cctools build scripts

Categories

(Firefox Build System :: General, enhancement)

enhancement
Not set
normal

Tracking

(firefox55 fixed)

RESOLVED FIXED
mozilla55
Tracking Status
firefox55 --- fixed

People

(Reporter: glandium, Assigned: glandium)

References

Details

Attachments

(1 file)

No description provided.
Blocks: 1356140
Blocks: 1356683
Comment on attachment 8858685 [details] Bug 1356933 - Use tooltool-download.sh helper in cctools build scripts. https://reviewboard.mozilla.org/r/130686/#review133510
Attachment #8858685 - Flags: review?(cmanchester) → review+
Pushed by mh@glandium.org: https://hg.mozilla.org/integration/autoland/rev/14010d77d56a Use tooltool-download.sh helper in cctools build scripts. r=chmanchester
backed out for invisible bustage, which was supposed to be triggered by the same patch, but didn't happen https://hg.mozilla.org/integration/autoland/rev/92f94fc0993b
Flags: needinfo?(mh+mozilla)
So, somehow those jobs, on top of being hidden on treeherder, don't have access to relengapi...
Flags: needinfo?(mh+mozilla)
Interesting mess. - The linux build failed because of a missing tooltool-downloads: public in the taskcluster definition - The mac build failed because the cctools it uses has a rpath pointing to /home/worker/workspace/clang/lib, which doesn't exist after the patch, since clang is now unpacked in /home/worker/workspace/build/src/clang. A few funny notes about the whole situation: - Cross-mac builds work with the same cctools because they set LD_LIBRARY_PATH - Ironically, after the patch, cctools has a rpath pointing to /home/worker/workspace/build/src/clang/lib, and it all worked fine on my try pushes because... I had bug 1313111 pushed along, making the mac cctools build use the result of the linux cctools build... Let's go with a LD_LIBRARY_PATH in the meanwhile...
Filed bug 1357317 for a long term solution.
Attachment #8858685 - Flags: review+ → review?(cmanchester)
Can you autoland this when you r+ it?
Flags: needinfo?(cmanchester)
Comment on attachment 8858685 [details] Bug 1356933 - Use tooltool-download.sh helper in cctools build scripts. https://reviewboard.mozilla.org/r/130686/#review134400
Attachment #8858685 - Flags: review?(cmanchester) → review+
Yes!
Flags: needinfo?(cmanchester)
Pushed by cmanchester@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/678b26af371a Use tooltool-download.sh helper in cctools build scripts. r=chmanchester
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla55
Product: Core → Firefox Build System
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: