Page MenuHomeFreeBSD

Bug 206940 - zpool (command terminology) misused to describe a pool
ClosedPublic

Authored by sd_beastie.io on Apr 30 2016, 2:46 PM.
Tags
None
Referenced Files
F132269695: D6163.id16746.diff
Wed, Oct 15, 9:01 AM
Unknown Object (File)
Sat, Oct 11, 8:17 AM
Unknown Object (File)
Tue, Oct 7, 2:23 AM
Unknown Object (File)
Tue, Sep 30, 1:00 AM
Unknown Object (File)
Sat, Sep 27, 2:30 AM
Unknown Object (File)
Sat, Sep 27, 2:24 AM
Unknown Object (File)
Sat, Sep 27, 2:22 AM
Unknown Object (File)
Sat, Sep 27, 2:19 AM
Subscribers

Details

Summary

Change incorrect term zpool to pool. It should be 'pool' to be correct even though informally, everyone refers to the pool as zpool.

  • change main term from zpool to pool and fix up references
  • put in a few other igor cleanups
Test Plan

igor and full doc build

Diff Detail

Repository
rD FreeBSD doc repository - subversion
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

sd_beastie.io retitled this revision from to Bug 206940 - zpool (command terminology) misused to describe a pool.
sd_beastie.io updated this object.
sd_beastie.io edited the test plan for this revision. (Show Details)
sd_beastie.io added reviewers: allanjude, wblock.
sd_beastie.io set the repository for this revision to rD FreeBSD doc repository - subversion.
en_US.ISO8859-1/books/handbook/zfs/chapter.xml
2440 ↗(On Diff #15771)

This is output from a command and should not be changed. Or rather, changed in the source, but not here.

3479 ↗(On Diff #15771)

The ID change is okay, but it is important to build the document to make sure that xrefs to the previous name aren't broken.

3537 ↗(On Diff #15771)

Better to use <command> here rather than the man reference:

in <command>zpool create</command>.  All vdevs
3538 ↗(On Diff #15771)

Moving the "must be" to this line is a whitespace-only change, and should not be mixed with a content change. But the suggestion for the previous line makes that unnecessary anyway.

Done - patch coming soon.

en_US.ISO8859-1/books/handbook/zfs/chapter.xml
3479 ↗(On Diff #15771)

xref checked by full build.

3538 ↗(On Diff #15771)

This still ends up being on the next line b/c of the command start and end tags.

sd_beastie.io marked an inline comment as done.

Updated diff based on review comments

en_US.ISO8859-1/books/handbook/zfs/chapter.xml
2268 ↗(On Diff #16746)

Not your fault, but the "no files can be" is just unpleasant. Better:

Files cannot be
3538 ↗(On Diff #15771)

No, wrapping that line is a whitespace-only change. So you have to break the formatting rules while making the content change:

in <command>zpool create</command>.  All vdevs must be
at least 128&nbsp;MB in size.</para>

That too-long line will be fixed in the followup whitespace-only commit.

sd_beastie.io marked 4 inline comments as done.

Updated based on review feedback.

en_US.ISO8859-1/books/handbook/zfs/chapter.xml
3538 ↗(On Diff #15771)

Got it.

This revision is now accepted and ready to land.May 24 2016, 2:04 AM
allanjude edited edge metadata.

fine with me

This revision was automatically updated to reflect the committed changes.