Skip to content

Add build script to remove x86_64 and i386 archs from frameworks #1126

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

Closed
wants to merge 64 commits into from

Conversation

josep1972
Copy link

Added a Run Script step in Build Phases of the Loop target that removes simulator architectures

@ps2
Copy link
Collaborator

ps2 commented Oct 10, 2019

Can you describe the issue that this fixes, and steps to test?

@josep1972
Copy link
Author

josep1972 commented Oct 10, 2019

This update addresses a failure when creating IPA for distribution. Specifically, when Frameworks contain sim archs (x86_64 and i386), the IPA generation process fails. This update removes the archs when Archiving and IPA generation.

There are two ways to test if the issue has been addressed:

  1. Archive and then Validate App or
  2. Archive and then create app for Ad-Hoc distribution.

@josep1972 josep1972 requested a review from ps2 October 14, 2019 16:48
@shahzadpandher
Copy link

I had the same issue but I was able to export it as ipa as said above

Jose Paredes and others added 20 commits March 31, 2020 20:22
This reverts commit f51aed7.

Revert
…o ab-remoteCarbsAndBolus

merge with LoopKit auto-bolus
This reverts commit f51aed7.

revert
This reverts commit d90118c.

revert
This reverts commit ab73c09.

revert

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.
Dev merge with ab otp
This reverts commit 5c341ae.

revert

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.
@josep1972
Copy link
Author

josep1972 commented Apr 1, 2020

merged level with LoopKit dev to fix conflicts

@josep1972 josep1972 closed this Apr 1, 2020
@josep1972
Copy link
Author

Close due to bad merge

@josep1972 josep1972 deleted the dev branch April 1, 2020 06:40
dekimsey added a commit to dekimsey/Loop that referenced this pull request May 8, 2020
This is an updated apply of PR LoopKit#1126 which addresses the issue reported
in LoopKit#1117
dekimsey added a commit to dekimsey/Loop that referenced this pull request Jan 4, 2021
This is an updated apply of PR LoopKit#1126 which addresses the issue reported
in LoopKit#1117
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.

None yet

4 participants