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

Add improved support for simple iOS sticker packs #824

Merged

Conversation

scelis
Copy link
Contributor

@scelis scelis commented Apr 6, 2020

In the process of updating some of our iOS sticker packs to use XcodeGen, I ran into two issues:

  1. By default, messages applications for simple iOS sticker packs created in Xcode do not include a compile sources phase. While XcodeGen supports this for the sticker pack extension, it does not for the messages application itself. The first commit in this PR addresses this issue and allows the target to skip the compile sources phase if the application has no sources.

  2. In order for the run scheme to function properly, launchAutomaticallySubstyle must be set to 2. Doing so will change the executable dropdown from the sticker app to Ask on Launch. Without this change, running the scheme will cause the simulator to open but hang instead of launching Messages.

Screen Shot 2020-04-06 at 8 08 44 AM

The second commit in this PR addresses that issue and updates the TestProject fixture which needed to be changed.

Since none of this is obvious, I added an example project for a simple sticker pack to Examples.md.

scelis added 4 commits April 6, 2020 08:10
By default, simple iOS sticker packs created in Xcode do not include
a compile sources phase. This change will allow messages applications
with an empty list of sources to skip that phase entirely.
This is especially important for simple iOS sticker packs that require
this run scheme setting to be set to 2 in order to run the scheme
properly.
@brentleyjones
Copy link
Collaborator

Thanks @scelis! Can you add a changelog entry and some tests for this?

@scelis
Copy link
Contributor Author

scelis commented Apr 6, 2020

@brentleyjones Done!

Copy link
Collaborator

@brentleyjones brentleyjones left a comment

Choose a reason for hiding this comment

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

Thanks @scelis!

@brentleyjones brentleyjones merged commit 0328fa0 into yonaskolb:master Apr 6, 2020
@scelis scelis deleted the support-simple-ios-sticker-apps branch April 6, 2020 18:39
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.

2 participants