Closed Bug 1343713 Opened 3 years ago Closed 3 years ago

Avoid mozconfig failures when tooltool packages have not been unpacked yet

Categories

(Firefox Build System :: General, defect)

defect
Not set

Tracking

(firefox55 fixed)

RESOLVED FIXED
mozilla55
Tracking Status
firefox55 --- fixed

People

(Reporter: glandium, Assigned: glandium)

References

Details

Attachments

(2 files)

For bug 1313111, I want to introduce a mach command to, first, replace tooltool (a wrapper of some sort), that would be augmented to download taskcluster artifacts from toolchain tasks for toolchains that come from there.

Using mach ends up evaluating mozconfigs, and doing that before the tooltool packages have been downloaded and unpacked currently fails because the mozconfigs exit with errors when the toolchain directories are not there (In most case, this is because of a (cd $path; pwd) construct for windows)
Comment on attachment 8842679 [details]
Bug 1343713 - Avoid mozconfig failure when sccache is not there.

https://reviewboard.mozilla.org/r/116442/#review118690

Things are definitely better now that we do sensible checks in moz.configure!
Attachment #8842679 - Flags: review?(ted) → review+
Comment on attachment 8842680 [details]
Bug 1343713 - Avoid mozconfig failures when clang-cl/msvc are not present.

https://reviewboard.mozilla.org/r/116444/#review118692

So this will get re-evaluated in `mach build` after your tooltool-replacement runs and the tools are present, right?

It'd be nice to figure out some less-ugly ways to deal with the mozconfig logic these days. Using shell scripts is just not that good.
Attachment #8842680 - Flags: review?(ted) → review+
Pushed by mh@glandium.org:
https://hg.mozilla.org/integration/autoland/rev/1dcda6d41df1
Avoid mozconfig failure when sccache is not there. r=ted
https://hg.mozilla.org/integration/autoland/rev/bc2f8a0c7d9a
Avoid mozconfig failures when clang-cl/msvc are not present. r=ted
https://hg.mozilla.org/mozilla-central/rev/1dcda6d41df1
https://hg.mozilla.org/mozilla-central/rev/bc2f8a0c7d9a
Status: NEW → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla55
Product: Core → Firefox Build System
You need to log in before you can comment on or make changes to this bug.