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

Update dexlib2 to 2.4.0 #9

Open
auermich93 opened this issue Dec 11, 2020 · 4 comments
Open

Update dexlib2 to 2.4.0 #9

auermich93 opened this issue Dec 11, 2020 · 4 comments

Comments

@auermich93
Copy link

Hi Lanchon,

would you mind to update dexlib's version to 2.4.0?

@Lanchon
Copy link
Member

Lanchon commented Dec 13, 2020

hi,

the project is stalled a bit, and i'll try restarting it. but first order of things would be new dexpatcher-gradle plugins. then possibly move dxp-tool out of beta. then support dexlib2 2.4.0. at that time i'll look at all the commits (there are a few) and will need to adapt my code. but mostly it will be dxp-tool affected; multidexlib2 might probably not need changes.

all this will surely take some time. what you can do in the meantime is swapping the dexlib2 dependency from underneath multidexlib2 with the newer one. gradle allows this; google how. this has a good chance of success, but testing would be your responsibility. i just built multidexlib2 against dexlib2 2.4.0 and it built without compile time issues.

@auermich93
Copy link
Author

I did this already in the meantime, there seems to be no conflicts with multidexlib2 and dexlib2 2.4.0. I have updated apktool to the latest version, which requires dexlib2 2.4.0, thus my request for update. The only changes that come with 2.4.0 are these hidden API restrictions as far as I have observed and they don't affect multidexlib2.

@Lanchon
Copy link
Member

Lanchon commented Dec 17, 2020

there are a bunch of changes and dxp will require rework. mdexlib2 might not, because i bugged JF to make some changes to dexlib2 so that clients need less internal knowledge. (before that, every new dex file structure required client changes to acount for the possibility of overflow in the pool of the new structure.)

@auermich93
Copy link
Author

Updated to 2.5.2 locally, works without any issue.

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

No branches or pull requests

2 participants