Great! Then I will close this revision.

Ping 2.

Ping.

Ping.

In D55632#1307092, @jilles wrote:

Is signal(3) the right reference? C code had better use sigaction(2) instead, or the list might be moved to a new signal(7) page?

In D56648#1303087, @ziaee wrote:

In D56648#1303053, @tembun_bk.ru wrote:

Can we have it in 15.1? I guess it would be reasonable, since we already have groups(7) there. They look better together.

I actually did not merge groups.7 into 15.1. This was my mistake. Let me see if I can address that.

Can we have it in 15.1? I guess it would be reasonable, since we already have groups(7) there. They look better together.

Address @kib's comments:

Sorry to bump this, but I found that this change wasn't MFC'ed as it planned to be. I think we can have this in -STABLE as well.

Ping.

Reference users.7 in intro.7.

In D56661#1297878, @jilles wrote:

The #! mechanism is already described (more briefly) in lib/libsys/execve.2 lines 73...100. That duplication is certainly undesirable.

I don't really have an opinion whether the text in lib/libsys/execve.2 needs to be extended or moved to a new page share/man/man7/script.7.

Forgot to include users.7 in MAN variable in Makefile.

Add chgrp(1), id(1) and newgrp(1) in groups(7) SEE ALSO section.

Simplify sentence in HISTORY section.

In D56512#1294510, @mhorne wrote:

Based on the original submission it is not a typo (D48877). It took one year for the merge to happen. I suggest just to leave it as is.

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

Use recommended wording for non-standard options.

In D56090#1289449, @ziaee wrote:

Sorry to backtrack here (and to let this slip for a while), but I did a deeper dive in preparation for merging this. Looking at our utilities that are specified in POSIX.1-2024:

xargs:
options are non-standard FreeBSD extensions which may not be available on other operating systems.

ls:
are non-standard extensions.

what:
The -q option is a non-standard FreeBSD extension which may not be available on other operating systems.

cut:
The -w flag is an extension to the specification.

pax:
operations are extensions to the POSIX standard.

date:
With the exception of the -u option, all options are extensions to the standard.

patch:
The flags [-BCEFfstVvxz] and [--posix] are extensions to that specification.

env:
options are non-standard extensions supported by FreeBSD, but which may not be available on other operating systems.

find:
are extensions to IEEE Std 1003.1-2001 (“POSIX.1”).

Looking at GNU coreutils (which doesn't provide STANDARDS or COMPATIBILITY sections), they also have both long-options and -v.

The extended documentation for the mdoc language suggests we use are extensions to that specification..

https://mandoc.bsd.lv/mdoc/style/standards.html

Take mhorne's suggestion.

Move non-POSIX options block in the STANDARDS section.

In D56090#1282965, @ziaee wrote:

The COMPATIBILITY section is a GNU-ism.

Correct grammar.

Fix a typo: 'signle' -> 'single'.

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

Is it ready to be committed?

I suppose it can be landed now?

Address @jilles comments.

Thank you for the review, @jilles!

• Reword sentence about non-POSIX extensions (chose second variant suggested by ziaee).
• Change missed occurrences of 'Non-standard' -> 'Non-POSIX'.

Ping 2.

Change 'nonstandard' -> 'non-POSIX'.

Rebase the patch.

Grammar improvements:

• Prefer American English: 'non-standard' -> 'nonstandard', 'behaviour' -> 'behavior'.
• 'a nonstandard local command' -> 'the nonstandard local command'.

Shouldn't .Dd be bumped in this case?

In D55169#1262390, @kib wrote:

In D55169#1262376, @tembun_bk.ru wrote:

Thank you! Do you think that kld(4) would be an appropriate place for this description? I guess I can incorporate it there.

Current kld(4) page needs a complete rewrite. Starting from the removal of LKM references, which were not relevant since 4.x.
The renovation for kld(4) is technically complicated work, requiring a lot of code reading to provide good manual.

In D55169#1262234, @kib wrote:

In D55169#1262072, @tembun_bk.ru wrote:

In D55169#1262058, @emaste wrote:

How about just load kernel module?

That's fine with me. However @kib already suggested to provide more information on what the 'kld file' is: https://reviews.freebsd.org/D55170#1261544. I thought maybe we can put this information in kld(4) for instance and use it as reference in kld*(2) man pages?

No, technically we load 'kld file', which is linked into the executing kernel. The file might contain some number of modules (from 0 to arbitrary), which are used as units of initialization.

Gentle ping.

In D55169#1262058, @emaste wrote:

How about just load kernel module?

Correct wording.

• Remove redundant 'of the function' part.
• Change "it's" -> "it is".

@ziaee Since https://reviews.freebsd.org/D49895 has been MFC'ed, I believe this change now has to be MFC'ed too.

Don't touch kldfind(2), leave changes only for kldload(2).

Add MLINKS: wheel.7 and operator.7.

Ping. Any other comments? :)

Rebase the patch.

Rebase the patch.

Instead of trying to mark everything under /usr/lib/debug/boot obsolete, prevent /usr/lib/debug/boot itself from being marked obsolete.

In D55077#1259991, @markj wrote:

In D55077#1259978, @tembun_bk.ru wrote:

So should /usr/lib/debug/boot directory itself be included in DEBUG_DIRS?

No, I think it shouldn't. So I guess the proper thing would be to exclude \! -d /usr/lib/debug/boot/* \! -d /usr/lib/debug/boot. Maybe there's a cleaner way to write that.

With respect to the change itself, I do not see why make delete-old should touch anything under /usr/lib/debug/boot. I believe that the intent of the current code is to ignore anything under /usr/lib/debug/boot/*. What exactly is the problem being solved?

In D55077#1258804, @tembun_bk.ru wrote:

In D55077#1258798, @emaste wrote:

Kernel debug files aren't controlled by WITH_/WITHOUT_DEBUG_FILES so should not be removed here

Oh, I see, these seem to be controlled by WITH[OUT]_SPLIT_KERNEL_DEBUG. But where we should put the code that will mark files in /usr/lib/debug/boot/{kernel,modules} 'obsolete'? I guess it must be somewhere in sys/, but I'm not sure where exactly.

• 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.

In D54758#1259273, @kevans wrote:

I was looking over this again- I think its' technically incomplete because we're missing its complement over in /usr/share for the system-distributed files: http://people.freebsd.org/~kevans/zfscompat.diff -- I haven't had a chance (machine) to test it, but it'd probably be good to land these together if you can confirm.

In D55091#1259161, @emaste wrote:

I don't think there's any expectation that the rc scripts will be portable, but when there's no material difference (e.g. in performance, clarity, maintainability, etc.) I agree we should indeed prefer the portable way.

Rebase the patch.

In D55077#1258798, @emaste wrote:

Kernel debug files aren't controlled by WITH_/WITHOUT_DEBUG_FILES so should not be removed here

Ping.