Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

bug: onesignal - The 'Pods-App' target has transitive dependencies that include #5142

Closed
nowrap opened this issue Oct 15, 2021 · 7 comments 路 Fixed by #5143
Closed

bug: onesignal - The 'Pods-App' target has transitive dependencies that include #5142

nowrap opened this issue Oct 15, 2021 · 7 comments 路 Fixed by #5143

Comments

@nowrap
Copy link

nowrap commented Oct 15, 2021

Bug Report

Capacitor Version

馃拪   Capacitor Doctor  馃拪 

Latest Dependencies:

  @capacitor/cli: 3.2.5
  @capacitor/core: 3.2.5
  @capacitor/android: 3.2.5
  @capacitor/ios: 3.2.5

Installed Dependencies:

  @capacitor/android: not installed
  @capacitor/cli: 3.2.5
  @capacitor/core: 3.2.5
  @capacitor/ios: 3.2.5

[success] iOS looking great! 馃憣

Platform(s)

ios

Current Behavior

A npx cap sync creates this error:

[!] The 'Pods-App' target has transitive dependencies that include
statically linked binaries:
(/mobileapp/ios/App/Pods/OneSignal/iOS_SDK/OneSignalSDK/Framework/OneSignal.framework)

This is related to:
#5099
#5137

A downgrade to @capacitor/cli version 3.2.4 solves this issue

Expected Behavior

That the sync runs without error

Code Reproduction

Assuming cocoapods are installed and synced:

ionic start myApp tabs --type vue

cd myApp
npm install @capacitor/ios
npx cap add ios

npm install onesignal-cordova-plugin@2

npm run build
npx cap sync

Other Technical Details

npm --version output: 8.1.0

node --version output: v14.18.1

pod --version output (iOS issues only): 1.11.2

macOS: 11.6

xcode: 13.0

Additional Context

@jcesarmobile
Copy link
Member

I can reproduce now, I was installing version 3 and the issue is not reproducible there.
We can't really filter per plugin version, so I'll just add it anyway despite it's not needed for version 3.

@nowrap
Copy link
Author

nowrap commented Oct 15, 2021

Ok, thx, for testing and adding.
Version 3 is the latest of the onesignal plugin but we didn't make the migration yet coz of the major changes.

Maybe it's an idea to create an option to add a plugin per project configuration somehow to this static exception list instead of having it globally defined in the cli project?

@nunoarruda
Copy link

nunoarruda commented Oct 25, 2021

I'm having the same issue 馃憞

Screen Shot 2021-10-25 at 15 50 44

How do we fix this for onesignal-cordova-plugin v2.x? I can't upgrade to onesignal-cordova-plugin v3.x just yet.

@jcesarmobile
Copy link
Member

You have to wait for next Capacitor CLI release or downgrade to previous version

@nunoarruda
Copy link

nunoarruda commented Nov 22, 2021

This problem is back, at least for me, after upgrading @capacitor/cli from v3.3.1 to v3.3.2:

Screenshot 2021-11-22 at 13 55 16

Git shows me that the CLI is trying to delete the following line in the Podfile:

Screenshot 2021-11-22 at 13 57 23

Workaround: Downgrade @capacitor/cli to v3.3.1.

@jcesarmobile
Copy link
Member

Please, when you find a bug, report an issue, don't comment on closed issues because they are not monitored, I usually unsubscribe after closing the issues.

In 3.3.2 we made the CLI to autodetect plugins that needs a static framework if they have use-frameworks="true" in the pods tag, and since OneSignal plugin had it, I removed it from the list.
But turns out version 2 didn't use the pods tag, so we'll need to put it back in the list.

Anyway, since 3.3.0 you can add any plugin you want to the static list from the capacitor config file without waiting for us to add it, you don't need to downgrade. Use the staticPlugins array inside the cordova object https://capacitorjs.com/docs/config

@ionitron-bot
Copy link

ionitron-bot bot commented Nov 10, 2022

Thanks for the issue! This issue is being locked to prevent comments that are not relevant to the original issue. If this is still an issue with the latest version of Capacitor, please create a new issue and ensure the template is fully filled out.

@ionitron-bot ionitron-bot bot locked and limited conversation to collaborators Nov 10, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants