-
-
Notifications
You must be signed in to change notification settings - Fork 33
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
Support for Samsung Galaxy Grand Prime VE #140
Support for Samsung Galaxy Grand Prime VE #140
Conversation
Added grandprimevelte to the list, but marked it as `untested`.
Hmmm, OAI / adb gets stuck after executing the |
Does the device to |
Yes it shows up for Log for install view:
|
Curious issue... 🤔 I had something similar after trying too many things with adb and fastboot. I was already worried my device/usb-ports/cables are broken, but a PC system reboot fixed that issue. That also seems to be a solution for some people with this issue online (for example XDA). |
I will try a reboot later and maybe switching the port... |
Ok, I tried it - nothing changes... |
Ohhh I think I know why it fails - TWRP 2.8.7.0 does not know e.g. the command |
This only appears on TWRP 2.8.7.0 - I force-flashed a new unofficial version manually. |
I acquired a newer unofficial version of TWRP, but OAI won't let me flash it. Is there an option to bypass this block? |
The very basic check on the recovery compatibility with the device is performed by looking for the device code and "twrp" in the file name. So if your TWRP image follows this convention it should let it flash it. |
Ok, that worked, but |
@tsterbak Is there a variable in which the current device is stored? |
You could try to replace |
In the views you can access it by |
Maybe, there should be a warning somewhere in general, that there might be issues with old twrp versions possibly not supporting all features. |
Ok, yeah with |
I tried to do it myself, but I only get My changes:
|
def __init__(self):
self.state.config.device_code = device_code Is not necessary, and I also don't know where this function is going? if self.state.config.device_code==grandprimevelte:
for line in run_command("adb shell twrp wipe data", bin_path):
yield line
else:
for line in run_command("adb shell twrp format data", bin_path):
yield line I assume this is in the tooling module. There you have no access to the components of the views and the state since I try to keep the direct device interactions as separate from the rest as I could manage. |
Uhhhhhm I must also say, that I am a total beginner in python and only hack myself around things... If it would be possible, please make the changes yourself. I am also not really available for coding until next Sunday so yeah... |
I will try it myself now once... |
@tsterbak Ok, I am not really getting anywhere... try:
for line in run_command("adb shell twrp format data", bin_path):
yield line
except:
print("Data partition can not be formatted, attempting to wipe it.")
for line in run_command("adb shell twrp wipe data", bin_path):
yield line |
Hey @MagicLike, I implemented a small change where, if |
I am gonna try it out rn! |
@tsterbak Ok, I tested it yesterday multiple times and the log does not show any indication, that there was a retry with |
Ah wait! I made a mistake... I'll fix the code in the other branch now. |
Hmmm, the behaviour is still the same... |
Thanks for testing again. |
Sadly not... |
Hm, that's a pity. So hard to tell what is going on... |
I actually don't know, but I tried to run the command seperately and it returns no error... |
Did it work now? |
Not quite - I am gonna do another test on a fresh codebase directly pulled from main and insert the config and tooling changes. Let's see... |
We'll, let's see. Otherwise I have to find a way to iterate on this faster to figure out what is going wrong and how to fix it... |
Here is the complete log - it still does not seem to do anything with the data partition. Maybe make it standard to first wipe it and - if supported - format it? |
I guess that would be a good option... (?) |
I'm a bit reluctant to change the current command sequence since it has been tested with a lot of devices. But maybe this is the way to go. I gave it another try and make it explicitly fail if the command is unknown. And then resolve that by retrying. Hopefully this works... |
Uhhhhmmmm
|
Ahhh, stupid mistake... Sorry for wasting your time... |
Should be fixed now. |
No problem, without me "wasting" time, the issue wouldn't have been discovered 😁 |
Gonna test it out later today! |
@tsterbak It finally works!
|
So please wait with merging both PRs! |
@tsterbak Uhhhhh this happens, when I want to click continue after installing:
|
Oh, I see... It should be fixed now :) |
Everything works, flashing is fine - ready to merge! 😃 |
If `adb twrp format data` fails, retry with `adb twrp wipe data`. This should address issues with old versions of TWRP as found in #140.
Added a config for the Samsung Galaxy Grand Prime VE (
grandprimevelte
), but I was not able to test it yet, as the device is not officially supported by LineageOS.