Sponsored by: The FreeBSD Foundation
MFC after: 1 week
Details
Details
Diff Detail
Diff Detail
- Repository
- rG FreeBSD src repository
- Lint
Lint Skipped - Unit
Tests Skipped - Build Status
Buildable 68995 Build 65878: arc lint + arc unit
Event Timeline
| usr.sbin/sndctl/sndctl.8 | ||
|---|---|---|
| 186 ↗ | (On Diff #167420) | I think xo_options is really what's relevant to the reader of sndctl. That links to libxo(3) if they're curious. |
| usr.sbin/sndctl/sndctl.8 | ||
|---|---|---|
| 186 ↗ | (On Diff #167420) | I followed what w(1) and wc(1)'s man pages say. |
| usr.sbin/sndctl/sndctl.8 | ||
|---|---|---|
| 186 ↗ | (On Diff #167420) | I think it's wrong there too, but that isn't a good argument for why we should do it here. What does the reader or soundctl might want to see also? xo_options explains how to actually use it. |
| usr.sbin/sndctl/sndctl.8 | ||
|---|---|---|
| 186 ↗ | (On Diff #167420) | I think xo_options(7) is the reasonable think to refer to, since this isn't a programing manual about how the internals of sndctl(8) work, but how to use the utility with --libxo, which is explained in xo_options(7). This isn't wrong. |
| usr.sbin/sndctl/sndctl.8 | ||
|---|---|---|
| 186 ↗ | (On Diff #167420) | That is what I am saying. You should link to xo_options(7). The diff is currently linking to libxo(3) instead. |
Comment Actions
Refer to xo_options(7), not libxo(3).
| usr.sbin/sndctl/sndctl.8 | ||
|---|---|---|
| 186 ↗ | (On Diff #167420) | ...Scratch that. I am an idiot. I thought you said the opposite, and I also thought the patch did the right thing. |
| usr.sbin/sndctl/sndctl.8 | ||
|---|---|---|
| 186 ↗ | (On Diff #167420) | Actually it happens to everyone! |