DexPatcher / multidexlib2

Multi-dex extensions for dexlib2
https://dexpatcher.github.io/
GNU General Public License v3.0
68 stars 34 forks source link

Update dexlib2 to 2.4.0 #9

Open auermich93 opened 3 years ago

auermich93 commented 3 years ago

Hi Lanchon,

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

Lanchon commented 3 years ago

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 commented 3 years ago

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 commented 3 years ago

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 commented 3 years ago

Updated to 2.5.2 locally, works without any issue.