This is an archive of the discontinued Mercurial Phabricator instance.

Differential D3958

Allow to run setup.py with python 3 without a mercurial checkout
ClosedPublic

Authored by glandium on Jul 16 2018, 8:43 PM.

Details

Reviewers
indygreg
Group Reviewers
hg-reviewers
Commits
rHGdbbe45ae5ad1: setup: allow to run setup.py with python 3 without a mercurial checkout
rHG7d8aa0746b71: Allow to run setup.py with python 3 without a mercurial checkout
rHGd5b31ae9cc0a: Allow to run setup.py with python 3 without a mercurial checkout
Summary

Some people may want to test mercurial in a python 3 environment through e.g. pip, in which case setup.py doesn't run in a mercurial checkout, so the hack in setup.py to allow python 3 cannot be overcome.

This change allows a manual override with the HGPYTHON3 environment variable.

Additionally, when for some reason the version is unknown (for crazy people like me, who have a git checkout of the mercurial repo), the version variable ends up being an unicode string, which fails the isinstance(version, bytes) assertion. So fix that at the same time.

Diff Detail

Repository
rHG Mercurial
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

glandium created this revision.Jul 16 2018, 8:43 PM
indygreg accepted this revision.Jul 16 2018, 8:47 PM

I'm OK with this undocumented hack. And there are cases where we may want to test setup.py outside the context of a checkout as well. The important thing is end-users not running Mercurial with Python 3 unknowingly.

The only thing I could bikeshed here is the naming. History tells us that someone will find this hack and e.g. post it to stackoverflow without the context that Mercurial is unsupported on Python 3. Perhaps a name like HGALLOWUNSUPPORTEDPYTHON3 would be better?

I'll queue this for now. We can change the name easily enough if others want that.

This revision is now accepted and ready to land.Jul 16 2018, 8:47 PM
Closed by commit rHGd5b31ae9cc0a: Allow to run setup.py with python 3 without a mercurial checkout (authored by indygreg). · Explain WhyJul 16 2018, 8:48 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathPackages
Msetup.py (4 lines)

Diff 9615

setup.py

# We don't yet officially support Python 3. But we want to allow developers to # We don't yet officially support Python 3. But we want to allow developers to
# hack on. Detect and disallow running on Python 3 by default. But provide a # hack on. Detect and disallow running on Python 3 by default. But provide a
# backdoor to enable working on Python 3. # backdoor to enable working on Python 3.
if sys.version_info[0] != 2: if sys.version_info[0] != 2:
badpython = True badpython = True
# Allow Python 3 from source checkouts. # Allow Python 3 from source checkouts.
if os.path.isdir('.hg'): if os.path.isdir('.hg') or 'HGPYTHON3' in os.environ:
badpython = False badpython = False
if badpython: if badpython:
error = """ error = """
Mercurial only supports Python 2.7. Mercurial only supports Python 2.7.
Python {py} detected. Python {py} detected.
Please re-run with Python 2.7. Please re-run with Python 2.7.
""".format(py=sys.version_info) """.format(py=sys.version_info)
])) ]))
try: try:
oldpolicy = os.environ.get('HGMODULEPOLICY', None) oldpolicy = os.environ.get('HGMODULEPOLICY', None)
os.environ['HGMODULEPOLICY'] = 'py' os.environ['HGMODULEPOLICY'] = 'py'
from mercurial import __version__ from mercurial import __version__
version = __version__.version version = __version__.version
except ImportError: except ImportError:
version = 'unknown' version = b'unknown'
finally: finally:
if oldpolicy is None: if oldpolicy is None:
del os.environ['HGMODULEPOLICY'] del os.environ['HGMODULEPOLICY']
else: else:
os.environ['HGMODULEPOLICY'] = oldpolicy os.environ['HGMODULEPOLICY'] = oldpolicy
class hgbuild(build): class hgbuild(build):
# Insert hgbuildmo first so that files in mercurial/locale/ are found # Insert hgbuildmo first so that files in mercurial/locale/ are found