Open
Bug 1226938
Opened 9 years ago
Updated 2 years ago
Update clang-format binaries for mach
Categories
(Firefox Build System :: General, defect)
Firefox Build System
General
Tracking
(Not tracked)
NEW
People
(Reporter: poiru, Unassigned)
Details
mach currently downloads an outdated version of clang-format, which lacks many of the Mozilla style improvements I've worked on (see bug 961541).
Right now[0], we download the binaries from https://people.mozilla.org/~ajones/clang-format/, which seems questionable. gps, do we have a better place for storing binaries?
[0]: https://dxr.mozilla.org/mozilla-central/source/tools/mach_commands.py#362
Flags: needinfo?(gps)
Reporter | ||
Comment 1•9 years ago
|
||
Perhaps we should create something like http://archive.mozilla.org/pub/gecko-tools/?
Comment 2•9 years ago
|
||
I'm almost positive I raised objections to the people.mozilla.org URL when I initially reviewed this code. By all means we should establish something more official.
archive.mozilla.org certainly seems reasonable.
Other build peers have done more with uploading tools than I have. They might be able to suggest an existing and appropriate location, perhaps even tooltool.
Component: mach → Build Config
Flags: needinfo?(gps)
Comment 3•9 years ago
|
||
What upstream clang-format minimum version is usable for Mozilla style formatting? Perhaps we could add a prompt to tell the user to install via the system's package manager so that we don't need to host the binaries ourselves?
Flags: needinfo?(birunthan)
Comment 4•9 years ago
|
||
The binaries should be uploaded to tooltool. See https://wiki.mozilla.org/ReleaseEngineering/Applications/Tooltool#How_To_Upload_To_Tooltool
Reporter | ||
Comment 5•9 years ago
|
||
(In reply to Ting-Yu Lin [:TYLin] (UTC+8) from comment #3)
> What upstream clang-format minimum version is usable for Mozilla style
> formatting? Perhaps we could add a prompt to tell the user to install via
> the system's package manager so that we don't need to host the binaries
> ourselves?
We have to use a consistent version of Clang-Format as its output changes from version to version. I haven't had much free time lately, but I'll try to get our version of Clang-Format upgraded in the near future.
Sorry for the (very) late reply!
Flags: needinfo?(birunthan)
Updated•7 years ago
|
Product: Core → Firefox Build System
Updated•2 years ago
|
Severity: normal → S3
You need to log in
before you can comment on or make changes to this bug.
Description
•