Commit b3af92d
authored
Add Universal Build Script (#432)
* Add Universal Build Script
• Now that we are generating universal static & dynamic frameworks for iOS, it is good to combine the logic for generating a framework into a single build script
• In the past, it was a bit confusing. We had a runscript on our Static Lib target that was actually producing a universal static framework and copying it to our repo under the /iOS_SDK/Framework folder.
• The motivation for this change is that beyond just having dynamic/static flavors of our framework now, we will also be splitting up our SDK into two separate frameworks (one for the Extension service, and one for the main iOS SDK) so instead of having multiple build scripts with a lot of duplicated logic, we just use a single shell script with build settings arguments
• This change does multiple things:
1. We now have two aggregate targets. Aggregate targets don't directly generate an output like a normal Xcode target does. Instead, they run a script and that script produces output using xcodebuild CLI
2. Each aggregate target uses custom Build Settings to determine if it produces a static or dynamic framework, where the framework should be copied to in the repo directory, what the name of the framework should be, and what source target is used.
* Fix Preprocessor Macro
• Previous commit made some changes to build configurations which ended up removing a preprocessor definition from UnitTests build settings.
• This preprocessor definition (OS_TEST) is used to change the timing of some HTTP reattempt logic1 parent 40e068a commit b3af92d
File tree
5 files changed
+274
-160
lines changed- iOS_SDK
- OneSignalDevApp/OneSignalDevApp.xcodeproj
- OneSignalSDK
- OneSignal.xcodeproj
- OneSignalFramework
5 files changed
+274
-160
lines changedLines changed: 46 additions & 0 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
6 | 6 | | |
7 | 7 | | |
8 | 8 | | |
| 9 | + | |
| 10 | + | |
| 11 | + | |
| 12 | + | |
| 13 | + | |
| 14 | + | |
| 15 | + | |
| 16 | + | |
| 17 | + | |
| 18 | + | |
| 19 | + | |
| 20 | + | |
| 21 | + | |
9 | 22 | | |
10 | 23 | | |
11 | 24 | | |
| |||
286 | 299 | | |
287 | 300 | | |
288 | 301 | | |
| 302 | + | |
| 303 | + | |
| 304 | + | |
| 305 | + | |
| 306 | + | |
289 | 307 | | |
290 | 308 | | |
291 | 309 | | |
| |||
303 | 321 | | |
304 | 322 | | |
305 | 323 | | |
| 324 | + | |
306 | 325 | | |
307 | 326 | | |
308 | 327 | | |
| |||
544 | 563 | | |
545 | 564 | | |
546 | 565 | | |
| 566 | + | |
| 567 | + | |
| 568 | + | |
| 569 | + | |
| 570 | + | |
| 571 | + | |
| 572 | + | |
| 573 | + | |
| 574 | + | |
| 575 | + | |
| 576 | + | |
| 577 | + | |
| 578 | + | |
| 579 | + | |
| 580 | + | |
| 581 | + | |
| 582 | + | |
| 583 | + | |
547 | 584 | | |
548 | 585 | | |
549 | 586 | | |
| |||
574 | 611 | | |
575 | 612 | | |
576 | 613 | | |
| 614 | + | |
| 615 | + | |
| 616 | + | |
| 617 | + | |
| 618 | + | |
| 619 | + | |
| 620 | + | |
| 621 | + | |
| 622 | + | |
577 | 623 | | |
578 | 624 | | |
579 | 625 | | |
| |||
0 commit comments