Differential D51080
Update multiple driver manual pages to use sysrc(8) instead of rc.conf(5) Authored by felix.the.red_gmail.com on Sat, Jun 28, 4:05 AM. Tags None Referenced Files
Details The preferred style to load kernel modules is to use sysrc(8) From the original PR: PR:268827 $ find . -type f -regex '.*\.[1-9]$' -print0 | xargs -0 grep -n "kld_list"
Diff Detail
Event TimelineComment Actions I almost think the kld_load lines (and the kldload now mentioned in some but not others) need to go from all of them and the manual procedure should be documented in one place elsewhere? Lastly there's other stuff in the diff which is not relevant for this change and should be another review? Also please upload a diff using -U10000 wen doing manually so there is context. Comment Actions Nack. The synopsis sections for drivers show lines of configuration, as it says and has said in intro.4 since the beginning. Consistency is what makes manpages strong. We should not mix commands in there. I presented at BSDCan, and everyone seemed very happy with it, what I have been doing and intend to continue to do. You can see this in vt(4), uftdi(4), iwx(4), and many others. Comment Actions Intro(8), or rc(8), or rc.conf(5) should explain that you can use service(8) or sysrc(8) to edit rc.conf. Comment Actions The uftdi(4) in current is the best example of what I want to do. How to use things needs to be described in DESCRIPTION, for consistency with the entire body of every section of manual pages. SYNOPSIS needs to list the possible ways to use it, like the entire body of every section of manual pages. Comment Actions Based on review from @ziaee, I think we should abandon this revision and close the PR. |