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

Implements workaround for generic archive bug. #4118

Closed
wants to merge 5 commits into from
Closed

Implements workaround for generic archive bug. #4118

wants to merge 5 commits into from

Conversation

jcampbell05
Copy link
Contributor

Workaround for CocoaPods/CocoaPods#4178

In certain project configurations Xcodebuild creates a generic archive with the iOS App with Apple Watch App and the Apple Watch app in the products folder.

This is a small tweak to remove any Watch IPAs in the root of the archive (i.e not inside of a iOS App) since in pretty much all cases a developer wouldn't want / need this.

Hopefully this will increase the chances that gym will pick the correct IPA file when this bug occurs.

end
end


Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Extra blank line detected.

@fastlane-danger-bot
Copy link

      <tr>
    <td>:white_check_mark:</td>
    <td data-sticky="true"><del><code>gym/spec/xcodebuild_fixes/generic_archive_fix_spec.rb:8</code><hr /><pre>expected: true<br />     got: false<br />(compared using ==)</pre></del></td>
  </tr>
        :white_check_mark: Good on 'ya.
    
  </th>
 </tr>

Generated by 🚫 danger

@@ -3,6 +3,9 @@ class Manager
def work(options)
Gym.config = options

puts ":I"
exit
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks like debug code that snuck in? ;)

@jcampbell05
Copy link
Contributor Author

@mfurtak All up to date, let me know if you are happy and then before we merge I'll confirm that this still works in real-life :)

@mfurtak
Copy link
Contributor

mfurtak commented Apr 11, 2016

This looks good to me! If you can give it your final checks and then squash the commits, we'll get it in 👍

@jcampbell05
Copy link
Contributor Author

jcampbell05 commented Apr 22, 2016

Should be good to go :) - just ran it and it works.

@mfurtak mfurtak self-assigned this Apr 22, 2016
@jcampbell05
Copy link
Contributor Author

@KrauseFx Just dange not liking the squash

@mfurtak
Copy link
Contributor

mfurtak commented Apr 25, 2016

Merged as #4325 🚀

@mfurtak mfurtak closed this Apr 25, 2016
@jcampbell05 jcampbell05 deleted the b/ImplementWorkaroundForGenericArchivesBug branch May 5, 2016 19:55
@fastlane fastlane locked and limited conversation to collaborators Feb 4, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants