Thank you for the quick edits. Approved by manpages for that part.

Can you run textproc/igor and "mandoc -Tlint" over the man page and see what warnings/errors they give you? Thanks!

OK from manpages.

Comments addressed.

Update with Kristof's comments.

OK from manpages.

Some comments. I'm more inclined on putting it into section 8 of the man pages. Going from the description of sections in man(1), it's more geared towards sysadmins rather than being a general FreeBSD command to run.
Have you run textproc/igor over it and "mandoc -Tlint" for any errors/warnings it might find?

Approved. Don't forget to bump the .Dd to today. Thank you!

A small typo fix, the rest looks good.

I will polish it a bit more and commit it.

A bit more description about the shared folders would be useful here. For example, describe that users need to create the folder (shared_folder_name) in their VM first and where the mount_vboxfs command should be run (host or VM). Thanks!

Looks good to me.

Looks good to me.

Not a bug, closed.

This typo has been fixed in HEAD r342579 already. Thanks for letting us know anyway.

OK for the man page.
Regarding @0mp's comment about the reference from openssl(1): we can do that later after this change has been committed. It's not critical to do that with this change.

Who's gonna commit this?

Looks good to me, thanks for doing the work!

Looks good to me. Thanks, @jhb, for taking over the mentorship!

Can you run textproc/igor and "mandoc -Tlint" over your man page change? I found one issue, maybe those tools have more. Thanks!

Looks good, approved!

Thanks for the quick writeup. I've only a few minor corrections.

Provide a link to the (small) word list in the FDP.

A few minor suggestions to be more precise.

OK from manpages. Thanks for doing the work.

Good catch, approved!

Approved for commit

OK, sounds good. We can try that in the next quarter.

I've reviewed the report and found a few minor issues.
Overall, it takes a lot of time to read the whole piece. I could have found some of the bugs earlier when they were still individual reports that are easier to go over.
Other than that, thanks for your efforts putting it together!

Definitely worth having those examples in that man page. Thanks for the work!

Good work, approved!

Update pf.conf.5 with suggestions from @0mp.

Good to go!

Small nit, the rest is fine. Thanks for adding the example, I think it is really useful to have.

Looks good, approved!

Looks good to me. Thanks!

Clarify some more.

This newer version is better as it gives an alternative for people who need that Monday-starts-the-week functionality. Approved!

Some countries start their week on Sunday, so I guess cal originated there and does not take other countries into account.
That being said, the change itself looks fine. Bumping the .Dd for this content change not withstanding. ;-)

Good catch, approved!

Go ahead...

OK from my point of view.

Hi @sobomax,
can you run textproc/igor and "mandoc -Tlint" over your man page change? It will give you a few suggestions (not many). Thanks!

Superfluous word.

I agree this is a historical leftover and does not apply anymore.

Looks good!

Sorry for the wait. Looks good to me. I can get it committed.

Nice, thanks for those changes. I can commit them later today.

OK from manpages. Can't say much about the content itself, but syntax looks good.

Looks good, OK to commit.

Just the normal <a href=...>blah</a> magic. See events2015.xml for an example.

Looks good from manpages. I guess bumping the .Dd to the date of the commit does not hurt.
I look forward to that feature. Thanks!

Link suggestion.

Just a small typo.

OK as well. Thanks for doing the work. Let me know if I should commit it (giving proper attribution, of course).