Closed
Bug 1471486
Opened 6 years ago
Closed 6 years ago
Have mach detect rustfmt and use that for bindgen
Categories
(Firefox Build System :: General, enhancement)
Firefox Build System
General
Tracking
(firefox63 fixed)
RESOLVED
FIXED
mozilla63
Tracking | Status | |
---|---|---|
firefox63 | --- | fixed |
People
(Reporter: xidorn, Assigned: xidorn)
Details
Attachments
(2 files)
I think there is a value to have rustfmt run automatically in local build as well.
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Assignee | ||
Updated•6 years ago
|
Attachment #8988074 -
Flags: review?(core-build-config-reviews)
Comment 3•6 years ago
|
||
mozreview-review |
Comment on attachment 8988075 [details] Bug 1471486 part 2 - Use RUSTFMT env in stylo build script. https://reviewboard.mozilla.org/r/253330/#review259964 Code-wise looks good, but note that glandium asked for it to be optional in https://bugzilla.mozilla.org/show_bug.cgi?id=1432153#c4 explicitly. So you may want to check with him or what not.
Attachment #8988075 -
Flags: review?(emilio) → review+
Updated•6 years ago
|
Attachment #8988074 -
Flags: review?(core-build-config-reviews) → review?(nfroyd)
Comment 4•6 years ago
|
||
mozreview-review |
Comment on attachment 8988074 [details] Bug 1471486 part 1 - Detect rustfmt like other rust executables. https://reviewboard.mozilla.org/r/253328/#review260006
Attachment #8988074 -
Flags: review?(nfroyd) → review+
Assignee | ||
Comment 5•6 years ago
|
||
glandium, since emilio said you asked running rustfmt on generated files to be optional in bug 1432153 comment 4, what do you think about this? This is still optional, via explicitly clearing RUSTFMT in mozconfig, or just remove rustfmt command from PATH. Do you think it's enough for what you were concerned in bug 1432153 comment 4?
Flags: needinfo?(mh+mozilla)
Assignee | ||
Comment 7•6 years ago
|
||
OK, thanks!
Pushed by xquan@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/799ee5183d48 part 1 - Detect rustfmt like other rust executables. r=froydnj https://hg.mozilla.org/integration/autoland/rev/765b972ec1c1 part 2 - Use RUSTFMT env in stylo build script. r=emilio
Comment 9•6 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/799ee5183d48 https://hg.mozilla.org/mozilla-central/rev/765b972ec1c1
Status: NEW → RESOLVED
Closed: 6 years ago
status-firefox63:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla63
You need to log in
before you can comment on or make changes to this bug.
Description
•