cast isspace() arg to unsigned char in previous commit

handle extraneous trailing ';' characters that are followed by whitespace

mhfixmsg now removes an extraneous trailing semicolon from header
parameter lists.

Use Nmh-Attach for attach header field name, and accept Attach.

Have to check for missing type parameter in message/related part
even though it's required according to RFC 2387 Sec. 3.1.

Expanded explanation of mhfixmsg return status in its man page.

Look at type of parent multipart/related when determining whether
a non-plain text part already has a text/plain sibling.

Added support for multipart/related to MIME parser.

Added some checks to mhfixmsg.c to avoid dereferecing null pointers
after a failed transformation.

Fixed double fclose() in InitMultiPart() and leaks introduced in
commit 5be8db81.

prevent spurious errors when requesting specific message parts

no one should be reporting NOTOK for failure to display parts that
weren't actually requested.  failure should be reserved for parts that
we really attempted to display.

(this fix is an extension of c9794733.)

Updated simple integration example in mhfixmsg(1) man page
and added return status values.

Updated test-mhfixmsg to support -[no]changecur.

Added -[no]changecur switches to mhfixmsg(1).

Updated test-repl to compare with new -help message.

Fixed repl help message to show "-[no]cc all|to|cc|me" instead of
different options for -cc and -nocc.

Fixed leak in content_charset():  if get_param() found the charset,
it returns a copy.

Moved mhstore declarations from uip/mhn.c, uip/mhstore.c, and
uip/mhstoresbr.c to h/mhparse.h.

Removed "If this draft originated on the local host, then" phrase
from mh-alias man page.  Alias resolution applies no matter where
the draft originated.

Changed content_charset() to not cache, so the user can always, and
should, free the return value.

Cleaned up leaks from calls to content_charset() in mhfixmsg.

Added deallocation of cts to freects_done().

Fixed repl(1) man page to show "msg" instead of "msgs" because
it only allows reply to one message.

Fixed mlistfree() so that it doesn't dereference a free'd list node
in order to find the next node.

Fixed commit 6cc9e85e73ff6f6c67ae6e645d184bbf57ecc2f0 to properly
return other than the first mbox/address if there's a match in
getmymbox/getmyaddr.

Fixed memory management in mhstore.  The main problem was a double
free when both -file and -outfile switches were used on a MIME
message.

Added initial "MH-Profile-Version: 1.0\n" line to newly created
profiles.  The main purpose of this first line is to fool file(1).
Without it, if the first line of the profile is Path:, file 5.19
reports its type as message/news.  With it, it reports the type as
text/plain.

Renamed getmymbox mh-format escape function to getmyaddr, and
added new getmymbox that returns the user's address with any
personal name.

Added getmymbox function escape.

Fixed memory leak in do_name() when multiple dates are parsed.

Added free_fs(), corresponding to new_fs().

Fixed dst format escape to return 1 instead of 16, to agree with its
documentation.

Added mhstore-store-text/html formatting string to mhn.defaults.

Fixed typo in mhstore man page.

Reworked charstring_buffer_copy() to not use strdup() so that
charstrings with null bytes can be supported.  Added
charstring_append_cstring().

Added mh_xcalloc().

The units of the zone format escape are minutes, not hours.

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

no-op to force check the buildbots.

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

mhn.defaults: make fetch less agressive on FreeBSD.

Drop the -a flag to fetch.  Its retry logic has a bug where it will
go into an endless loop when trying to re-fetch an object behind
an https:// link when it is unable to validate the X.509 certificate
chain.

Use "mhparam etcdir" instead of hard-coded etc in build_nmh.

Clean atags file as part of maintainer-clean.

Add a missing option to mhn fetch command for FreeBSD.

I *always* forget the '-o -' to redirect to stdout :-P

On FreeBSD, use fetch(1) to retrieve external content through URLs.

fetch(1) is part of the base system, so it's guaranteed to always be there.
And it is more likely to have been configured for any local site
customizations (local caches, proxies, etc).

If building from a non-master git branch, include the branch name in the version string.

New locations for config files and support binaries, and smarter default for install root directory.

The default locations for config files and back-end executables has
changed.

Config files have moved from ${sysconfdir} to ${sysconfdir}/nmh, and
the back-end programs from ${libdir} to ${libexecdir}/nmh.

The only user-visible change is the addition of a new mhparam(1)
component 'libexecdir', which replaces the previous 'libdir'.  User
scripts invoking 'mhparam libdir' will need an update.  'libdir'
has been kept around for the time being, to give people a chance
to update their scripts.  It returns the same value as 'libexecdir'.
'libdir' has been marked deprecated in the mhparam(1) manpage and
the release notes; it will be removed in a couple of releases.

In conjunction with this change there was an opportunity to do a
bit of Makefile cleanup.  The old 'auxexec' construct has been
replaced with a new scheme that leverages automake's templating.
The auxexec_* references have been replaced with nmhlibexec_* forms,
and a corresponding nmhlibexecdir variable defined as ${libexecdir}/nmh.

Likewise, etcdir has been replaced with nmhetcdir.

These name changes have percolated downstream to some extent, primarily
in the man pages and test scripts.

Finally, configure has been made a bit smarter about setting the
default ${prefix}.  If it finds an existing nmh installation, it
will set the default ${prefix} to match, falling back to /usr/local/nmh.
configure looks for an existing installation by searching $PATH for
a viable mhparam binary.

Fixed commit 2adafe760a9c45b417727a3f5d9481de26471a8b to not
try to allocate a charstring_t of length INT_MAX with -width 0.

Simplified test-mhl-flags a bit.

Add a 'guide' file for acme(1).

If we ever find more than one acme user working on the source,
the single-guide-file premise might need a review.

Ultimately, there should be a way for the configure script to
search for and import a personal guide file from the developer's
$HOME or some such.  The next person to come along can figure
that out.

git: ignore top-level tags files

atags is the output from a ctags variant that generates its
output in a format recognized by Plan 9's acme editor.
(ftp://orthanc.ca/unix/atags/ for details.)

FreeBSD packages destination directory fixup.

If the ports collection is not installed, assume ports/packages
install under /usr/local.

Also, when querying <bsd.port.mk>, use the value of the $(LOCALBASE)
macro. ($(PREFIX) is for port-specific overrides of $(LOCALBASE).)

For FreeBSD builds, also search the ports tree for includes and libs.

Added checks of return value of fwrite() calls where missing.

Fixed formatting of rtrim/nortrim in mhl man page.

An "rtrim" flag has been added to mhl to remove any trailing
whitespace from filtered text lines.  A corresponding "nortrim" flag
has also been added.

If a component has trailing whitespace, e.g., body:component="> ",
mhl now trims that whitespace off when filtering blank text lines.

Fixed all anomalies detected by clang static analyzer (with
default checkers on Linux).  One was notable:  there was a break
missing from a switch case in fmt_scan.c, which caused the result
of the sday function escape to be garbage.

Fix typo in man page

On some platforms, -D_FORTIFY_SOURCE=2 warns about ignoring the
return value from mbtowc() even when it is called to reset the
shift state.  So wrap that in an if statement with a null body.

Added -D_FORTIFY_SOURCE=2 now that the build is clean with it.
Removed -Wno-unused-result because it shold no longer be necessary.
Adjusting all settings of CPPFLAGS to have consistent form.

Added a couple of new directories and a note about valgrind
warnings from debuginfo to README.developers.

Check return values of system calls in uip/ and test/ .c files, found
with gcc's -D_FORTIFY_SOURCE=2.

Quote "$SEARCHPATH" in mhn.defaults.sh to allow spaces, etc., in
PATH components.

Doubled size of buffers used by m_getfld(), scan, and fmttest to
8192, on any platform, so that -width 0 and -outsize max are
likely to include at least some part of the body of each message.

Fixed typo in mh-profile.man.

Don't let scan() use too large a width (BUFSIZ for now) to prevent
allocating a huge output buffer.

A value of 0 for the width switch of scan(1), inc(1), ap(1), dp(1),
fmttest(1), and mhl(1) now means as many characters as the format
engine can produce [Bug #15274].  That amount is limited by internal
buffers.

Fixed read of body in fmttest to limit size to that of rbuf, if
smaller than outwidth, to avoid overrunning it.

fmt_scan() no longer subtracts 1 from the width.  This has the effect
of no longer counting the trailing newline in the output of scan(1),
inc(1), and the other programs that rely on it.

Very very rough cut at trying to parse email addresses with Bison.

Check return values of system calls in several sbr .c files, found
with gcc's -D_FORTIFY_SOURCE=2.  Updated checks added previously to
mhfixmsg.c to be consistent.

Moved reverse_alternative_parts() from mhfixmsg.c to mhparse.c
and added declaration to mhparse.h.

Removed msh relics from scansbr.c, including a global.

Added size_t cast of another MB_CUR_MAX to silence the compiler
on FreeBSD 9.

Added cast of MB_CUR_MAX to size_t to silence compiler warning on
FreeBSD.

Dynamically allocate space for the output of fmt_scan(), using
charstring_t, so a fixed sized output buffer is no longer needed.
Also got rid of scanl global.

Added const to last argument of trace_cb signature, it'll be
needed for use with charstring.

Added charstring "class".

Refer to RFC 2046 instead of 1521 in mhlist.man.

Changed dist_contrib_DATA to dist_contrib_SCRIPTS so that all
of the scripts will be installed with execute permissions.

Updated mhfixmsg(1) man page to refer to mh-mkstemp(1) instead of
mktemp.

Checked return value of write() calls in mhfixmsg.c,
found with gcc's -D_FORTIFY_SOURCE=2.

Removed h/msh, hopefully the last msh relic.

ncurses package is required to run all of test suite on Linux and Cygwin.

Print information about the compiler toolchain on Darwin and FreeBSD.

Add new build tool: tools/showbuildenv

This prints some basic information about the build environment.
It's intended use is to print some information about the specific
build environment for each of the builds in the buildbot cluster.
In particular, the patch level of the OS, and the versions of
3rd-party packages installed on the system, compiler versions, etc.

Removed second argument from m_Eom() in m_getfld.c because
it was unused.

Here's a better fix to m_Eom() in m_getfld.c than commit
d2520ac7054ad75d60342606bf13c821305d958c.  The comparison
of the return value of Getc() with EOF must be as an
integer, not a char.

Removed eom_action from m_getfld.c because it was only used by
m_eomsbr(), which was removed by commit
68e8c25f906e7353269502d2292c8e99aa8c6605.

Removed mhlsbr().  It was only used by msh, too.

Removed m_eomsbr() from m_getfld.c, and its scan_eom_action()
wrapper.  It was only used by msh, which was removed by commit
e6917522a770cf2dba9997ca047977bc55fac061.

Added -debug switch to pick(1) and deprecated $MHPDEBUG.

Fixed refile(1) -retainsequences when the source and destination
folders are the same.

Separated out list of programs that are required to build from a
source code snapshot from those that are required to build from a
distribution.

Fixed sed usage in flex fixup to not use -s (or -e, because
even that's not needed).

Removed "true" from end of flex fixups.  I don't know why it was
there.

Apply flex fixup to dtimep.c with flex 2.5.37 as well as 2.5.36.

When mhparam(1) is going to output the value for spoollocking,
call mts_init() first in case it was set in mts.conf.

Updated test-spoollocking to restore mts.conf when it's finished
(successfully) so as to not upset subsequent tests.  Also fixed
removal of existing spoollocking option each time through.