X Tutup
The Wayback Machine - https://web.archive.org/web/20201204225805/https://github.com/PythonTurtle/PythonTurtle/pull/153
Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

pip install wxPython before building the Windows EXE #153

Open
wants to merge 1 commit into
base: master
from

Conversation

@cclauss
Copy link
Contributor

@cclauss cclauss commented Sep 25, 2020

This pull request pip installs wxPython before pyinstaller creates the Windows EXE file.

@cclauss cclauss changed the title pip install wx Python bfore building the Windows EXE pip install wxPython bfore building the Windows EXE Sep 25, 2020
@cclauss cclauss changed the title pip install wxPython bfore building the Windows EXE pip install wxPython before building the Windows EXE Sep 25, 2020
@cclauss cclauss mentioned this pull request Sep 25, 2020
@bittner
Copy link
Member

@bittner bittner commented Sep 25, 2020

Just some minor feedback: (please don't be offended!)

This pull request pip installs wxPython

This describes the "what". That can be seen by the code changes anyway.

Try to describe the "why" instead! – It should explain what lead to this PR. Which problem do the changes herein solve. Maybe even refer to evidence, e.g. a failed build or a discussion that triggered making the changes.

@bittner
Copy link
Member

@bittner bittner commented Sep 25, 2020

Side note:

As a reference, the build on AppVeyor creates a working Windows EXE, IIRC. Only the MSI was "empty" (which is discussed in detail in #107 (comment)).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants
You can’t perform that action at this time.
X Tutup