Page MenuHomeFreeBSD

bsdinstall: Provide help text for partitioning options
ClosedPublic

Authored by woodsb02 on Jan 17 2020, 7:12 AM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Nov 26, 5:54 AM
Unknown Object (File)
Sun, Nov 24, 1:49 AM
Unknown Object (File)
Sat, Nov 23, 5:06 AM
Unknown Object (File)
Fri, Nov 22, 6:12 PM
Unknown Object (File)
Fri, Nov 22, 5:01 PM
Unknown Object (File)
Tue, Nov 19, 2:19 AM
Unknown Object (File)
Thu, Nov 14, 1:00 PM
Unknown Object (File)
Thu, Nov 7, 3:04 AM
Subscribers

Details

Summary

bsdinstall: Provide help text for partitioning options

Includes commentary of when ZFS or UFS are recommended.

Also hoist the options text to the top of script as variables
(will help with future international translations).

Test Plan

Test run bsdinstall - works ok

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 28717
Build 26732: arc lint + arc unit

Event Timeline

Use dialog menu tag variables in subsequent case statement
(rather than re-typing the variable contents)

This revision is now accepted and ready to land.Jan 18 2020, 12:29 AM

Modify help text inline with discussion on freebsd-arch@ mailing list.

This revision now requires review to proceed.Jan 25 2020, 3:58 AM

Looks good. I like the pointer to the tuning guide.

This revision is now accepted and ready to land.Jan 25 2020, 4:05 AM

msg_partitioning_ufs_help: Update text to avoid implying multiple disks can be selected

This revision now requires review to proceed.Jan 25 2020, 4:08 AM
This revision is now accepted and ready to land.Jan 25 2020, 4:09 AM
rgrimes added a subscriber: rgrimes.

Other than perhaps a uniformity issue, that you can ignore if you wish, this looks fine to me.

usr.sbin/bsdinstall/scripts/auto
56

To be more uniform could this be "Guided UFS Disk Setup" as the zfs desc above explicity mentions ZFS too.

dteske requested changes to this revision.Jan 26 2020, 3:24 AM
dteske added inline comments.
usr.sbin/bsdinstall/scripts/auto
52–63

The variable names here need to be adjusted.

msg_partititoning_zfs="Auto (ZFS)"

For example should be:

msg_auto_zfs="Auto (ZFS)"

So that when used in code, the code reads like the text. Also, when searching the code, one can prepend "msg_", change to lower-case, and replace non-printable characters and spaces into underscore to find uses of the text.

Also, should the text be re-usable elsewhere, it is not tied to the partitioning menu. This is so that when someone performs international translation, fewer translations have to be made. For example, there is no reason to have:

msg_partitioning_shell="Shell"

When the following will do:

msg_shell="Shell"

Which makes any singular occurrence of the word "Shell" accessible under the "msg_shell" variable. Unnecessarily localizing individual words and phrases to a particular dialog, function, or module creates duplication and extra work for translators.

This revision now requires changes to proceed.Jan 26 2020, 3:24 AM
woodsb02 marked 2 inline comments as done.
  • Mention "UFS" in description of "Auto (UFS)" option (recommended by rgrimes)
  • Adjust variable names to read like the text (recommended by dteske)
  • Sort msg_ variable definitions in alphabetical order (as per scripts/zfsboot)

Thanks Rod and Devin for your comments - I have incorporated them

This revision was not accepted when it landed; it landed in state Needs Review.Jan 27 2020, 10:46 PM
This revision was automatically updated to reflect the committed changes.