-
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
Workaround for https://github.com/fastlane/fastlane/issues/5163 #5626
Conversation
I've got the same issue, and this change fixes it for me using Xcode 8 beta 4 👍 |
Very interesting approach, definitely something we could try 👍 |
I'm hitting this problem in my project, but with
It still hangs sometimes. |
So it seems like this doesn't actually fix the issue 😢 |
Going to close this in favor of #5188 |
I support with @IngmarStein, #5188 seems to have nothing to do with this "always hangs with Xcode 8". |
@ohwutup the underlying bug hasn't been fixed in Xcode 8 Beta 6, but @tokorom's fix still works. |
@ohwutup Petitioning for reopening. :) |
+1 |
@IngmarStein thanks for sharing, will try it |
It worked the first 2 times and then failed (by hanging again). Ok, now the fourth time works. |
Interesting. With my Core Data based project, I can easily run |
I have this running behind Jenkins, that may be the added variable. |
@lyndsey-ferguson is it possible that you're running into #4059 then? A combination of this PR and #5188 may be required for a complete fix. |
@IngmarStein Thanks for investigating and putting this together 👍 I'm going to take this as-is, but do a follow-up to put this behavior behind a feature switch so that people can opt-in to the change. ✅ |
Seems to work in my projects and Xcode 7.3.1. Thanks. |
@IngmarStein the commit made by @fcy works for me. I put this in my Gemfile: |
After talking with @KrauseFx, we decided not to put this workaround behind a feature switch. It'll go out live for everyone by default with the next release. 👍 |
@lyndsey-ferguson Thanks for chiming in with your test results. 👍 We'll work on merging a retry solution next! |
Hi all, this has been released as part of The retry solution from #5188 is also a part of that release. Thanks very much for your help! Please let us know how it looks 👍 |
To test the latest release, please run Please let us know if that fixes the issue for you, so we can push new releases for gym and scan 🚀 |
Hi all, To get updated and try them out, please run |
Doesn't this ignore the -clean parameter, and break incremental builds? |
@jacobjennings Could you submit a new issue for this? 👍 |
This looks like a good workaround to me.