Scanning an empty mbox file is no error.
Of course there won't be a delimiter, but mail drop files are frequently
empty, though existing.
This solution is more of a hack, but it works. Improvements welcome!

Default scan format: Added leading zeros, which I've forgotten.

Added my own scan format: scan.meillo

Added HH:MM to the default scan format.

Minor layout fix in man page mhparam(1).

Removed -(no)push and -(no)forward flags from send, spost and whatnow.
We don't send directly (with SMTP) anymore. We only execute sendmail,
which, on modern systems, takes few time. Thus we do it directly
instead of having more complexity for the rare wish to run sendmail in
the background and also be unable to deal with possible errors directly.

Put structure into anno(1) and its man page.
The ``newly'' added features (-list, -delete, ...) were simply added
on top, but without rethinking and adjusting anno(1)'s structure. (I
agree, however, that it was the right decision to add these features
to anno(1) instead of writing another tool.) I've reworked anno now to
give it a clear structure. And I've reworked its man page to propagate
this structure.

Updated man page mh-mail(5).

Added missing NULL sentinels in switches array.

mmh.sh: Minimal change.

Added wrapper mmhwrap for parallel installations with nmh or a small $PATH.
It might be helpful.

Folded fileproc to constant `refile'.
Also moved sbr/refile.c into uip/whatnow.c

Migrated man page post(8) to spost(8).
It would be nice to have it improved a bit more.
Spost is located in section 8 (maintenance commands) of the manual,
but it is located in the bindir, rather than the sbindir. This is
on purpose. The sections of the manual are mostly informational (today),
whereas directories are either in $PATH or not, and thus cause a
functional difference. Hence, I don't blindly link sbindir to section 8.

mhl: removed -folder switch and folder output for messagename pseudo component.

Removed dead code within never set define WHATNOW.

Trash folder replaces rmmproc. Rework of rmm(1) and refile(1).
Removed several switches from refile and changed its behavior.
(Although, rmm does not use backup prefixes anymore, other tools
still do.)

Fixed programming mistakes, detected by cppcheck.
Two memory leaks and a possibly not terminated string.
(There also is a false positive in sbr/path.c.)
(sbr/dtimep.c has warnings, but I believe lex is correct. ;-) )

Removed -realsize from man page because mhshow doesn't support it.

Pulled in from nmh. Thanks to David Levine.

Enhance fix of 0b81a300d9e7cf78b6bc11d8c870f57fa81fcdc4.
Ken Hornstein did the same fix for nmh. He showed me that I had missed
the second, similar, case. Thanks.

Automated generation of mh-chart.man7.
Moved mh-chart from section 1 to section 7, in the same go.

Based upon a pull-in from nmh. Thanks a lot to David Levine.
Autogenerating is the only sensible approach to mh-chart.

Trivial change.

Add some more default MIME content suffixes.
Some of them originated from:
    cat /etc/mime.types |
    perl -ane 'if ($F[1] ne "") {print "mhshow-suffix-$F[0]:.$F[1]\n"}'

Pulled in from nmh. Thanks to Ken Hornstein, David Levine and Steve Rader.

Clean up prototypes.

Pulled in from nmh. Thanks to Ken Hornstein.

Make sure we don't send Envelope-From if we're using spost.

Pulled in from nmh. Thanks to Ken Hornstein.

Create proper prototypes for formataddr().

Pulled in from nmh. Thanks to Ken Hornstein.

Fixed mhstore to honor nmh-storage profile entry even with -auto [Bug #35303].

Pulled in from nmh. Thanks to David Levine.

Fixed two fd leaks in mhparse.c.
1) In openQuoted(), the decoded contents file was opened twice in
   succession. The second open was added by a patch in March 2000.
2) openBase64(), openQuoted(), and open7Bit() could open the contents
   file, but it never would be closed. Added code to each to close the
   file if the function opened it.
[Bug #24004: (debbug 344182)]

Pulled in from nmh. Thanks to David Levine.

Replaced UTC-8 with UTF-8.

Pulled in from nmh. Thanks to David Levine.

Fixed a bunch of minor man page formating problems.
(Revealed by `groff -ww'.)

Pulled in from nmh. Thanks to David Levine.

Fixed mhlist -verbose to work with all content types [Bug #35219].

Pulled in from nmh. Thanks to David Levine.

Added -verbose/-noverbose to mhlist synopsis

Pulled in from nmh. Thanks to David Levine.

Escape spaces in DEFAULTS sections of man pages so they no longer are lost.

Pulled in from nmh. Thanks to David Levine.

Removed ignored -verbose (and -noverbose) options to mhstore.
(In contrast to nmh, we also change the code.)

Pulled in from nmh. Thanks to David Levine.

Fixed example to show that mhparam preserves case of component name.

Pulled in from nmh. Thanks to David Levine.

Fixed `mhbuild -check': use the correct file.
mhbuild -check would always fail because it tried to build the digest
for the decoded contents file (cefile), which can be null. If it is, use
the contents file itself.

Pulled in from nmh. Thanks to David Levine.

Make -volume switch actually work (it always got overridden).
As far as I can tell, this never worked. Found by the nmh test suite!

Pulled in from nmh. Thanks to Ken Hornstein.

Suppress echoing of man page build lines.
One of them says "warning", and it's handy to check that build output
doesn't say that.

Pulled in from nmh. Thanks to David Levine.

Updated docs to reflect that all of the lists are on nongnu.org.

Pulled in from nmh. Thanks to David Levine.

Report an error when the width used by putaddr is less than the label width.

Pulled in from nmh. Thanks to Ken Hornstein.

new: Improve the fix with the NULL-sentinel.
With multiple sequence arguments on the command line you might end up
with garbage in the sequence[] array (it needs the last argument to be
a NULL. So do two things: make the first two arguments NULL, and as we
add arguments to sequence[] make sure the list has a NULL on the end.
This is another problem discovered by the nmh test suite.

Pulled in from nmh. Thanks to Ken Hornstein.

Correct the maximum number of sequences in the mark(1) man page.
Also add a note explaining the somewhat low limit.

(Note: mmh's value is different to nmh's.)

Pulled in from nmh. Thanks to Ken Hornstein.

Show assignment of configure variables using VAR=VALUE instead of in the env.

Pulled in from nmh. Thanks to David Levine.

We have only numeric timezone support (e.g. +0200) from now on.

That's an mmh decision. Pulled in from nmh is the following part:

Removed use of tzset()/tzname in dtimezone(). They caused the returned
timezone string to be for the user's current timezone, but the function
is supposed to return the string representation of the timezone of its
argument.

Thanks to David Levine.

new: sequences[0] needs to be initialized to NULL.
This was caught by the nmh test suite.

Pulled in from nmh. Thanks to David Levine.

Added error message if COMPLETION-BASH is source directly.

Pulled in from nmh. Thanks to David Levine.

Exit with error message if no library could be found with tgetent().

Pulled in from nmh. Thanks to David Levine.

Replace ics.uci.edu/~mh/ links with corresponding rand-mh.sf.net links.
The former ones are dead.

Pulled in from nmh. Thanks to David Levine.

abs() is part of C89.

FreeBSD needs the HAVE_STRUCT_TM_TM_GMTOFF test.
FreeBSD has a timezone() function, not a timezone global variable
(that's visible with our default configuration).
Undid most of 0e4bc6fa7fa8ad74a1c3e855411bdc0e63bc6a2e.

Pulled in from nmh. Thanks to David Levine.

uip/termsbr.c: remove references to PC.

Pulled in from nmh. Thanks to Lyndon Nerenberg.

s/AC_CHECK_NETLIBS/NMH_CHECK_NETLIBS/, since it's not part of autoconf.

Pulled in from nmh. Thanks to Ken Hornstein.

Removed the undocumented `select' option of slocal(1) and thus removed utmp.

Removed HAVE_STRUCT_TM_TM_GMTOFF support.
timezone, tzname, and tzset() are all POSIX.

Pulled in from nmh. Thanks to David Levine.

Removed unused parameter of replout().

Changed types and added casts so that build is clean with gcc -Wsign-compare.

Pulled in from nmh. Thanks to David Levine.

Proper field initialization.
gcc -Wmissing-field-initializers noticed several struct initializations
that didn't explicitly list all fields. They were of no consequence because
they were for static data, so they were initialized properly.
Also, changed the initialization of global.c_ovtxt from "" to NULL because
free_queue () freed it if non-NULL, though I think the "" was always
overwritten.

Pulled in from nmh. Thanks to David Levine.

Documented support of OURDEFS to pass compile flags to configure.

Pulled in from nmh. Thanks to David Levine.

Add some comments to the format compiler and engine.

Pulled in from nmh. Thanks to Ken Hornstein.

Update README.developers with the correct version of autoconf we need.

Pulled in from nmh. Thanks to Ken Hornstein.

Remove some unused macros.

Pulled in from nmh. Thanks to David Levine.

Completely remove the use of TIME_WITH_SYS_TIME.
Removed the autoconf test but never removed the code, and this caused
some problems on systems that needed some prototypes in <time.h>.

Pulled in from nmh. Thanks to Ken Hornstein.

Remove OS-specific defines. Remove GECOS ampersand-magic, along with it.

Pulled in from nmh. Thanks to Ken Hornstein.

Clean up time handling; always assume we have tzset().
Also assume that the right prototype is in time.h (both are valid in POSIX).

Pulled in from nmh. Thanks to Ken Hornstein.

We are making POSIX signal support a requirement.
Remove all support for other signal interfaces.

Pulled in from nmh. Thanks to Ken Hornstein.

Switch from using RETSIGTYPE via autoconf to void.
POSIX says that's the type to use for a signal handler.

Pulled in from nmh. Thanks to Ken Hornstein.
(btw: `to pull in' describes my action better than `to merge'.)

The result from the call to parse_mime() was ignored.
Detected by `gcc -Wempty-body'. It has been that way since the beginning
of history and was probably of no consequence, but fixed anyway.

Merged from nmh. Thanks to David Levine.

Removed hard-coded -I.. and -I../.. from INCLUDES.
It was redundant given -I$(top_srcdir).

Merged from nmh. Thanks to David Levine.

Added Content-Description header with the filename of the attachment.
This seems to be common practice now and gives mhlist something useful
to display.

Merged from nmh. Thanks to David Levine.

Added removal of autom4te.cache directory to mostlyclean-local.

Merged from nmh. Thanks to David Levine.

Remove RPATH code though. Discussions on nmh-workers have convinced me.

[2012-01-05] Lyndon Nerenberg:
> [2012-01-05] Ken Hornstein:
>> RPATHS    - Construct Return-Path headers from "From " lines.
>>             I say keep it, since it's been around forever (a fair
>>             amount of code, actually).
> I'm going to argue for the removal of this code, as well. The
> Return-Path header should be inserted by the delivering MTA, and I
> can't think of any modern MTA that doesn't.

Added datarootdir definitions to silence warnings from configure.

Merged from nmh. Thanks to David Levine.

Removed check for st_blksize; it's required by POSIX.
It's unclear why the it was there at all as the define wasn't even used!

Merged from nmh. Thanks to Ken Hornstein.

Switch to standard autoconf macro for checking d_type field in struct dirent.

Merged from nmh. Thanks to Ken Hornstein.

Remove #ifdef around use of sigsetjmp().

Merged from nmh. Thanks to Lyndon Nerenberg.

Remove overrides for 'broken' stat macros. Remove redundant SEEK_* macro defs.

Merged from nmh. Thanks to Lyndon Nerenberg.

Nuke directory processing conditionals; use dirent.h.

Merged from nmh. Thanks to Lyndon Nerenberg.

Clean up process wait to use POSIX waitpid() interface.

Merged from nmh. Thanks to Lyndon Nerenberg.

Make use of mkstemp() unconditional (vs. mktemp()).

Merged from nmh. Thanks to Lyndon Nerenberg.

No longer check for the mkstemp() function in the mkstemp library.

Merged from nmh. Thanks to Ken Hornstein.

Remove compatibility functions that are always provided by the POSIX host env.

Merged from nmh. Thanks to Lyndon Nerenberg.

Well, if the nmh folks now believe that snprintf() can be expected to be
available on recent hosts, we remove it again. ;-)

Ignore OSPEED/ospeed in termcap. Nothing current should need this.
Remove autoconf ospeed checks.

Merged from nmh. Thanks to Lyndon Nerenberg and Ken Hornstein.

Collapse termio/termios/sgtty terminal interface down to Posix termios.
No longer check for termio.h or termios.h

Merge from nmh. Thanks to Lyndon Nerenberg and Ken Hornstein.

Eliminate '#ifdef HAVE_FOO' for Posix-mandated FOOs.
Update configure file for removal of POSIX-mandated header files.

Merged from nmh. Thanks to Lyndon Nerenberg and Ken Hornstein.

Deprecate uname() in favour of (Posix) gethostname().
Merged from nmh. Thanks to Lyndon Nerenberg and Ken Hornstein.

Moved #endif so that lockit () is only declared with DOT_LOCKING,
to avoid compile warning without it.

Merged from nmh. Thanks to David Levine.

Changed second argument of calls to part_ok() and type_ok() from 0 to 1,
to match what store_multi() in mhstoresbr.c does.

A test case is below.  Without this fix, the following command:

  mhshow -part 1.1 -form mhl.null

produced no output.  mhlist showed a part 1.1:

 msg part  type/subtype              size description
1231       multipart/mixed            276
     1     multipart/related          107
     1.1   text/plain                  29

and "mhstore -part 1.1" properly stored it.  With this
fix, mhshow behaves properly and shows the part.

Analogous behavior happened with -type text/plain.

test case:

MIME-Version: 1.0
Content-Type: multipart/mixed; boundary="NextPart_001"
Content-class: urn:content-classes:message
Date: Thu, 19 May 2011 00:00:00 -0000
From: sender@example.com
To: recipient@example.com

This is a multi-part message in MIME format.

--NextPart_001
Content-Type: multipart/related;
type="multipart/alternative";
boundary="NextPart_002"

--NextPart_002
Content-Type: text/plain;
charset="utf-8"

This is the text/plain part.

--NextPart_002--

--NextPart_001--

Merged from nmh. Thanks to David Levine.

Replaced cvs with git references.
Merged from nmh. Thanks to David Levine.

Removed unused local variables to eliminate warnings from gcc 4.6.2.
Merged from nmh. Thanks to David Levine.

There's no need for killpg(). We can use kill() instead.
(Also removed a now unused define.)

Removed conflict(8), because checking passwd, /var/mail, etc. is not our job.
Conflict did check for:
- duplicate users
- duplicate groups
- existence of group members
- valid group ids
- orphaned maildrops
... and by the way for valid format of alias files.
The latter task can be covered by ali(1) without address arguments.
All the other tasks have nothing to do with the MUA. Actually, most of
them can be covered with shell one-liners.

Removed the now irrelevant parts of the ``history section'' in m_getfld.c.

CHANGE OF THE DECADE: I finally gave m_unknown() a name: thisisanmbox().
The ``magic invocation'' of this function informs m_getfld() that it is
reading a packed mbox file and not an MH-style (one-file-per-message) file.
m_getfld() needs to care for message delimiters then. The ``magic'' of
m_unknown() had vanished some time ago, already. Back then, it needed to
guess which kind of mailbox format the file is. With dropping the support
for MMDF (^A^A^A^A) maildrops, we converted m_unknown() into a plain dump
function ... and now, it also got a boring name. Well ...

s/unixbuf/fromline/ and minor refactoring.

Replaced msg_style with the boolean ismbox.
We only need to differ between mbox and MH-default (one msg per file).

We don't need m_msgdef.c separately and some minor rework in m_getfld.c.

Removed support to handle ARPAnet (RFC733) addresses containing ` at '.

Removed personal flavor for ISI.
Logically: Don't add myself more than once with `repl -cc me' and the like.

Merged folder_pack.c into folder.c.

Replaced folder_exists() by a call to create_folder().
Changed inc(1) to create non-existing folders with -silent, instead of
silently failing in this case. I consider the new behavior to follow the POLS.

Fixed `inc -silent' by not building a scan line if we don't want to print it.
scan() is still ugly and I'm still not pleased with the scan/inc merge,
but I don't see a better solution yet.