Page MenuHomeFreeBSD

lock(1): Add EXAMPLES section
ClosedPublic

Authored by fernape on Dec 10 2020, 3:58 PM.

Details

Reviewers
yuripv
Group Reviewers
manpages
Commits
rS368556: lock(1): Add EXAMPLES section
Summary

Add simple example showing the use of the flags: p, t, v

Test Plan
  • mandoc -Tlint clean
  • igor clean
  • aspell happy
  • man ./lock.1 renders the page properly

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

This revision is now accepted and ready to land.Dec 10 2020, 4:00 PM
gbe added inline comments.
usr.bin/lock/lock.1
77 ↗(On Diff #80525)

I would reword this sentence in "5 minutes and disabling switching of virtual terminals".

usr.bin/lock/lock.1
77 ↗(On Diff #80525)

Oh, it is actually a "for" meaning, that in 5 minutes it will be unlocked even if the password was not typed in:

$ date && lock -t 1 -p
Thu Dec 10 22:03:41 CET 2020
lock: fernape using pts/13 on XXXX. timeout in 1 minute.
time now is Thu Dec 10 22:03:41 CET 2020
Password:lock: timeout
$ date
Thu Dec 10 22:04:47 CET 2020
usr.bin/lock/lock.1
77 ↗(On Diff #80525)

Ok, I misunderstood :-)

Since I don't want to repeat "and", what about I commit this one?

Lock the terminal for 5 minutes, disable switching virtual terminals and use the user's login password to unlock:
usr.bin/lock/lock.1
77 ↗(On Diff #80525)

May be "require" instead of "use"?

s/use/require as suggested by yuripv@

This revision now requires review to proceed.Dec 11 2020, 6:32 AM
fernape added inline comments.
usr.bin/lock/lock.1
77 ↗(On Diff #80525)

May be "require" instead of "use"?

Sounds better. Thanks!

This revision was not accepted when it landed; it landed in state Needs Review.Dec 11 2020, 7:28 PM
This revision was automatically updated to reflect the committed changes.
fernape marked an inline comment as done.

Thank you both for the review!