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

Remove Jetifier from Build #13378

Closed

Conversation

jamesonwilliams
Copy link
Contributor

@jamesonwilliams jamesonwilliams commented Jan 23, 2024

First time contributor checklist

Contributor checklist

  • Pixel 7 API 34 emulator
  • My contribution is fully baked and ready to be merged as is
  • I ensure that all the open issues my contribution fixes are mentioned in the commit message of my first commit using the Fixes #1234 syntax

Description

This removes Jetifier from the build.

2 out-of-date dependencies were bringing in Android Support libraries as transitive dependencies. This required Jetifier to be run. More recent versions of these libraries have been released, now directly using AndroidX equivalents. By upgrading these dependencies, Jetifier is no longer needed.

Old Dependency New Dependency Comment
com.pnikosis:materialish-progress:1.5 com.pnikosis:materialish-progress:1.7 Minor version bump 1.5 -> 1.7, see code diff
com.davemorrissey.labs:subsampling-scale-image-view:3.10.0 com.davemorrissey.labs:subsampling-scale-image-view-androidx:3.10.0 No commit hash for updated code, but new published artifact is discussed here

materialish-progress and subsampling-scale-image-view were bringing in
Android Support libraries as transitive dependencies. This required
Jetifier to be run as part of the build. More recent versions of these
dependencies have been released, which now use AndroidX directly. By
upgrading these dependencies, Jetifier is no longer needed to build
Signal.
@jamesonwilliams jamesonwilliams marked this pull request as ready for review January 23, 2024 05:32
@greyson-signal
Copy link
Contributor

Thank you so much! This has been on my wishlist for a while, glad we can finally do it :) This will go out in our 6.46 release.

TwoLeaves pushed a commit to TwoLeaves/Signal-antisocial that referenced this pull request Feb 7, 2024
materialish-progress and subsampling-scale-image-view were bringing in
Android Support libraries as transitive dependencies. This required
Jetifier to be run as part of the build. More recent versions of these
dependencies have been released, which now use AndroidX directly. By
upgrading these dependencies, Jetifier is no longer needed to build
Signal.

Resolves signalapp#13378
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants