-
Notifications
You must be signed in to change notification settings - Fork 36
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
Add workaround for Samsung devices #6
Comments
Thanks for the report. I don't own any Samsung device, so I had no idea. Before I decide what to do about it, I'd like to understand more how this even works. I already found some information but if you have any useful resources, you're welcome to share them. |
I haven't tried but I'm just asking, |
I don't really know... Haven't got around to investigate this further. |
I'm using an S22 SM-S901W Snapdragon on android 14. Whenever I type in termux-adb it seems to be working. I'm still doing some research for my S10 G973U which is also Snapdragon android 12 and don't want to screw anything up. Haven't even plugged it in yet so here's hoping it's good news... Feel free to throw some advice my way since I don't even know if it'll work at all. All I want for Christmas is my OEM and bootloader unlocked! |
I can't run it anyway on my phone (Samsung A24 - Android 14)...
|
This looks unrelated to Samsung. Do you have the latest version running? It's possible I'll need to do a new build. This linking issue happens from time to time when Termux updates the protobuf library. |
"pkg install android-tools" don't forget to split screen the first time you connect. Also after you do "adb pair ip:port pairing code" you have to run "adb connect ip:port". Every time after that all you need to do is turn on wireless debugging and "adb connect ip:port" |
Sorry @nohajc. Don't mean to step on any toes here, just trying to help. Rish is pretty easy to use to get a shell too |
I don't think the issue has anything to do with wireless debugging either. You should be able to use termux-adb for normal USB debugging. These linking errors usually appear when there's an update of termux dependencies and it just means I have to compile my code again. Unfortunately, google's protocol buffer C++ library doesn't keep any binary compatibility. |
Anyway, this time I cannot reproduce it. I just did |
i tried turning off 'Auto Blocker' (incase it is a security layer related issue) but it didn't help either... Termux version: v0.118.0+8e3a898 |
Is this some beta or a custom build? When I run
|
a debug build from GitHub actions' artifact of commit 8e3a898 |
That will be the problem. The way I build and release the package now, I can only support one revision at a time. That means it works with the current stable but it will most likely stop working soon as the changes you're building against make it to a release. At that point I'll make sure I update my repo too. You'd basically have to do your own termux-adb build to have it running now. I'm not happy about the situation but unfortunately didn't have time to come up with a proper solution yet. If termux-adb became part of the official Termux repository, it would solve all these issues. I didn't have such ambition so far though... |
Hi @nohajc , |
Hi, I'm going to look into it. |
Ok, first, apologies for any confusion as I was previously referring to version 0.2.2-1 which is in fact not released. I did however try with 0.2.1-3 and stable Termux (after pkg upgrade) and it doesn't give me the linker error. Can you verify you're running Termux from F-Droid and not any custom build? Also, which version of libprotobuf do you have? It should be
|
Yes, I am using official Termux app from F-Droid
|
Ok, turned out I was using a different repository mirror which wasn't exactly up to date. I switched to the official one, upgraded and now I can replicate your issue. I'll make a new build ASAP. Thanks for reporting! |
@huynhtanloc2612 Try to upgrade now. Version |
Great! Previous problem was fixed. There is no error now. I will do more tests with usb cable tomorrow (because I don't have any cable with me now) and let you know if it is ok. |
I made another release (this time really |
I have just tested with cable and saw that termux-adb (v0.2.1-4) could detect and connect with other devices successfully and adb commands work correctly. |
30 seconds seems like too much... One thing you could try is to
On the first run, termux-adb should detect all connected devices relatively fast. If the server is already running, devices are checked periodically in the background but it should be more often than 30 s. Sometimes, the phone's battery saving features can mess with the clock (like stopping the ticks or slowing them down) but this typically happens to me only when the display is off. I'll try to play with it and see if I can reproduce any of that with the new version. |
I have tried above suggestion but device detection time is no change (still very slow) |
That was my theory too. Unfortunately, such problems may be very difficult to work around within Termux. I suppose if somebody made a native ADB app designed for Android, it might give better results. |
Can we feedback or suggest Termux developers about this issue of ADB so they can give good solution to overcome it and suggest integrating termux-adb to official Termux repo...as well? I think it will be great. |
@nohajc I have just found that the slowness happened with |
Can I ask you to report it in the termux repo? With the information about your device etc. I can try to talk to someone about integration to the official repository in the meantime. Thanks! |
Yes, I have just reported this termux/termux-api#680 |
Due to some issue with the way samsung devices handles things. Unsure what exactly if I'm being honest
ANDROID_NO_USE_FWMARK_CLIENT=1 fakeroot
is needed to run termux-adb or adb in general through termux. Initially I didn't need it but now I notice it fails if I don't use that. I'm wondering if it would be possible to add some sort of detection for Samsung devices and have that automatically appended?If not at least add a footnote to the readme about it.
EDIT:Fold 4 user
The text was updated successfully, but these errors were encountered: