Page MenuHomeFreeBSD

lock(1): Add EXAMPLES section
ClosedPublic

Authored by fernape on Dec 10 2020, 3:58 PM.
Tags
None
Referenced Files
F163113834: D27541.id80593.diff
Mon, Jul 20, 4:43 AM
F163079174: D27541.id80575.diff
Sun, Jul 19, 9:48 PM
F163062149: D27541.id.diff
Sun, Jul 19, 6:04 PM
F163061380: D27541.id80525.diff
Sun, Jul 19, 5:56 PM
Unknown Object (File)
Sun, Jul 19, 12:26 PM
Unknown Object (File)
Sun, Jul 19, 6:37 AM
Unknown Object (File)
Sat, Jul 18, 12:17 PM
Unknown Object (File)
Sun, Jul 12, 8:48 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
Lint Not Applicable
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!