pygame-community / pygame-ce

🐍🎮 pygame - Community Edition is a FOSS Python library for multimedia applications (like games). Built on top of the excellent SDL library.
https://pyga.me
821 stars 130 forks source link

pypy, windows appveyor, bdist_wheel (423) #403

Closed GalacticEmperor1 closed 11 months ago

GalacticEmperor1 commented 1 year ago

Issue №423 opened by illume at 2018-03-19 13:09:17

Some findings.

Comments

*mattip commented at 2018-03-24 18:31:17*

Nice progress. A couple of questions:

*stuaxo commented at 2021-09-20 00:36:49*

Does anybody have some info about the needed PyState_XXX APIs ?

There is an issue on Pypy asking to implement those APIs here - https://foss.heptapod.net/pypy/pypy/-/issues/3438


*ankith26 commented at 2021-12-12 06:35:07*

Hmm, I guess we could workaround and not have to use PyState_* API at all in the first place.

the python 3 docs states that PyState_AddModule must be called before PyState_FindModule, and this interface is all new for python 3.3 and higher. It seems PyPy does not yet implement either function. So both are needed for pygame?

That's a good catch, thanks! We use PyState_FindModule in display and freetype modules but don't use PyState_AddModule, this could be a reason for some of our display bugs

EDIT: Oh NVM seems like PyState_AddModule gets called implicitly when the module is imported


*illume commented at 2022-01-22 20:04:44*

There’s a module clean up PR now… probably to be metered soon… and I wonder if that changes things for this issue. Perhaps display and free type modules are now ok.


*illume commented at 2022-01-22 20:07:06*

There was an update on the pypy issue above about the pystate APIs being added to a pypy milestone. Fingers crossed?


*stuaxo commented at 2022-01-24 21:40:41*

It's been pushed to the next release, but that seems reasonable, I've been keeping up with the updates on this side and the pypy side, let's see how it goes.

zoldalma999 commented 11 months ago

I'll close this, as we do not use appveyor anymore, and most things seem fixed here anyway.