I'm not able to test this on 11-STABLE, but this looks good to me as an unused environment variable should be harmless.

I won't update the patch here given it's already submitted but I'm submitting a fix to HEAD now.

fstat.1:

• fix minor grammer issue
• bump Dd

Updated diff again to remove whitespace diff.

Diff updated.

Update diff after jilles@ review.

Remove an unintended whitespace diff.

In D13040#275505, @mat wrote:

The version of the software does not change, you should not do the PORTVERSION -> DISTVERSION change here, but wait for an update that does change the version.

s/DISTVERSION/PORTVERSION/

s/DISTVERSION/PORTVERSION/

Added latest poudriere log.

Fix a mismerge.

Attached poudriere log with fixed mismerge.

Sync ports tree. Fix a mismerge.

Attached poudriere log file.

Sync ports tree.

Attached poudriere log.

Fix PORTREVISION (should be 5, not 6).

Remove a comment which is not valid any more.

Add missing tabs.

Change post-install targets to do-install.

I submitted the changes in https://svnweb.freebsd.org/base?view=revision&revision=325049

Update the date of the manpages, as requested by bcr@.

Resolve comment

Update diff with mat@'s second review.

Update diff after mat@'s review.

So has a consensus been reached finally? I can go either way, although dropping the revision and leaving things as-is is obviously the easiest way ;-).

Sorry I'm still learning how the whole thing works :-).

cy@, I checked pam_rhosts(8) and it's not obvious to me that it's strictly related to rcmds, although I can totally understand they were used alongside.

Update rc.d script.

Fix a "Subsonic" leftover in the description.

I've updated etc/mtree/BSD.tests.dists but I checked on various boxes I own, there's nothing related to rcp in /usr/tests/.

Removed rcp from etc/mtree/BSD.tests.dist.

FYI this was submitted as https://svnweb.freebsd.org/base?view=revision&revision=322277

In D11743#243391, @emaste wrote:

For reference it looks like these were considered r-cmds as of rS183242. I wonder if we should keep a knob to turn them on/off.

I think stack protection has already been disabled in the very low level stuff. This change is fairly non-intrusive and I think he ready for further testing. Go ahead and commit please.

Update license comment as in D11444.

Any update?

Improve the comment about the license.

Thanks Allan!

Set back LICENSE=NONE as no license doesn't look right.

Remove license altogether.

Ah I missed the LGTM. Sorry the UI is confusing.

Ping :-)