search

traviscross / mtr

Official repository for mtr, a network diagnostic tool
http://www.bitwizard.nl/mtr/
GNU General Public License v2.0
2.64k stars 337 forks source link

add --max-display-paths option #430

Closed bewing closed 2 years ago

bewing commented 2 years ago

It is not unknown to have datacenters with over 8 ECMP paths between two hops. Expand the maximum number of ECMP hosts stored in addrs to 128, and add a new option to control the number to display, defaulting to the old MAX_PATHS of 8

rewolff commented 2 years ago

Hey. Sorry about that, but I'm afraid I saw your other patch first and clicked "merge" on that one. Now this one has conflicts. I think it's easy, but I would like it if you could double check it as you're the one that changed that line.... :-) P.S. I fully agree with the patch and the philosofy.

bewing commented 2 years ago

Wouldn't have mattered which merged first, the other would've needed a rebase/manual resolution. Pushed fixed changeset.

rewolff commented 2 years ago

Thanks!