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’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(android): native lib causes app compilation to fail #42

Merged
merged 1 commit into from
Jan 14, 2025

Conversation

OS-pedrogustavobilro
Copy link
Contributor

Using OSInAppBrowserLib-Android with 1.x plugin, that assumes Capacitor 7 compileSdk 34, causes apps to fail compilation due to transitive dependencies requiring compileSdk 35:

Dependency 'androidx.core:core-ktx:1.15.0' requires libraries and applications that
      depend on it to compile against version 35 or later of the
      Android APIs.

By using @aar, the transitive dependencies are no longer passed to the plugin / app.

This would cause issues if some of the dependencies were missing, but they're all declared by capacitor, and using different version didn't affect the native Android app at all, from my tests.

@OS-pedrogustavobilro OS-pedrogustavobilro requested a review from a team January 14, 2025 10:16
@OS-pedrogustavobilro OS-pedrogustavobilro self-assigned this Jan 14, 2025
@OS-pedrogustavobilro OS-pedrogustavobilro merged commit 8a57a22 into 1.x Jan 14, 2025
4 checks passed
@OS-pedrogustavobilro OS-pedrogustavobilro deleted the fix/transitive-dependency branch January 14, 2025 15:48
IT-MikeS pushed a commit that referenced this pull request Jan 15, 2025
## [1.0.3](v1.0.2...v1.0.3) (2025-01-15)

### Bug Fixes

* **android:** native lib causes app compilation to fail ([#42](#42)) ([8a57a22](8a57a22))
* **android:** Unknown Kotlin JVM target 21 ([#41](#41)) ([78245bc](78245bc))
* Remove unnecessary permissions from AndroidManifest.xml ([#39](#39)) ([a7218d5](a7218d5))
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants