Page MenuHomePhabricator Dev

No bug: Move build environment checks to configure.py. r?#release-managers! r=phab-email-qa-reviewer-1
ClosedPublic

Authored by sheehan on Jan 21 2022, 6:09 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Oct 11, 10:10 AM
Unknown Object (File)
Thu, Sep 24, 5:29 AM
Unknown Object (File)
Jul 20 2026, 10:49 AM
Unknown Object (File)
Jun 29 2026, 3:22 AM
Unknown Object (File)
Jun 29 2026, 3:21 AM
Unknown Object (File)
Jun 24 2026, 9:05 PM
Unknown Object (File)
Jun 8 2026, 8:20 AM
Unknown Object (File)
Jun 5 2026, 7:58 AM
Subscribers

Details

Reviewers
phab-email-qa-reviewer-1
phab-email-qa-author
Group Reviewers
release-managers
Bugzilla Bug ID
1631233
Summary

We also avoid creating the virtualenv before those checks, and remove
the long useless check for build residue in the source directory (these
files will not have been created because builds that would create them
have been forbidden for years)

Original Revision: https://phabricator-dev.allizom.org/D3243

Depends on D3250

Diff Detail

Repository
rMC m-c
Branch
default

Unit TestsFailed

TimeTest
9,051 mscode-review::mercurial
WARNING: The code review bot failed to apply your patch.
patching file configure.py
Hunk #1 FAILED at 41
1 out of 2 hunks FAILED -- saving rejects to file configure.py.rej
abort: patch failed to apply