ReVanced / revanced-patcher

💉 ReVanced Patcher used to patch Android applications
https://revanced.app
GNU General Public License v3.0
2.41k stars 200 forks source link

feat: Incompatible package annotation #261

Closed LisoUseInAIKyrios closed 7 months ago

LisoUseInAIKyrios commented 7 months ago

Type

Functionality

Issue

Some universal patches fail to work with some apps. Some of these incompatible apps are well known, such as using change package name with YouTube, yet patching still shows success and the patched app initially appears to work (until the user opens the app settings and it crashes).

Feature

Add an additional annotation, or additional Patch annotation field, to indicate incompatible apps/versions. Make it like the existing compatibility field, but for Incompatible apps.

Motivation

Stop users from applying universal patches that are known not to work correctly.

Additional context

No response

Acknowledgements

oSumAtrIX commented 7 months ago

If a patch is not constrained at all, it is supposed to be compatible with all packages. This means, that if it fails on an app, it is considered to be an issue with the patch, which needs to be fixed. In some scenarios this may be impossible. For example YouTube fails to work after being patched without the MicroG patch, but you can't just blacklist all apps

LisoUseInAIKyrios commented 7 months ago

Should the patch be modified to work with YT?

Or should users be allowed to continue this use case of applying rename package and creating a half broken patched app?

An alternative is the rename package app could use its own blacklist, and it throws a patch exception if it's asked to patch YT.

LisoUseInAIKyrios commented 7 months ago

The main use case here was the incompatibility of change package name and YouTube. But now the patch works correctly, so there isn't much of a need for this request anymore.

Closing this as resolved