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

Fix the python3 directory so scripts work on Win32 #6472

Merged
merged 2 commits into from
Aug 30, 2019

Conversation

earlephilhower
Copy link
Collaborator

No description provided.

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.
@earlephilhower earlephilhower added this to the 2.6.0 milestone Aug 30, 2019
@earlephilhower
Copy link
Collaborator Author

Fixes #6467

I'm trying to create clean VMs to ensure a fresh install works under both Windows and Linux. From what I can see, I'm not sure how the Linux side was working since it was installing the python3 symlink under tools/python/python3 and platform.txt called for tools/python3/python3. Very strange...

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.
The get.py renamer uses some logic which results in the Win32 extracted
directory always ending up as `tools/python` and not `tools/python3`.
Adjust the zip archive and title to work around this (and the IDE
proper) issue.
@earlephilhower
Copy link
Collaborator Author

Verified a build on install into a clean Win32 and Ubuntu 18.04 VM. Good to go.

@dok-net
Copy link
Contributor

dok-net commented Aug 31, 2019

@earlephilhower Please update the gitignore entry, too.

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

Successfully merging this pull request may close these issues.

2 participants