Skip to content

Skip test_shell on macOS ARM #292

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Oct 20, 2021
Merged

Skip test_shell on macOS ARM #292

merged 1 commit into from
Oct 20, 2021

Conversation

serhiy-storchaka
Copy link
Member

No description provided.

@serhiy-storchaka serhiy-storchaka mentioned this pull request Oct 20, 2021
Merged
@ambv ambv requested a review from pablogsal October 20, 2021 18:15
@pablogsal
Copy link
Member

Is there any BPO for this problem?

@serhiy-storchaka
Copy link
Member Author

bpo-45502. Currently test_shell doesn't test with dbm.ndbm and dbm.gdbm (it runs instead repeatedly for dbm.dumb). After fixing it (python/cpython#29003) one of buildbots is crashed.

@serhiy-storchaka
Copy link
Member Author

https://buildbot.python.org/all/#/builders/721/builds/50

@pablogsal pablogsal merged commit 8919e65 into python:main Oct 20, 2021
@zware
Copy link
Member

zware commented Oct 20, 2021

test_shell or test_shelve?

@serhiy-storchaka serhiy-storchaka deleted the macos-arm-skip-test_shell branch October 20, 2021 19:28
@serhiy-storchaka
Copy link
Member Author

test_shell or test_shelve?

Damn! Thank you for catching this @zware!

@vstinner
Copy link
Member

Why skipping the test in the buildbot configuration, rather skipping it in the Python test suite directly?

@pablogsal
Copy link
Member

Why skipping the test in the buildbot configuration, rather skipping it in the Python test suite directly?

Other MacOS systems can run the test without problem. We need to investigate what happens on M1

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ambv ambv ambv approved these changes

@pablogsal pablogsal Awaiting requested review from pablogsal

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@serhiy-storchaka @pablogsal @zware @vstinner @ambv