1234754 - ./mach build directory broken on Windows.

Status: RESOLVED WONTFIX

Description

[Mark Hammond [:markh] (PTO until Dec 4)]

Created attachment 8701335 [details]
|./mach build services -v| output

This may or may not be related to bug 1234439.

STR:
* open services/sync/Weave.js and add a dump() statement at the top of the file.
* |./mach build services && ./mach run| - note the dump statement is not printed.
* |./mach build && ./mach run| - the dump is printed.

Previously either |./mach build services| or |./mach build services/sync| would do the right thing. Unlike bug 1234439, no error is printed and the build thinks it worked, but the js changes aren't being picked up.

Comment 1

[Mike Hommey [:glandium]]

Since bug 1229241, EXTRA_COMPONENTS (which Weave.js is) are treated by install manifests, at the top-level, which means mach build subdirectory doesn't do anything for them.

mach build subdirectory has never done everything people expects from it, and with time, it does less and less. This is not something we're going to change. Bug 1234439 is unfortunate, but mach build binaries and mach build faster are the only currently really supported ways of shortcutting the build system.