Page MenuHomeFreeBSD

Some fixes to bsdinstall(8) documentation
ClosedPublic

Authored by arrowd on Dec 26 2019, 9:57 AM.
Tags
None
Referenced Files
Unknown Object (File)
Oct 24 2024, 8:41 PM
Unknown Object (File)
Sep 23 2024, 1:11 AM
Unknown Object (File)
Sep 22 2024, 11:19 PM
Unknown Object (File)
Sep 22 2024, 7:24 PM
Unknown Object (File)
Sep 22 2024, 3:38 AM
Unknown Object (File)
Sep 17 2024, 10:15 PM
Unknown Object (File)
Sep 15 2024, 3:27 PM
Unknown Object (File)
Sep 8 2024, 2:33 AM
Subscribers

Details

Summary
  • Mention bootconfig target in TARGETS section.
  • Fix example for DISTRIBUTIONS variable. The .tgz suffixes are required.
  • Document PARTITIONS variable, which is only mentioned in the examples, but doesn't have its own point.

Diff Detail

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

Event Timeline

Found two issues. Maybe textproc/igor and "mandoc -Tlint" will find some more.

usr.sbin/bsdinstall/bsdinstall.8
122

s/approprate/appropriate/

268

You need to have a line break after a sentence stop.

arrowd marked 2 inline comments as done.
arrowd edited the summary of this revision. (Show Details)
  • Address comments.
  • Add gpart bootcode command invocation into UFS example.
  • mandoc -Tlint clean.
  • igor complains about date being not today (line 28) . Should I change it?
0mp added inline comments.
usr.sbin/bsdinstall/bsdinstall.8
265

This has been committed in rS354967

268

How about "The partitioning of the disk onto which...". This way it would match descriptions of other environment variables in this section.

  • mandoc -Tlint clean.
  • igor complains about date being not today (line 28) . Should I change it?

Yes, but we usually change it once all issues are cleared up and then it becomes the date of the commit. For people who don't have a commit bit, it will be done by the person doing the commit.

arrowd marked 2 inline comments as done.

Address comments

Apart from a couple of missing words, looks fine.

@bcr, could you approve it? I cannot approve doc changes on my own just yet.

bsdinstall.8
122 ↗(On Diff #66114)

... the appropriate partition ...

271 ↗(On Diff #66114)

of the

This revision is now accepted and ready to land.Feb 6 2020, 4:37 PM

Looks good. Don't forget to bump the .Dd and set an MFC timeout of your choice.

This revision was automatically updated to reflect the committed changes.