Closed Bug 1732948 Opened 3 years ago Closed 3 years ago

Request a "mach create-mach-environment" if Mach venv out-of-date

Categories

(Firefox Build System :: Mach Core, enhancement)

enhancement

Tracking

(firefox96 fixed)

RESOLVED FIXED
96 Branch
Tracking Status
firefox96 --- fixed

People

(Reporter: mhentges, Assigned: mhentges)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

Note: currently, mach_venv.up_to_date() takes ~500ms to run, which is a lot for every Mach invocation. Let's land this after performance there improves (probably by removing the pip list call, using hashes instead, etc).

Blocks: 1717051

As we leverage the Mach environment more, it becomes increasingly
important that it isn't out-of-date on developer machines.

Add an up_to_date() check during Mach initialization.
To minimize the cost to startup, I'm skipping the "pip list" check.

This change required moving virtualenv from mozbuild to mach to
make it available during the early stage of Mach init.

Attachment #9243772 - Attachment description: Bug 1732948: Assert that the Mach venv isn't out-of-date during init → WIP: Bug 1732948: Assert that the Mach venv isn't out-of-date during init
Attachment #9243772 - Attachment description: WIP: Bug 1732948: Assert that the Mach venv isn't out-of-date during init → Bug 1732948: Assert that the Mach venv isn't out-of-date during init
Assignee: nobody → mhentges
Status: NEW → ASSIGNED

There's a r+ patch which didn't land and no activity in this bug for 2 weeks.
:mhentges, could you have a look please?
For more information, please visit auto_nag documentation.

Flags: needinfo?(mhentges)
Flags: needinfo?(ahal)

Probably landing this today, it was a dependent on the stack :)

Flags: needinfo?(mhentges)
Flags: needinfo?(ahal)
Pushed by mhentges@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/e68fffdead19
Assert that the Mach venv isn't out-of-date during init r=ahal
Status: ASSIGNED → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → 96 Branch
Regressions: 1738947
Regressions: 1739067
Regressions: 1739124
See Also: → 1741632
See Also: 1741632
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: