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

python3-pgzero: update to 1.2.1. #29720

Closed
wants to merge 2 commits into from
Closed

Conversation

pbui
Copy link
Contributor

@pbui pbui commented Mar 24, 2021

Also update related dependency python3-pygame to 2.0.0. This now uses SDL2 rather than SDL. The only package that depends on python3-pygame appears to be pysolfc and I have tested that it still runs with this update to python3-pygame.

General

Have the results of the proposed changes been tested?

  • I use the packages affected by the proposed changes on a regular basis and confirm this PR works for me
  • I generally don't use the affected packages but briefly tested this PR

Comment on lines 17 to 19
do_check() {
: requires video device
}
Copy link
Member

Choose a reason for hiding this comment

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

make_check=no or maybe extended instead of defining an empty function, but you can probably try wrapping the test command in xvfb-run?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Ah, Ok. I didn't know about make_check and have updated the template to use that. I tried to use xvfb-run, but I could not get that to work.

hostmakedepends="python3-setuptools"
depends="python3-setuptools python3-pygame python3-numpy"
depends="python3-setuptools python3-pygame>=2.0.0 python3-numpy"
Copy link
Member

Choose a reason for hiding this comment

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

I don't think we need to specify the version here, since it will come in a single update to the dependency chain.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Ok, I have removed this.

short_desc="Zero-boilerplate games programming framework based on Pygame (Python3)"
maintainer="Peter Bui <pbui@github.bx612.space>"
license="LGPL-3.0-or-later"
homepage="https://pygame-zero.readthedocs.io/en/stable/"
distfiles="${PYPI_SITE}/p/pgzero/pgzero-${version}.tar.gz"
checksum=91e641d545c6235a24719dea0fd83c8429e92a8b5ab1756bef145128e8db9017
checksum=8cadc020f028cbac3e0cbd3bb9311a1c045f1deedac7917ff433f986c38e6106
make_check=no
Copy link
Member

Choose a reason for hiding this comment

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

Do leave a comment as to why tests are disabled :)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done!

Comment on lines -21 to -24
pre_build() {
# Remove the C source to re-cythonize, which avoids some API changes
rm src_c/pypm.c src_c/_sdl2/{sdl2,audio,video}.c
}
Copy link
Member

Choose a reason for hiding this comment

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

Did they stop shipping these files or did re-generation stop being necessary?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I believe the re-generation is no longer necessary. I checked the ArchLinux package and they do not have this there and it appears to work without this.

@ericonr ericonr closed this in 2c9e49e Apr 4, 2021
hazayan pushed a commit to hazayan/void-packages that referenced this pull request Apr 15, 2021
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jul 4, 2021
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.

None yet

2 participants