Skip to content

Fix for return type of 'getInstalledPackages' changing in SDK 37 - #641

Open
jackyzy823 wants to merge 1 commit into
MMRLApp:masterfrom
jackyzy823:android17
Open

jackyzy823 wants to merge 1 commit into
MMRLApp:masterfrom
jackyzy823:android17

Conversation

@jackyzy823

Copy link
Copy Markdown

Reference: MuntashirAkon/AppManager@836c724


Off-topic:

This PR is based on an old commit (based on the latest fdroid one versionName: v34296-release versionCode: 34296 commit: ef01dae).

Newer version source code requires GitHub token to access GitHub package maven repo for dev.mmrlx:terminal and dev.mmrlx:webui.core to compile. (which i think is not allowed for f-droid and actually so)

@artemnadein

Copy link
Copy Markdown

No interface method getInstalledPackages(JI)Landroid/content/pm/ParceledListSlice; in class Landroid/content/pm/IPackageManager; or its super classes (declaration of 'android.content.pm.IPackageManager' appears in /system/framework/framework.jar)

X3.a.b(SourceFile:16)
w5.s1.s(SourceFile:111)
f7.a.i(SourceFile:8)
H8.K.run(SourceFile:111)
C1.a.run(SourceFile:268)
O8.j.run(SourceFile:2)
O8.a.run(SourceFile:92)

Pixel 10, Android 17 (CP2A.260805.005)

@omegavschaos

Copy link
Copy Markdown

If it can help this fix work great on the AppManager.
If someone had a build to test I would love to test it.
If not I'll try to build the pull request myself next week.

@DerGoogler

Copy link
Copy Markdown
Collaborator

@omegavschaos

Copy link
Copy Markdown

Debug and release version seems to works well.
No crash at start anymore.
Modules are seen.
I just tried to update a module and it never really try.
Here is the full install log

- Installing [color=primary]zygisk-detach-v1.24.0-35.zip[/color]

And the installation is marked complete without doing it.
I tried from magisk application and it works normally.
I don't know if this behaviour was present before or not.
I think it's another issue.

Thanks for the release (and the whole project).

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.

4 participants