Page MenuHomeFreeBSD
Differential D19437

explain fsck -f more in detail
ClosedPublic
Actions

Authored by wosch on Mar 3 2019, 4:47 PM.
Tags
None
Referenced Files
F114603882: D19437.diff
Mon, Apr 14, 8:28 AM
F114587336: D19437.id.diff
Mon, Apr 14, 5:06 AM
F114551209: D19437.id54725.diff
Sun, Apr 13, 10:07 PM
F114545522: D19437.id54642.diff
Sun, Apr 13, 9:02 PM
Unknown Object (File)
Sun, Apr 13, 8:55 PM
Unknown Object (File)
Sun, Apr 13, 6:43 AM
Unknown Object (File)
Sun, Apr 13, 6:40 AM
Unknown Object (File)
Mon, Mar 24, 6:44 AM
View All 84 Files
Subscribers
0mp
emaste
imp

Details

Reviewers
mckusick
0mp
imp
Group Reviewers
manpages
Commits
rS368378: MFC r344922 (by wosch): explain ``fsck -f'' more in detail
rS344922: explain ``fsck -f'' more in detail
Summary

As discussed in https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=223491 you may need to run fsck -f on a file system with a corrupt journal due a lying disk. The current man page is not clear about whats happens here. This patch adds a brief explanation.

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

wosch created this revision.Mar 3 2019, 4:47 PM
Herald added a reviewer: manpages. · View Herald TranscriptMar 3 2019, 4:47 PM
Herald added a subscriber: imp. · View Herald Transcript
0mp requested changes to this revision.Mar 3 2019, 4:59 PM
0mp added a subscriber: 0mp.

Also, please remember to bump the date of the manual page.

sbin/fsck/fsck.8
147

As per style.mdoc(5) it would be better to stylize it like this:

Running
.Dq Li fsck -f
ignores ...
This revision now requires changes to proceed.Mar 3 2019, 4:59 PM
mckusick added a comment.Mar 3 2019, 9:28 PM

Your change looks good. I'll approve once you update the suggested mandoc refinement.

emaste added a subscriber: emaste.Mar 4 2019, 3:52 PM

Thanks, this seems reasonable to me too.

wosch updated this revision to Diff 54725.Mar 5 2019, 7:09 AM

follow style.mdoc(5) rules

0mp accepted this revision.Mar 5 2019, 7:02 PM

OK from manpages (keep in mind I am only a ports committer ;)

This revision is now accepted and ready to land.Mar 5 2019, 7:02 PM
imp accepted this revision.Mar 5 2019, 8:58 PM
mckusick accepted this revision.Mar 7 2019, 12:15 AM

Looks good, thanks for clarifying -f.

Closed by commit rS344922: explain ``fsck -f'' more in detail (authored by wosch). · Explain WhyMar 8 2019, 10:03 AM
This revision was automatically updated to reflect the committed changes.
gbe added a commit: rS368378: MFC r344922 (by wosch): explain ``fsck -f'' more in detail.Dec 6 2020, 7:39 AM

Revision Contents
Changeset List

PathSize
sbin/
fsck/
10 lines

Diff 54725

View Options

sbin/fsck/fsck.8

Loading...