Closed Bug 1384422 Opened 7 years ago Closed 7 years ago

Migrate clang to toolchain dependencies

Categories

(Firefox Build System :: General, enhancement)

enhancement
Not set
normal

Tracking

(firefox56 fixed)

RESOLVED FIXED
mozilla56
Tracking Status
firefox56 --- fixed

People

(Reporter: glandium, Assigned: glandium)

References

Details

Attachments

(2 files)

      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: 7 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.