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.

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.

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

switches need to be renumbered following removal of some switches so,
e.g. inc -version was not working

Massive garbage collection.  Remove functionality for APOP, RPOP,
all NNTP support (including bboards) and MPOP.

* uip/whatnowsbr.c: enforce requirement of at least one file
  argument to attach command.

man/send.man: added name= portion to Content-Type
headers to reflect 2006-10-24 change to uip/sendsbr.c.
That change allowed mhstore to use the (file)name when
extracting attachments.

Remove RCS keywords, since they no longer work after git migration.

Whoops, forgot to bracket the calls for TLS shutdown with a #ifdef TLS_SUPPORT

Update for git conversions (and cleanup of autogen.sh to use autoreconf)

Change configure.in to emit a warning when an argument is given to
--with-cyrus-sasl.

Fix the test for sasl.h to look in the correct place.

(Minor) cleanup of some of the autoconf code with respect to the handling
of compiler flags for SASL, and also rototill (and simplify) the autoconf
handling for the same.

Add TLS support for the SMTP MTA.

man/slocal.man: fix formatting error that meant a subsection heading
wasn't displayed as such. Followon cleanup: since the new heading is too
wide to fit in the margin, tidy the margin to 4 characters wide so we
are consistent with how all the headings are displayed.

configure.in: add a level of expansion for mandir when printing the
end-of-configure install path summary.

man/rmm.man: fix error in synopsis: the option is [no]unlink, not [no]link.

Silence a (harmless) uninitialized variable warning.

* uip/.cvsignore:  added "new".

#ifdef AI_ADDRCONFIG since older IPv4-only systems may not define this flag,
causing files that use to not compile.

Fixed variable declaration so older compilers will not complain.
I.e. Older compilers may not support C99, so variables must be
declared at the beginning of a block.

* Bug #15213, #18635: The use of the insecure m_scratch() and
  m_tmpfil() functions have been replaced by m_mktemp()
  or m_mktemp2() functions (defined in sbr/m_mktemp.c).
  The new functions use mkstemp() to securely create
  temporary files to avoid the numerous race conditions
  that exist with the old functions.  This does assume
  that mkstemp() is available.  Unsure if we need to
  create an alternative implementation if mkstemp() is
  not available.  More information about new temp file
  functions in m_mktemp.c, including the support for
  MHTMPDIR, TMPDIR, and TMP envvars.

NOTE: The files sbr/m_scratch.c and sbr/m_tmpfil.c should be removed.
Until more testing verifies changes to support new functions have not
broken functionality, the files are be left in the tree, but comments
have been added stating the functions in them should not be called.

* mts/smtp/smtp.c: added SASL support if mts configuration
  setting is set to "sendmail".  This is useful if sendmail
  conf option is to a custom script that creates a proxy
  connection to an smtp server.

* sbr/mts.c: added support for MHMTSCONF and MHMTSUSERCONF
  envvars.  The former specifies an alternative system
  mts.conf to use.  The later specifies a user-specific
  mts.conf to use.  This one will be read after the system
  conf, so the user's conf only needs to set options they
  want to override.  The MHMTSUSERCONF allows a user to set
  personal alternative mail submission methods w/o affecting
  other users on the system.

* uip/whom.c: added SASL-based options so address checking
  can work against a server that requires SASL.