Skip to content

Commit

Permalink
RDART-1039: Drop catalyst support. Flutter doesn't support it (#1696)
Browse files Browse the repository at this point in the history
* Drop catalyst support. Flutter doesn't support it

* Update CHANGELOG
  • Loading branch information
nielsenko authored May 28, 2024
1 parent 7384897 commit 413e21a
Show file tree
Hide file tree
Showing 6 changed files with 5 additions and 23 deletions.
8 changes: 1 addition & 7 deletions .github/workflows/binary-combine-ios.yml
Original file line number Diff line number Diff line change
Expand Up @@ -15,23 +15,18 @@ jobs:
with:
name: librealm-ios-device
path: binary/ios

- name: Fetch simulator build
uses: actions/download-artifact@v4
with:
name: librealm-ios-simulator
path: binary/ios
- name: Fetch catalyst build
uses: actions/download-artifact@v4
with:
name: librealm-ios-catalyst
path: binary/ios

- name: Build .xcframework
run: |
xcodebuild -create-xcframework \
-framework ./binary/ios/Release-iphoneos/realm_dart.framework \
-framework ./binary/ios/Release-iphonesimulator/realm_dart.framework \
-framework ./binary/ios/Release-maccatalyst/realm_dart.framework \
-output ./binary/ios/realm_dart.xcframework
rm -rf ./binary/ios/Release-*
Expand All @@ -48,5 +43,4 @@ jobs:
name: |
librealm-ios-device
librealm-ios-simulator
librealm-ios-catalyst
failOnError: false
2 changes: 1 addition & 1 deletion .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -53,7 +53,7 @@ jobs:
with:
runner: macos-12
binary: ios
build: '["ios-device", "ios-simulator", "ios-catalyst"]'
build: '["ios-device", "ios-simulator"]'

build-android-combined:
name: Build combine Android
Expand Down
1 change: 1 addition & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -12,6 +12,7 @@
### Internal
* Using Core 14.7.0.
* Disabled codesigning of Apple binaries. (Issue [#1679](https://github.com/realm/realm-dart/issues/1679))
* Drop building xcframework for catalyst. (Issue [#1695](https://github.com/realm/realm-dart/issues/1695))

## 2.3.0 (2024-05-23)

Expand Down
9 changes: 0 additions & 9 deletions packages/realm_dart/CMakePresets.json
Original file line number Diff line number Diff line change
Expand Up @@ -185,15 +185,6 @@
"xcode_destination": "generic/platform=iOS Simulator"
},
"configuration": "Debug"
},
{
"name": "ios-catalyst",
"displayName": "Catalyst",
"inherits": "ios",
"environment": {
"xcode_destination": "generic/platform=macOS,variant=Mac Catalyst"
},
"configuration": "Debug"
}
]
}
2 changes: 1 addition & 1 deletion packages/realm_dart/dev/lib/src/build.dart
Original file line number Diff line number Diff line change
Expand Up @@ -38,7 +38,7 @@ enum Architecture {
enum iOSSdk {
iPhoneOS('device'),
iPhoneSimulator('simulator'),
; // flutter doesn't support maccatalyst (yet?)
; // flutter doesn't support catalyst (yet https://github.com/flutter/flutter/issues/33860?)

final String? _cmakeName;
String get cmakeName => _cmakeName ?? name;
Expand Down
6 changes: 1 addition & 5 deletions packages/realm_dart/scripts/build-ios.sh
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@ function usage {
echo ""
echo "Arguments:"
echo " -c : build configuration (Debug or Release)"
echo " <platforms> : platforms to build for (catalyst, ios, or simulator)"
echo " <platforms> : platforms to build for (ios, or simulator)"
echo " "
echo "Environment variables:"
echo " REALM_USE_CCACHE=TRUE - enables ccache builds"
Expand Down Expand Up @@ -66,10 +66,6 @@ for platform in "${PLATFORMS[@]}"; do
cmake --build --preset ios-device --config $CONFIGURATION
FRAMEWORKS+=(-framework ./binary/ios/$CONFIGURATION-iphoneos/realm_dart.framework)
;;
catalyst)
cmake --build --preset ios-catalyst --config $CONFIGURATION
FRAMEWORKS+=(-framework ./binary/ios/$CONFIGURATION-maccatalyst/realm_dart.framework)
;;
simulator)
cmake --build --preset ios-simulator --config $CONFIGURATION
FRAMEWORKS+=(-framework ./binary/ios/$CONFIGURATION-iphonesimulator/realm_dart.framework)
Expand Down

0 comments on commit 413e21a

Please sign in to comment.