Page MenuHomeFreeBSD

Give example of deleting partitions and partitioning scheme
ClosedPublic

Authored by danfe on Sep 12 2017, 10:14 AM.
Tags
None
Referenced Files
Unknown Object (File)
Dec 23 2023, 9:20 AM
Unknown Object (File)
Dec 2 2023, 12:41 PM
Unknown Object (File)
Nov 23 2023, 5:15 PM
Unknown Object (File)
Nov 15 2023, 3:27 PM
Unknown Object (File)
Nov 11 2023, 8:32 AM
Unknown Object (File)
Nov 7 2023, 5:19 AM
Unknown Object (File)
Nov 7 2023, 5:00 AM
Unknown Object (File)
Nov 6 2023, 5:33 PM

Details

Summary

Our users (including myself) often get confused and frustrated when trying to delete partition table and getting Device busy error because they forgot (or do not ever know that they have) to delete all its partitions first. Manual page mentions this briefly, but does not stress it out enough. Personally I always forget this annoying feature and have to google the answer every time I want to delete partition table on some drive.

Provide working example and explanation of Device busy error and how to properly deal with it.

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Skipped
Unit
Tests Skipped
Build Status
Buildable 11512

Event Timeline

This revision is now accepted and ready to land.Sep 12 2017, 10:21 AM

Still waiting for a review/clearance from manpages...

bjk requested changes to this revision.Sep 16 2017, 7:04 PM
bjk added a subscriber: bjk.

manpages here.
Just a few minor tweaks for silly parts of the English language (inline).

You could update the date in .Dd to match the commit date when committing if you want.

There is also a general trend to avoid using "you" and addressing the reader directly, though I am not going to insist that you rewrite the whole thing to comply with that.

/usr/src/sbin/geom/class/part/gpart.8
1330

Add an 'a' for "If you get a"

1332

"with the"

1345

"invoke the"

1347

"with the"

This revision now requires changes to proceed.Sep 16 2017, 7:04 PM
danfe marked 4 inline comments as done.

I wholly agree with you that addressing the reader this way better be avoided, but I failed to come up with you-less wording (I've tried to dance off e.g. the "If one gets..." but it sounded/looked even worse).

Eventually I've decided it was OK here in the EXAMPLES section, and also because you is already (ab)used earlier in the manual page (a NOTE before SYSCTL VARIABLES section). If someone comes with a better wording we can always improve it in the future revisions. For the time being, I feel this change is good enough and useful even in its current form (after fixed articles).

I wholly agree with you that addressing the reader this way better be avoided, but I failed to come up with you-less wording (I've tried to dance off e.g. the "If one gets..." but it sounded/looked even worse).

Eventually I've decided it was OK here in the EXAMPLES section, and also because you is already (ab)used earlier in the manual page (a NOTE before SYSCTL VARIABLES section). If someone comes with a better wording we can always improve it in the future revisions. For the time being, I feel this change is good enough and useful even in its current form (after fixed articles).

I agree that it is useful even in its current form, thanks for putting it together.

This revision is now accepted and ready to land.Sep 19 2017, 12:02 AM

It would be good to note also about the error when you can not modify partition table until it will be recovered.
This is frequent problem when GPT is marked as CORRUPT.

Noted @ae; I'll think about how to put this piece of information into the manpage and come up with another diff. Meanwhile let's close this one first.

This revision was automatically updated to reflect the committed changes.
yuripv_gmx.com added inline comments.
head/sbin/geom/class/part/gpart.8
1350 ↗(On Diff #33224)

This block closer seems to be superfluous:

mandoc: gpart.8:1350:2: ERROR: skipping end of block that is not open: Ed

Yuri, can you provide a patch and run it through the manpages? I'm myself not a mandoc expert of any kind...

wblock added inline comments.
head/sbin/geom/class/part/gpart.8
1328 ↗(On Diff #33224)

This should be title-capitalized.

.Ss Deleting Partitions and Partitioning Scheme
1329 ↗(On Diff #33224)
If a
1331 ↗(On Diff #33224)
error is shown when trying to destroy a partition table, remember that
1332 ↗(On Diff #33224)
all of the partitions must be deleted first with the
1335 ↗(On Diff #33224)
In this example,
1337 ↗(On Diff #33224)
has three partitions:
1345 ↗(On Diff #33224)

Please don't use that halting "Alternatively," start to a sentence.

Rather than deleting the partitions and then destroying the partitioning scheme, the
.Fl F
option can be given with
.Cm destroy
to automatically delete the partitions first.

Oh, and as far as the use of "you", we do have specific guidelines to avoid it: https://www.freebsd.org/doc/en_US.ISO8859-1/books/fdp-primer/writing-style.html

"Write in a formal style. Avoid addressing the reader as “you”. For example, say “copy the file to /tmp” rather than “you can copy the file to /tmp”."

@wblock, while your comments are certainly valid, they arrived a bit too late (special note on the "usage of you": yes, I know and agree this is a bad style, but I failed to come up with you-less wording, see the earlier comments).

As a manpages member and committer, I guess you can simply fix those shortcomings in one of the next usual sweep over the manpages.