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

Added static framework build settings presets #819

Merged
merged 1 commit into from
Apr 2, 2020

Conversation

acecilia
Copy link
Collaborator

@acecilia acecilia commented Apr 2, 2020

Added static framework build settings presets: the lack of SKIP_INSTALL: 'YES' was resulting in invalid archives when archiving. See #358 and https://stackoverflow.com/a/26563115/3203441 for further references

@acecilia
Copy link
Collaborator Author

acecilia commented Apr 2, 2020

Also may solve #813

Copy link
Owner

@yonaskolb yonaskolb left a comment

Choose a reason for hiding this comment

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

Awesome thanks @acecilia!

CHANGELOG.md Outdated
@@ -4,6 +4,7 @@

#### Fixed
- Fixed issue which caused watch app schemes to be generated incorrectly, preventing these apps from launching. [#798](https://github.com/yonaskolb/XcodeGen/pull/798) @daltonclaybrook
- Added build presets for the target type `framework.static`: due to the lack of them, archiving a static framework was resulting in an invalid archive. See https://github.com/yonaskolb/XcodeGen/pull/358 and https://stackoverflow.com/a/26563115/3203441 for further references. [#819](https://github.com/yonaskolb/XcodeGen/pull/819) @acecilia
Copy link
Owner

Choose a reason for hiding this comment

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

The context here is great! We can leave it in this PR though

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Context removed from CHANGELOG.md👌

…LL: 'YES' was resulting in invalid archives when archiving. See yonaskolb#358 and https://stackoverflow.com/a/26563115/3203441 for further references
@yonaskolb yonaskolb merged commit 8c21380 into yonaskolb:master Apr 2, 2020
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

2 participants