Reuse the clang-tidy tc job to install clang-format

RESOLVED FIXED in Firefox 58

Status

()

Core
Rewriting and Analysis
RESOLVED FIXED
19 days ago
17 days ago

People

(Reporter: sylvestre, Assigned: sylvestre)

Tracking

(Blocks: 1 bug)

Trunk
mozilla58
Points:
---

Firefox Tracking Flags

(firefox58 fixed)

Details

MozReview Requests

()

Submitter Diff Changes Open Issues Last Updated
Loading...
Error loading review requests:

Attachments

(1 attachment, 1 obsolete attachment)

Comment hidden (empty)
Comment hidden (mozreview-request)
Comment hidden (mozreview-request)

Comment 3

18 days ago
mozreview-review
Comment on attachment 8925479 [details]
Bug 1414781 - Rename some stuffs from clang-tidy to clang-toolchain as we are using it also for clang-format now

https://reviewboard.mozilla.org/r/196608/#review202110

Calling this clang-toolchain would be confusing, since it's *not* using the clang toolchain artifacts.
Attachment #8925479 - Flags: review?(mh+mozilla) → review-

Comment 4

18 days ago
mozreview-review
Comment on attachment 8925480 [details]
Bug 1414781 - Also install clang-format

https://reviewboard.mozilla.org/r/196610/#review202112
Attachment #8925480 - Flags: review?(mh+mozilla) → review+
Comment hidden (mozreview-request)
(Assignee)

Updated

18 days ago
Attachment #8925479 - Attachment is obsolete: true

Comment 6

18 days ago
Pushed by sledru@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/dd930811ae49
Also install clang-format r=glandium

Comment 7

17 days ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/dd930811ae49
Status: NEW → RESOLVED
Last Resolved: 17 days ago
status-firefox58: affected → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla58
You need to log in before you can comment on or make changes to this bug.