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 www.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.

In-sourced copyip() and could make the code simpler.
The function had had a bad name and it's use in mhl.c was complex, anyway.

closefds() doesn't seem relevant or large enough to be a function.
I should beware in-sourcing too much ...

Replaced atooi() with strtoul(..., 8). That's part of C89 and thus okay for us.

whatnow: alist will now always list in long and numbered format.

etcpath(): The unchanged argument is the ultimate fall-back.

Merged annosbr.c into anno.c, as it is no longer used be the other tools.

We need to fix `anno -list -number'. Seems anno needs some more rework.

whatnow: call external anno(1) instead of internal annotate().

whatnow: Some minor refactoring.

forw,repl: Replaced internal annotate() call with invokation of anno(1).

anno: Can annotate files now too. This feature is yet hidden and will change.

Fixed building in a separate build directory.
Like this:
mkdir /tmp/foo
cd /tmp/foo
/path/to/sources/configure -q
make
If you want to know more, search for VPATH.

Make mhn.defaults a static file and removed the scripts that built it.
This concept its likely to change in future anyway.

Removed unused defines.

mhmail: If spost fails, preseverve draft by hand, not with mv(1).

makedir(): Removed the strange RUID!=EUID code and the access(2) call.
Dan couldn't find a reason for the code neither, hence I assume it's save
to simply remove it.
    Ummm, why do we want to avoid creating directories
    with the effective user ID?  None of the nmh tools are
    installed such that the effective should be different
    from the real, and if some parent process made the two
    be different, I don't see why it should be our job to
    enforce the real UID.  Also, why the heck do we call
    the mkdir executable rather than the library function in
    this case??  If we do want to call the mkdir executable,
    we should at least be giving it -p (and change the single
    chmod() call below) so it can successfully create nested
    directories like the above code can.
    -- Dan Harkless <dan-nmh@dilvish.speed.net>
Concerning access(2): We should better always just try the action instead
of first asking if we are allowed to do it.

scan(): Symbolic names instead of magic numbers.

Removed the trivial peekc() and put the code at the appropriate places.

Removed now unused OfficialName(). (Had been for UUCP, AFAIK.)

Remove accidentally added files.

Removed unused fdcompare().

Insourced push() into send.

scansbr: Minor refactoring.