Garbage collect autoconf support for checking for the header file for initgroups
and snprintf() prototypes.

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

Finally able to get rid of acconfig.h!  Since we're doing slightly better
in the autoconf universe, rename configure.in to configure.ac, the "more
correct" name for the configure script template.

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

Removed unused bcmp, bcopy, bcpy, and bzero #defines.

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

Removed traces of bboards and bbc from mh-profile and msh man pages.  Added see also of msh to packf man page.

Wrapped #include of config.h with #ifdef HAVE_CONFIG_H, just in case someone ever wants to build without using configure.

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

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

Added note about fileproc and mhlproc mh-profile entries.

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

Quoted argument to AC_LINK_IFELSE to silence authoheader warnings.

Merge branch 'fileproc_mhlproc_to_post'

Move definition of LINK to mh header file, and garbage collect ATTVIBUG code.

Hardcode the defaults for Msg-Protect and Folder-Protect.  Also fix the
documentation (had the wrong value for the default for Msg-Protect).

Huh, apparantly I got rid of any support for POPSERVICE over a year
ago!  Good riddance, I say.

Garbage collect all of the builtin ftp client support.

Garbage-collect MHRC (and make it the default).

Garbage-collect SLOCAL_MBOX (and make it the default).

Garbage-collect RPATHS support.

Huh, turns out this define hasn't worked in forever ... the original
changes were made to the lexer output, and silently vanished a decade
ago when code was reorganized from zotnet to sbr!

Remove last vestiges of #ifdef BANG

Remove DBMPWD, make it the default.

Added datarootdir definitions to silence warnings from configure.

Fixed trivial compile warnings.

Not sure why we're checking for st_blksize; it's required by POSIX and
we're not even using the define!

Switch the use of LOCKDIR in acconfig.h to --enable-lockdir.

Merge branch 'master', remote-tracking branch 'origin'

Garbage collect all of the old UCI #ifdef'd code.

Update this with mh-e changes.

Feed fileproc and mhlproc from rcvdist, send, and whatnow to post.

Feed fileproc and mhlproc from rcvdist, send, and whatnow to post.

Bring in all of the mh-e support that is actually used to be compiled by
default, and garbage collect the stuff that is not used.

Feed fileproc and mhlproc from rcvdist, send, whatnow, and whom to post.

Whoops, for right now we do need those signal function checks.

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

Remove #ifdef around use of sigsetjmp().

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

Include <stdio.h> for snprintf() prototype.

Nuke directory processing conditionals; use dirent.h.

Add back missing include of <sys/wait.h>

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

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

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

This patch also removes the m_scratch() and m_tmpfil() functions.
They have been deprecated for a while, and are no longer used in any of
the code base.

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

Get rid of checking for vfork(), since we no longer use.

Replace <arpa/ftp.h> with "h/arpa_ftp.h".

Import non-Posix <arpa/ftp.h> for uip/ftpsbr.c.

Remove HAVE_ARPA_FTP_H and arpa/ftp.h references.

Remove autoconf ospeed checks

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

No longer check for termio.h or termios.h

Collapse termio/termios/sgtty terminal interface code down to
the Posix termios interface.

Update configure file for removal of POSIX-mandated header files.

Eliminate '#ifdef HAVE_FOO' for Posix-mandated FOOs.

No longer need to check for the existance of sys/utsname.h

Deprecate uname() in favour of (Posix) gethostname().

Sigh.  I put the documentation about the -tls switch in the long description,
but not in the command summary.

Updated pending-release-notes

Added note about cpio failing to unpack an nmh tarball.

Switch to using buffered I/O when doing TLS (but only for output).

Updated these files since we've finally got 1.4 out the door.

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

Reworked nmh.spec to support rpmbuild from source RPM.

Removed unused vmh.c and wmh.c.

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

Replaced cvs with git references.  I'm not sure about the cgit
URL at the very end.  Also removed mention of source directories
that were deprecated and removed long ago.

Added --with-rpmdir configure option so that user can set location of RPM build root.

Added comments about source of description, etc.

Replaced use of GNU make shell function with backticks.  These were
for the rpm target, which would most likely only be build on Linux.
So they probably would have been OK, but just in case.

The docs/Makefile install target didn't do anything before.  Added
the files that go into doc to it, and removed them from nmh.spec.
nmh.spec should now require zero maintenance.

Added RPM/ so that git status doesn't mention that it's an untracked file.

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

Moved version number from nmh.spec to Makefile.in, to minimize
maintenance for nmh.spec.

Make sure to not reset hghnum (in the Maildir code) if we are incorporating
mail from a POP mailbox.

Added first cut at nmh.spec, along with rpm: target in Makefile.in.

Removed netdb.h because we no longer distribute it.

Removed unused local variables to eliminate warnings from gcc 4.6.2.

Make sure to ignore the nmh dist file.

Since we're now moving past 1.4, bump this.

Add support to inc for incorporting email from Maildir dropboxes.

Add missing #include for <sys/socket.h> to bring in definitions of PF_UNSPEC.

Signed-off-by: Ken Hornstein <kenh@pobox.com>

Whoops, user & port were in the wrong order so msgchk didn't work for POP
connections.

Updated the developer README with a few notes about the git conversion.

Signed-off-by: Ken Hornstein <kenh@pobox.com>

Signed-off-by: Ken Hornstein <kenh@pobox.com>

More documentation about the hooks support.

Clean up code in inc.c

Clean up the confusing use of a variable 'i' in inc.c.
The case where it was being used as 'an error occurred
which means we should not truncate the mailspool' is now
called 'incerr' and merged with the equivalent 'p' variable
used in the POP-specific code. Cases where it was being
used as a random temporary are still called 'i' but now
declared with a much more restricted scope.
This fixes a compilation warning for some compilers.

Signed-off-by: Peter Maydell <peter.maydell@linaro.org>

Suppress tsort diagnostic waffle

tsort produces waffle to standard error if it detects loops in its input.
For our purposes in ordering object files this is harmless, so just send
it all to /dev/null.

Signed-off-by: Peter Maydell <pmaydell@chiark.greenend.org.uk>

Fix unused-function warnings in lex output

Fix the unused-function warnings in the output of lex. In particular, we
silence a warning about yyunput being static but unused by exposing it as
a global function. This costs us a few hundred bytes in the binary but
the other available solutions are flex-specific or more complicated.
YY_NO_UNPUT used to silence this but no longer works in recent flex.
On the other hand YY_NO_INPUT does work to silence warnings about input().

Signed-off-by: Peter Maydell <pmaydell@chiark.greenend.org.uk>

Fix used-uninitialised warning in fmt_rfc2047.c

Fix a used-uninitialised warning in fmt_rfc2047.c.
I don't think it can actually be used uninitialised, but the
code is too complicated for the compiler to realise this.

Signed-off-by: Peter Maydell <pmaydell@chiark.greenend.org.uk>

Fix used-uninitialised warning in new.c

Fix a used-uninitialised warning in new.c by having the check_folders()
routine return first=last=NULL for empty folder lists. This is a more
sensible API anyway. (It would not actually have been possible for
the code to try to use last while it was uninitialised, as it happens.)

Signed-off-by: Peter Maydell <pmaydell@chiark.greenend.org.uk>

Fix crash if fnext/fprev are given an empty folder list

Add a check for a NULL pointer, which stops fnext/fprev
from segfaulting if given an empty folder list (for example
via 'fnext -folders emptyfile').

Signed-off-by: Peter Maydell <pmaydell@chiark.greenend.org.uk>

Fix warning in sbr/folder_addmsg.c by code cleanup

Clean up some rather oddly written code in folder_addmsg()
so it is easier to understand for both humans and for the
compiler (and the compiler thus stops complaining about an
uninitialised variable).

Signed-off-by: Peter Maydell <pmaydell@chiark.greenend.org.uk>

Fix so this compiles without SASL support.

Damn you, Oliver Kiddle, for finding all of the bugs I add to nmh!

All of the arguments to AC_ARG_ENABLE need to be quoted, otherwise the
last action doesn't make it into the configure script.

Make sure we ignore config.h.in~

fix typo and clarify two sentences in mhpath man page

improve mhparam man page

fix badly formatted code found by Markus Schnalke

propagate and check return status of close() in slocal

document msg parameter to folder