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

fix(ios): update objc module template clang settings #11283

Merged
merged 2 commits into from Oct 30, 2019

Conversation

janvennemann
Copy link
Contributor

JIRA: https://jira.appcelerator.org/browse/TIMOB-27410

Optional Description:
Updates the Objective-C module template with the most recent default clang settings from Xcode 11. This includes CLANG_ENABLE_MODULES=YES which is required for compatibility with TitaniumKit, since it contains module imports. The Swift template already contains these settings.

Also bumps the deployment target to 9.0, to match our app project template.

@build
Copy link
Contributor

build commented Oct 16, 2019

Messages
📖

💾 Here's the generated SDK zipfile.

📖

✅ All tests are passing
Nice one! All 4411 tests are passing.
(There are 479 skipped tests not included in that total)

📖 ✊ The commits in this PR match our conventions! Feel free to Rebase and Merge this PR when ready.

Generated by 🚫 dangerJS against c75e3cd

Copy link
Contributor

@vijaysingh-axway vijaysingh-axway left a comment

Choose a reason for hiding this comment

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

CR passed.

@ssekhri
Copy link

ssekhri commented Oct 28, 2019

@janvennemann with this PR the clang setting to Enable Modules (C and Objective-C) in xcode project for the module does gets set to Yes by default. However following the test instructions and test code mentioned in the ticket https://jira.appcelerator.org/browse/TIMOB-27410 we still do get the same error with both Xcode 10.3 and 11.1.

@janvennemann
Copy link
Contributor Author

@ssekhri did you update the sample module? Also make sure to update the modules titanium.xcconfig, since they use a hardcoded value for the TITANIUM_SDK path.

@ssekhri
Copy link

ssekhri commented Oct 29, 2019

FR Passed. The clang setting to Enable Modules (C and Objective-C) is set to Yes by default. The module builds successfully.
Verified on:
Mac OS: 10.14.5
SDK: 8.3.0.v20191016042241
Appc CLI: 7.1.1
JDK: 1.8.0_162
Node: 10.5.0
Studio: 5.1.4.201909061933
Xcode: 11.1

@sgtcoolguy sgtcoolguy merged commit da07971 into tidev:master Oct 30, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants