don't rc_post if rc_stop failed.
"I agree with the direction" sthen@
ok robert@

in libkvm with the stable-ABI versions that are currently named
KERN_PROC2, kvm_get{proc,argv,envv}2().  The latter names and symbols
will continue to be supported for a few releases.
Committing now that they ports people have had a couple releases
to update pkgs that usd the old functions

Use $() constructs as it's safer than ``.
Both req. by deraadt@
Use pkill instead of pgrep in rc_check so that we don't need to fiddle
with redirections in reload.
prodded by schwarze@
ok robert@

ok robert@

rc_stop so scripts don't need to set it.
ok sthen@ (on a much bigger diff) robert@

rc_cmd start/stop actions. This way when rc.d(8) scripts override these
functions, we don't loose rc_{pre,post}.
Add a max 5 secs loop after rc_stop in the rc_cmd top action. This seems
to be a good default for returning to command line only after the daemon
has really stopped. This fixes "restart" for some daemons and allows to
properly stop some others at shutdown time.
Note that this is just a best-effort default, some daemons may need a
lot more time to shutdown but this case is usually handled in the
rc.d(8) script itself and we obviously do not want to hang the shutdown
process.
Call rc_cmd start/stop in restart and _not_ rc_start/rc_stop which can
get overriden in a script.
discussed with and inputs from sthen@ and schwarze@
ok sthen@ robert@

a daemon in the background.
man page tweaks from schwarze@
discussed with an ok schwarze@ robert@

ok guenther

argv and don't suppress the handling of leading '-' in optstring when
POSIXLY_CORRECT is set.
Based on patch from Eric Blake.
ok and manpage update from millert@, manpage ok jmc@

invokes handlers registered with __cxa_atexit().
"seems right" deraadt@

ok robert@ espie@ miod@

by default, adjust comments in sysctl.conf;  ok markus, tedu, djm, miod

prodded by espie@
ok miod@

in a better place to make it more readable.
ok jmc@ and miod@

the \: roff escape is an optional line break.

non-routeable prefixes. While there sort list.
Diff provided by Andre Keller.

time.  We've been summing the one from the previous build (then additional
goo I run has been cleaning it up..)

we run netstart
ok tedu, djm liked it too

in particular a single write will result in a single re-key event,
rather than 64 writes causing 64 re-keys -- wasting the kernel's time.
ok guenther

easily use it in rc scripts instead of copying/pasting the same huge
command lines everywhere.
ok robert@ sthen@

let's use it. If not, then fall back to using daemon.
While here, do some reordering.
"I like this" sthen@, ok robert@

ok jmc@ jsing@

with POSIX 2008
ok millert@

As schwarze@ also stated, this will prevent anyone from calling it
(which would be an error) and makes it clearer that rc.subr is not just
a functions container.
No functionnal change, unless you were calling rc_conf from an rc script
which would have been wrong anyway.
ok schwarze@ robert@

script; instead, rc.subr can call rc_conf itself right after reading in
rc.conf.
from schwarze@ (committing on his behalf)
ok robert@

local_rcconf would get overwritten by flags from the rc script itself.
Based on an original idea and diff from robert@, create an rc_conf
function that we add in the rc_script after the defaults daemon*
variables.
This way we can use defaults variables names in other part of the script
(when defining a specific pexp for instance).
While here, simplify setting up the default daemon variables so that we
don't need to do any substitution in rc_start.
rc scripts must include daemon variables before calling rc_conf. All
other locally modified variables (pexp, rc_reload, local additions...)
must come after.
feedback from and works for sthen@
ok robert@

ok kjell otto miod

OK jsing@

ok deraadt@ djm@

path and realpath are the same.  This matches the manual and avoids
a bogus cast to non-const.  OK jsing@, previous version OK kettenis@

ok deraadt

tested for a while by me.
ok otto@

ok sthen@ deraadt@

constant expressions with the help of gcc post-3.3.
ok millert@, mikeb@.  been in snaps for weeks.

Also noticed and agreed by eric@
ok robert@

the last su(1) commit

corresponding variable to NO.
e.g.
rc_reload=NO
This will save us the trouble of copying the same rc_reload() function
(or others) all over the tree.
ok robert@