Removed references to bboards, as we don't support them anymore.

Broke long lines.

Don't call writev() anymore.
It might be a sexy syscall, but it's not portable and more complex than
the mainstream solution. For sexy source program, go elsewhere!

Removed outc(), which only called putchar(), while having the same signature.
I substituted all outc() calls with calls to putchar().

Removed the space between function names and the opening parenthesis.

Reformated comments and long lines

Removed the draft message in favor for a consistent draft folder facility
- reworked sbr/m_draft.c (of course)
- removed the -draftfolder/-draftmessage/-nodraftfolder switches
- removed the -draft switch from anno and refile
- no more need to question on existing draft message - yay!
Everything is more consistent now. Cornercases went away. :-)

(Also fixed the switch numbering, as already done for other files earlier.)

Fixed broken swit numbering (there are more files to fix)

Removed the whom program, which relayed on post(8) functionality.
We can later add one again, which only expands aliases but does not
require the MTA to support -bv.

Removed the split feature of send.
It would split large messages into several partial ones, using MIME.
But if a message contains one large file attached, it won't solve the
problem. We better go without this feature and leave it to the user.

Removed unneeded sbr code.
memmove() and strerror() removed again. ;-) Mmh takes C89, at least, for
available.

Rearranged whitespace (and comments) in all the code!
The primary goal was to only indent with tabs and not mix them with spaces.
Also, no more tabs should appear past the first printable char on each line.
Now editing the sources will be much easier, no matter which editor one uses,
and rearranging won't break aligned code.
(Beware, the changeset is huge, but the `diff -w' changes are moderate.)

Removed mhn, as it was already replaced by mhlist/mhshow/mhstore.

Removed msh, vmh and wmh.
The latter ones were long time obsolete; now they went away.
Msh has hardly to do with everyday mmh usage, and we don't support bboards
anymore.
This one changeset saves us about 7,000 lines of code. *yay*

Removed unused and not compiling program wmh.
It was added 1986, and is not even mentioned in nmh's history.
The website is (was?) http://wmh.sf.net

Removed obsolete MTS docs from README.developers

Clarification fix in man/mhparam.man, related to -debug.

Reworked LINK, now named `altmsglink', similar to BACKUP_PREFIX.

Moved default file/folder modes from acconfig.h to config/config.c

Removed --with-{editor,pager} configure options.
We keep defaulting to vi and more, but search in PATH at run time.
Nowadays, both programs should be available via PATH everywhere.
Otherwise, the the path in the profile.

Documented -debug switch or mhparam(1)

Moved sendmail item to better place in man page mh-profile(5)

Removed configure option --with-hash-prefix and moved backup-prefix to profile

Renamed configure.in to the more modern name configure.ac

Nicer structure in summary printing of the configure script.

Moved the sendmail path config line from mts.conf to the MH profile
This another step in getting rid of the mts stuff.

Removed TLS support. We don't need it anymore as we stopped talking SMTP
Libcrypto and thelike went away also, as they were needed only for TLS.

Removed SASL support. We don't need it anymore because we talk SMTP no more

Removed the MTS and post; now we always use /usr/sbin/sendmail -t
Mmh won't talk SMTP (nor POP).
Also moved OfficialName() from /mts/smtp/hosts.c to sbr/addrsbr.c.

Removed --with-smtpservers configure option and thelike

Removed all POP support
To retrieve mail from POP servers use external tools, e.g. fetchmail.

We'll always use sendmail (spost) now

Install to /usr/local/mmh

This is the birth of mmh. :-)

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.

* uip/mhlistsbr.c, uip/mhlsbr.c, uip/picksbr.c: cast
pointers to unsigned long before casting to unsigned int
to avoid warnings on 64-bit about casting from pointer
to int of different size.

* uip/new.c: cast folder_len to int to avoid warning on
64-bit about passing a size_t to an int conversion specifier.

* uip/inc.c: gcc 4.4.1 noticed that maildir could have been
used before initialization.  In fact, if that code was ever
reached, maildir would never have been initialized.  It's in
an adios call, so replaced with NULL.

* mts/smtp/smtp.c: In sm_auth_sasl (), removed zeroing of
sasl_inbuffer because it could be used in sm_fget ().  Also,
removed allocation of sasl_inbuffer because that was done in
either smtp_init () or sendmail_init ().

Remove duplicate sm_fgetc() declaration

Fix warning in getaddrinfo() call.
Add missing support for -port in whatnow.
Make sure the SASL input buffer is allocated when using the sendmail mts.

* etc/mhn.defaults.sh: Never try to make mhshow start xterm.

Note 'new' commands.

* test/runtest, test/tests/inc/test-deb359167,
test/tests/inc/test-eom-align, test/tests/manpages/test-manpages:
Load common.sh via absolute path, otherwise some /bin/sh
(e.g. dash) can't load it.

* uip/Makefile.in, uip/new.c, test/tests/new/test-basic: Add new
program, and fn/fp/unseen symlinks.

* test/{runtest,setup-test}: Move MH profile under Mail directory
so each test script will have its own to muck with, if needed.

* h/Makefile.in, h/crawl_folders.h, sbr/Makefile.in,
sbr/crawl_folders.c, uip/folder.c: Extract the folder crawling
code from folder.c into new crawl_folders function, using a
callback to assemble the folder info in folder.c.  Drop compare
function and use strcmp instead.  Rename addfold and addir to
add_folder and add_children (add dir vs. add folder?
confusing names).

Move freeaddrinfo() to inside of our loop; this prevents a memory leak
and also prevents it from being called with a NULL pointer.

Massive overhaul of networking code.  Changes:

- All networking interfaces now converted to new APIs (getaddrinfo() and
  the like).  All networking now supports IPv6 (with the exception of
  the built-in FTP client).
- SASL security layers are now supported for SMTP.
- Command-line and profile-based selection of the SMTP server and port.

* sbr/m_getfld.c: fix two bugs which could cause us to walk off
the beginning of the stdio or prefix-string buffer when checking
for presence of the end-of-message delimiter in some situations.
This might cause inc to dump core if you were unlucky. (This was
Debian bug 359167.)
* test/tests/inc/{md5sums,msgheader.txt,test-deb359167,
test-eom-align,fromline.txt,filler.txt,deb359167.mbox}: new tests
both for the specific problem and to try to check various alignments
of the eom string with the stdio buffer ending.
* test/common.sh: Added some functions for doing progress indicators.

test/common.sh: new file for common utility functions for the test scripts.
Moved findprog out of manpage test script into this new file.

test/runtest: automatically run setup-test if it hasn't already
been done. Improve error handling in test script with 'set -e'.

* h/mh.h: raised NAMESZ from 128 to 999.  RFC 2822 limits line
lengths to 998 characters, so a header name can be at most that long.
m_getfld limits header names to 2 less than NAMESZ, which is fine,
because header names must be followed by a colon.  Add one for
terminating NULL.

* sbr/m_getfld.c: in warning message, use NAMESZ-2 instead of
NAMESZ-1 bytes because the message says "exceeds", not
"is greater than or equal to".

* docs/MAIL.FILTERING: added note on removing procmail -f or
-r options to try to suppress generation of a "From " line on
incoming messages.

+mhn.defaults.sh.cfg

* test/setup-test: Don't run autogen.sh unless needed.  Build in a
blddir under the temporary directory so as not to interfere with
whatever the developer may be doing in the source directory.
If the temporary directory already exists, rebuild and install it.

This is what happens with last-minute "safe" improvements: actually point
chset to something in the new merged InitText!

* sbr/ambigsw.c: Send print_sw output to stderr.  This avoids
strange problems with scan `pick -bogus`, not to mention being
nicer anyway.

* sbr/getans.c, sbr/print_help.c, uip/msh.c: Send print_sw output
to stdout.

* h/prototypes.h, sbr/print_sw.c (print_sw): Add FILE * argument
and send output there rather than to stdout.

* test/tests/pick/test-stderr: Add test that error messages don't
end up going to stdout.

* uip/pick.c: Ensure valid sequence name as soon as we parse it from
command, instead of searching the messages first.

* uip/pick.c: Print matching messages immediately, instead of
waiting until the very end to print anything.

* uip/Makefile.in, uip/popi.c: Remove popi.c, which has been dead
since PatchSet 257 (2000/06/12 11:07:34 UTC).

* h/mhparse.h: Move interfaces also used by mhbuildsbr.c here
from mhparse.c.

* uip/Makefile.in (mhbuild): Build with mhparse.o.

* uip/mhbuildsbr.c: Drop the duplication of mhparse.c.

* uip/mhbuild.c, uip/mhlist.c, uip/mhn.c, uip/mhshow.c,
uip/mhstore.c, uip/mhtest.c: Drop check swdeclaration, which is
now in mhparse.h.

* uip/mhparse.c: Move some declarations to mhparse.h, and remove
'static' from the definitions here.
(get_ctinfo): Take magic mhbuild support from mhbuildsbr.c's copy,
and adapt callers.
(incl_name_value, extract_name_value): Move from mhbuildsbr.c to
support get_ctinfo change).
(get_content, open7Bit): Add Content-Disposition support, added to
mhbuildsbr.c's copy in mhbuildsbr.c r1.12 (PatchSet 549).
(InitText): Default to CHARSET_UNSPECIFIED instead of "US-ASCII",
as mhbuildsbr.c's copy did.  Explain termproc.
(openBase64, openQuoted, open7Bit): Fix comment from r1.5:
suffixes, not prefixes.
(openMail): Explain showproc.

* test/tests/mhshow/test-cte-binary: Add test for
Content-Transfer-Encoding: binary (Debian #136976).

* test/tests/mhbuild/test-forw: Add an mhbuild test, testing forw
with and without RFC-934 mode.  More mhbuild tests would be nice.

Oops, meant to use $failed (number of failed checks) as the exit code.

Add descriptive headers, as other tests have.

* test/tests/bad-input/test-header: Add test for it.

* sbr/m_getfld.c: If we reach the end of the line without finding
a ':' when parsing a header field, treat that line as the
beginning of the body rather than blowing up.  These messages are
usually spam, but it's nice to be able to at least scan them.

* test/tests/mhshow/test-qp: Test various valid and invalid
escape sequences.

* uip/mhparse.c (openQuoted): Simplify the decode-or-show for loop by
peeking ahead to the next byte(s) when encountering '=', and just let
invalid escape sequences through as literals (fixes bug #15245).