bsdinstall: add variables to allow skipping over some phases of the install

[brd]

Summary

Some phases of the installer are not always needed so allow setting the
following varibles to skip their respective phases:

BSDINSTALL_KEYMAP_SKIP
BSDINSTALL_TIME_SKIP
BSDINSTALL_SERVICES_SKIP
BSDINSTALL_HARDENING_SKIP
BSDINSTALL_USERS_SKIP
BSDINSTALL_MANUAL_SKIP

If this seems reasonable I have ideas for a few more

[dteske]

Can we discuss perhaps using K&R style naming?

That is to say, BSDINSTALL_SKIP_* instead of BSDINSTALL_*_SKIP -- it would go a long way to being able to more-easily grep-out the skippable options and also open the prefix to being used in other parts of bsdinstall that are unrelated (read: other scripts/functions)

[brd]

In D35357#801606, @dteske wrote:

Can we discuss perhaps using K&R style naming?

That is to say, BSDINSTALL_SKIP_* instead of BSDINSTALL_*_SKIP -- it would go a long way to being able to more-easily grep-out the skippable options and also open the prefix to being used in other parts of bsdinstall that are unrelated (read: other scripts/functions)

I was thinking in a follow up commit about adding more variables, BSDINSTALL_TIME_NAME and BSDINSTALL_TIME_DESC to populate the menu in finalconfig(). Doing it that way seemed a bit nicer to me, but I don't feel strongly about it. So instead we could use BSDINSTALL_SKIP_*, BSDINSTALL_NAME_* and BSDINSTALL_DESC_*.

[dteske]

In D35357#801773, @brd wrote:

In D35357#801606, @dteske wrote:

Can we discuss perhaps using K&R style naming?

That is to say, BSDINSTALL_SKIP_* instead of BSDINSTALL_*_SKIP -- it would go a long way to being able to more-easily grep-out the skippable options and also open the prefix to being used in other parts of bsdinstall that are unrelated (read: other scripts/functions)

I was thinking in a follow up commit about adding more variables, BSDINSTALL_TIME_NAME and BSDINSTALL_TIME_DESC to populate the menu in finalconfig(). Doing it that way seemed a bit nicer to me, but I don't feel strongly about it. So instead we could use BSDINSTALL_SKIP_*, BSDINSTALL_NAME_* and BSDINSTALL_DESC_*.

I would say that if BSDINSTALL_NAME_TIME is used outside of a "time" module then this would be proper, but if there is some "time" module (e.g., "bsdinstall time") then I would prefer that all module-specific variables be named "BSDINSTALL_{MODULE}_*" and reserve "BSDINSTALL_*" to properties that have scope outside of a single module or group of modules.

[brd]

Swap the name of the variables

[brd]

@dteske ping?

[allanjude]

Reviewed by: allanjude

[dteske]

Great work. I gave it a thorough check, inside and out. Green-lit to go.