Skip to content

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