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

select proper bitness from pypy externals for windows #637

Merged
merged 3 commits into from
Jun 10, 2021

Conversation

altendky
Copy link
Contributor

@altendky altendky commented May 26, 2021

PyPy went to 64 bit between 7.3.3 and 7.3.4 which happened between the last good build de0cc3e and the first bad one at a22932e. We need to update to use the 64-bit PyPy externals reference to get the needed libraries for building cryptography. Also, 7.3.4 dropped Python 3.6 support so our 3.6 tests will continue with 32-bit PyPy 7.3.3.

@altendky altendky marked this pull request as draft May 26, 2021 01:24
@altendky altendky changed the title use win64_14x from pypy externals select proper bitness from pypy externals for windows May 26, 2021
@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@altendky altendky marked this pull request as ready for review May 26, 2021 01:56
@altendky
Copy link
Contributor Author

@dhoomakethu, here's a Windows PyPy fix up.

@altendky altendky mentioned this pull request May 26, 2021
5 tasks
@dhoomakethu
Copy link
Contributor

@altendky its still failing for windows 3.7x.

@altendky
Copy link
Contributor Author

It is. I should probably take another pass at clearing that flakiness, but it is not related to the PyPy fix implemented here.

@dhoomakethu
Copy link
Contributor

@altendky
Copy link
Contributor Author

@dhoomakethu, yes, that is CPython and thus unrelated to the new PyPy release being 64-bit. This PR fixes the PyPy issue. I would recommend merging this fix and addressing the flaky asyncio test setup separately. I'll see if I can find some time today to look at the flaky stuff.

@altendky
Copy link
Contributor Author

altendky commented Jun 3, 2021

For linkage, see #639 for work on the flaky CPython asyncio test.

@dhoomakethu dhoomakethu added the 3.x label Jun 6, 2021
@dhoomakethu dhoomakethu merged commit bdc99ae into pymodbus-dev:dev Jun 10, 2021
@altendky altendky deleted the patch-9 branch June 10, 2021 20:01
camtarn pushed a commit to camtarn/pymodbus that referenced this pull request Dec 6, 2021
* use win64_14x from pypy externals

* dynamically select pypy externals bitness

* refer to pypy as python in commands
camtarn added a commit to camtarn/pymodbus that referenced this pull request Dec 6, 2021
* use win64_14x from pypy externals

* dynamically select pypy externals bitness

* refer to pypy as python in commands
camtarn added a commit to camtarn/pymodbus that referenced this pull request Dec 6, 2021
* use win64_14x from pypy externals

* dynamically select pypy externals bitness

* refer to pypy as python in commands
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Apr 21, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants