Details
- Reviewers
rpokala 0mp - Group Reviewers
manpages - Commits
- rS338054: Add config.lua(8) to the tree
Diff Detail
- Repository
- rS FreeBSD src repository - subversion
- Lint
Lint Not Applicable - Unit
Tests Not Applicable
Event Timeline
Just a note: Anywhere I wrote {kernel} is because I couldn't come up with a better markup. I have no idea what the ideal syntax is for "this is a path with this variable substituted in" rather than "this is a path with this variable name literally in it"
stand/lua/config.lua.8 | ||
---|---|---|
5 ↗ | (On Diff #40672) | Per discussions over the last few months, remove "All rights reserved." |
109 ↗ | (On Diff #40672) | I think config.loadKernel needs a re-write. As it stands, if says that if you don't specify the new kernel to load, it will attempt to load a module named kernel, and that it will try {kernel} (with no /boot/ prefix) twice. |
143 ↗ | (On Diff #40672) | "restore the environment" to what? |
I rewrote some things around all of your comments, but it's early. =) The config.loadKernel was mostly-ish correct, but I rewrote the {kernel} instances to {other_kernel} and rewrote other parts for clarity. I also added some paragraph breaks for additional clarity (hopefully).
I've reviewed the mdoc side of this change. Looks fine!
BTW, should we add some cross-references from other manpages to config.lua(8)?
Thanks! I'll add all appropriate cross-references once I've got the complete set of manpages up.