Do not recommend sleeping when using vipw(8) non-interactively

[0mp]

Summary

vipw.8: Do not recommend sleeping when using vipw(8) non-interactively

Before r231383, pw_edit(3) was unable to detect file changes reliably
without an explicit waiting via sleep(1) for example.  Since then,
however, pw_edit(3) has a nanosecond precision of detecting file
modifications, so calling "sleep 1" when using a non-interactive editor
with vipw(8) is no longer necessary.

Keep the note about the modification time, as it might still be useful
to users.

I am not exactly sure if this change is correct. A nanosecond precision is probably good enough for most users.

[pstef]

I would move the "The mechanism for checking for password file modifications requires that the modification time of the password file changes." part under a NOTES section and then remove the BUGS section.

[bcr]

Is this one still active?

[0mp]

In D25338#675153, @bcr wrote:

Is this one still active?

As long as I can get someone to review and accept the change :)

In D25338#559975, @pstef wrote:

I would move the "The mechanism for checking for password file modifications requires that the modification time of the password file changes." part under a NOTES section and then remove the BUGS section.

mdoc(7) does not have a NOTES section in its standard. It should live in the BUGS section.

Is the change good otherwise?

[pstef]

In D25338#770884, @0mp wrote:

In D25338#559975, @pstef wrote:

I would move the "The mechanism for checking for password file modifications requires that the modification time of the password file changes." part under a NOTES section and then remove the BUGS section.

mdoc(7) does not have a NOTES section in its standard. It should live in the BUGS section.

Is the change good otherwise?

I'm OK with the change for as long as it keeps the piece of information that I mentioned. I'd prefer under an IMPLEMENTATION NOTES or CAVEATS section, but if not there, let it be BUGS.

[pauamma_gundo.com]

In D25338#770884, @0mp wrote:

In D25338#675153, @bcr wrote:

! In D25338#559975, @pstef wrote:

I would move the "The mechanism for checking for password file modifications requires that the modification time of the password file changes." part under a NOTES section and then remove the BUGS section.

That should be "requires that ... change" (no "s" suffix). See https://en.wikipedia.org/wiki/English_subjunctive#Finiteness.

[0mp]

• bring back the note about time