seq_nameok(): Distinguish three identical error messages.

Otherwise `illegal sequence name' doesn't say what's wrong.

test-charset: Divulge mhshow output if iconv_open(3) doesn't gripe.

Andy Bradford was finding the grep failed, but was left with no
explanation that's what happened, or what the file that didn't match
contained.

%(divide): Avoid SIGFPE on integer divide of INT_MIN by -1.

Dividing by zero isn't the only cause of SIGFPE.
    fmttest -raw -format '%(num -2147483648) %(divide -1)' foo
Also, add FIXME comment suggesting the user should be told of overflow
rather than silently using zero as the result.

Use trim_suffix_c() to remove optional trailing character.

I'm unsure if one of the cases can know the string's length is long
enough to read the char before the NUL.  Using trim_suffix_c()
simplifies and removes the possibility.

mhfixmsg.man: Replace UTF-8 `§' with `\(sc' troff escape.

man page source is ASCII.  GNU groff's troff's source is ISO 8859-1.
The `§' in UTF-8 looks like `§' in ISO 8859-1.  If those two bytes make
it to a UTF-8 terminal then it appears as `§', but target something
else, e.g. PDF, and the flaw shows.

Also adjust test-manpages to grep for bytes other than HT, and SPACE to
tilde, in the built man pages.  A test of the source could be also be
done for the Makefile's `all' target, but this one should remain to spot
the build process breaking things, e.g. non-ASCII in $prefix.

Fixed RFC reference in mhfixmsg man page.

RFC 5322 has line length limits.

fmt_scan(): Avoid undefined behaviour from overlapping strncpy(3).

M. Levinson pointed out in private email that pcc(1) tickles a bug that
causes undefined behaviour due to part of `buffer[]', accessed by `str',
being strncpy(3) back to its start.  Fix by using memmove(3) to slide
`str' back to `buffer's beginning whenever it may have wandered off.

Fixes e8635a8a that added `%(unquote)', but using `%(trim)' twice was
also faulty in the same way and pre-dates git.

mhshow: Avoid SEGV when user's command has two or fewer words.

After argsplit() returned a NULL-terminated vector, the existing NULL
was overwritten with another and the vector index then incremented.
This made the test for whether vec[2] was available out by one resulting
in NULL being used and printed.

This plucking of the third word is a faulty assumption as argsplit()
doesn't always return ["sh", "-c", ...], but add a new test script for
it to avoid regression and so that the script can be expanded with more
related tests.

Fixes 7559e1ebf.

test-whom: Send whom's stderr down pipe; don't discard.

Fixes 2a02eef3.

test-msgchk: Send msgchk's stderr down pipe; don't discard.

Present since original 26ac907a.

test: Send command's stderr down pipe; don't discard.

Fixes 880ff2a4 that turned `foo >$actual 2>&1' into
`foo | sed ... >$actual 2>&1'.

test/common.sh.in: Unset libcurl(3)'s proxy environment variables.

Debian's packager, Alexander Zanger, reported test/oauth/test-* problems
due to environment variables for the system's network proxies.  David
Levine suggested the unsetting of them.  The tests only make HTTP
connections so just the variables affecting those are unset.

test-subpart: Send mhshow's stderr down pipe; don't discard.

Fixes 2f1e191a.

test-textcharset: Send mhshow's stderr down pipe; don't discard.

Fixes 67bb0fc0.

Removed quotes around "%{charset}" in mhbuild-convert-text.

They broke test/repl/test-convert on Cygwin.  It looks like parameter
quoting differs between mhbuild and mhshow.

Fix to commit 47629e9e87a324531be881cebfca38dd38f8afaa.

Removed quotes from charset argument to iconv.

Fix to commit 47629e9e87a324531be881cebfca38dd38f8afaa.  Ken
noticed that it broke test/repl/test-convert on MacOS X.

Filter test output to remove added whitespace.

Ralph noticed that lynx added whitespace to the (empty) converted
html document, breaking the test.

Fix to commit 47629e9e87a324531be881cebfca38dd38f8afaa.

Wrapped uses of %{charset} in mhn.defaults with double quotes.

Add test-fmttest for cpstripped(), cptrimmed(), and cpnumber().

Many more tests could be done, and some, like those below, would produce
known erroneous results, but this is a start and helps spot regressions.

 create mode 100755 test/format/test-fmttest

fmttest: Allow `-outsize 0' to mean no output, not default size.

`0' might be arrived at by a script tracking remaining space.

Add Arch Linux to the often-built-and-tested distro list.

And correct a `Mac OS' to have an `X', and `HPUX' to be `HP-UX'.

Also trap ambiguous post switch.

Fix to commit d321a6ee82fce5d187f4342199c090909a72e9b8.

Fixed rcvdist to pass switch arguments to post(1).

Looks like this has always been broken.  Found by Steven Winikoff
when he tried to pass a -port switch, to override the default
submission port.

This is intended to be a temporary fix.  The post switches should
be factored out into one place.

Removed escape before newline.

It was causing a *roff font selection code to appear in the
formatted man page.

Fix to commit 95e0df2af.  That commit removed a space after the
escape, which used to cause a double space in the formatted output.

Added mhfixmsg -[no]decodeheaderfieldbodies switches.

Note that mhshow-suffix/mhshow- matching is case-insenstive.

implement the long-neglected 'folder -noprint'

Removed sysconfdir overrides from SPECS examples.

Add support for setting nmh directories.

Add new precious variables (nmhlibexecdir and nmhetcdir) that will
permit the user to specify these directories exactly.  They default to
$(libexecdir)/nmh and $(sysconfdir)/nmh, respectively.

Removed version number from OpenBSD, it's in the "recent" section.

Moved OpenBSD back to "routinely built and tested" category.

Updated platform lists in MACHINES.

Removed sbr/dtimep.c from CLEANFILES.

automake puts it in maintainer-clean-generic, which is where it belongs.
Reversal of commit 1353a95720e4049e5f8357cd009b76c0450ec39f.

Added note to commit checklist to start message with one-line synopsis.

Also, removed obsolete reference to Lyndon's buildbot.

test-mhfixmsg: pwd(1)'s `-P' needed to avoid failure with symlinks.

Steven Winikoff reported to nmh-workers that one of this file's tests
failed in 1.7.  It attempts to manipulate the output of mhpath(1)
assuming it starts with the result of `pwd', but symlinks in the latter
may cause this to fail.  Use POSIX's `-P' option to ensure no symlinks,
as done in some other tests.

Use ISO-2022-KR instead of EBCDIC-US in test-charset, because
Robert Elz reported that it is installed on his NetBSD host.
Also, don't run that check without iconv.

Fix tests that assume the backup prefix is a comma.

Steven Winikoff reported to nmh-workers that some tests failed in 1.7
configured with --with-hash-backup because they assume the backup prefix
is comma rather than hash.  Alter those tests to use `mhparam sbackup',
as other tests already do.  PATH is already set to use the mhparam being
tested.

test-mhfixmsg used find(1) to check for leftover files that globbed
`mhfix*' or `,mhfix*'.  Changed to just `*mhfix*' as I think any file
containing that would be unwanted, and it will also spot mhfixmsg
hard-coding the comma prefix.

NEWS: Encourage nmh-announce subscription.  Capitalise `Gmail'.

Cherry-picked to 1.7-RC3-6-g3d2c68e3.

mhshow/test-charset: Protect question mark from shell globbing.

Reworked parameter value tests to reflect Ralph's (re-?) discovery
that GNU iconv elides the ? from charset names.  nmh uses it as a
replacement character when decoding fails.

Makefile.am: Remove `-I./sbr' from sbr_libmh_a_CPPFLAGS.

Doesn't seem to be useful.  Present since the switch to automake, so
possibly a leftover from experimentation.

fmttest.c: Clear output buffer after each write.

Otherwise each write outputs the whole buffer again, including what's
already appeared.  Instead of the three lines 1, 2, 3, one sees 1, 1, 2,
1, 2, 3.

Rewrite cpnumber() to fix `%2(msg)' with 123 overflowing to `?23'.

Reported in 1.7 by Ken on nmh-workers yesterday.
`It seems this was introduced in commit 92128dacf'.
We now use the C library to format the number, and then manipulate the
result.  A temporary charstring_t is no longer used.
Also adds left justification using negative width.

Normalize some #include paths.

Include the subdirectory so these files will compile from an object build.

Move etc/rmmproc.messageid to fix its install permissions.

And removed duplicate docs/contrib/replaliases to fix its install permissions.

fmt_new.c: Move interface to own file.

fmt_addr.c: Move interface to own file.

datetime.c: Make rrule_clock() static;  no external callers.

datetime.c: Move interface to own file.

crawl_folders.h: Base header on implementation.

Normally done as part of extracting the new header file from a larger
one, but this file already existed so it had to be moved as one commit
and altered in the next, otherwise git would break its history.

crawl_folders.h: Rename to be alongside implementation.

oauth.c: Alter permissions from 0755 to 0644.

Possibly introduced by 380046d2.

h/rcvmail.h: Delete.  Contained just a couple of pointless macros.

RCV_MOK was 0, RCV_MBX was 1.  They were used for exit(3) and done().
`exit(0)' is clearer than another layer of abstraction, and the commands
that used them didn't seem to document the exit statuses anyway.  The
included by other means, at least on one platform.

popsbr.h: Base header on implementation.

Normally done as part of extracting the new header file from a larger
one, but this file already existed so it had to be moved as one commit
and altered in the next, otherwise git would break its history.

popsbr.h: Rename to be alongside implementation.

popsbr.h: Remove pop_fd() and pop_set() prototypes.

Function definitions removed in 4ea2f92f.

popsbr.c: Add `static' to function definitions to match prototypes.

parse_proxy(), command(), and multiline() all have prototypes claiming
they're static in popsbr.c, but their later definitions lack that
qualifier.

picksbr.h: Rename to be alongside implementation.

dtime.c: Move interface to own file.

h/prototypes.h: Remove mhlsbr() prototype; no such function.

Function deleted in c808c5d5.

whatnowsbr.c: Move interface to own file.

whatnowproc.c: Move interface to own file.

sendsbr.c: Move interface to own file.

distsbr.c: Move interface to own file.

mhlsbr.c: Move interface to own file.

annosbr.c: Move interface to own file.

scansbr.c: Move interface to own file.

Merging in content of existing h/scansbr.h, though that only contained
one of the three prototypes.  The other two were in h/prototypes.h.

m_name.c: Move interface to own file.

m_gmprot.c: Move interface to own file.

m_getfld.c: Move interface to own file.

getarguments.c: Move interface to own file.

read_switch_multi*_readline.c: Move interface to own file.

read_switch_multiword.c: Move interface to own file.

read_yes_or_no_if_tty.c: Move interface to own file.

read_switch.c: Move interface declaration to own file.

concat.c: Move interface to own file.

seq_setunseen.c: Move interface to own file.

seq_setprev.c: Move interface to own file.

seq_setcur.c: Move interface to own file.

seq_save.c: Move interface to own file.

seq_read.c: Move interface to own file.

showfile.c: Move interface to own file.

smatch.c: Move interface to own file.

getpass.c: Move interface to own file.

r1bindex.c: Move interface to own file.

refile.c: Move interface to own file.

ruserpass.c: Move interface to own file.

remdir.c: Move interface to own file.

snprintb.c: Move interface to own file.

strindex.c: Move interface to own file.

ssequal.c: Move interface to own file.

credentials.c: Move interface to own file.

encode_rfc2047.c: Move interface to own file.

fmt_rfc2047.c: Move interface to own file.

client.c: Move interface to own file.

closefds.c: Move interface to own file.

copyip.c: Move interface to own file.

cpydata.c: Move interface to own file.

cpydgst.c: Move interface to own file.

discard.c: Move interface to own file.