-
Notifications
You must be signed in to change notification settings - Fork 106
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
Destination host unreachable - not working #17
Comments
Hi, Do window prints summary about the round-trip-time for either or both of success or failure cases? If so, please paste them here. |
D:>ping 140.80.159.35 Pinging 140.80.159.35 with 32 bytes of data: Ping statistics for 140.80.159.35: |
It should return false for unreachable host after this patch :D |
thanks a lot! |
it is wrong solution. _Russian output windows_: Статистика Ping для 80.91.249.69: _German_ Ping-Statistik für 140.80.159.35: _English_: Reply from 140.80.159.35: bytes=32 time<1ms TTL=128 Ping statistics for 140.80.159.35: Chinese:??? We discussed just on other ping project the windows outputs: The solution with works good even for linux. |
Oops. I will write another patch for that issue. Thanks for the explanation. |
I call search() instead of match() since we don't care the TTL contents atm. Let me know whether this patch works or not. |
Please fire another ticket if there are any new problems. close #17 |
My english windows returns following if destination not reachable:
'''
C:>ping 192.168.1.72
Pinging 192.168.1.72 with 32 bytes of data:
Reply from 192.168.1.69: Destination host unreachable.
Reply from 192.168.1.69: Destination host unreachable.
Reply from 192.168.1.69: Destination host unreachable.
Reply from 192.168.1.69: Destination host unreachable.
Ping statistics for 192.168.1.72:
Packets: Sent = 4, Received = 4, Lost = 0 (0% loss),
'''
And
lines[t].indexOf ('(0%')>0
returns true on the "Packets: Sent = 4, Received = 4, Lost = 0 (0% loss),"
The text was updated successfully, but these errors were encountered: