Closed Bug 1385388 Opened 7 years ago Closed 7 years ago

Compilation failure doesn't tell me the exact command that's failing

Categories

(Firefox Build System :: General, defect)

56 Branch
x86_64
macOS
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED INVALID

People

(Reporter: kats, Unassigned)

References

Details

Attachments

(1 file)

Attached file Build output
Usually when the build fails it spits out the command that failed. However right now I'm getting a compilation failure on OS X and the build system isn't giving me exact command. This makes it hard to debug. I tried --verbose but that didn't help. See attached output.
Try -v instead of --verbose.
Same result. I also tried `mach -v build -v`, the only difference there was mach printed the environment variables at the beginning.
FWIW, the command that was failing was sccache - see bug 1385386 comment 1. I discovered that by running `make` in $OBJDIR/gfx/vr/openvr/
Here's the problem:

 0:03.10 Adding client.mk options from /Users/kats/zspace/mozilla-wr/.mozconfig:
 0:03.10     MOZ_MAKE_FLAGS=-s -j4

Your mozconfig is shooting you in the foot by a) ignoring your -j1 and b) ignoring your -v by adding -s
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → INVALID
We may want to nuke MOZ_MAKE_FLAGS from mozconfigs. Pretty much the only uses of it I see are for -s and -j. These both can be controlled via `mach build` arguments and 99% of the time the defaults (-j == #cores and -s) are what the user wants.
(In reply to Gregory Szorc [:gps] from comment #5)
> 99% of the time the defaults (-j == #cores and -s) are
> what the user wants.

Not if you're using IceCC [1].

[1] https://github.com/icecc/icecream
I've said it before and I'll say it again: we need to make the build system icecc aware so this stuff "just works." That may be bug 1313519.

At some point, people running -j128 because icecc will start complaining because you launch 128 expensive local processes and DoS your machine. I'm kinda surprised all the Rust we're adding hasn't made this a problem yet...
> I'm kinda surprised all the Rust we're adding hasn't made this a problem yet...

We're fortunate that either cargo is not very good at parallelising, or the crate dependencies make massive parallelism impossible at the moment. (or both)
(In reply to Mike Hommey [:glandium] from comment #4)
> Your mozconfig is shooting you in the foot by a) ignoring your -j1 and b)
> ignoring your -v by adding -s

Whoops, sorry! I verified that removing the stuff from my mozconfig fixes both (a) and (b). The -v option does indeed report the compilation failure command.
Product: Core → Firefox Build System
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: