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

Install optional dependencies during test stage #123

Open
sethmlarson opened this issue May 7, 2024 · 2 comments · May be fixed by #129
Open

Install optional dependencies during test stage #123

sethmlarson opened this issue May 7, 2024 · 2 comments · May be fixed by #129
Assignees

Comments

@sethmlarson
Copy link
Collaborator

The test phase of the build pipeline is missing:

  • libmpdec
  • tkinter

It would be nice to copy what's being done on the CPython repository so we can properly run the entire test suite.

@sethmlarson sethmlarson self-assigned this May 7, 2024
@JelleZijlstra
Copy link
Member

For what it's worth I looked through https://github.com/python/release-tools/actions/runs/8992970829/job/24703944115 and looked at all skipped tests. Most are Windows-specific and obviously can't run on the Linux runner, but I noticed the following that sound like they could possibly work (in addition to decimal and tkinter):

test_ioctl skipped -- Unable to open /dev/tty

  • Maybe depends on actual TTY?
test_generated_cases skipped -- cases_generator directory could not be found
test_clinic skipped -- clinic directory could not be found
  • Maybe these only work if running in the whole repo?
test_dbm_gnu skipped -- No module named '_gdbm'
test_dbm_ndbm skipped -- No module named '_dbm'
  • Judging from the devguide, something like apt-get install libgdbm-dev should work

@erlend-aasland
Copy link

See also .github/workflows/posix-deps-apt.sh in the cpython repo.

@sethmlarson sethmlarson linked a pull request May 9, 2024 that will close this issue
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 a pull request may close this issue.

3 participants