799648 - Move MozbuildObject functionality into mach mixins

Status: RESOLVED FIXED

(Firefox Build System :: Mach Core, enhancement)

Description

[Gregory Szorc [:gps]]

A consequence of mach's birth as an alternative front-end to mozilla-central's build system has left it tightly coupled with mozbuild, the Python package containing mozilla-central's build system Python code.

This coupling isn't desired and is constraining the usefulness of mach to other projects, like B2G.

I'll be dropping a bunch of patches that reshape mach into a standalone piece of software. Most of these patches are simply lifting code from one package to another. So, the review burden shouldn't be too high.

Comment 1

[Gregory Szorc [:gps]]

Attachment: Part 1: Move logging and process execution into mixin classes, v1

This patch moves logging and process execution functionality from mozbuild into mach.

In the new world, useful mach features are exposed as mixin classes. These are classes which simply provide useful methods but aren't important for class hierarchy. If someone is interested in e.g. launching a process, they inherit from the ProcessExecutionMixin.

This patch breaks some random mach commands in the tree which rely on _run_command(), which has been renamed to run_process(). These will all be updated in a subsequent patch which mass fixes all the breakage as part of the porting.

Comment 2

[Jeff Hammel]

Comment on attachment 669704 [details] [diff] [review]
Part 1: Move logging and process execution into mixin classes, v1

This is fine for now.  I would like to get a lot of this to mozlog and then use that

Comment 3

[Gregory Szorc [:gps]]

Attachment: Part 2: Move log manager into mach, v1

Move the core log management code into mach.

This file can eventually be uplifted to mozlog with little effort. Let's wait until mach stabilizes first, however.

Comment 4

[Gregory Szorc [:gps]]

Attachment: Part 3: Move config module into mach, v1

Moving the config module into mach. Pretty straightforward.

This actually breaks gettext integration. I have a solution for that in the next patch...

Comment 5

[Gregory Szorc [:gps]]

Attachment: Part 4: Disable mach config files, v1

This is a temporary patch to disable config file integration with mach.

The main reason for this is config files currently don't do anything. The only setting is the number of threads to pass to make. And, since no make targets that any mach commands currently execute need this, it is worthless and just leads to confusion.

The secondary (and arguably as important) reason is that settings integration is not very well thought out. It needs some core improvements to integrate better with command providers and with a yet-to-be-refactored MozbuildObject. Stay tuned.

Comment 6

[Gregory Szorc [:gps]]

https://hg.mozilla.org/mozilla-central/rev/dda561124c61
https://hg.mozilla.org/mozilla-central/rev/66d59a4d5a1b
https://hg.mozilla.org/mozilla-central/rev/0ccb7a8a8e5b
https://hg.mozilla.org/mozilla-central/rev/c0977caba1d6

Comment 7

[:Ms2ger (he/him; ⌚ UTC+1/+2)]

https://hg.mozilla.org/mozilla-central/rev/07919b62b9cf