Migrate clang to toolchain dependencies

RESOLVED FIXED in Firefox 56

Status

enhancement
RESOLVED FIXED
2 years ago
2 years ago

People

(Reporter: glandium, Assigned: glandium)

Tracking

unspecified
mozilla56
Dependency tree / graph

Firefox Tracking Flags

(firefox56 fixed)

Details

Attachments

(2 attachments)

No description provided.
Data for the migration helper attached to bug 1313111:
KNOWN_TOOLCHAINS = {
    'linux64-clang': '52f3fc23f0f5',
    'win32-clang': '44dee70d525e',
    'win64-clang': 'cd3ed31acefd',
}

This leaves out fuzzing and static analysis builds, that are using, respectively, clang 4.0.1 and clang 3.8.
Err, same player shoot again:
KNOWN_TOOLCHAINS = {
    'linux64-clang': '52f3fc23f0f5',
    'win32-clang-cl': 'cd3ed31acefd',
    'win64-clang-cl': '44dee70d525e',
}
Comment on attachment 8890225 [details]
Bug 1384422 - Use toolchain dependencies for clang.

https://reviewboard.mozilla.org/r/161328/#review167096

::: taskcluster/ci/toolchain/linux.yml
(Diff revision 2)
>              - 'build/build-clang/**'
>              - 'taskcluster/scripts/misc/tooltool-download.sh'
>          toolchain-artifact: public/build/clang.tar.xz
>      toolchains:
>          - linux64-gcc
> -        - linux64-sccache

This hunk feels out of place. I trust you to triage appropriately before landing.
Attachment #8890225 - Flags: review?(gps) → review+
Comment on attachment 8890226 [details]
Bug 1384422 - Remove clang entries from tooltool manifests.

https://reviewboard.mozilla.org/r/161330/#review167098
Attachment #8890226 - Flags: review?(gps) → review+
Pushed by mh@glandium.org:
https://hg.mozilla.org/integration/autoland/rev/cc2502e91e16
Use toolchain dependencies for clang. r=gps
https://hg.mozilla.org/integration/autoland/rev/9ea98e47b682
Remove clang entries from tooltool manifests. r=gps
https://hg.mozilla.org/mozilla-central/rev/cc2502e91e16
https://hg.mozilla.org/mozilla-central/rev/9ea98e47b682
Status: NEW → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla56
See Also: → 1380171
Product: Core → Firefox Build System
You need to log in before you can comment on or make changes to this bug.