Variable IOS_USE_PRECOMPILED_FIRESTORE_POD does not work on 14.2.1-cli
See original GitHub issueBug report
CHECKLIST
- I have read the issue reporting guidelines
- I confirm this is a suspected bug or issue that will affect other users
- I have reproduced the issue using the example project or provided the necessary information to reproduce the issue.
- I have read the documentation thoroughly and it does not help solve my issue.
- I have checked that no similar issues (open or closed) already exist.
Current behavior:
Using the 14.2.1-cli build, the pre-compiled Firestore pod is always used regardless of the variable IOS_USE_PRECOMPILED_FIRESTORE_POD
.
The non-cli 14.2.1 version works fine.
Comparing plugin.xml
between 14.2.1-cli and 14.2.1 reveals a number of differences including:
14.2.1-cli:
<pod name="FirebaseFirestore" git="https://github.com/invertase/firestore-ios-sdk-frameworks.git" tag="9.1.0"/>
14.2.1
<pod name="Firebase/Firestore" spec="9.1.0"/>
Expected behavior:
The variable IOS_USE_PRECOMPILED_FIRESTORE_POD
should control whether the pre-compiled Firestore pod is used.
Steps to reproduce:
Add 14.2.1-cli into a project without specifying the variable (defaults to false).
Check the PodFile - it states:
pod 'FirebaseFirestore', :tag => '9.1.0', :git => 'https://github.com/invertase/firestore-ios-sdk-frameworks.git'
Environment information
- Cordova CLI version
11.0.0
- Cordova platform version
ios 6.2.0
- Dev machine OS and version, e.g.
- OSX
Monterey 12.5.1
- OSX
- Node JS version
16.17.0
Issue Analytics
- State:
- Created a year ago
- Reactions:2
- Comments:7 (1 by maintainers)
Should be fixed in
cordova-plugin-firebasex@15.0.0-cli
which has just been released to npmHi @RobYed. Since release 14.2.0, the pre-compiled version is not the default anymore. As explained upper, there is a merge issue. If you want the same behavior on the cli, instead of hook, juste adapt the plugin.xml to the right pod (as here)…
In all case, you’ll need a fork before the fixing release.
EDIT : I just detect that the current
cli_build
branch has already been fixed with the PR you are waiting for. https://github.com/dpa99c/cordova-plugin-firebasex/pull/757