Removed myuid and mygid variables, they're no longer used and
the Solaris C compiler complained about their initializers.

Cast wchar_t to long unsigned int to deal with differing types of
wchar_t across systems.

Suppress warning from test-manpages that groff can't break a line if
the user's prefix is too long.

The Solaris compiler was right about unreachable statements.
Fixed all of them in uip/.

Added l modifier to X printf conversions of wchar_t's.

Removed unused vec[0] relic from before argsplit conversion.

Added trailing newline to sed input in mhmail when processing
-headerfield argument, to squelch complain from Solaris sed.

Set MHTMPDIR to try to keep all test-related files together.

More test-mhmail probing.

Try this test-mhmail probe.

More test-mhmail probing.

Modified test-mhmail probe.

Added temporary probe to test-mhmail.

Convert post over to use new argsplit code.

Handle the case where (on Solaris) the width of U+2019 is 2 (which seems
TOTALLY bizarre to me, but there you have it).

Rework multibyte test so it will calculate the width of all of the
Unicode characters we are using.

In setup_test(), set PATH after install.

Added set -x to test-ali.

Maybe the mkdir in setup_test() needs a -p?

More test-ali diagnostics for Solaris.

Use test -t 1 instead of test -w in test-pick.

More setup_test() and test-ali attempts on Solaris.

Removed the ls of folder from setup_test().

Try full path to folder in setup_test().

Maybe the PATH needs to be exported again in setup_test()?

So folder does exist the first time through setup_test().
Let's see its stdout.

Suppressed display of info message when test-eom-align isn't
run with stdout to a terminal.

Temporarily added ls of folder to setup_test().

In setup_test(), create the Mail directory if it doesn't
already exist.

Ignore the return status of diff, so we don't exit if diff fails when using
set -e.

Add some diagnostics to determine what is going wrong.

It looks like the first test fails on Solaris because it
can't find folder.  So, added a test/sleep before its first
use, in setup_test().

If the multibyte width test fails, dump the values of wcwidth() for the
complete BMP.

Added diagnostic to setup_test() to try to figure out
why the first test fails on the Solaris buildbot.

Restored test-ali to its prior condition.  It failed on the
Solaris buildbot because it was the first test.

Make sure we created the test file even if we don't have iconv support;
it's needed for two tests.

Move ICONV_ENABLED so it always gets initialized even if the header check
for iconv.h fails.

If we don't have iconv, skip the multibyte test that requires character
set conversion.

Finally switch over from vfork() to fork() completely.

More code converted to use argsplit().

Fix configure tests for -Wfoo option support.

Drooped support for -Wunused-macros and -Wno-unused-macros.  The
warnings serve no useful purpose, and causing subsequent configure
checks for -Wfoo flags to get messed up.

Split each test of -Wfoo into a separate check. Include -Werror in
the individual flag tests to ensure the test compile fails if the
other flag isn't supported.  Without this, clang issues a diagnostic
message about the flag being ignored, but otherwise carries on, fooling
configure into thinking the flag is in fact supported.

Merge branch 'master' into savannah

Update manpages to use .TP for tagged paragraphs (part I).

This replaces .fc/.nf and friends with the man(7) .TP macro.

This commit covers about half of the man pages.  The rest will
follow in a couple of days.

Update manpages to use .TP for tagged paragraphs (part I).

This replaces .fc/.nf and friends with the man(7) .TP macro.

This commit covers about half of the man pages.  The rest will
follow in a couple of days.

Update the manpage template documentation.

Update the section ordering to follow the UNIX order.  This
pulls nmh-specific sections into the body of the manpage, and
pushes FILES, SEE ALSO, and BUGS to the end.

Clarifies a few other minor points.

More probing of test-ali on Solaris:  exit before the
setup_test call, which has the folder invocation.

OK, test-pick should be good on Solaris now.

Added exit to test-ali, temporarily, to isolate failure on
Solaris.

More homing in on the test-pick warning.  Also, removed
warning to user that the tty test is being skipped, it's
not that important.

Removed an used sed invocation from test-mhmail.  I don't
know if it was causing the warnings on Solaris, we'll see.

Fix some warnings for Solaris.

it looks like the write to /dev/tty is causing the warning from
test-pick on Solaris, here's an attempt to squelch that.

Removed sizes from mhlist output.  Apparently tar files are
so much smaller on Solaris that they cause the check of its
output to break.  The test also compares the contents of
the tar file, so it's still adequate.

Redirected stderr to /dev/null in the test for presence of
/dev/tty.  Solaris warns that it can't create the device.
This may not fix it, but it should help isolate which part of
the test is causing the warning.

Set the locale for one of the "pick" invocations to UTF-8, so we can decode
the UTF-8 header correctly in the test.

Use cmp(1) to compare acheieved and expected test results.

'diff -c' on Solaris 11 generate a spurious 'No differences encountered'
message (on stdout, no less!) when the compared files are identical. This
changes the test logic to compare with 'cmp -s', and then perform a
'diff -c' if the results don't match expectations.

Wrap 'pick -subject foobar 13' test with set +e/-e.

The command was failing silently on FreeBSD, leading to
premature script termination.

Change the MD5FMT macro in configure.ac to not emit filename.

How hard can it be to write a portable shell function that does nothing?

Pretty hard, apparently :-P

Hush progress indicator during tests when stdout is not a tty.

This eliminates 100 lines of logfile spam during buildbot runs.

Merge branch 'master' of ssh://git.sv.gnu.org/srv/git/nmh into nmh

IPPROTO_TCP requires <netinet/in.h>

At least on BSD it does.

More changes to use argsplit() for helper programs.

Switch the editor invocation over to use argsplit().

The start of the use of argsplit() to process command strings.

Changed m_getfld() buffer size to 4096 from 8192, to benefit
scan(1).  We could consider using a larger buffer size when
readying message bodies.  But I expect any benefit would be
overshadowed by the way parse_mime() currently reads MIME
messages:  it reopens the file for each part.

Added m_getfld_track_filepos() for callers to indicate that they
may interleave fseek*/ftell* calls with m_getfld() calls.  This
eliminates the need for m_getfld() to call fseek() when called
by inc(1) and scan(1).

Added scan_reset_m_getfld_state(), for msh only.  It needs to
call that so that it can support more than one operation on its
mailbox.

Merge branch 'm_getfld'

In the sortm warning message, say "file modification time"
instead of "file mtime", per Paul F.'s suggestion.

Changed sortm warning message from an amonish() to and advise(),
and included the remedy of using the file mtime, per suggestions
of Norm and Ralph.

Fixed a couple of calls to m_getfld():  they were in while loops
and the loop bodies were missing brackets.

Deprecated msh(1).

In build_nmh's output from the test suite:  retain
colorization and list any test failures.

Enabled colorized test output (PASS/FAIL/summary) by default
via "color-tests" automake option.  To disable at runtime,
either set your AM_COLOR_TESTS environment variable to "no",
or provide that on the make check command line.

Reworked the new documentation in m_getfld.c.

Handle the case in the test suite where wcwidth() can return different
values for combining Unicode codepoints on different operating systems.

Fix the format engine so it properly keeps track of multibyte characters
and column positions when processing components and other functions.

When working on this code I realized we have nothing that checks to make
sure right justification of format function output works properly, so a
test for right justification is also included.

Added functions for inc, msh, rcvtty, and scan to access
scansbr's gstate instance so it can be hidden, except for
msh's uses of m_getfld().

Fix a bug leading to message corruption with inc when incorporation more
than one message at a time.

When I reworked the scan() code to remove the use of stdio internals,
two code paths were created.  The inc code path consumed another buffer
from the nxtbuf list for processing the message body.  At the end of
scan() there was code to push back all of the consumed buffers to the
nxtbuf list.  The extra buffer consumed in the inc code path wasn't
being pushed back, and as a result the nxtbuf list would end up with
duplicate pointers in it.  This would result in two components sharing
the same text buffer, which caused the message corruption.

When I looked at this closer, I realized that there was no reason to
push those buffers back to the list; nxtbuf gets reset to compbuffers
at the start of every call to scan(), and nxtbuf isn't used after the
buffers are pushed back.  Some researched revealed that this was the
same back in the original MH code.  I decided it was simplest to simply
delete the buffer recycling code.
(cherry picked from commit 22d2acba60374851c291260ff6aa716e9bf44a6d)

Brought over changes to test-pop from master to m_getfld branch.

Added scan and inc from mbox with multiple messages.
(cherry picked from commit c4c53777458bfcfd288d45b7708e52eb47c56405)

Added test-msg-buffer-boundaries.
(cherry picked from commit dbd0fa0a56a222bd118fea627eba53e5d9811d01)

Brought over changes to test-pop from m_getfld branch.

Changed m_getfld() to automatically allocate a new state
instance as needed.  Callers no longer use
m_getfld_state_init().

Removed unnecessary m_getfld_state_destroy() from scan.c.

Reworked enter_getfld() and Peek().

Fix a bug leading to message corruption with inc when incorporation more
than one message at a time.

When I reworked the scan() code to remove the use of stdio internals,
two code paths were created.  The inc code path consumed another buffer
from the nxtbuf list for processing the message body.  At the end of
scan() there was code to push back all of the consumed buffers to the
nxtbuf list.  The extra buffer consumed in the inc code path wasn't
being pushed back, and as a result the nxtbuf list would end up with
duplicate pointers in it.  This would result in two components sharing
the same text buffer, which caused the message corruption.

When I looked at this closer, I realized that there was no reason to
push those buffers back to the list; nxtbuf gets reset to compbuffers
at the start of every call to scan(), and nxtbuf isn't used after the
buffers are pushed back.  Some researched revealed that this was the
same back in the original MH code.  I decided it was simplest to simply
delete the buffer recycling code.

Plugged small memory leak in m_unknown().

Replaced hard-coded value with MAX_DELIMITER_SIZE in m_getfld.c.

Removed the static m_getfld() state instance and replaced
with functions for callers to create and destroy their own
instances.  This also allows inc, msh, and pick to call
m_getfld() with an initial file stream position other than 0.

There is now a global state instance so that scan can call
m_unknown() on maildrop files.  It'd be nice to get rid of
that.

Fix memory leak in arglist code

Added scan and inc from mbox with multiple messages.

Updated test-inc-scanout on master to match version of
m_getfld branch.

Added a FILE * to the m_getfld_state struct so that most
functions don't need to pass it explicitly.  Note that
m_getfld() must still be passed the FILE * because some
callers, e.g., those in mhparse.c, reopen their input file
and therefore pass different FILE *'s for the same input.

Moved all file static data in m_getfld.c into the
m_getfld_state struct.  Changed MSG_INPUT_SIZE from
hard-coded number to BUFSIZ.

Added check of -pack to test-pop.

Fixed test-header to work on Cygwin, which has a BUFSIZ of
only 1024.  That caused an error message to be truncated.

Beginning of implementation of new argsplit() function to handle arguments
to profile-defined commands.

Removed all traces of m_getfld() FLDEOF and BODYEOF states.
They were never set by m_getfld(), so all caller code that
supported them was completely unexercised.