-
Notifications
You must be signed in to change notification settings - Fork 5.6k
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
Fix Xcode 8 hung-up by making Helper.backticks retriable #6000
Conversation
Retriable is a simple DSL to retry failed code blocks with randomized exponential backoff time intervals. This is especially useful when interacting external api/services or file system calls. See more about retriable: https://github.com/kamui/retriable/blob/master/README.md
#5626 is merged. And it's much better! |
We may still want to consider this PR, since I think adding retry behavior may be an important fix for a different problem with Overall the PR looks pretty good, though I think we need to consider whether we're willing to add another dependency to solve the problem. |
I see. I reopen this for now. |
UI.command_output(result) if print | ||
return result | ||
def self.backticks(command, print: true, retriable_options: {}) | ||
require 'retriable' |
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.
I'd love to keep the old code, and only use the new one if retriable_options
is passed. Something like a big if else
in the beginning of the method
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.
Let's wait for @mfurtak's opinion first on if we want to add the dependency, before updating the PR, sorry for chiming it like this
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.
OK, Thanks 😇
Thank you too 🚀 |
This is for #4059 #5163 #5603
This will solve these issues!
The problem is caused by Xcode 8
Ref: https://forums.developer.apple.com/thread/50372
How to reproduce
See sample project: https://github.com/nafu/Xcode8HungUpTestWithCoreData/tree/master
Notes
I know there is already PR for this #5188, but I try to take another option.