-
-
Notifications
You must be signed in to change notification settings - Fork 794
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: App update installation always fails on first try #2361
Comments
Is it #2224? Please provide the exact installation failure error |
#2224 has been marked as spam and the thread has been locked. But I don't think the issue has been resolved. This just happened to me |
Just wondering, what phone model and Android version are you guys running? @JimKnoxx and @pramodhrachuri . Looking at #2224, it seems like an issue that people with Samsung phones are running into. Could help to isolate and reproduce the issue. |
I am using a Pixel 7 with Android 15. I got a few more data points in the meantime.
I believe this happens only when we are trying to update the patched app. Let's say I already have Gmail-v1.2.3-revanced installed, and we try to patch+reinstall gmail-v1.2.3-revanced again, and this bug comes out. This case is not unrealistic. There could be scenarios where patches for a specific app have been updated, but the base APK is the same. Edit: do you know how to get this issue reopened? |
Also using a Pixel 7 with Android 15, not rooted. I repatch YouTube whenever there is a new "ReVanced Patch Update", so that my YouTube has the newest features. So most times same APK, other patch version. |
I do the same haha. That's how I landed here. It's a bit of a corner case but not unrealistic. |
@pramodhrachuri i wonder why it consistently bugs out the first time and works the second time on your Galaxy Tab. In #2224, it seems to be the opposite where the first time it succeeds, but the second time crashes. So to be clear, it doesn't bug out at all on the Google Pixel 7? Even when trying to repatch the same APK with different patches? The strange thing is that the flutter file picker selects the file, but seems to be passing a null pointer to the android activity that handles installing the file. Maybe similar to miguelpruivo/flutter_file_picker#397 or miguelpruivo/flutter_file_picker#1025 The behavior is flaky enough that it would be nice if it was fixed though. |
I am able to replicate it in Pixel 7 too! Yay! @shurane my apologies for the confusion. I meant 2nd time too. I used 'reinstall' instead of the number. Let me write it as a timeline instead.
|
That's helpful, @pramodhrachuri Actually just ran into this too on my phone, a Pixel 9 Pro with Android 16: So I guess I can see if I can get it to be more reproducible and get some logs out of it. |
Been thinking about this and reading through other bug reports on I think the reason for a sometimes null pointer error might be related to how Android handles intents/launching other activities. It's not guaranteed the current activity will stick around while the file/image/etc is being loaded from the file picker. So when the activity is paused or destroyed to save memory and then resumed later, it needs to restore its state somehow. flutter_file_picker may need a way to retrieve "lost data" in case the activity calling it gets destroyed. It seems like image_picker gives a way to do that here: https://pub.dev/packages/image_picker#handling-mainactivity-destruction. Edit: I bet a way to simulate this is to have the |
How about logging the reason for "Installation Cancelled"? I'm not an Android dev. But to me it looks like "Installation Cancelled" message is a 'catch all errors' message. |
Bug description
I noticed for some time now, that the first time I am updating a already patched app, that the installation will fail. If I reselect the app and retry patching it works fine.
The log files shows two errors, hopefully they can help narrow down the problem!
I added the Log from the ReVanced Manager as a file, because the text is too long. I only copied the errors to the patch log part of this form.
revanced-manager_logcat_20241223125625606308.log
Version of ReVanced Manager and version & name of app you are patching
Manager v1.23.2
YouTube v19.47.53 (happend to older versions too)
Installation method
None
ReVanced Manager logs
Patch logs
Acknowledgements
The text was updated successfully, but these errors were encountered: