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

TST: unbreak tests on FREEBSD #23684

Closed
wants to merge 5 commits into from
Closed

TST: unbreak tests on FREEBSD #23684

wants to merge 5 commits into from

Conversation

mattip
Copy link
Contributor

@mattip mattip commented Dec 7, 2020

In PR gh-22088 I added extended a test that FREEBSD is not happy with.

This should unbreak the build on FREEBSD. I don't know why the original PR did not trip this, perhaps the buildbot was not running? Once the build is unbroken, I would be happy to hear from FREEBSD people why the test failed, it should not.

@fochoao
Copy link

fochoao commented Dec 7, 2020

Add it on a Tuple ir List.

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.
@vstinner
Copy link
Member

vstinner commented Dec 8, 2020

Well, I would prefer to fix the issue that skipping it. Otherwise, there is a risk of forgetting this bug.

@vstinner
Copy link
Member

vstinner commented Dec 8, 2020

It is ok is FreeBSD and AIX buildbots are broken for a few days.

@mattip
Copy link
Contributor Author

mattip commented Dec 8, 2020

Closing in favor of fixing the problem in gh-23708

@mattip mattip closed this Dec 8, 2020
@fochoao
Copy link

fochoao commented Dec 29, 2020

The code can be made with a function with tuples or lists, instead of writing the same 5 or 6 times.

I can do this though, if You want this tell me.

@mattip
Copy link
Contributor Author

mattip commented Dec 29, 2020

@fochoao what do you mean? I see this comment from 22 days ago as well. Can you be more specific: which function?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
awaiting review tests Tests in the Lib/test dir
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants