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

Update iOS Xcode project #1182

Merged
merged 14 commits into from
Jul 20, 2017
Merged

Update iOS Xcode project #1182

merged 14 commits into from
Jul 20, 2017

Conversation

PeqNP
Copy link
Contributor

@PeqNP PeqNP commented Oct 3, 2016

Update the Xcode project to use the new "improved" method for including image resources.

These changes include updates to how the app icon and launch resources are included in the project. These changes also reduce the compile time for subsequent builds to a few seconds if ccache is installed.

  • AppIcon is now in Images.xcassets
  • Launch icon is now in Images.xcassets
  • Now uses LaunchScreen.xib to reduce the amount of assets required by project to a single image
  • Introduced ccache to all libmoai targets as well as both the moai-ios and moai-ios-static projects. This reduces build times to a few seconds after the first build. You must have ccache installed to use this feature. If ccache is not found, it falls back to the default compiler.

Lastly, I have set both projects to always require fullscreen and to hide the status bar. The supported orientations have also been set to portrait and both landscape orientations. Obviously these can be changed. I believe these settings seem reasonable, giving that the projects used by MOAI are primarily games.

@moai-sdk
Copy link

moai-sdk commented Oct 3, 2016

Can one of the admins verify this patch? Type 'test' in a comment to start test.

8 similar comments
@moai-sdk
Copy link

moai-sdk commented Oct 3, 2016

Can one of the admins verify this patch? Type 'test' in a comment to start test.

@moai-sdk
Copy link

moai-sdk commented Oct 3, 2016

Can one of the admins verify this patch? Type 'test' in a comment to start test.

@moai-sdk
Copy link

moai-sdk commented Oct 3, 2016

Can one of the admins verify this patch? Type 'test' in a comment to start test.

@moai-sdk
Copy link

moai-sdk commented Oct 3, 2016

Can one of the admins verify this patch? Type 'test' in a comment to start test.

@moai-sdk
Copy link

moai-sdk commented Oct 3, 2016

Can one of the admins verify this patch? Type 'test' in a comment to start test.

@moai-sdk
Copy link

moai-sdk commented Oct 3, 2016

Can one of the admins verify this patch? Type 'test' in a comment to start test.

@moai-sdk
Copy link

moai-sdk commented Oct 3, 2016

Can one of the admins verify this patch? Type 'test' in a comment to start test.

@moai-sdk
Copy link

moai-sdk commented Oct 3, 2016

Can one of the admins verify this patch? Type 'test' in a comment to start test.

…Build with DWARF only in Debug mode (reduces compile times).
@PeqNP
Copy link
Contributor Author

PeqNP commented Oct 3, 2016

test

@halfnelson
Copy link
Contributor

test

On 4 Oct 2016 5:38 am, "Eric Chamberlain" [email protected] wrote:

test


You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
#1182 (comment), or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAhpdHF_x7nlXd7ipPH-X3ruYGb27R0Yks5qwVmugaJpZM4KMyAZ
.

@seclorum
Copy link
Contributor

seclorum commented Oct 4, 2016

test

1 similar comment
@ConghuZhao
Copy link
Contributor

test

@seclorum
Copy link
Contributor

I'm not seeing any reason not to merge this. Objections to just doing it? Would make life easier for new recruits.

@seclorum seclorum merged commit 88d42e0 into moai:develop Jul 20, 2017
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.

5 participants