Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Can't run it. Device 'd:' not found #1

Closed
ewoks opened this issue Mar 31, 2017 · 5 comments
Closed

Can't run it. Device 'd:' not found #1

ewoks opened this issue Mar 31, 2017 · 5 comments
Assignees
@rom1v
Labels
bug

Comments

@ewoks
Copy link

ewoks commented Mar 31, 2017 •
edited

// I wrote following line in terminal, next 3 lines are output written by ghirehtet:
./gnirehtet rt
Starting gnirehtet...
'adb' -s 'd:' reverse tcp:31416 tcp:31416
error: device 'd:' not found
// Dialog should appear but it doesn't

Siera macOS 10.12.4,
adb 1.0.39
Android 5.0.1, Samsung (I know, I am sorry)

// If I manually install, than execute:
adb reverse tcp:31416 tcp:31416
adb shell am startservice -a com.genymobile.gnirehtet.START
// => Dialog appears as expected
@rom1v
Copy link
Collaborator

rom1v commented Mar 31, 2017 •
edited

It seems that you executed the command with an additional argument having value d::

./gnirehtet rt d:

Did you do that?

EDIT: OK, I reproduced the problem on MacOS. Thanks for the report. I'm working on it.

@ewoks
Copy link
Author

ewoks commented Mar 31, 2017

not really, that is why I am surprised as well where 'd:' is coming from

@ewoks
Copy link
Author

ewoks commented Mar 31, 2017

Tested with other non Samsung device (N5): Same error

rom1v added a commit that referenced this issue Mar 31, 2017
@rom1v
Fix "getopt" usage in gnirehtet script
ee407b9 
The syntax of the shell command "getopt" may differ on different
systems.

Provide the parameters the way which is supported by both versions:
<https://linux.die.net/man/1/getopt>
<https://developer.apple.com/legacy/library/documentation/Darwin/Reference/ManPages/man1/getopt.1.html>

Related to <#1>.
@rom1v
Copy link
Collaborator

rom1v commented Mar 31, 2017 •
edited

Commit ee407b9 should fix the issue.

You can download the updated version of the script: gnirehtet.

Please confirm that it solves the problem for you 😉

@rom1v rom1v self-assigned this Mar 31, 2017
@rom1v rom1v added the bug label Mar 31, 2017
@ewoks
Copy link
Author

ewoks commented Mar 31, 2017

I can confirm that problem is not there anymore on macOS and stdout is showing:
2017-03-31 13:15:36.122 I Main: Starting server...
ticket can be closed and new script packed in the latest release ;) thanks

@rom1v rom1v closed this as completed Mar 31, 2017
@bunam bunam mentioned this issue Apr 4, 2017
Closed
@ReimuDaze ReimuDaze mentioned this issue Feb 13, 2018
Open
@caburet caburet mentioned this issue Nov 7, 2018
Open
@andrei-lazarov andrei-lazarov mentioned this issue Jun 10, 2019
Closed
@alligatorme alligatorme mentioned this issue Feb 23, 2020
Closed
@liquidnight99 liquidnight99 mentioned this issue Mar 3, 2020
Open
@ghost ghost mentioned this issue Apr 21, 2020
Open
@simonferndriger simonferndriger mentioned this issue Jan 5, 2021
Open
@EisigesEis EisigesEis mentioned this issue Jul 27, 2021
Open
@billybhaub billybhaub mentioned this issue Oct 10, 2021
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@rom1v rom1v

Labels
bug
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@rom1v @ewoks