Distribution will be done after all of the lualoader manpages are created.
Details
- Reviewers
cem rpokala - Group Reviewers
manpages - Commits
- rS330268: Add menu.lua(8), but do not add to distribution
Diff Detail
- Repository
- rS FreeBSD src repository - subversion
- Lint
Lint Skipped - Unit
Tests Skipped - Build Status
Buildable 15218
Event Timeline
stand/lua/menu.lua.8 | ||
---|---|---|
1 | Don't forget the SPDX tag. | |
34 | You can drop this fragment; I think it's okay to just start with the name of the file. | |
38 | influenced by loader environment variables. As opposed to environment variables at build-time. That might be clear to most people, but make it explicit. | |
51 | Since "entries" starts with a vowel sound, this should be "contain an". | |
57 | I think "If the value of the entries key is itself a table" reads a bit cleaner. | |
76 | You should define or describe "carousel". | |
85 | "A table (or a lambda that returns a table) of the possible choices for this carousel." I think that reads a little better. | |
88 | Same. | |
168 | I think the ".It Pa /boot/lua/menu.lua" and ".Nm" lines aren't meant to be here...? |
stand/lua/menu.lua.8 | ||
---|---|---|
177 | My eyes are rendering this as: The loader(8) /boot/lua/loader.lua loader(8) bootstrapping script. Which still doesn't look right. |
I've ripped out the FILES section entirely. This was a holdover from the 4th manpages, but it likely belongs elsewhere that describes the entirety of lualoader. There's not exactly only one loader that this stuff will be applicable to anyways, so it wasn't entirely accurate.