GRDB.swift: Xcode 13 beta 5 fails to build GRDB in Release configuration with CocoaPods
What did you do?
Compile a release build on XCode Version 13.0 beta 5 (13A5212g)
Fully aware it’s a beta and likely to break, but just incase it’s a long term change. It seems to be different from #984, but feel free is close if it’s a duplicate.
What happened?
Compliation failed, please see attached screenshot of logs. Happy to provide anything else that may be useful. It seems the
#if canImport(Combine) at the top of DatabaseMigrator.swift seems to be the cuprit.
Environment
GRDB flavor(s): (GRDB) GRDB version: 5.9.0 Installation method: CocoaPods Xcode version: Version 13.0 beta 5 (13A5212g) Swift version: 1.26.9 Apple Swift version 5.5 (swiftlang-1300.0.29.102 clang-1300.0.28.1) Platform(s) running GRDB: iOS macOS version running Xcode:: 11.5.2 (20G95)
About this issue
- Original URL
- State: closed
- Created 3 years ago
- Comments: 15 (7 by maintainers)
Commits related to this issue
- Bump required iOS version to 13.0 https://github.com/groue/GRDB.swift/issues/1033 — committed to lcharlick/GRDB.swift by lcharlick 3 years ago
- Temporary fix to use GRDB with Xcode 13 RC when we don't need Combine support but must deploy to iOS 10.3. Reference: https://github.com/groue/GRDB.swift/issues/1033 — committed to mnvr/GRDB.swift by mnvr 3 years ago
- Require iOS 13 to get around Xcode 13 release build bugg See https://github.com/groue/GRDB.swift/issues/1033 — committed to Filibaba/GRDB.swift by simme 3 years ago
- Bump min iOS version to 11.0 This drops support for 32 bit devices Fix #1059 Fix #1033 — committed to groue/GRDB.swift by groue 3 years ago
- Bump min iOS version to 11.0 This drops support for 32 bit devices Fix #1059 Fix #1033 — committed to groue/GRDB.swift by groue 3 years ago
My steps:
Package.swift, changing line 9 to.iOS("13.0"),Uh, I just realised I didn’t need to use a fork for a team of one. But yeah, for a team to use you’ll want a fork, so that you can commit the change and use the fork as the project dependency instead of the main repo.
@groue Thank you Groue, thought i’d leave the workaround for others, I had a hunch is was on apple to fix. Many thanks.