-
-
Notifications
You must be signed in to change notification settings - Fork 697
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
bug: *patch name* failed app.revanced.patcher.patch.PatchException: '*patch name*' depends on 'SettingsPatch' that raised an exception #1861
Comments
Make space |
If you mean storage space on my device, then I still have a good 30GB free - so I'm not sure this is the cause. |
If you have an SD card, move the app to internal storage |
It's all on internal, no SD card in the phone. |
Add a video recording of the following:
|
az_recorder_20240408_223500.mp4 |
Do you have a PC for debugging the issue? |
Yes - let me know what you need doing to debug. |
Clone the repository, open it in Android Studio, connect your phone over ADB. Sync the project using Gradle (you will need to authenticate to GitHub packages, follow the building docs in this repository), once it's finished, next to the run icon, press on debug. After the application finishes building, it'll launch on your connected phone. Everything will be slower, since it's running in debug mode, but check if the issue is still occuring |
Different issue now - see stacktrace. It took me a few attempts to get a successful build (haven't done too much Android development or touched Gradle much), but I did eventually get there. I don't think I did anything to cause this, but feel free to tell me otherwise. --edit: formatting
|
This error occurs when theres not enough space. Something on your end is blocking the storage. |
Interesting. I'll experiment on my device and report back if I find anything. Just to confirm, this wouldn't be anything to do with permissions would it? I haven't ever seen ReVanced manager ask for storage access/perms and I don't see an option to enable it in the Android permission settings for the app. Sadly I think the fix for this on my end may well be a rather overdue fresh install of Android. |
The app uses it's internal storage. Permissions shouldn't play a role. What you can do on your end to see the exact error is to attach a debugger and break where the exception is thrown which should tell you the exact io/directory exception. For that you have to open the Gradle submodule inside this flutter project under app/ in Android Studio. Locate the line I linked and you should be able to set a breakpoint |
Top class. I'll have a deeper dive when I've got some more time in a couple of days. On the plus side, I've managed to successfully patch an APK using the Pixel VM in Android Studio. Certainly been an interesting way to spend an evening for someone who's never touched app dev before. Thanks again. |
@NullChips |
Bug description
Hi - trying to patch YouTube, but I seem to be getting hit with an absolute wall of error messages, see logs below.
Most patches are saying that they 'depend on 'SettingsPatch' that raised an exception', followed by a massive stacktrace. I am using the APK downloaded from APKpure that Revanced Manager recommends.
Steps tried:
-Clearing and deleting Revanced Manager cache/data as well as reinstalling.
-Clearing and deleting YouTube cache/data.
-Removing YouTube updates.
-Disabling YouTube app.
I have shortened the patch logs section somewhat - most of it appears to be repeats of the same error, repeated for each patch. The full patch logs can be viewed here if needed: https://pastebin.com/2hV7h45V
Version of ReVanced Manager and version & name of app you are patching
Revanced Manager: 1.20.1
Attempting to patch YouTube 19.09.37
Installation method
None
ReVanced Manager logs
Patch logs
Acknowledgements
The text was updated successfully, but these errors were encountered: