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

Windows embedable support #2353

Merged
merged 21 commits into from
Jun 25, 2022
Merged

Windows embedable support #2353

merged 21 commits into from
Jun 25, 2022

Conversation

reksar
Copy link
Contributor

@reksar reksar commented Jun 9, 2022

  • ran the linter to address style issues (tox -e fix_lint)
  • wrote descriptive pull request text
  • ensured there are test(s) validating the fix
  • added news fragment in docs/changelog folder
  • updated/extended the documentation

After these steps #1774 (comment)
the embeddable Python should works fine with virtualenv.

Virtualenv now ships zipped std lib packages along with other dll and pyd files.

@reksar reksar changed the title Windows embedable support #1774 Windows embedable support Jun 9, 2022
Copy link
Contributor

@gaborbernat gaborbernat left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You'll need to fix the tests first here 👍

@reksar
Copy link
Contributor Author

reksar commented Jun 9, 2022

You'll need to fix the tests first here 👍

Of course. Just give me some time)

The importlib accepts a Path-like objects from Python 3.6
It is possible to make pass CPython3 tests under Python 2,
but it's better to disable it instead of decreasing the
readability and performance of Python 3 style.
@reksar
Copy link
Contributor Author

reksar commented Jun 24, 2022

Hm. Looks like last PR must fix 1 test here.

@reksar reksar requested a review from gaborbernat June 24, 2022 21:58
@gaborbernat
Copy link
Contributor

Can you rebase the PR on main?

@reksar
Copy link
Contributor Author

reksar commented Jun 24, 2022

I expected the error to occurs in this PR #2363 and I fix it. But it seems we need to merge that PR and restart tests here to see logs and catch the error. New logs should tell us more about the error, at least the path to non-existent file.

@gaborbernat
Copy link
Contributor

Merged now so you'll need to rebase against main 👍

Copy link
Contributor

@gaborbernat gaborbernat left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@reksar
Copy link
Contributor Author

reksar commented Jun 25, 2022

Wow) I expected an exception, but OK)

@gaborbernat gaborbernat merged commit 805dcff into pypa:main Jun 25, 2022
@matthuisman
Copy link

with this PR, I'm now getting
"'PythonInfo' object has no attribute 'version_nodot'"
I have opened an issue below for it
#2367

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.

4 participants