Update with suggestions by Dru.

"Make it so" from manpages.

OK from manpages.

Two small suggestions, nothing major.

This change probably warrants an UPDATING entry and entries to obsolete files?

A few suggestions. Overall, I like the text and the addition to the handbook.

You need to bump the .Dd at the beginning of the man page when making the commit as this is a content change. The change itself looks good from the manpage side of things.

OK from manpages.

OK from manpages. Don't forget to bump the .Dd at the beginning of the man page to the date of the commit as this is a content change.

Can you run "mandoc -Tlint" and textproc/igor on your manpage change? It'll probably tell you a few things (no, not next week's lottery numbers).

OK from manpages. Thanks!

Approved by manpages, too.

OK from manpages, others may have comments on the content itself.

Did you ran the man page through "mandoc -Tlint" and textproc/igor to see if they spot any errors?
I found just one issue. Good to have a man page for it and even better to upstream it!

I'm OK with the doc side of things and I also find the exceptions here reasonable. I'll wait a bit before approval to give others time to comment.

Now it's good and I can approve it from manpages. ;-)

Two comments for the updated version.

Looks good (or better). Approved!

Found a small bug, nothing major. Did you give the man page the usual "mandoc -Tlint" and textproc/igor treatments?

Looks good, approved!

That's a good change to have, so hereby approved!

Looks good to me, approved by manpages.

Belatedly mark comments as done.

Removed the email= part.

ping? I think we can add it and see how it works out regarding the data collection we are trying to do.

Short and sweet and ready for commit from manpages. ;-)

Minor nit, the rest is fine.

Looks good, go ahead!

A few easy fixes for both man pages (I annotated only, but they apply to both).
Thanks for the writeup.

Can you bump the .Dd of ctm.1 for this content change? The rest of the man page looks fine, thanks!

OK from manpages.

Nice change. Can you bump the .Dd of the man page as this a content change. Thanks!

OK from manpages. Thanks for the work!

Approved!

Opps, did not want to select approve here, simply to add a comment.

How about we split this up into two sections: one with instructions for users on UFS and the other for those on ZFS? Each would have the individual instructions (like mounting) for the filesystem specific parts and a general section (before and after) that is applicable to both (non-filesystem specific).

Adding Sato-San and Warren to help shed a light on the issue about the entities (which is the proper file they should be added to?).

Hmm, OK. Their choice, so let's document it. ;-)

Update with suggestions from Brooks:

• use "the curl package"
• use \ to wrap long lines

Is this a mix up?

These are good additions to the man page!

Ah yes, this is a good low-hanging fruit. We have a lot of mentions specific versions in the handbook and I've often thought about generalizing them ("use the latest version from the Ports Collection") or writing some kind of tool that would report if there is a new version in ports available.

Looks good, thanks for the work!

Still OK from manpages.

Nice, thanks for these updates!

Approved for commit. Thanks for reminding.

Approved!

Excellent, ready for commit.

Well, in that case, you need to wrap the last word ("213" or even "room") to the next line, but the </site> tag needs to be there, no space.

@0mp corrected me that there is no .Dd needed. There was no change in the syntax in the man page itself, just whitespace fixes. In those cases, you can leave the .Dd as it is.

OK from manpages (don't forget to bump the .Dd, even for such a small content change).

Two things, nothing major. Thanks for the help in getting this documented.

Nice, thanks for working on this. Approved!

OK from manpages. Can you bump the .Dd to the date of the commit for this content change? Thanks.

BTW, have you run textproc/igor and mandoc -Tlint on your changes? Could provide some valuable help and suggestions for fixes.

Can you bump the .Dd at the beginning of share/man/man7/hier.7 for this content change? Thanks!

OK from manpages.

Hi Rodrigo,

You need to bump the .Dd at the beginning of the document to the date of the commit since this is a content change.

I think it's fine for a first entry. We can always make changes to the wording. Approved!

Veni, vidi, approvi (I came, I saw, I approved). ;-)

Cool, thanks for working on it.
OK from manpages.

Good to go! Thanks.

Great, then I shall approve it. ;-)

Can you test also with textproc/igor and "mandoc -Tlint" ? Thanks!

Fine for me.

Found a typo.

Good catch!

Looks good, thanks for doing the work!

Two fixes for the man page.

A suggestion on grouping related things together. I like the overall format for the example template.

Looks good to me (in case you need approval).