./mach build directory broken on Windows.

RESOLVED WONTFIX

Status

RESOLVED WONTFIX
3 years ago
9 months ago

People

(Reporter: markh, Unassigned)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

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.
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.
Status: NEW → RESOLVED
Last Resolved: 3 years ago
Resolution: --- → WONTFIX

Updated

9 months ago
Product: Core → Firefox Build System
You need to log in before you can comment on or make changes to this bug.