Fixed the trap in mhmail.

Cleaned up uip/mhmail.  Added chmod +x of spost in configure.ac.

Manpage cross-reference cleanups:  convert '.BR cmd (section)' to '.IR cmd (section)'
throughout the text.

A couple more manpage typo cleanups.

Typo.

Manpage SEE ALSO cleanup.

Convert to '.IR cmd (section)' format for manpage references.

Fixed a couple of inline '.BR cmd (section)' instances as well.

Removed HISTORY from new(1); this stuff doesn't belong in manpages.

Elide spurious empty lines in manpage source files.

Ensure the .TH macro is the first line of the manpage source files. Some
versions of man(1) get upset if .TH doesn't come first.

Update the manpage authors guide, primarily for .TP, but also fixes
a couple of other incorrect style recommendations.

These updates are incomplete ... more to follow.

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

Incorporate Paul Fox's changes that group the commands by functional groups.

Move the "generated code" warning comment below the .TH macro to work around some buggy
man(1) implementations.

Removed all the .RS/.RE indenting and replaced the use of .fc and tab indenting
with .TP macros.

Move the BUGS section below SEE ALSO to follow the usual section ordering convention.

Fixed up some italic vs. bold usage.

Use '.IR foo (section)' in the SEE ALSO section.

Removed some irrelevant text, along with a small amount of word-smithing.

Unset PAGER in test/common.sh so that make distcheck passes.

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

Add new make targets 'man' and 'cleanman' to build and clean just
the man/ sub-directory.  I also removed the '@' supressors for the
internal manpage targets.  There's no net benefit to hiding this stuff.

Cleaned up some printf's.

Updated sendfiles(1) man page to reflect upcoming code update.

Some version of man(1) barf if the first line of the manpage is not
the .TH macro.  Re-order the generated mh-chart.man so that the 'auto-generated'
comment appears after the .TH macro.

Slightly reformat the FINDCOMP macro to quell a clang warning about
an empty for()-loop body.

automake 1.12 generates some (apparently) harmless warnings, but the build
fails becuase we invoke automake with -Werror.  Remove -Werror from the
automake options until the autotools gurus can patch things up.

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

Allow $PAGER to override the compiled-in default pager command.

Removed unused function seek_home().

The first alias contained in a blind list is now expanded.  The
mh-alias(5) man page was updated to show that blind lists must not
be terminated with, or contain, a trailing semicolon [Bug #15604].

Changed error message to say that blind lists aren't
compatible with sendmail/pipe.

Fixed "mark -sequence cur -delete all" to work for cur as
well as any other sequence, to allow clearing of the current
message indication.

Don't allow blind aliases with sendmail/pipe mts.  Before this change,
they would be fed to sendmail, which would reject them anyway.  This
change catches that in post and presents a more helpful error message
to the user (at the "What now?" prompt, if used).

A possible future enhancement might be, with sendmail/pipe, to expand
out blind aliases and put the addresses in Bcc: header fields, but
then the recipients would get messages with the Blind-Carbon-Copy
indication.  (Note that Dcc isn't supported with sendmail/pipe,
either.)

Just for information:  old spost fed the addresses of blind aliases to
sendmail, making them visible to all recipients.  Oops.

Removed potential buffer overflow in ali.c by replacing array of
hard-coded maximum size with dynamically-sized array.

Fixed sortm(1) verbose message and man page to reflect that with
-limit 0, the sort is only on the text field.

Added -noall/-all switches to sortm(1).

Fixed typo in mh-alias man page.

Added newline to "send" that's piped to dist.  Old 2.6 Linux
needs it.

Added test-anno, test-dist, test-msgchk, and test-rcvtty.

Changed nmh-fcc to fcc.  See etc/forwcomps.  Another catch by the
test suite!

Updates to test/common.sh.in:
1) Added function check_for_hard_links.  Moved this code out of
   test-refile so that other tests can use it.
2) Added mmdfldir and mmdflfil settings to the mts.conf for tests, to
   support test-msgchk.

Removed double quotes around case arguments in test scripts.

When determining the path of the script inside mhmail, handle case
where $0 is just the script name, without any directory.  Bourne
shell does that if the script is found using PATH.

Disabled echo of rm -f lines in Makefile to quiet builds and
installs a bit.

Replaced spost with script that exec's post -mts sendmail/pipe.

Removed mhmail.c because it was no longer needed and could get in
the way of make if its timestamp was newer than that of the mhmail
script.  Thanks to Johan Viklund for reporting this.

For the mhstore -clobber test, cd to the Mail subdirectory so that
the stored files will get cleaned up by the exit trap in common.sh.

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

Fix a comment typo.

With mhstore -clobber auto/suffix, when looking for a new filename,
actually open(2) the file with O_CREAT | O_EXCL to avoid the race
condition when figuring out the filename first and opening it later.

Added "-d --" to invocation of ls when figuring out what files
to attach in whatnow.

Fixed description of n with mhstore -clobber auto/suffix.

Clarified description of clobber -auto/-suffix per Ralph's suggestion.

Added -clobber switch to mhstore(1) [Bug #11160].

Changed ; to && in shell command, per Ralph's suggestion.  There
were two remaining places that still used ";".  Though they
weren't of consequence, it's the right thing to do.

Allow whatnow to accept "l" as an abbreviation for "list", instead of
requiring at lest "li".

Filter out all Nmh-* headers in post(1).  Do that silently for empty
ones (no header body), and warn about non-empty ones.

Added check in send(1) of attach (default Nmh-Attachment) headers to
ensure that only plain files are attached.  Otherwise, it is a fatal
error.  Note that whatnow's attach will continue to allow attachment
of directories because it expands those out to their contents.  It
doesn't check what the contents are, though.  That's why we needed to
add this check.

Also added a check to skip attach headers that are empty so that
they can be put in drafts, per Norm's request.

Updated docs/README-ATTACHMENTS, mainly to reflect that no setup is
required with nmh 1.5 and beyond.  And the user documentation is in
the send(1) and whatnow(1) man pages.

Added references to the whatnow(1) man page description of its
attach support to the nmh(7) and send(1) man pages.

Move chdir() call so -recurse option works again.

New test for folder -recurse

Change the Bcc: code in post so it includes the From: header from the original
message.

Cleaned up message_id().

Removed chmod of bccfil from make_bcc_file().  It was done before
bccfil was defined so it had no useful effect.  m_mktemp2() sets
the umask so it wasn't necessary, anyway.

Cleaned up test-utf8-body a bit.

Added mhbuild: -nocontentid to profile used for tests, to simplify
output checking.

Added default_content_type argument to
make_mime_composition_file_entry().  It's used only to force message
bodies to always use text/plain content type, which allows mhbuild to
do the right thing with them.  Thanks to Steve Rader for reporting
that UTF-8 characters in message bodies had previously caused them to
be encoded as octet-streams (with attachments managed by send), and to
Ken for coming up with the fix.

Removed nmh-from condition from rcvdistcomps.

Fixed explanation of why the unset SHELL test is uselessif /bin/sh is bash.

Reworked test-rcvdist so that it sees the output of post.

Added test of whatnow with SHELL unset to test-ls, though it doesn't
reveal anything on Linux, at least.

whatnow cooks up strings for executing external commands, and then
feeds these strings to popen or system.  These command strings rely on
$SHELL being present - and if that is not the case, then we get weird
error messages ("sh: -c not found" or similar).  (Because both
system() and popen() start up fine with the std shell, but their arg
string to parse doesn't include a cmdname/path before the -c.)

As far as i understand the POSIX standard, SHELL is recommended but
not actually mandatory, so i think it would be good to handle this a
bit more robustly: by setting SHELL to /bin/sh if not present.

Tidied up for loop in message_id().

With -messageid random, make the part after the @ more resemble a
hostname by inserting a couple of dots, and using - and _ instead of
+ and /.

Added -messageid switch to send(1) and post(8).  This allows selection
of the style to use for generated Message-ID and Content-ID header
fields.  The default localname style is pid.time@localname, where time
is in seconds, and matches previous behavior.  The random style
replaces the localname with some (pseudo)random bytes and uses
microsecond-resolution time.

Here's a better test of Resent-From: in test-rcvdist:  use the user's
actual localmbox.

Added check to notpost for presence of Resent-From:.

If post or send fail, exit with their exit status.

Removed #ifndef RAND/#endif that no longer had any effect.

Fixed the default rcvdistcomps and rcvdistcomps.outbox to to insert
the Resent-From: header.

Removed debugging statement and added comments to the duplicate check
of readconfig ().

Added -mts switch to post, send, and whom.  Replaced test-sendmail-pipe
with test-mts.

Removed -m from sendmail invocation with sendmail/pipe.  Modern
sendmail and postfix, at least, ignore it.

Correction to previous commit, 704bdde497da69854846d51d86c440fc980c5489,
to restore the behavior when using SMTP and with any blind addresses.
Don't quit the SMTP session between sending the message and bcc's, but
instead reset it.

Removed use of the ONEX sendmail verb.  It is obsolete,
according to http://smtpfilter.sourceforge.net/esmtp.html.
And was removed over 10 years ago from sendmail (8.12),
according to http://www.irbs.net/internet/postfix/0111/2237.html.

Renamed "sendmail" mts method to "sendmail/smtp", allowing "sendmail"
as an alias.  Renamed "pipe" to "sendmail/pipe".

spost(8) has been merged into post(8).  Its functionality is
enabled by selecting the "pipe" mail transport method in
etc/mts.conf, which in turn can be configured using
--with-mts=pipe.  The spost -noalias, -backup/-nobackup,
-push/-nopush, and -remove/-noremove switches are not
supported by post.  Note that spost did not support -whom or
Dcc, and neither does post when using mts: pipe.  spost is
now deprecated.

Added test-packf.

test-prompter:  removed an unused input that was piped to the
program because it sometimes caused printf to complain about a
write error.

Filtered out "From " lines when comparing mboxes because their
timestamps sometimes differed by a second.

Updated pending-release-notes.

Rather than fix the reported bug, backslash in commented lines
in alias file is interpreted, I documented that behavior in
the mh-format man page and etc/MailAliases.  This behavior is
unfortunate, but it is a result of the implementation of nmh's
vfgets() function.  That function is used by other callers
that expect the behavior.  [Bug #23553]

Added test-dp, test-fmtdump, test-prompter, test-rcvstore,
test-rcvpack, and test-rcvdist.

Applied patch #5571, Propagate Mail-Followup-To.

Also clarified man page:  replgroupcomps won't be used if you
specify another forms file on the command line or in your profile.

Added note that mhmail does not read context file itself.

Removed unused #ifdef NRTC and reference to its contents in the
rcvdist man page.

Updated man page descriptions of Msg-Protect:  it defaults to 0600 and
is not supported on some filesystems, such as FAT32.

Wrapped some shell variables with quotes so the tests have a chance of
running if the current directory has a character such as a space in
it.  Though the tests themselves need the same help for them to work.
Also, added mention of Heirloom shell to test/README.

Replaced utilities that operate on pbm files with those that operate
on pnm files, per the bug report referenced below (from Dec 2005!).
And changed the mhshow-show-image/x-pbm mime type to
mhshow-show-image/x-pnm.  [Bug #15152]

Removed BUGS section at end of rcvstore(1) man page.  It
said to not use rcvstore if you use the "Unseen-Sequence"
profile entry because the context (sequences file, actually)
could be corrupted if multiple nmh processes tried to update
it simultaneously.  The locking added by commit
82a21b6f3cddf8ab048dcb02c080ea9797c73c5a serialized that
simultaneous update. [Bug #4361]

Removed the one remaining unused #ifdef ISI.

Simplified getm() a bit, removing a goto.

Fixed ismymbox() to look at Local-Mailbox as well as
Alternate-Mailboxes.  The problem was that when an alternate
mailbox was added to the list, the node for the local
mailbox information was lost.  Added check to test-mymbox.
Added test-repl, because this is where it was first noticed.
Also enhanced output with MHWDEBUG set to show the entire
list, not just each alternate mailbox as it is added to the
list.  [Bug #36635].

Deprecated mhtest.

Removed unused UUCPDIR and UUCPFIL #defines.

Added test-sortm.  Also clarified some details in the sortm man page.