In D32130#1139636, @oaccounts_chingarande.com wrote:

I am taking over this issue and will be responsible for this. let me know how I can assign my self this ticket

• remove ping6.8.gz from ObsoleteFiles.inc

In D42203#966915, @lwhsu wrote:

We also need to remove OLD_FILES+=usr/share/man/man8/ping6.8.gz from ObsoleteFiles.inc.

In D42203#966915, @lwhsu wrote:

We also need to remove OLD_FILES+=usr/share/man/man8/ping6.8.gz from ObsoleteFiles.inc.

Oh and we also have to close it, because it did land! ugh Phabricator is very Project-Management-y :)

In D40188#920799, @jamie wrote:

Commited in eb5bfdd06565. I forgot to add the review to the commit message :-/

In D38826#920142, @ihor_antonovs.family wrote:

IMHO this is superseded by https://reviews.freebsd.org/D40188

In D42203#962979, @bcr wrote:

Why do you need a docs review for this?

This has landed in 5ac2a874d070, closing it.

• Remove the memo and add jail_conf_dir

make sure that zfskeys, ip6addrctl, jail and ipfw_netflow return non-zero when not running

In D37383#849570, @lwhsu wrote:

Don't forget to modify usr.sbin/service/service.8 :)

In D32130#729639, @fernape wrote:

In D32130#729265, @antranigv_freebsd.am wrote:

Hey @0mp , what I did not understand is that if you already changed the code in the Diff or should I resend the patch with your changes using arc? Sorry, first time seeing this :) Let me know!

arc patch D32130 shows that it is not changed ;-)
You need to resend it.

Hey @0mp , what I did not understand is that if you already changed the code in the Diff or should I resend the patch with your changes using arc? Sorry, first time seeing this :) Let me know!

Update the date

• Update date

• Add help menu

• Remove the whitespace

• update manpage for gstat.8

In D24570#721083, @dvl wrote:

Is this at the stage where I could start using it on a test system to provide feedback?

Update the man page, also fix the paths

Change from jail.d to jail.conf.d

Update rc.conf.5

@kp should we merge this or it's stale? :) Just reminding.

Hey @jamie thanks for the feedback.

While I understand @pi 's point, I think jail.conf.d is better. I'm also planning to submit my base-based written in shell jail orchestration tool, using jail.d would be better for integrations (e.g. hooks, scripts, etc).

Fixed styling issue at line 3848

Update the man page according to textproc/igor

Update the date in the man page.

Fixed an unintended slash in rc script

Hmm, I think it looks better this way?

Correct the documentation

Use .Pa macro for file system path.

Updated man pages as well.

More description regarding fscale

addressing jhb's comments

Following style(9) guidelines

Note in the comments that only ACPI reported batteries are supported

1. I know someone in the city who has a Pinebook, should I try running on it? 2) So now I should just set a comment that says this is ACPI only? 😄

Update display.c to follow style guidelines.

Trying to comply to style(9)