RxFirebase-Deprecated icon indicating copy to clipboard operation
RxFirebase-Deprecated copied to clipboard

Updates to Swift 3.0 and RxSwift 3.0

Open kzaher opened this issue 8 years ago • 9 comments

Hi,

I've made some adaptations for Swift 3.0 and RxSwift 3.0.

First commit is updating Podfiles. Second commit are real changes.

I think there are still issues with CocoaPods and Firebase, but at least it now compiles because I've manually patched the Pods projects.

kzaher avatar Dec 07 '16 11:12 kzaher

I think it should now be ok, I've fixed linker errors.

kzaher avatar Dec 07 '16 13:12 kzaher

I'm planning to use this library this addition will be awesome. Any thoughts about when will it be merged?

leoneparise avatar Dec 09 '16 21:12 leoneparise

👍 I'm quite interested in using RxFirebase and contributing to it. A work project already uses Firebase and RxSwift, but not the two together, so we're a prime candidate to adopt this. That said, the work represented in this PR is a must since I've already pushed us through the migration to Swift 3, etc.

jwhitley avatar Dec 16 '16 00:12 jwhitley

Hi guys,

this PR has issues that it causes duplicate symbols warning. I think we are still waiting for firebase dynamic library dependency.

Until then you can just copy paste those critical couple of lines.

kzaher avatar Jan 01 '17 20:01 kzaher

Maybe it would help to bring this issue up on the Firebase google group (https://groups.google.com/forum/#!forum/firebase-talk)? I can do it but I'll need a clear understanding of the issue... it seems that the warnings arise from the Firebase framework being included in both the RxFirebase framework and also the App...

acchou avatar Jan 03 '17 20:01 acchou

Yes @acchou. It duplicate the links because Firebase is distributed as a static library. When it's linked to RxFirebase to generate the .framework package it includes all code from Firebase inside. I tried different build configurations without success.

My final solution was to include the RxFirebase code into my project directly and use Firebase from Cocoapods.

leoneparise avatar Jan 05 '17 15:01 leoneparise

I solved it with a work a round to only include the RxFramework pod. I did create my own fork (pietgk/RxFramework) with this pull request included and referencing it locally:

pod 'RxFirebase', path: '~/RxFirebase/RxFirebase.podspec'

Does anyone know if there are any references that a framework version for the firebase libs is planned?

pietgk avatar Feb 27 '17 14:02 pietgk

Hey guys, are you going to merge this?

OSemenovBoyarka avatar Jul 26 '17 13:07 OSemenovBoyarka

We can't merge it until Firebase releases a dynamic library.

kzaher avatar Jul 26 '17 13:07 kzaher