Make sure the valgrind suppression file is honored, and add some extra
suppressions needed when running under the Debian fakeroot environment.

Fix error message to correctly report the program that failed to execute
when doing HTML conversion.
Also, remove perl function prototypes as they are not considered best
practice.

Rearrange all callers of post(8) so that the filename is always last.

mhparse.c: verify all MIME-Version: headers encountered

in addition to verifying, silently ignore duplicate version headers.

Merge remote-tracking branch 'origin/convertargs'

use %l uniformly on all the text-mode browser display strings

otherwise behavior changes depending on which browser was present
when nmh is configured/installed.

Filter converted text/html to remove blank lines and leading space
that lynx inserts.  Also, changed ISO-8859-1 to US-ASCII because it
is, so it doesn't need to be converted.

Removed m_chkids(), per suggestion of Marcin Cieslak.

Removed unnecessary #include of h/signals.h.

Removed #include of signal.h because it's in h/signals.h.

Reset done in inc_done() to prevent endless loop when setegid()
fails (in which case adios() is called, which calls done()).

That didn't work, so removed offset.

Use +0000 isntead of -0000 offset because FreeBSD displays it that way.

Replaced setgid() calls with setegid() so that it works with dot
locking on FreeBSD.  setegid() should be supported on modern POSIX
systems.

Added showmimeproc to test/common.sh.in so that test-show will test
it in the workspace, not the user's installation.  Also added all of
the other procs defined in config/config.c that are currently unused,
just in case they some day are.

Removed a few more msh (MS_MSH and mshproc) and vmh (vmhproc) relics.

Replaced individual assignment of array elements, one of which was
out of range, with array initialization.

Fixed the error message in lockit() when it fails to create tmp file.

Fixed "inc" to be installed setgid mail with dot locking.  MAILGROUP=1
was not set in the config.h because we no longer use "LOCKTYPE"
configure variable.

update tests for new message separator header from mhshow

also added test names to tests, in any file i touched

update man pages for new mhshow message separator header

add new message separator header to mhshow

new switches to mhshow (-headerform and -[no]header) control whether
and what is displayed before the display of every message's content.
the default separator looks like "[ Message folder:msg ]".  the
implementation parallels the -markform feature.

since mhshow is often started by show, show now passes the -[no]header
options through to mhshow.  (also, show no longer attempts to pass
such options through to cat (in the case of -noshowproc)).

fix typo in mh-format.man

Test that show always shows message numbers.

show shows message numbers in two different formats when showing one
non-MIME, multiple non-MIME messages, or one MIME message.

show does not show any message numbers when showing multiple messages
any of which is MIME.

This is not a new bug, just one that has bothered me since I started
using mhshow with 1.6.

test/common.sh: allow giving individual tests a name

man pages -- clarify what -part and -type do when used together

Merge remote-tracking branch 'origin' into convertargs

Added support to mhfixmsg -reformat for multipart/related.  If it has
only a single part that is not text/plain and can be converted to
text/plain, a text/plain part is added and the type of the part is
changed to multipart/alternative.  If the multipart/related has more
than one part but does not have a text/plain part, try to add one.

Fixed test-convert to work on Cygwin and other platforms that
default to UTF-8 locale.

Merge remote-tracking branch 'origin' into convertargs

Removed unused local variable from move_preferred_part().

pending-release-notes: add mhshow's "-prefer", and mh-format's %(kibi/kilo)

test-subpart: add mhlist and mhshow tests for -type/-part/-prefer

update mhshow, mhlist, and mhstore man pages for -prefer switch

implement -prefer, for multipart/alternative preferences

mhshow.man: more detail on use of -part and -type switches

mhshow: ensure that -part always shows every part requested

previously, "mhshow -part 1 -part 1.3" would show 1.1 but not 1.3.

mhshow: fix display of alternatives after certain nested multiparts

adjust return codes so that we now have a clear indication of whether
there was a hard error (NOTOK), a failure to display due to type
mismatch (DONE), or complete success (OK).  use those codes in
show_multi_internal() to decide whether we're finished with this
multipart or not.

prior to this change, it was impossible to show part 1.2 of a
message constructed like the following, because the disinction
between (the new, as described above) DONE and OK states was
lost while processing multipart/related part 1.1.
 msg part  type/subtype              size description
  28       multipart/mixed           1040
     1     multipart/alternative      623
     1.1   multipart/related          275
     1.1.1 text/html                  105
     1.2   text/plain                  33
     2     text/plain                  83

this fix is related to 9cc2510 and c9794733.  hopefully it's right
this time.

mhshow: simplify code -- the DONE status was adding no value

drop the second argument to part_ok(), since it is always 1

the second argument hasn't been used since "historical" times.
also add some commentary to part_ok().

Add a note about new bindir option to mhparam.

'mhparam bindir' now prints NMH's idea of $(bindir).

Added a line break to mhbuild man page because the
%docdir%/contrib/replaliases line caused format complaints from groff.

Merge remote-tracking branch 'origin' into convertargs

Fixed check of return value of show_content_aux().

Have show_content_aux2(), and therefore show_content_aux(), return
the status of the process used to show, instead of (usually) OK.

(cherry picked from commit a70006bbdf676639961877b02a19e9e1f1d0ec78)

Have show_content_aux2(), and therefore show_content_aux(), return
the status of the process used to show, instead of (usually) OK.

Fixed typo in mh-format.man page, which I wouldn't have noticed
if I didn't go back in to update its date.

Added multiply format function, requested by Norm.

Merge remote-tracking branch 'origin' into convertargs

Pass along arguments to calaccept(), etc., functions in replaliases.

Fix to commit 6b073f7b67a0afd64012ff7d00818ec1bd07b5f2 to properly
detect a Content-Type header in output from converted content.

Another fix to convertarg handling of multipart/related.  Two,
actually.  One is a simple bug fix, the other restricts concatenation
to text/plain, not just text, parts.

Specifically look for Content-Type header in converted content.

Fixed convertarg handling of text part in multipart/related.

Forgot to commit test-mhical.

Added mhical(1), to display, reply to, and cancel iCalendar (RFC 5545)
event requests.

Added yacc/bison support to configure.ac.

Added rtm alias.

More cleanup of the cp in tests/common.sh.in.

Added -h and -p options to replaliases.  The -p argument can be useful
with improperly structured messages, such as those that use
multipart/related when they should have used multipart/alternative.
Alos, removed -editor mhbuild because it precludes use of WhatNow
attach.

Replaced explicit list of files to be copied in test/common.sh.in
with *.  It wasn't trivial, per the comments in the file about
portability to posh, but close.

OK, this fixes the problem referenced in the last commit.  It
removes the copy of the etc files only during make distcheck.

Reverted commit a2b00e7053ca3be3d1389588580c71498222a267.  It caused
make check to pull etc files from where they were already installed,
not from the test directory.

Removed export of most of the variables in test/common.sh.in.  The
test scripts source common.sh so they don't need them to be exported.

Removed copy of all of the etc files from test/common.sh.in.
Instead, for make distcheck, pass the location of the installed etc
directory via DISTCHECK_CONFIGURE_FLAGS, configure, and
TESTS_ENVIRONMENT to a new nmhetcdirinst variable.  Outside of make
distcheck, nmhetcdir is valid by itself.

Rearranged the variable settings at the top of test/common.sh.in.

Changed direct access to etc files in a couple of tests to be the
installed (for the test) versions, not the copies made by common.sh.

Replaced nmhexecdir with nmhlibexecdir in test/common.sh.in (and
test-mhparam) for consisitency with the Makefile.

Added -fixtype switch to mhfixmsg(1).  It ensures that each part of
a message has the correct MIME type shown in its Content-Type header.

Added const to char * arguments of uprf().

Added ct_str_type() and ct_str_subtype() functions, to translate
MIME type and subtype names to the the internal MIME parser codes.

Merged two adjacent if (verbose) blocks.

Fixed memory leak in mime_type().  get_file_info() always allocates
its return value.

Always initialize those two variables made static in commit
771706ac9fed2b902bd4d3ddccb15ee27b0f58ae in case they were
set in previous calls.

And removed another improperly copied comment.

Removed improperly copied comment from test-mhfixmsg.

In expand_pseudoheader(), set Content-Type to 7-bit for ASCII text.

Added mention of profile components that are described
in other man pages.  Suggested by Michael Richardson.

Rearranged code to avoid "might be clobbered by longjmp" warnings from
gcc 4.9.2

Removed unnecessary #includes of setjmp.h.

Removed widths from mhbuild-convert-text/html lines that should
have been removed by commit 5ab36412b5fc49feb41fa816129c68fd6d946e8b.

Fixed use of text_plain_ct in mhbuildsbr.c so that it works with
multiple text parts.  Found by Clang static analyzer.

Set COLUMNS in test-cd if using readline so that its wrapping doesn't
disturb the test output.  The problem arose with a long workspace path.

posh didn't like use of file wildcard in common.sh.in.

Removed R from default PARINIT environment variable so that par(1)
does not consider an input word that is too long to be an error.

Fixed format engine output of negative number with 0 fill character.
Improper output was reported by Bob Carragher.

Added -convertargs switch to repl(1), to pass arguments to programs
specified in the user's profile or mhn.defaults to convert message
content.

Rearranged mhn.defaults.sh a bit:  consolidated web-browser specific
settings and put most of the mhstore-store- directives in a here doc.

Clarified in repl(1) man page that -noformat is only the
default when no -filter switch is used.

Only remove extraneous trailing semicolon from Content-Type and
Content-Disposition headers because those are the only ones that
parse_mime() warns about, but it does appear on others.

Terminate last arg in proxy argv n popsbr.c:parse_proxy().

This bug seems to have existed since this code was born.  I guess
others have been lucky?

Set all Content structure elements to NULL as they are free()'d; in a few
cases a content structure is reused and you can get into problems with
double-free()ing elements depending on your MIME composition content syntax.

Merge commit '8206fbf', due to my screwup of committing it on a detached
HEAD after a git-bisect.

Turns out those warnings for ali(1) actually happen when the prefix is
33 characters, not 35.

Explicitly depend on $(srcdir)/config/version.h rather than depend
on VPATH.

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.