1844925 - Win32 2012 Rust Tests & searchfox idx should use sccache for real

Status: RESOLVED FIXED

(Firefox Build System :: Task Configuration, task)

Description

[Sylvestre Ledru [:Sylvestre]]

Bug 1844725 highlighted that sccache is configured for these tasks but it isn't use for real.
The build system probably needs some help with it!

Comment 1

[Mike Hommey [:glandium]]

Attachment: Bug 1844925 - Make it an error to have both SCCACHE_DISABLE and USE_SCCACHE set.

Comment 2

[Mike Hommey [:glandium]]

Attachment: Bug 1844925 - Disable sccache for searchfox by not enabling it in the first place.

Comment 3

[Mike Hommey [:glandium]]

Attachment: Bug 1844925 - Enable sccache on windows rust tests.

Comment 4

[Mike Hommey [:glandium]]

So, there actually are good reasons for sccache not to be enabled on these tasks.

• searchfox wouldn't be indexing the things that come from the cache.
• we don't have a storage set up for azure.

Comment 5

[Mike Hommey [:glandium]]

Attachment: Bug 1844925 - Error out when sccache is enabled but it can't actually be used.

Comment 6

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/433d189857bd
Make it an error to have both SCCACHE_DISABLE and USE_SCCACHE set. r=firefox-build-system-reviewers,ahochheiden
https://hg.mozilla.org/integration/autoland/rev/7b9a47f9a338
Disable sccache for searchfox by not enabling it in the first place. r=firefox-build-system-reviewers,ahochheiden
https://hg.mozilla.org/integration/autoland/rev/778036c0146f
Error out when sccache is enabled but it can't actually be used. r=firefox-build-system-reviewers,ahochheiden

Comment 7

[Cristina Horotan [:chorotan]]

https://hg.mozilla.org/mozilla-central/rev/433d189857bd
https://hg.mozilla.org/mozilla-central/rev/7b9a47f9a338
https://hg.mozilla.org/mozilla-central/rev/778036c0146f

Comment 8

[Cristina Horotan [:chorotan]]

https://hg.mozilla.org/mozilla-central/rev/433d189857bd
https://hg.mozilla.org/mozilla-central/rev/7b9a47f9a338
https://hg.mozilla.org/mozilla-central/rev/778036c0146f