groups.7: Standard group names

[tembun_bk.ru]

Summary

I'm not 100% sure whether every statement and group description is true for FreeBSD and its current state. Please, point out mistakes in my assumptions.

The group list is sorted by GID. All the group names from /usr/src/etc/group are described, except "uucp": honestly, I don't know what it is used for and what to say about it.

PR: 264966
Obtained from: NetBSD

[tembun_bk.ru]

share/man/man7/groups.7

4–5

What should I do with the license? The page was not only imported, but changed for FreeBSD. I also did change some of the original parts.

208

I'm not sure if I'm right about this one.

247

I'm not quite sure about this one too. If I understand correctly, it must be somehow related to group "uucp" (which is missed), since on my machine /dev/cuau2 is owned by used "uucp" and in group "dialer".

253

I'm not sure if it's really unused now. Didn't find much about it in /usr/src.

265

As above.

269

Didn't have any experience with it, just tried to deduce what it is used for :)

301

That's only my guess described here. Maybe there is a different reason why there are "nogroup" and "nobody".

[tembun_bk.ru]

'December' instead of 'Dec'.

[tembun_bk.ru]

Add groups(7) cross-references into other man pages.

[tembun_bk.ru]

I'm also going to port users(7) man page. The structure and the overall idea of these two pages are very similar; do you have some notes/advises that I can learn from this page and apply to users(7)?

[tembun_bk.ru]

@ziaee I've been thinking: maybe it would be a nice idea to create hard links for this manual page: wheel.7 and operator.7? wheel and operator are the most noticeable groups and people frequently have questions about their purpose, so it would be nice to be able to do man wheel or man operator. We have the similar thing for builtin.1 which has a link to every builtin command: https://cgit.freebsd.org/src/tree/share/man/man1/Makefile?id=4b0d5d1d6a7c9773e38882feb4747a76b37a645c#n13

[ziaee]

Sorry I haven't had time to review this yet!

I don't know about adding mlinks. We can't really do it consistently without creating a lot of collision in the apropos database, which is hard for beginners to navigate.

share/man/man7/groups.7

4–5

The license always stays, that's what the first clause therein says (lines 9 and 10), but add a SPDX tag at the top so we can index it mechanically.

[tembun_bk.ru]

share/man/man7/groups.7

4–5

That should be 'SPDX-License-Identifier: BSD-2-Clause', right?

[imp]

share/man/man7/groups.7

4–5

Yes. I believe this is the standard BSD-2-Clause, or close enough, that this is the right thing.

[tembun_bk.ru]

Add SPDX-License-Identifier.

[tembun_bk.ru]

In D54114#1244827, @ziaee wrote:

Sorry I haven't had time to review this yet!

I don't know about adding mlinks. We can't really do it consistently without creating a lot of collision in the apropos database, which is hard for beginners to navigate.

@imp (sorry, I'm not sure if you can answer this, but since you've been active on this revision) do you think that adding mlinks would be worth it or it is better to avoid such hard links?

[tembun_bk.ru]

Update diff with -U9999.

[tembun_bk.ru]

Ping.

[tembun_bk.ru]

• Document that games group is used by hgame(4) (since b6cad334e64).
• Correct several mandoc -T lint warnings.
• Document that bind(8) was removed from the base in FreeBSD 10.0.
• Replace reference to unbound(8) with local-unbound(8).
• Remove references to setuid(7), since NetBSD has this page, but FreeBSD does not.

[tembun_bk.ru]

Add MLINKS: wheel.7 and operator.7.

[tembun_bk.ru]

Could somebody please review this? Or maybe tag appropriate people?

[tembun_bk.ru]

share/man/man7/groups.7

41–43

May be that would be relevant to mention that this behaviour can be changed with pam_group(8)? From su(1):

PAM is used to set the policy su will use.  In particular, by default only users in the “wheel” group can switch to UID 0 (“root”).  This group requirement may be changed by modifying the “pam_group” section of /etc/pam.d/su.  See pam_group(8) for details on how to modify this setting.

[tembun_bk.ru]

Rebase the patch.
Don't add operator.7 in MLINKS, since this man page already exists.