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 Skipped - Unit
Tests Skipped - Build Status
Buildable 17115
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 | ||
|---|---|---|
| 6 | Per discussions over the last few months, remove "All rights reserved." | |
| 110 | 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. | |
| 144 | "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.