Skip to content

Commit

Permalink
Setup: Fix unicode path related installation errors (#5678).
Browse files Browse the repository at this point in the history
Pip prefers to install sdists by building a .whl then
installing that. If the `wheel` package is not installed
then it defaults to the older `python setup.py install`
method. This older method breaks if it encounters unicode
paths so ideally we want to `wheel` installed before
attempting to install PyInstaller itself. PEP517's
build-system.requires adds this option.

Prevents errors like #5670.
  • Loading branch information
bwoodsend committed Mar 30, 2021
1 parent d71a9ff commit 6df79e6
Show file tree
Hide file tree
Showing 2 changed files with 12 additions and 0 deletions.
2 changes: 2 additions & 0 deletions news/5678.bugfix.rst
@@ -0,0 +1,2 @@
Fix installation issues stemming from unicode characters in
file paths.
10 changes: 10 additions & 0 deletions pyproject.toml
Expand Up @@ -64,3 +64,13 @@
directory = "build"
name = "Bootloader build"
showcontent = true

[build-system]
# Tells pip to install wheel before trying to install PyInstaller
# from an sdist or from Github.
# Installing without wheel uses legacy `python setup.py install`
# which has issues with unicode paths.
requires = [
"wheel",
"setuptools",
]

0 comments on commit 6df79e6

Please sign in to comment.