Closed Bug 1525876 Opened 1 year ago Closed 1 year ago

Could the FILES_PER_UNIFIED_FILES setting inherit its value from parent directories?

Categories

(Firefox Build System :: General, enhancement)

enhancement
Not set
normal

Tracking

(firefox67 fixed)

RESOLVED FIXED
mozilla67
Tracking Status
firefox67 --- fixed

People

(Reporter: bbouvier, Assigned: bbouvier)

Details

(Keywords: in-triage)

Attachments

(2 files)

I've had to push two follow-up fixes to a busted inbound tree because of non-unified build. The first one was because I didn't try FILES_PER_UNIFIED_FILES=1 in js/src/moz.build; fine. But the second one should not have happened: it is because now every subdirectory of js/src/ has its own moz.build file with its own FILES_PER_UNIFIED_FILES variable. So now, to test non-unified compilation, we need to change N+1 moz.build files where N is the number of subdirectories with their own moz.build files.

Instead, could we just have the subdirectories inherit the FILES_PER_UNIFIED_FILES variable from the parent dir? That would make it easier to test non-unified build.

(Since my goal is really to make non-unified build testing easier, anything that goes in that direction would be helpful too, independently of the chosen solution)

export in js/src/moz.build might be able to be used for this.

Before this patch, changing FILES_PER_UNIFIED_FILES in the directory would
require changes to all the files that defined the sandbox variable, which is
a bit misleading. Exporting the variable prevents this, and it is safe to use
because it doesn't escape the scope of the js/src build directory.

Pushed by bbouvier@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/1bb3f6cdfa73
Export FILES_PER_UNIFIED_FILES in js/src/moz.build to avoid redefining it; r=froydnj
https://hg.mozilla.org/integration/autoland/rev/9b872b266b2e
Move ENABLE_WASM_CRANELIFT build test in the wasm mozbuild; r=tcampbell
Status: NEW → RESOLVED
Closed: 1 year ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla67
Assignee: nobody → bbouvier
You need to log in before you can comment on or make changes to this bug.