Moved run_without_input() from test-version-check to common.sh.in.

Removed unnecessary warning about missing semicolon.

The warning was for Content-Type and Content-Disposition, and was
incorrect.  Thanks to Tom for providing output with it.  Also
removed multiple trailing newlines from warning output, thanks to
Ralph for tracking those down.

Added test of encoded Resent-To: and Resent-cc:.

No longer need trailing space after CFLAGS variable assignment.

Reorder configure options to be before variables

Reorder the configure options generated by build_nmh so they
will be before the variables, otherwise they end up IN things like
CFLAGS.  Fixes bug #56680.

Fix handling of fullname

Make it that if a Local-Mailbox profile entry was NOT specified
that the fullname would be generated BEFORE the local mailbox was
generated, so the user's name ended up in the email address properly.

show and mhl now decode more addresses in header fields.

Added decoding of To:, Cc:, Resent-To:, Resent-Cc:, and
Resent-From: addresses to mhl.format and mhl.headers.  Thanks to
Conrad Hughes for suggesting it for the first two components and
to Ralph for suggesting it for the Resent- components.

Handle teen ordinal suffixes correctly.

Create new mh-format function %(ordinal)

Implement a new mh-format function %(ordinal).  This will output
the appropriate ordinal prefix (st, nd, rd, st) based on the value
of the "num" register.

Support the TLS SNI extension

Add support for sending the TLS Server Name Indicator extension
during TLS connection negotiation.

Print port number in the connection error message

If a connection to a host fails, print the port number in the brief error
message.  This will hopefully eliminate problems when nmh is using a default
port number that users are not expecting.

- Fixed bcc to work with sendmail/pipe [Bug 55700].

And better documented that dcc doesn't work with sendmail/pipe.

Commit ddf3a8574f65 is a fix to commit af586ebe59b7.

Fixed inc(1) and %(me) to not obey Local-Mailbox profile component.

Thanks to Martin McCormick for reporting and Ken for diagnosing.

Added missing ')'.

Clean up more whitespace errors

Get rid of more whitespace errors; hopefully I got them all.

Remove the #ifdef UK for the default scan format

Remove the #ifdef UK that lets you select an alternate default
scan format; this is left over from old mhconfig program.

Fix whitespace errors

Remove final traces of TMA support

TMA (the TTI Trusted Mail Agent) was an early implementation of
encrypted mail and is long gone.  There were a few vestigial bits
of this support left in nmh, and this is finally removing them.
This includes the check for the "Encrypted" header field in various
scan formats and some code to return the code SCNENC from scan()
function where it would have special handling by programs like inc(1).

Added links to David Malone's MH 6.x archives.

Cleanup MIME header handling in mhbuild

Per discussion on nmh-workers, cleanup the handling of MIME headers
in mhbuild.  The following logic should now apply.

If mhbuild is run with -auto AND a MIME-Version header is detected, then
exit silently with no error (and leave the draft untouched).  Otherwise,
if a MIME-Version header or ANY Content-* header is found, exit with an
error.  Note that this means if mhbuild is run with -auto then an error
will be returned if any Content-* headers are seen.

Previously, mhbuild would silently eat any Content-Type headers seen
in an existing draft; this behavior was brought forward from MH 6 and
no one could explain the reason for this behavior.  This behavior has
been removed.

Added several historical source archives from the early days of MH.

SRI-NOSC, 2.9BSD, mh-jun-1982, and mh-nov-1983 predate those under
RCS/CVS source code control.

Thanks to M. Levinson, David Malone, Chad, Noel Chiappa, and Ralph
for advising of their discoveries of the various archive locations.

Removed docs/historical/mh-6.8.5/miscellany/mem/a.out because SPARC
executables shouldn't be archived.

Improve fmttest output

Improve the fmttest output; make it so more arguments to format
instructions are printed and clean up some of the argument handling.

Clarify %(profile) escape

Note that %(profile) can also pull entries from the context as well
as the profile

Remove support for message/partial

Improve SASL snoop output

Generate more information in the netsec layer when -snoop is enabled.
Specifically, enumerate the supported SASL mechanisms by the client and
server and explain which layer is using encryption.

Permit the matching of canonical MIME multipart boundaries

Allow the matching of MIME multipart boundaries if the message is
in canonical form (\r\n line endings).

Updated name of generated file in comment of generation script.

Removed test that hasn't been added to repo.

Fix to commit 3f845b1b0.

Commented out full output device test in test-scan.

Reverts commit d9b33232b.  It would pass on platforms that don't
have /dev/full, but was expected to fail.

Removed redundant initialization of local variable tls.

Fix to commit ec173fd2c.  gcc noticed that it, in effect, it
was declared as an int but initialized the second time with
a bool value.

Remove MIME content caching code

Remove all of the MIME content caching code.  Besides being of dubious
value and having a number of 'interesting' security implications,
it apparently only would cache content if you ran it using mhn
(the necessary entry point was not copied over when mhn was split
apart) and it would only work on the top-level MIME content of a
message.

Removal of Content-MD5 support

The generation and verification of Content-MD5 headers is no longer
done.  FINALLY.

Fix file descriptor leak in ruserpass()

Mention inc(1)/msgchk(1) changes

Set the close-on-exec flag for sockets

Since we no longer use closefds(), start cleaning up stray descriptors.
Start by making sure that any network sockets we create have the
close-on-exec flag set on them.

Merge branch 'pop-stls'

Loosen up valgrind suppression

Change the valgrind suppression rules so they exclude more things
on MacOS X.

Garbage collect closefds()

Remove the closefds() function; it was used sporadically and was
causing problems on High Sierra.  In the future look at opening
more descriptors with FD_CLOEXEC.

Final cleanup of POP STLS support

Perform final cleanup for support for the STLS command in POP.  Add
necessary switches to msgchk and add information to man pages.

Output the header name for the PUTADDR instruction

Add basic support for the STLS command in POP

Add support for the STLS command in POP (sometimes called STARTTLS).
Functionality not yet complete.

Added -checkbase switch to mhfixmsg(1).

Renamed build_multipart_alt() to build_multipart().

And some other minor code cleanup.

Add support for %(trimr) format function

Add support for a %(trimr) format function, which behaves exactly
like %(trim) but also returns a string.  Turns out this is literally
a one-line change that does not require a new format instruction.

Replaced use of hard-coded sizes with sizeof the char arrays.

Fix to commit 23024ffad75d997bf2a85b3a65fbfdfd35f8150c.

Added CONTEXT section to fmttest man page.

Added -attendee switch to mhical(1).

This allows the user to select which of their attendee addresses they are
responding about, when they have more than one in the invitation.
Added corresponding -a switch to calaccept, etc., functions in replaliases.
Updated mhical to retain DESCRIPTION lines.  Gmail puts a link to the event
DESCRIPTION line along with a message to no edit that section, though it
seems to accept responses if the DESCRIPTION line was removed.
Incremented mhical version number to 0.5.

Reduced quantities of data that could be written to a few buffers.

gcc 8 noticed that snprintfs could have overrun the buffers.

Increased sizes of a couple of buffers.

gcc 8 noticed that snprintf could have overrun them.  It might be
better to reduce the sizes of buffers used for display names, etc.
Or ever better, use dynamically sized buffers.

Resized buffer, and moved it to the only block where it's used.

gcc 8 noticed that snprintf could have overrun it.

Resized buffer and provided timezone its own buffer.

gcc 8 noticed that snprintfs could have overrun them.

Added full output test to test-scan.

From commit f54674f1b5d07148d9655fc3b337ad5f725272a7.  The test
fails now, so added to XFAIL_TESTS.

Revert "scan(1) now checks for full output device [Bug #50925]."

This reverts commit f54674f1b5d07148d9655fc3b337ad5f725272a7.

scan(1) now checks for full output device [Bug #50925].

Added fflush(3) after every output line:  may slow down scan on
output with large number of lines.

Added PROFILE COMPONENTS section and corrected CONTEXT.

Corrected comment: whom(1) does not use context_foil().

Increased sizes of a couple of buffers.

gcc 8 noticed that snprintfs could have overrun them.

Check for /etc/centos-release, on CentOS.

ruserpass.c: Return the password typed by the user.

Darren Provine reported that entering the password didn't work.  This
was due to ruserpass() failing to set the password.  Fixes 1d4977af.

mhlsbr.c: Use variable for strncpy(3)'s size, not sizeof.

Silences gcc 8.1.1's warning that the size of the source was being used,
not the destination.  The destination has just been allocated to be the
size of the source so no overflow can occur.  Move existing variable
that later holds the buffer size to before the allocation and strncpy so
it can be used for those too.

post.c: Remove `pure' attribute from void-returning chkadr().

Spoted by gcc 8.1.1.  Fixes 93a18dd7.

rcvtty.c: Use struct utmpx's ut_line[], not a copy.

Silences gcc 8.1.1's warning that strncpy(3)'s length is the sizeof of
the source, not the destination, by removing the strncpy and passing the
source to alert() where it's only read, not written.

man: Use `.B' instead of `.BR' when only one argument.

The sendmail config component/switch can be used with sendmail/smtp mts.

As well as with sendmail/pipe.

Change the default editor from 'vi' to 'prompter'.

Absent any profile environment variable setting, the default editor
for commands like 'comp', 'forw', and others is now the nmh program
'prompter'.

Fixed mhshow/test-charset to work on all tested platforms.

Restores the functionality of commit f81046da6.  Also, skips the
last two tests if iconv isn't enabled.  And removes the test
files if the last test, without iconv_elides_question_marks, behaves
as expected.  Finally, some comments have been updated.

Avoid free() in format engine for now.

This free() call messes up the buffer handling in scansbr.c:scan();
for now we're going to live with the leak and fix this properly when
we normalize the format engine's memory handling.

cpstripped(): Replace malloc()s with statics based on MB_LEN_MAX.

MB_CUR_MAX from stdlib.h isn't a compile-time constant so can't be used
for the size of a couple of static char arrays so malloc() was used at
run time, with the tiny claims never being freed.  valgrind(1) noticed
and David suggested MB_LEN_MAX from limits.h instead;  that header's
contents are suitable for #if expressions and thus compile-time
constants.

Remove extra semicolons at end of C statements.

trunccpy(): Fix typo in comment describing function.

mhshowsbr.c: Delete single-use global int `nolist'.

Was only used in one place for its value that was always zero.

Add a few more MacOS X valgrind suppressions

Clean up memory leaks by callers of scan().

Make sure we reuse the "scanl" argument to the scan() function, as that's
the signal to have scan() reuse it's buffers.  Otherwise repeated calls
to scan will leak a TON of memory.  Reported by Ralph Corderoy.

Compile the header format for each message.

Per Ken's suggestion, compile_header() and compile_marker() are now
called for each message.  That's done with reset_comptable disabled.
So, per Ralph's suggestion, fmt_free(NULL, 1) is called after all
messages are shown to free up memory held by the component table.

Generalized gdbm 1.13 Memcheck:Param suppression a bit.

gdbm 1.13-3 (Fedora 27) doesn't show the __write_nocancel call.

Fixed mhshow part markers when displaying multiple messages.

Added test/mhshow/test-markers, it reveals the problem.

Suppress unused parameter warning in builds without Cyrus SASL support.

Support building without Cyrus SASL or TLS.

If the user doesn't specify -y, and specifies n for either Cyrus
SASL or TLS when asked.

Removed unused hf_encoding from struct hfield.

I could not find any evidence of it ever being used.

Removed comment about testing noiconv portion of get_param_value().

I'm not sure if it ever did, but it doesn't seem to now.

test-charset: Fix test when HAVE_ICONV is false.

When the Content-Type's charset parameter's encoding is `invalid', it
makes no difference whether HAVE_ICONV is defined or not as the
parameter's value can't be decoded.

decode_rfc2047(): Stop unused-variable warning if HAVE_ICONV false.

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.