normalize and update Xcode templates #306

merged 2 commits into from Mar 3, 2013


None yet
2 participants

pizthewiz commented Mar 2, 2013

the Xcode project templates had varying default settings and in one case, would be called out as an issue prompting for update. the PR normalizes and updates the settings to be more consistent with each other and Apple's templates with a focus on the project language code, bundle identifier's reverse-DNS format, bundle versions, copyright year and application icon.

the Xcode project compatibility version was also bumped from 3.1 to 3.2 as Xcode 4.4 is already required for the branch and opening a 3.1 compatibility version project in Xcode 4.6 will call it out as an issue (misidentified as a warning in the commit message).

pizthewiz added some commits Mar 2, 2013

@pizthewiz pizthewiz upgrade compatibility to 3.2 to avoid warning
the entire appRewrite branch requires 4.4 as it is, so there really isn't a backward compatibility issue to contend with. additionally new tinderbox-created projects opened in 4.6 (possibly earlier too) will display a warning to upgrade, might as well save the trouble directly from the source.
@pizthewiz pizthewiz normalize and modernize template settings
standardize language code, bundle identifier reverse-DNS format, bundle versions, copyright year and application icon.

pizthewiz commented Mar 2, 2013

one could also make the case for adding the typical DEBUG=1 macro for the Debug configuration, but i'm not certain yet that a similar macro doesn't exist within Cinder.


andrewfb commented Mar 3, 2013

I had changed the compatibility to 3.1 at a user's request (AppCode apparently requires it) but at the time didn't connect that XCode would start complaining. I think that's a rare enough use case (and one addressable by manually editing the App template) that we should restore 3.2. There's been discussion of using Apple's suggested DEBUG=1 macro, which I'll integrate shortly. Thanks for the changes.

@andrewfb andrewfb added a commit that referenced this pull request Mar 3, 2013

@andrewfb andrewfb Merge pull request #306 from pizthewiz/xcode-templates
normalize and update Xcode templates

@andrewfb andrewfb merged commit c401763 into cinder:appRewrite Mar 3, 2013

pizthewiz deleted the pizthewiz:xcode-templates branch Mar 3, 2013

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment