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.

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.

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 test-msg-buffer-boundaries.
(cherry picked from commit dbd0fa0a56a222bd118fea627eba53e5d9811d01)

Brought over changes to test-pop from m_getfld branch.

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.

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.

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

Check that a character array isn't empty before looking for its
next-to-last element.

In build_nmh, removed configuration of pager and editor.
And replaced use of which(1) with a self-contained function.

Support the VISUAL and EDITOR environment variables, and remove support
for configuring a default editor with --with-editor.

Remove the --with-pager configure option and replace it with a default
of "more".

We never use the autoconf checks for errno.h, crypt.h, or netdb.h, so
remove them.

At this point we're assuming at least C89 support, so there's no reason
we should check for support for const and standard header files.

Convert the MIME content cache switches over to the smatch() New World Order.

Added explicit third initializer to some swit struct values
to silence gcc -Wmissing-field-initialziers warning.

Convert all argument processing over to using X-Macros for cleanup.
Based on a suggestion by Ralph Corderoy.

Fix bug where a "cur" sequence that was outside the folder allocation limits
could trigger a core dump on some systems.

Removed support for #ifdef LBL.  It was broken anyways,
sometime between MH 6.8.5 and the Apr 1999 entry into CVS.
Also removed the addtoseq format function that could only be
accessed with it.

Fixed test-utf8-body to actually contain the UTF-8 character
instead of the single byte, 8-bit character.  This was caught
using Heirloom shell with a UTF-8 locale:  when it wrote the
test draft from a here document, it converted the single byte
character to UTF-8.  This verifies that behavior:

LANG=en_US.utf-8 /usr/lib/heirloom/5bin/sh -c "od -x <<EOF
`printf '\xA1'`
EOF
"

Expanded warning message for invalid Content-Transfer-Encoding.

Added reference to RFC-1864 to mhbuild man page.

Remove support for -ebcdicsafe/-noebcdicsafe switches in various utilities.

Create a dependency for mh-chart.man on all of the man page sources so
it gets rebuilt correctly if the man pages change.

In cpstripped() and cptrimmed(), if a multibyte character is
found, only count it as taking up one character in the
destination buffer (if it has enought room).  This way,
scan(1)'s output won't be jagged if there are any subjects,
for example, that have multibyte characters.

Added a test for this to test-scan-multibyte, and moved the
test for an invalid multibyte sequence from test-scan to it.

Removed empty file name .null from the historical mh-6.8.5 tree.

Fixed decoding of header fields when they contain a character that
can't be converted.  Added test case to test-scan.  valgrind also
noticed that things went amiss.

Remove test .netrc file at the end of test-pop.

Switch to using native automake rules for cscope.

Change TLS support to only link the TLS libraries with the programs that
need them.

Clean up (and hopefully improve) the autoconf iconv tests.

Still MORE autoconf cleanup.

Autoconf/automake cleanup

Document removal of --enable-nmh-debug

Finally deprecate --enable-nmh-debug

Note changes to autoconf requirements here.

We are now requiring autoconf 2.68 and automake 1.13.

Note that the minimum required version of automake is 1.12.

Ignore top-level ar-lib file.

This is detritus from the AM_PROG_AR macro I added to configure.ac.

Add an AM_PROG_AR program check to configure.ac  This quells a
warning when configuring on FreeBSD 10.

Tweaked last commit to test/common.sh.in to eliminate a mv.

Added note about PATH when running "make check" on Cygwin.

Fixed test-slocal to work with mmdfldir set to /dev/null, and on
Cygwin, which needs the full path to tee(1).

Removed a few #ifndef JLR/#endif pairs because they're not needed.

Removed some dead code, all inside #if 0's.  I tried to not remove
anything useful, so there are a few #if 0's remaining.

Import a copy of Markus Schnalke's master's thesis: The Modern Mail Handler.

(Added with Markus' permission.)

Remove the peeking into the stdio internals inside of scan().  Hm, this
was more complicated than I originally thought.

Added release notes for releases 1.4 and 1.3 to tail of NEWS.

mhshow:  properly treat unknown multipart subtypes as "mixed"

Added test of -nosequence to test-pick.

pick: add -nosequence command line switch

Note in dist, mh-profile, and repl man pages that the @ link
is only created with -atfile.

Fix to 25581a94c5113eb78b2baf7110408df96efc4418: always set $editalt,
even without -atfile.

Added docs/contrib/vpick.

Cleaned up sendfiles(1) man page just a bit.

Corrected the last addition to man/nmh.man.  Might be nicer to explain
the warty truth from the off.

Updated the TODO file.

Copied atexit() code from fakesmtp.c to fakepop.c so that its
pid file gets cleaned up.  Also copied the sig handler.

Fixed formatting glitch in msh.man.

Added description of @folder format to nmh.man.

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

show: remove unused variable

this should have been part of the previous removal of unreachable code.

show:  remove unreachable code

the stanza in question has been impossible to reach for some time.  i
believe its purpose was to give a non-mh display proc full pathname
arguments to work with.  since we've chdir'd to the folder, giving it
message numbers (i.e., relative pathnames) is fine.

show: remove MAXARGS limit when running any of the helper procs

use app_msgarg() to maintain both the msgs[] and vec[] string vectors.

forw: remove MAXARGS limit when running mhlproc

use app_msgarg() to maintain both the msgs[] and vec[] string vectors.

burst: eliminate use of MAXARGS

use the already available app_msgarg() utility to make this easy.
(the MAXARGS limit was highly unlikely to be exceeded, in practice.)

Update about changes to rmmproc support.

app_msgarg: de-obfuscate

add some whitespace and break things up.  and add comment to reflect
that this might be (and is) used for more than just msg parameters.

Artificial argv limits.

when i skimmed the code earlier, i was looking at all the uses of
MAXARGS -- many of which are used to size local arrays.  converting
them all to be dynamically-sized might be a fair amount of work.

but looking at rmm.c, refile.c, and folder_delmsgs(), i don't think
that issue applies.  the patch below fixes the problem, and makes
rmmproc and refile do the right thing (well, at least, the modern
thing).  NB: i didn't test to the limit where execvp should return
E2BIG.

as for the rest of mh -- it's certainly possible to hit the MAXARGS
limit with other commands (show, for instance), but i suspect the
frequency is much lower, and those, too, can be fixed as needed.

paul

>From 9636bc0e1697829966f51d2c301cd5d730c38c0a Mon Sep 17 00:00:00 2001
From: Paul Fox <pgf@foxharp.boston.ma.us>
Date: Mon, 26 Nov 2012 20:49:55 -0500
Subject: [PATCH] rmm/refile: remove the MAXARGS limit when running rmmproc

Since pick now uses decode_rfc2047, we need to be sure to link in the
iconv library (if we need it).

In pick(1), attempt to decode each header field as if it were
MIME encoded.

In pick(1), attempt to decode each header field as if it were
MIME encoded.

A new test for the POP support in inc.  Very simple for now, but it seems
to at least check the basic functionality.

Add support for a -nosasl switch.

Fixed pick(1) to properly unfold multiple-line header fields
by removing newlines instead of replacing them with spaces
[Bug #15215].

The big news for this update is nmh 1.5.

Fix the race condition with fakesmtp so it works without needing retries.

Document changes to inc.

Document the -port switch to inc.  Also, fix the -port switch to inc so
it actually works.

Added test-slocal.

slocal used to support two different formats for three of
its arguments:

  [address info sender]
  [-addr address] [-info data] [-sender sender]

But it hasn't properly supported the first form since at
least April 1999.  So, I removed that first form from the
man page and the broken support from the code.  Score
another for the test suite!

Added Message-Id to messages in test suite.

Fixed a couple of signed-unsigned comparisons.

Yozo TODA: fix for mhshow-charset- directives in .mh_profile being broken

Merge branch 'format-memory-rework'

Make sure we strip off a newline from components we want to display (we
were stripping off a newline from the wrong copy of the component).

Convert repl to the new world format API.

Support for the new world format API for comp and forw.

Whoops, with the new world order we need to make sure we allocate memory
for all strings in format instructions.

Convert scan() to the new world format API.

Change fmt_findcasecmp() to fmt_findcasecomp().  The new name is more
consistent.

Change API a bit.

Rename fmt_addcomp() to fmt_addcomptext().
Create new function fmt_addcompentry().

Remove unused variable.

Removed leading space from lines in mhbuild and rmm man pages
to correct formatting.

Finished work on rcvdist to convert to new world format API.