-
-
Notifications
You must be signed in to change notification settings - Fork 32.1k
gh-135101: When choosing the default simulator device, don't use simctl --set testing
#135102
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
Conversation
Most changes to Python require a NEWS entry. Add one using the blurb_it web app or the blurb command-line tool. If this change has little impact on Python users, wait for a maintainer to apply the |
!buildbot iOS |
🤖 New build scheduled with the buildbot fleet by @freakboy3742 for commit d07659e 🤖 Results will be shown at: https://buildbot.python.org/all/#/grid?branch=refs%2Fpull%2F135102%2Fmerge The command will test the builders whose names match following regular expression: The builders matched are:
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
My testing suggests this will work as intended; thanks for the report and fix!
@@ -127,7 +127,7 @@ async def async_check_output(*args, **kwargs): | |||
async def select_simulator_device(): | |||
# List the testing simulators, in JSON format |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
# List the testing simulators, in JSON format | |
# List the testing simulators, in JSON format. Use the full list, not just | |
# the testing set; on a fresh install (or in CI), the testing set may be empty. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Dangit... forgot to merge this suggestion. I've got some other cleanups I need to make in the general area; I'll add it in then.
Thanks @joerick for the PR, and @freakboy3742 for merging it 🌮🎉.. I'm working now to backport this PR to: 3.13, 3.14. |
…e `simctl --set testing` (pythonGH-135102) On a fresh Xcode install (including some CI provider configurations), there is no pre-existing testing set that can be used to identify simulator models. Use the default device set to detect available models instead. Live testing simulators are still created in the testing set. (cherry picked from commit dba9de7) Co-authored-by: Joe Rickerby <joerick@mac.com>
GH-135113 is a backport of this pull request to the 3.14 branch. |
…e `simctl --set testing` (pythonGH-135102) On a fresh Xcode install (including some CI provider configurations), there is no pre-existing testing set that can be used to identify simulator models. Use the default device set to detect available models instead. Live testing simulators are still created in the testing set. (cherry picked from commit dba9de7) Co-authored-by: Joe Rickerby <joerick@mac.com>
GH-135114 is a backport of this pull request to the 3.13 branch. |
…se `simctl --set testing` (GH-135102) (#135114) On a fresh Xcode install (including some CI provider configurations), there is no pre-existing testing set that can be used to identify simulator models. Use the default device set to detect available models instead. Live testing simulators are still created in the testing set. (cherry picked from commit dba9de7) Co-authored-by: Joe Rickerby <joerick@mac.com>
…se `simctl --set testing` (GH-135102) (#135113) On a fresh Xcode install (including some CI provider configurations), there is no pre-existing testing set that can be used to identify simulator models. Use the default device set to detect available models instead. Live testing simulators are still created in the testing set. (cherry picked from commit dba9de7) Co-authored-by: Joe Rickerby <joerick@mac.com>
…on't use `simctl --set testing` (pythonGH-135102) (python#135113) On a fresh Xcode install (including some CI provider configurations), there is no pre-existing testing set that can be used to identify simulator models. Use the default device set to detect available models instead. Live testing simulators are still created in the testing set. (cherry picked from commit dba9de7) Co-authored-by: Joe Rickerby <joerick@mac.com>
…on't use `simctl --set testing` (pythonGH-135102) (python#135114) On a fresh Xcode install (including some CI provider configurations), there is no pre-existing testing set that can be used to identify simulator models. Use the default device set to detect available models instead. Live testing simulators are still created in the testing set. (cherry picked from commit dba9de7) Co-authored-by: Joe Rickerby <joerick@mac.com>
This is a fix for the error seen in pypa/cibuildwheel#2443 .
I don't know what
--set testing
is doing, but the previous default deviceiPhone SE (3rd edition)
was not listed there either, and omitting the flag seems to list all the devices that the SDK knows about, rather than ones previously used for testing.cc @freakboy3742