dpogue opened a new pull request #1154:
URL: https://github.com/apache/cordova-ios/pull/1154


   ### Platforms affected
   iOS
   
   
   ### Motivation and Context
   This adds a `Package.swift` file to allow installing the CordovaLib library 
through Swift Package Manager.
   
   This also cleans up our Cocoapods spec file to remove the need to explicitly 
list each header.
   
   ### Description
   Swift Package Manager requires all the public headers to be in an `include` 
folder, so all the headers from `Classes/Public` were moved to 
`include/Cordova`.
   
   Also got rid of the precompiled header because SwiftPM doesn't know how to 
do that, and we were only precompiling Foundation and UIKit.
   
   Xcode 13 also adds a new warning when framework headers are used with 
relative imports `"myfile.h"` rather than framework imports `<myfile.h>`. I've 
cleaned up the imports for all the public headers. (Note: some of the plugins 
use private headers, which must retain the relative import syntax)
   
   
   
   ### Testing
   * Tested creating a new Swift project in Xcode, adding CordovaLib using 
SwiftPM and compiling successfully
   * Tested creating a new Objective C project in Xcode, adding CordovaLib 
using Cocoapods as a dynamic library and compiling successfully
   * Tested creating a new Objective C project in Xcode, adding CordovaLib 
using Cocoapods as a static library and compiling successfully
   * Existing test suite passes
   
   
   ### Checklist
   
   - [x] I've run the tests to see all new and existing tests pass
   


-- 
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]

Reply via email to