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

[TIMOB-20484] Hyperloop: iOS: Local-embedded frameworks not working #34

Merged
merged 2 commits into from Jun 8, 2016
Merged

[TIMOB-20484] Hyperloop: iOS: Local-embedded frameworks not working #34

merged 2 commits into from Jun 8, 2016

Conversation

cheekiatng
Copy link
Contributor

//For some reason, when using ticore and having custom headers, the original header search path goes missing.
//FIX ME
if(!this.builder.tiapp.ios['use-jscore-framework']) {
addParam('HEADER_SEARCH_PATHS', 'headers');
Copy link
Contributor

Choose a reason for hiding this comment

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

Use $(inherited), it should add it.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

$(inherited) is already added in line 1008. It won't work when ticore is enabled.

@cheekiatng
Copy link
Contributor Author

@hansemannn addressed comments

@@ -365,18 +365,25 @@ module.exports = {
ios: {
xcodebuild: {
flags: {
FRAMEWORK_SEARCH_PATHS: '/path/to/framework'
FRAMEWORK_SEARCH_PATHS: '../../src'
Copy link
Contributor

Choose a reason for hiding this comment

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

The src directory is placed in the same folder hierarchy as the appc.js, how does it come we specify we to ../../src then?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

It's with reference to the Xcode project in the build folder. Appc.js is really expecting an absolute file reference, we could code it such that no matter if developer puts anything here in appc.js, it will always check in ../../src anyway. What you think

On 5 Jun 2016, at 2:59 AM, Hans Knoechel <notifications@github.commailto:notifications@github.com> wrote:

In documentation/ios.mdhttps://github.com//pull/34#discussion_r65806223:

@@ -365,18 +365,25 @@ module.exports = {
ios: {
xcodebuild: {
flags: {

  •                                 FRAMEWORK_SEARCH_PATHS: '/path/to/framework'
    
  •                                 FRAMEWORK_SEARCH_PATHS: '../../src'
    

The src directory is placed in the same folder hierarchy as the appc.js, how does it come we specify we to ../../src then?

You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com//pull/34/files/030d77551ca3abe26dd5ff214887b96e2ecb98aa#r65806223, or mute the threadhttps://github.com/notifications/unsubscribe/AIBddPs7-5BNo4uhFSw575lAwVxKwQrwks5qIcsJgaJpZM4IrSUZ.

Copy link
Contributor

Choose a reason for hiding this comment

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

100 % agree, we should point to that directory by default.

@hansemannn
Copy link
Contributor

The PR is not working correctly for me: I'm trying to use the AccountKit SDK from Facebook in Titanium Mobile.

It generates some kind of metabase, but after running the build, the following error occurs when clicking "Third-Party Libraries > AccountKit". Other frameworks and classes work. May I have to change something else in the appc.js?

@cheekiatng
Copy link
Contributor Author

@hansemannn Where's the following errors? We can look through this when we meet.
But meanwhile, I think you have to merge this PR and resolve the ticket, so that QE has time to test for 5.4.0.Beta. We can work on the improvements and revising the c-spec on appc.js for 6.0.0 ok?

@hansemannn
Copy link
Contributor

Tested using our example projects, including third-party frameworks, Cocoapods dependencies and System frameworks. Approved 🚀

@hansemannn hansemannn merged commit 04cea24 into tidev:master Jun 8, 2016
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