Page MenuHomeFreeBSD

route.8: Describe metric modifier
ClosedPublic

Authored by pouria on Fri, May 15, 10:03 PM.
Tags
None
Referenced Files
F157942738: D57025.diff
Tue, May 26, 5:52 PM
F157912876: D57025.diff
Tue, May 26, 10:31 AM
F157912155: D57025.diff
Tue, May 26, 10:22 AM
Unknown Object (File)
Mon, May 25, 7:17 PM
Unknown Object (File)
Fri, May 22, 12:32 PM
Unknown Object (File)
Thu, May 21, 11:18 PM
Unknown Object (File)
Wed, May 20, 2:23 PM
Unknown Object (File)
Tue, May 19, 2:55 AM
Subscribers

Details

Summary

Describe -metric argument in the route manual.

Depends on D56335

Test Plan

mandoc lint:
mandoc -Tlint sbin/route/route.8

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 73137
Build 70020: arc lint + arc unit

Event Timeline

Thanks for doing this!

sbin/route/route.8
178

I don't have experience doing advanced networking, but should there also be some kind of structure in this area somewhere so it says something like:

route [-n] command [-net | -host] destination [gateway [-weight] [netmask]] [-fib number]

329

Maybe?

sbin/route/route.8
178

I don't have experience doing advanced networking, but should there also be some kind of structure in this area somewhere so it says something like:

route [-n] command [-net | -host] destination [gateway [-weight] [netmask]] [-fib number]

You're right, we should include -weight and -metric in that too.
but there are too many keywords too include AND you can use -metric without specifying -gateway during route del and vice versa.
So [gateway [-weight] [netmask]] won't help much.

329

I prefer to use word selected instead of preferred.
words like cost, metric, preference are similar but have their own technical meaning in routing stack.
I don't want to accidentally confuse users.

sbin/route/route.8
178

sounds like we need another list like what's on 335. we could do that in a followup commit, if you want?

329

okay, thanks for explaining!

sbin/route/route.8
178

Yes! That's a good idea.
I'll work on one in another revision.

This revision was not accepted when it landed; it landed in state Needs Review.Wed, May 20, 8:59 PM
This revision was automatically updated to reflect the committed changes.