Closed Bug 1671424 Opened 10 months ago Closed 10 months ago

Set the `PYTHON3` environment variable where appropriate for the build


(Firefox Build System :: General, defect)



(firefox84 fixed)

84 Branch
Tracking Status
firefox84 --- fixed


(Reporter: rstewart, Assigned: glandium)




(1 file, 1 obsolete file)

No description provided.
See Also: → 1669934

This is necessary because at various points we cross over between Python and shell scripts (because the migration in bug 1247781 isn't complete yet). The shell scripts generally use the PYTHON3 environment variable to determine which Python to use. In the absence of this environment variable, it will generally use the Python at the head of the PATH, which is usually the incorrect behavior.

Assignee: nobody → rstewart

mach configure currently runs the equivalent to make -f
This is history, and essentially does the following:

  • Create configure and js/src/configure from and
    js/src/ respectively.
  • Create the objdir.
  • Run configure from the objdir.

The configure script is, nowadays, only really used as a means to set
OLD_CONFIGURE (and also for people who want to run configure,
literally, as in the configure; make workflow). mach configure
actually doesn't need it. Neither does recursing into js/src require
js/src/configure, since bug 1520340 (and now as of bug 1669633, we
don't even recurse).

Because can actually derive OLD_CONFIGURE on its own
(except for js/src/configure, but mach configure doesn't run that),
we don't really need configure for mach configure.

So all in all, we're at a point in history where it's straightforward to
just initiate from mach configure, so we just do that.

And in the hypothetical case where the mach configure code is somehow
running in python2, we get the mach virtualenv python3 and use it to

Assignee: rstewart → mh+mozilla
Attachment #9181799 - Attachment is obsolete: true
Pushed by
Move configure execution from to `mach configure`. r=firefox-build-system-reviewers,rstewart
Flags: needinfo?(mh+mozilla)
Pushed by
Move configure execution from to `mach configure`. r=firefox-build-system-reviewers,rstewart
Closed: 10 months ago
Resolution: --- → FIXED
Target Milestone: --- → 84 Branch
You need to log in before you can comment on or make changes to this bug.