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 (checklist) #5925

Closed
5 of 6 tasks
TacoTheDank opened this issue Jun 4, 2022 · 2 comments · Fixed by #6982
Closed
5 of 6 tasks

Remove Jetifier (checklist) #5925

TacoTheDank opened this issue Jun 4, 2022 · 2 comments · Fixed by #6982
Labels
Good first issue Can be addressed without deep knowledge of the AntennaPod code. But exploring the code may be needed Type: Feature request

Comments

@TacoTheDank
Copy link
Contributor

TacoTheDank commented Jun 4, 2022

Checklist

  • I have used the search function for open and closed issues to see if someone else has already submitted the same feature request.
  • I will describe the problem with as much detail as possible.
  • This request contains only one single feature, not a list of multiple (related) features.

App version

2.6.0

Where did you get the app from

Google Play

Problem you may be having, or feature you want

As you probably know, the android.enableJetifier=true flag automatically migrates dependencies using old support libraries to AndroidX. However, this also slows down build times. We can speed up the building by being able to get rid of this flag (from some sources I've found, this provides a non-insignificant performance improvement).

Fortunately, there is a new gradle check in AGP 7.x.x that allows us to check what libraries are still using Jetifier. For this purpose, all I did was bump the AGP version to 7.2.1 and run ./gradlew checkJetifier. It returns these results (I cleaned up the output manually to make it easier to read):

The following libraries used by project ':app' depend on legacy support libraries.
To disable Jetifier, you will need to use AndroidX-supported versions of these libraries.

com.github.bumptech.glide:glide:4.8.0 -> com.android.support:support-annotations:27.1.1

com.joanzapata.iconify:android-iconify:2.2.2 -> com.android.support:support-v4:22.2.1

com.github.shts:TriangleLabelView:1.1.2 -> com.android.support:appcompat-v7:24.2.0

Suggested solution

Here are my suggestions for each library:

  • Glide - Update the library to at least 4.10.0, where they migrated to AndroidX.
  • Iconify - This one will be the most difficult as there is no direct solution or replacement (also, it is unmaintained and outdated). We will have to find some replacement for this library.
  • TriangleLabelView - As this library is composed of a single small class, I would suggest simply integrating the class into one of the .view class directories (either the app submodule or the ui:common submodule).

I made them checkboxes so they can be checked off as they're completed.

Screenshots / Drawings / Technical details

No response

@ByteHamster
Copy link
Member

Iconify - This one will be the most difficult as there is no direct solution or replacement (also, it is unmaintained and outdated). We will have to find some replacement for this library.

I think we only use that library in very few places. It can probably be replaced with TextView compound drawables in most cases

@TacoTheDank
Copy link
Contributor Author

Mm, sounds good. I'll look into that.

@ByteHamster ByteHamster added the Good first issue Can be addressed without deep knowledge of the AntennaPod code. But exploring the code may be needed label Feb 12, 2023
caoilte pushed a commit to caoilte/AntennaPod that referenced this issue Aug 12, 2023
caoilte pushed a commit to caoilte/AntennaPod that referenced this issue Aug 12, 2023
caoilte pushed a commit to caoilte/AntennaPod that referenced this issue Aug 13, 2023
caoilte pushed a commit to caoilte/AntennaPod that referenced this issue Aug 19, 2023
caoilte pushed a commit to caoilte/AntennaPod that referenced this issue Aug 27, 2023
- ic_info is a different size to md_info so needs its own ImageView
- We now need to hide the text and the icon when subscriptions are not
  filtered

Part of AntennaPod#5925
caoilte pushed a commit to caoilte/AntennaPod that referenced this issue Aug 27, 2023
- ic_info is a different size to md_info so needs its own ImageView
- We now need to hide the text and the icon when subscriptions are not
  filtered

Part of AntennaPod#5925
caoilte pushed a commit to caoilte/AntennaPod that referenced this issue Aug 27, 2023
- ic_info is a different size to md_info so needs its own ImageView
- We now need to hide the text and the icon when subscriptions are not
  filtered

Part of AntennaPod#5925
caoilte pushed a commit to caoilte/AntennaPod that referenced this issue Aug 27, 2023
- ic_info is a different size to md_info so needs its own ImageView
- We now need to hide the text and the icon when subscriptions are not
  filtered

Part of AntennaPod#5925
caoilte pushed a commit to caoilte/AntennaPod that referenced this issue Aug 27, 2023
- ic_info is a different size to md_info so needs its own ImageView
- We now need to hide the text and the icon when subscriptions are not
  filtered

Part of AntennaPod#5925
caoilte pushed a commit to caoilte/AntennaPod that referenced this issue Sep 16, 2023
caoilte pushed a commit to caoilte/AntennaPod that referenced this issue Sep 16, 2023
caoilte pushed a commit to caoilte/AntennaPod that referenced this issue Sep 16, 2023
caoilte added a commit to caoilte/AntennaPod that referenced this issue Sep 17, 2023
caoilte added a commit to caoilte/AntennaPod that referenced this issue Sep 24, 2023
caoilte added a commit to caoilte/AntennaPod that referenced this issue Sep 24, 2023
caoilte added a commit to caoilte/AntennaPod that referenced this issue Sep 24, 2023
caoilte added a commit to caoilte/AntennaPod that referenced this issue Sep 24, 2023
caoilte added a commit to caoilte/AntennaPod that referenced this issue Sep 24, 2023
wseemann pushed a commit to wseemann/AntennaPod that referenced this issue Oct 22, 2023
wseemann pushed a commit to wseemann/AntennaPod that referenced this issue Oct 22, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Good first issue Can be addressed without deep knowledge of the AntennaPod code. But exploring the code may be needed Type: Feature request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants