Patchwork D7258: makefile: use Python 3 by default (BC)

login
register
mail settings
Submitter phabricator
Date Nov. 5, 2020, 4:13 p.m.
Message ID <522bdbfc0434ff2cad2179c2b0514953@localhost.localdomain>
Download mbox | patch
Permalink /patch/47547/
State Not Applicable
Headers show

Comments

phabricator - Nov. 5, 2020, 4:13 p.m.
Herald added a subscriber: mercurial-patches.
indygreg edited the summary of this revision.
indygreg retitled this revision from "makefile: use Python 3 by default outside of Windows (BC)" to "makefile: use Python 3 by default (BC)".
indygreg updated this revision to Diff 23421.

REPOSITORY
  rHG Mercurial

CHANGES SINCE LAST UPDATE
  https://phab.mercurial-scm.org/D7258?vs=18278&id=23421

BRANCH
  default

CHANGES SINCE LAST ACTION
  https://phab.mercurial-scm.org/D7258/new/

REVISION DETAIL
  https://phab.mercurial-scm.org/D7258

AFFECTED FILES
  Makefile

CHANGE DETAILS




To: indygreg, #hg-reviewers, martinvonz, mharbison72, durin42
Cc: mercurial-patches, yuja, mharbison72, durin42, dlax, martinvonz, mercurial-devel

Patch

diff --git a/Makefile b/Makefile
--- a/Makefile
+++ b/Makefile
@@ -5,7 +5,16 @@ 
 # % make PREFIX=/opt/ install
 
 export PREFIX=/usr/local
+
+# Default to Python 3.
+#
+# Windows ships Python 3 as `python.exe`.
+ifeq ($(OS),Windows_NT)
 PYTHON?=python
+else
+PYTHON?=python3
+endif
+
 $(eval HGROOT := $(shell pwd))
 HGPYTHONS ?= $(HGROOT)/build/pythons
 PURE=