Skip to content

[3.8] bpo-28009: Fix uuid SkipUnless logic to be based on platform programs capable of introspection (GH-12777) #14115

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

Merged
merged 1 commit into from
Jun 15, 2019

Conversation

miss-islington
Copy link
Contributor

@miss-islington miss-islington commented Jun 15, 2019

uuid could try fallback methods that had no chance of working on a particular
platform, and this could cause spurious test failures, as well as degraded
performance as fallback options were tried and failed.

This fixes both the uuid module and its test's SkipUnless logic to use a
prefiltered list of techniques that may at least potentially work on that platform.

Patch by Michael Felt (aixtools).
(cherry picked from commit 3a1d50e)

Co-authored-by: Michael Felt aixtools@users.noreply.github.com

https://bugs.python.org/issue28009

… capable of introspection (pythonGH-12777)

uuid could try fallback methods that had no chance of working on a particular
platform, and this could cause spurious test failures, as well as degraded
performance as fallback options were tried and failed.

This fixes both the uuid module and its test's SkipUnless logic to use a
prefiltered list of techniques that may at least potentially work on that platform.

Patch by Michael Felt (aixtools).
(cherry picked from commit 3a1d50e)

Co-authored-by: Michael Felt <aixtools@users.noreply.github.com>
@miss-islington
Copy link
Contributor Author

@aixtools and @ncoghlan: Status check is done, and it's a success ✅ .

@miss-islington miss-islington merged commit f0e5c01 into python:3.8 Jun 15, 2019
@miss-islington miss-islington deleted the backport-3a1d50e-3.8 branch June 15, 2019 16:10
@miss-islington
Copy link
Contributor Author

@aixtools and @ncoghlan: Status check is done, and it's a success ✅ .

1 similar comment
@miss-islington
Copy link
Contributor Author

@aixtools and @ncoghlan: Status check is done, and it's a success ✅ .

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 this pull request may close these issues.

5 participants