Use the default visual and depth instead of "best"
Use Xrender for sclaing when library is available and server supports it
Set VisualIsRGB8 when possible

Don't require Git when building from tarball

Clear console abort flag after prompting for destination user

e.g. when running filelist.js as a timed event, you can't redirect stdout
(without running via jsexec), so let's support specifying an output
filename with an option (-out=filename).

It's unclear whether trailing whitespace is supposed to be significant or not
(FidoNet specs don't say), but I don't see any reason why "John Doe" and
"John Doe " should be considered unique senders or recipients. Remove any
trailing whitespace from message subjects too.

This change makes the trailing whitespace truncation in SMB hash functions
unnecessary (at least for SBBSecho-imported messages), but most (all?) other
message transports don't allow for this nonsense in the first place.

Log a warning message when either the "local" or "in-transit" flag has been
stripped from ("Sanitized") imported packed-messages.

Keyop reported a chksmb warning on a FidoNet message that contained the
'in-transit' attribute:
<Keyop> Message flagged as 'In Transit'     (?): 1
<Keyop> fidonet_fidosoft Message Base has Errors!

The message in question appears just to be a corrupted echomail msg coming
from HPT:
Sender           '56:43 '
To               Tommi Koivula
Subject          test
X-FTN-AREA       FIDOSOFT.HUSKY
X-FTN-TID        hpt/os2-wc 1.9.0-cur 2021-03-14

SBBSecho didn't use to convert/import the "in-transit" flag on imported
netmail or echomail messages, ... until commit 446ab0ef, which
was needed to support the TickFix robot. So now, let's just sanitize the
packed-message attributes a little sooner in the import process and log a
warning when such sanitization has taken place.

If a msgbase internal code is passed on the command-line, use that msgbase
instead of 'mail' (where the use of the 'in-transit' flag is normally expected
to be seen). For Keyop.

The notification of offline users was removed from the mail server 13 years
ago (commit 729eb16b) without a lot of fanfare or explanation. Add an
option for notification of offline users (in addition to online users) for
those sysops that want that behavior.

Use the userdat is_user_online() function in place of the getnodedat() loop,
which also wasn't considering nodes in "logon" status as having a user online.

Contrary to https://mosquitto.org/api/files/mosquitto-h.html#mosquitto_loop_stop, passing force=true is not enough for the mosquitto network thread to end, mosquitto_disconnect() must be called.

JSDOC fix-ups.

Mostly just bitfield->bit-flags.

A little more visiblity into the MQTT library (version, function return values)

Recently started truncating trailing whitespace from sender/recipient names
before calculating the CRC value (similar to what we do for message subjects
as of a year ago). To suppress errors when evaluating messages (with trailing
whitespace in sender or recipient namems) created or imported before this
change, use this new -N option.

Make it obvious when a message header value string has trailing white-space.
Make this function (and binstr()) thread-safe by eliminating the local static
variable.

This should prevent programs using rgbmap (ie: SyncTERM) from
having an exectuable stack on Linux.

And don't offset long(er) windows from edges to allow more options to be
on screen at once.

Tested with 50 column mode.

Maximize screen realestate for options/settings display:
- Don't place a margin around windows when in a narrow screen mode.
- Don't display shadows on windows when in a narrow screen mode.

This commit also fixes a bug in uinput() when left was non-zero and the total
width (left + width) would be too wide for the screen. This bug was apparent in
SCFG->Message Areas in 50 column mode (no Message Group window would be
displayed when a message group was selected).

Some BBS packages tack on trailing whitespace on names, e.g.
https://github.com/wwivbbs/wwiv/issues/1607

Fixes Windows build.

Right now, only supports publish, subscribe to come next.

Only included with the mosquitto lib support is enabled in the build.

Only on Linux, Documented as VSWTCH (also added), as a SysV-only
thing but actually defined a VSWTC, must be disabled (ie: set to
_POSIX_DISABLE) to not show up in stty as being modified.

*sigh*

I'm not convinced Linux developers want termios to work.

Be sure to set them first though, since they may overlay with
other ICANON entries.

The wrong characters in the wrong order.  I have no idea why they
even have the thing defined, but we'll just stop using it since
it seems BSD systems are the only ones where it's useful...
instead just set V* to C* except ERASE and ERASE2 which we set both
to CTRL-H

The the date/time display would overwrite parts of the application
title in narrower screen modes, so let's not do that.

No functional change, just a little clean-up.