Thanks!

Odd. Thanks.

OK; I'm going to abandon this review then and just hook up the page as is.
Thanks both.

In D30998#697446, @fernape wrote:

In D30998#697422, @ceri wrote:

In D30998#697421, @manu wrote:

Just realized that we install most OF_* manpages on all arches.
And always installing nvmem.9 would help people developing on amd64 for embedded (most people do that) so I think that this should always be installed.
Sorry for not noticing before.

That's ok; the work that @fernape is doing on installing all manpages anyway should take care of that and will be the default, so the logic here would then just allow people who didn't want the page to leave it out.

Since MAN_ARCH=all by default starting at 0a0f7486413c147d56808b38055c40c64cff61f5 maybe we could add || empty(${MAN_ARCH}) || ${MAN_ARCH} == "all" to that condition. That is what I plan to do with the remaining pages not covered by the initial commit.

In D30998#697421, @manu wrote:

Just realized that we install most OF_* manpages on all arches.
And always installing nvmem.9 would help people developing on amd64 for embedded (most people do that) so I think that this should always be installed.
Sorry for not noticing before.

Could you add egrep to the .Xr section please?

Can we try https://reviews.freebsd.org/F24494081 - this also removes text that clang is imparting to people anyway.

In D30807#695022, @kevans wrote:

In D30807#695258, @grembo wrote:

In D30807#695219, @kevans wrote:

So basically "blacklisted" was not the correct term to begin with.

This is my opinion, yes. I'm still searching for more ideal verbiage

If I would just start implementing this (without any external limitations I'm not aware of), I would probably do something like creating one directory called all and one called trusted with entries in trusted being symbolic links to certificates in all and check those into git.

caroot
|-- all
|   `-- GTS_Root_R1.pem
`-- trusted
    `-- GTS_Root_R1.pem -> ../all/GTS_Root_R1.pem

There are probably reasons why this isn't feasible though.

Fix build with dbaio's clean tree; address comments.

Keen to see some version of this committed, as the current instructions left my laptop unbootable.
Am testing the rest of this.

In D30867#694382, @dbaio wrote:

just to be clear, DDG still supports multiple sites.

This doens't work anymore:

site:docs.FreeBSD.org/mail,lists.FreeBSD.org/pipermail

This works:

(site:www.FreeBSD.org OR site:docs.FreeBSD.org OR site:lists.FreeBSD.org OR site:wiki.FreeBSD.org OR site:forums.FreeBSD.org)

go to https://www.freebsd.org or https://www.freebsd.org/search/ (first form) and search for pgp, this is working there.

Remove style(9) stuff that crept in.

Thanks!

I committed the reverse of this in c603905d8303ee561fa8c7f10b2c6b170d017ce2 as it was more consistent with the other files.

Thanks!

In D30718#691132, @dbaio wrote:

In D30718#691109, @blackend wrote:

In D30718#690992, @dbaio wrote:

This is nice to test/build a single document, only issue is that it won't use our template/css.
Just to know, these html's will be used where?

In D30732#690947, @dbaio wrote:

In D28062#690713, @debdrup wrote:

I don't really have time for this right now, so someone else can go ahead and commit it once it's been accepted.

Adding blackend as we had discussed the DOC_LANG separators some weeks ago.

missed gnu_getopts in two files.

Looks good - have also tested with some (fake) conflicting files and the right thing happens.

Change looks good, I think, but is there any reason not to renumber those "footnotes" at the same time so that they are vaguely in order and 2 and 7 aren't missing for no good reason?

gtg

In D30388#689940, @grembo wrote:

That sysctl is 2 by default, at least on -CURRENT.
Do we care that this is wrong on CURRENT (I don't think I do) ?

ok - a comment or two regarding how to use these can follow up. Thanks

In D30656#689103, @dbaio wrote:

In D30656#689079, @ceri wrote:

I don't understand why BIND and HOSTNAME aren't the same variable?

I imagine people want to bind it to a specific ip/interface and access through a hostname which is different than the ip address.

-b, --baseURL=""    hostname (and path) to the root, e.g.
http://spf13.com/


--bind="127.0.0.1"  interface to which the server will bind

adding @mat, could you please review this as well?

I don't understand why BIND and HOSTNAME aren't the same variable?

New rev works too

Good idea. Looks good.