JH7 commented on PR #1235: URL: https://github.com/apache/cordova-ios/pull/1235#issuecomment-1167382109
@delki8 `cordova-plugin-meteor-webapp` does not use `cordova-ios` directly. `cordova-ios` is rather installed by `meteor add-platform ios` (along with other Cordova packages). Currently, I'm applying these fixes by hand, meaning: I build my Meteor project with `meteor build`, then I open the built Xcode project. Fromt there, I manually edit the source files (`CDVWebViewEngine.m` and `CDVViewController.h`) by hand before I build the project in Xcode. So I think that you need some sort of Meteor project set up, I cannot say anything regarding testing directly on a Meteor checkout since I haven't tried that. I'm pretty sure there is a smarter way to do this - I did not try anything here since Meteor uses its bundled `npm` for the Cordova package installations (at least I guess so), so I don't know how to force Meteor to resolve the npm pacakge `cordova-ios` to my fork on GitHub for example. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
