MFC r346571,r346572:

Update the spelling of my name

Previous spellings of my name (NGie, Ngie) weren't my legal spelling. Use Enji
instead for clarity.

While here, remove "All Rights Reserved" from copyrights I "own".

MFC SVN r336350: Send sysrc(8) error message to stderr (not stdout)

PR: bin/229806
Reported by: Andreas Sommer <andreas.sommer87@googlemail.com>
Sponsored by: Smule, Inc.

MFC r334303: sysrc(8): Test variable names for invalid characters

PR: bin/187461
Reported by: ebay@looksharp.net
Sponsored by: Smule, Inc.

MFC r335277:

sysrc(8): Exit with failure on API error

Fix exit status when f_sysrc_set() fails. Errors in the underlying API
provided by bsdconfig(8) -- /usr/share/bsdconfig/sysrc.subr -- were not
being communicated back to the command-line. This was affecting ansible
modules using sysrc as they were not able to accurately test for error.

PR: bin/211448
Reported by: Christian Schwarz <me@cschwarz.com>
Sponsored by: Smule, Inc.

Copy head@r302406 to stable/11 as part of the 11.0-RELEASE cycle.
Prune svn:mergeinfo from the new branch, as nothing has been merged
here.

Additional commits post-branch will follow.

Approved by: re (implicit)
Sponsored by: The FreeBSD Foundation

Change my given name from "Garrett" to "Ngie"

A legal name change from "Garrett" to "Ngie", as well as a FreeBSD
account name change, is pending.

Approved by: re (hrs)

jls(1) -> jls(8)

MFC after: 1 month
Sponsored by: The FreeBSD Foundation

Revert r295169 and switch `\ ' to `" "'

MFC after: 3 days
X-MFC-to: stable/10
X-MFC-with: r295169, r295170, r295173, r295177

Fix a typo in a comment

Bump version to 7.1 for +=/-= fix

MFC after: 3 days
X-MFC-to: stable/10
X-MFC-with: r295169, r295170

For +=/-=, add . and / to convenience characters

sysrc(8) supports key+=value and key-=value, but can be told what the
delimiter is by being passed as char1 (e.g., "sysrc key+=",value" to use a
comma as the delimiter instead of space). For convenience, if the first char
is alpha-numeric, it is assumed you wanted whitespace as the delimiter.

However, if you naively (as I just did) execute:
sysrc rc_conf_files+=/etc/rc.conf.other
the result is unexpected.

This commit makes `.' and `/' in-addition to alpha-numeric first-characters
to cause the default of whitespace to be used as the delimiter. This also
means that you can no longer use these as a delimiter.

Replace (Qo \ Qc) with (Qo (space) Qc)

When using col(1) piped to vim(1) as pager for man(1), the former sequence
of (Qo \ Qc) renders as "" without the space. Replace with (Qo (space) Qc)
which renders properly in more (all?) pagers.

META MODE: Update dependencies with 'the-lot' and add missing directories.

This is not properly respecting WITHOUT or ARCH dependencies in target/.
Doing so requires a massive effort to rework targets/ to do so. A
better approach will be to either include the SUBDIR Makefiles directly
and map to DIRDEPS or just dynamically lookup the SUBDIR. These lose
the benefit of having a userland/lib, userland/libexec, etc, though and
results in a massive package. The current implementation of targets/ is
very unmaintainable.

Currently rescue/rescue and sys/modules are still not connected.

Sponsored by: EMC / Isilon Storage Division

Add sysrc(8) support for "rc.conf.d" file(s) when given "-s name" to
indicate service(8) script. While here, add "-l" option for listing the
set of configuration file(s) considered (in order; separated by space).
Also add "-L" for exploring all configuration files and "-E" to omit
files that don't exist from operations.

Differential Revision: https://reviews.freebsd.org/D3551
Reviewed by: allanjude
MFC after: 1 week
X-MFC-to: stable/10
Relnotes: yes

Remove redundant beforeinstall.

Fix mandoc(1) "WARNING: end of line whitespace"

MFC after: 3 days
X-MFC-to: stable/10

Remove non-functional examples.

MFC after: 3 days
X-MFC-to: stable/10

Minor code cleanups (no functional changes).

MFC after: 3 days
X-MFC-to: stable/10

Bump version for altered long-opts processing

MFC after: 3 days
X-MFC-to: stable/10

Simplify long-option processing

MFC after: 3 days
X-MFC-to: stable/10

Bump version for prior fix (SVN r287381)

MFC after: 3 days
X-MFC-to: stable/10

Style: Remove whitespace around brackets from function syntax options

MFC after: 3 days
X-MFC-to: stable/10

Comment for escape() function.

MFC after: 3 days
X-MFC-to: stable/10

Comment

MFC after: 3 days
X-MFC-to: stable/10

Properly escape arguments when moving into jail or chroot

MFC after: 3 days
X-MFC-to: stable/10

Style: comments

MFC after: 3 days
X-MFC-to: stable/10

Style consistency: add single space before each `;;' case entry

MFC after: 3 days
X-MFC-to: stable/10

Remove `SYSRC_' prefix from $SYSRC_VERBOSE (prefix unnecessary since
this is a non-inheritable attribute; was previously).

MFC after: 3 days
X-MFC-to: stable/10

Fix a few mandoc warnings.

MFC after: 1 week

Update usage statement to align with post-r279624 functionality.

MFC after: 3 days
X-MFC-to: stable/10 stable/9

sysrc(8): Add `key-=remove' and improve `key+=append' syntax

MFC after: 3 days
X-MFC-to: stable/10 stable/9

Add EXAMPLES-section entry for new key+=append syntax of sysrc(8).

MFC after: 3 days
X-MFC-to: stable/10 stable/9
X-MFC-with: 274068
Thanks to: @SeanChittenden

Add key+=append syntax to sysrc(8).

Reviewed by: shurd
MFC after: 3 days
X-MFC-to: stable/10 stable/9

Oops, forgot to bump manual date/copyright in r268860.

MFC after: 3 days
X-MFC-to: stable/10, stable/9, ports
X-MFC-with: r268860

Minor enhancements, bug fixes, and man-page adjustments.

+ Make `sysrc -x foo' produce error status if foo is unset.
NB: Reported by lme (everything else ``while I'm at it'')
+ Remove mention of SYSRC_VERBOSE from `sysrc --help' output.
NB: False documentation leftover from an ancient precursor.
+ Make `sysrc -qc foo' work the same as `sysrc -ic foo' when foo is unset
NB: For syntax convenience (my fingers know `-q' more than `-i').
+ Update description of `-c' flag in help message/manual.
+ Update description of `-q' flag in help message/manual.
+ Make `sysrc -vc foo' work as documented in the manual
NB: Show message stating whether foo is currently set.
+ Make `sysrc -vc foo=1' work as documented in the manual
NB: Show message stating how value of foo would be changed (if at all).
+ Remove odd usage of parentheses in `-R dir' section of manual.
+ Clarify syntax section of manual w/respect to sysctl(8) similarities.
+ Add new/missing people to the `THANKS TO' section of the manual.

Reported by: lme
MFC after: 3 days
X-MFC-to: stable/10, stable/9, ports

use .Mt to mark up email addresses consistently (part2)

PR: 191174
Submitted by: Franco Fichtner <franco@lastsummer.de>

Revert SVN r257830 -- that feature was annoying and was removed long ago.
Was a momentary memory lapse induced by old code that was lying in my ~/bin

Document SYSRC_VERBOSE enviroment variable in the manual.

Fix a bug with `-e' that broke sometime around when I added `-F'. Also,
revamp the `-e' output to truly be sh(1) compatible syntax (even if
passed in `-v' for verbose output).

Comments and whitespace.

Add a `-c' option for `check only' to determine if a value is set
(get command) or if the current value is the same as what is desired
(set command). In those cases, the return status is success.
Otherwise, if the variable is unset (get command) or set to a different
value that what is desired (set command), returns with error status.
This is useful for puppet integration as well as everyday scripting.

Add new `--version' long option for getting the version (good for
comparing against the ports revision).

Fix a bug that prevented `-d' from working as-documented in sysrc(8).
Rather, prevent a bunch of debugging information from spewing onto the
screen when using the `describe' flag (misinterpreted as `debug' by
the automatatic initialization routines).

Fix typos in the BSD License.

Update release info.

Update copyright/date on the back of r252798.

Update sysrc(8) manual to coincide with r252797:

Do not inherit $SYSRC_VERBOSE from operating environment. The concern is
that when a user (such as myself) which has SYSRC_VERBOSE=1 in his/her
~/.bash_profile or such that when they are told to execute a command like:

hostname `sysrc -n hostname`

NOTE: To activate a recently configured hostname.

If $SYSRC_VERBOSE is set, then POLA is violated because the output of sysrc
is indirectly influenced (making for an inconsistent experience).

Do not inherit $SYSRC_VERBOSE from operating environment. The concern is
that when a user (such as myself) which has SYSRC_VERBOSE=1 in his/her
~/.bash_profile or such that when they are told to execute a command like:

hostname `sysrc -n hostname`

NOTE: To activate a recently configured hostname.

If $SYSRC_VERBOSE is set, then POLA is violated because the output of sysrc
is indirectly influenced (making for an inconsistent experience).

Remove NO_OBJ from Makefiles that generate manuals because this causes the
GZIP compressed manuals to appear in ./src instead of the appropriate obj dir.

PR: conf/175844
Submitted by: Dominique Goncalves <dominique.goncalves@gmail.com>

Mirror the changes made in SVN r240798:

Replace "( : ${var?} )" syntax with better "[ ${var+set} ]" syntax.

Reviewed by: adrian (co-mentor)
Approved by: adrian (co-mentor)

mdoc: begin sentences on a new line.

Minor mdoc fix.

Remove trailing whitespace.

Fix HISTORY.

Sort manual page cross references.

Approved by: joel (mentor)

Remove whitespace at EOL

Approved by: joel (mentor)

Import sysutils/sysrc from the ports tree (current version 5.1). Importing
disconnected under the WITH_BSDCONFIG flag (a good idea since this version of
sysrc(8) indeed requires the `sysrc.subr' module installed by bsdconfig(8)).

Multiple reasons sysrc should not simply continue to live in ports. The most
important being that it is tightly coupled with the base.

Approved by: adrian (co-mentor)