replace obsolete autoconf macros

Fix various buffer overruns in fmt_scan.c; the bulk of this is passing
buffer length through to decode_rfc2047() and having that function do
sufficient bookkeeping to avoid running off the end of the buffer.

Fix a compile error if building nmh with --enable-apop.

Don't fclose() the file twice in error-exit paths from get_content()
(was causing crashes on multipart mails with invalid Content-Type headers)

Remove space between -o and output filename so it works on older versions
of flex as well as newer ones.

Add an AC_PREREQ so we don't just fail with an obscure error message if
using old 2.13 era autoconf (or an autoconf-version-guessing wrapper).

Update the instructions for subscribing to the nmh-workers list

fail when kpop connection attempted without KPOP support compiled in

fix for newer version of flex and remove autogenerated file from cvs

speed up flist by skipping stat on files with numbers as names

include new files in distribution

  A couple of questions from nmh's original FAQ were folded into
  the MH FAQ which is now a part of nmh.

    New Questions

1.09 What is the copyright status of nmh?
3.23 Why do folder and flist overlook some of my sub-folders?

  Changed Questions

1.02 current version? nmh history update, nmh 1.1-RC4, mailutils 0.6

Harald Geyer: back out fork/vfork workaround and handle the issue directly

use fork instead of vfork on Linux

test/report error writing to stdout

correct SASL include file locations

add bash completion definitions from Debian

fix a bug affecting AFS where nmh was setting the READONLY flag for a
folder even when you do have write access to the folder

Carl Mummert: add unquote() function for removing quotes from RFC-2822 headers

fill in a few of the missing ChangeLog entries

use iconv to convert RFC-2047 encoded headers to the character set used
by the current locale

fix Debian bug #202667: crash when a message's filename overflows an int
when converted

Updated config.guess and config.sub to the most recent versions
(from automake 1.9.5)

fix bug where inc crashed on failing to reopen the terminal

acknowledge that my change was based on an older patch from Michael Richardson

Added -proxy option to inc and msgchk

Added -proxy option to inc and msgchk

On systems where it is available, use nl_langinfo to get the character
set if MM_CHARSET is unset

US-ASCII is a subset of UTF-8 so can be handled directly when UTF-8 is
being used

Fix -part option to mhshow/mhlist/mhstore to find sub-parts of the specified
part

Added mail directory argument to folder_addmsg in order to make it possible
to provide a path to the ext_hook call that is mailpath-based.  A problem
existed when a folder was a symbolic link and the pwd call would return
the path relative to the filesystem, not to mailpath.  A new argument was
needed because there was otherwise no reasonable way to get that path.

Fixed problem where the refile hook was being called after a message was
renamed so that it wasn't around for the hook.  The hook is now called
before the message file is renamed.

Fixed typo in comment.
Fixed problem with refile hook when refiling across filesystems.

Removed unnecessary code.

Fixed wrong directory for hook when refiling with -src option.

Fixed calling of external hooks.

Fixed another weird bug caused by the static mailpath being overwritten.

Fixed bug that caused anno to mangle headers.

Fixed strange bug that prevented a lock from ever being obtained if getting
it failed the first time.  The problem was that the string of XXXXXX that
is required by mkstemp() was overwritten the first time through, and so
all subsequent times failed because mkstemp() failed.  The fix
reinitializes the tmp file string.

Fixed bug in which the static maildir was overwritted if a format string
was read from the profile.

Fixed bug that was producing an incorrect path for the external hook.

db configuration fix for Debian; yet another location for ndbm.

restored use of PUTDF() macro; was broken in versions 1.10-1.12;
replacement implementation filled on right instead of on left.

[bug #4302] errno is not always an extern int
The errno declaration is best left to errno.h; several systems use
#define errno (*(___errno()))
or similar

corrected termination condition for fill character introduced in last version

[bug #4296] PUTD() macro can't hack 10 digit values
Once the number exceeded 1000000000, the conversion would fail.  An execution of
 dp -format "%(clock{text})" "Tue, 01 Jul 2003 21:38:05 +0700" would erroneously print
749660476, instead of 1057070285.

The macros PUTD() and PUTDF() have been replaced by snprintf() calls;  These macros
should never have been written in the first place, when system functions exist to do the
work.

(prev version 1.10 was erroneously comitted).

[bug #4296] PUTD() macro can't hack 10 digit values
Once the number exceeded 1000000000, the conversion would fail.  An execution of
 dp -format "%(clock{text})" "Tue, 01 Jul 2003 21:38:05 +0700" would erroneously print
749660476, instead of 1057070285.

The macros PUTD() and PUTDF() have been replaced by snprintf() calls;  These macros
should never have been written in the first place, when system functions exist to do the
work.

[bug #4297] The strlen mh-format function can crash if 'str' is null.

note recent changes

fix handling of "-list" and "-seq" options, so behavior is orthoganal.
Previously, "pick -list -seq seqname" meant "pick -nolist -seq seqname".
Now "pick -list -seq seqname" and "pick -seq seqname -list" do the same thing.
Thanks to Robert Elz.

update 'mp->lowmsg' from 0 to 1 during regular 'inc' from mail spool;
avoids failure of 'folder_realloc()' for case of no messages in
folder, and more than 100 messages to incorporate.  Initially,
'folder_read()' will initialize 'mp->lowmsg' to 0, and 'mp->lowoff'
to 1.  So the call would be folder_realloc(mp, 1, 200), and the
comparison 'if (mp->nummsg > 0 && lo > mp->lowmsg)' will fail.

Removed space between command name and man section number (e.g.,
"show (1)" --> "show(1)", so xemacs can recognize them as links to other
man pages.

make 'spost' handling of "masquerade:" line of mts.conf containing
'draft_from' same as for 'post'.  Debian bug report 144098, filed by
Terran Melconian.

status of action wasn't saved for use by 'N' result, so when 'N' was
used slocal could perform the action even if the previous
action failed.  Debian bug report #149745, filed by Daniel Müller.

Increase timeouts to comply with RFC 1123; Debian bug report #152728 by Ian Jackson

added cross references

fixed typo:  example for scan.default didn't include proper terminating
%> for "%<{date} %|*%>"; reported as Debian bug #143427 by Peter Maydell

Apply bug fix reported to Debian (#136976) by Peter Maydell; allows
messages with "Content-Transfer-Encoding: binary" to be
displayed, per RFC 2045 section 6.2 paragraph 4

corrected typo for nroff directive

add command to remove $(libdir)/install-mh prior to creating the link

The first ``inc'' into an empty folder would not update the unseen
sequence.  This is because seq_list() will return NULL if mp->nummsg
is zero.  Insure that mp->nummsg is incremented.

Man page fix from Igor Sobrado

Added documentation of "Dcc", per recent discussion on mailing list
(main contributors Jerry Peek and Robert Elz)

Merge in changes from the 1.1 branch.

Fix from branch.

Quiet lock warning.

Merge in locking fixes into the main line.

When compiling format strings, nmh attempts to avoid multiple parsing
of address and date fields by only inserting calls to the parse
functions (FT_PARSEADDR and FT_PARSEDATE) for a given component once.
The problem with this method is that the initial invocation may
actually be on a code path that is conditionally executed.  This can
result cached copies of data from the fields in previous messages to
be used.

My solution is to move this optimization from compile time to run time.
Address and Date parsing calls (FT_PARSEADDR and FT_PARSEDATE) will
always be included.  Run time flags are used to prevent these functions
from being run more than once per component per message.

The c_flags field has being converted from a boolean to a bit-field to
facilitate maintenance of the new CT_PARSED value.  The result value
that used to be in this field is now the bit CF_TRUE and the
overloaded use of this field by scan() is now the CT_DATEFAB bit.

Some unneeded flags (CT_ADDRPARSE, CT_MYMBOX) have also been removed.

more detail for bugs fixed, so reader doesn't need to go to Bugzilla

update

restored use of %manext5% for man section, which was lost
in version 1.4

Updated description of default template to reflect current
change to In-Reply-To, and Fcc:

Default templates for 'comp', 'forw', 'dist', 'repl' now include
Fcc: +outbox

use symbols for date and version so values are
automatically updated during build.

bump date so everything gets a new timestamp for 1.1

fix bug #1393, "sortm core dumps", using patch provided by
Robert Elz, kre@munnari.oz.au.  The core dump occurred
when "Subject:" was immediately followed by an
alphanumeric, and "-textfield subject" were used
as arguments.

Re-write for clarity, using major contributions from
Bart Massey, and Jerry Peek.

made RFC 2822 compliant, using an "In-reply-to:" format
supplied by Jerry Peek.

prevent double free of 'ce->cd_file' or later use of invalid data

fix for bug #578 repl leaks umask; there are several other
places in the code where the umask value is not restored,
so there might be other similar bugs

Changed use of GNU "simply expanded variables" for MAN1, MAN5, MAN8, so
one makefile will work with SysV make (e.g., Solaris) and GNU make

allow gdbm/ndbm.h in addition to db1/ndbm.h; needs corresponding change in
configure.in

Allowed gdbm as substitute for Berkeley db1/ndbm.h libdb1 (check for
gdbm/ndbm.h, libgdbm now); send only stdout to /dev/null when checking for
'broken vi', this allows 'vim' on Solaris to work, since 'vim' closes stdin
and uses stderr for input when running a script.

Fix detection of non-mime messages when the charset is not enclosed
in double quotes.  (Not perfect yet, but 97% effective.)

Fixed a typo in the SRCS definition; had a .o where it should have been a .c

Fixed a bug in the waitpid version where it would return incorrectly if a
child process was interrupted.

Removed the reference to the library directory since this now goes in bin.

Changed to move install-mh to the bin directory instead of the lib
directory.  A link is made from the lib to the bin just in case.

The file README-ATTACHMENTS was removed - it has been moved to the docs
subdirectory.  I thought that this had already been done but I guess not.
It's unrelated to the rest of these changes.

The remainder of these changes affect the way that nmh is installed.  Nmh
has historically prompted the user for installation when any nmh command
is invoked if it doesn't think that it's installed.  This is a problem for
programs that want to, for example, interact with nmh programs via pipes.

These changes alter this behavior.  Now, when an nmh program thinks that
nmh isn't installed, it tells the user to run install-mh to install it.
A -check option has been added to install-mh that allows the existence of
an installation to be silently checked and indicated via the exit status.

The bulk of this change is in sbr/read_context.c.  In the process of making
these changes, I noticed a fair amount of useless code that I removed.  In
particular:

1. The program started with a test for whether or not defpath had already
been set.  This could only happen if context_read() was called more
than once, which it isn't.  This check also existed and was removed
from the following places:
sbr/context_del.c
sbr/context_find.c
sbr/context_replace.c
sbr/seq_read.c
sbr/seq_save.c
uip/flist.c
uip/folder.c
uip/rmf.c

2. A similar test for mypath was removed.  No need to test it since it
doesn't happen.  I'm just not a believer that

a = 1;

if (a != 1)
error();

is a good way to write code!

3. I removed the code makes a copy of the $HOME environment variable as
returned by getenv().  It's never changed, so it doesn't need copying.

4. I removed code that copied the pw_dir member of the passwd structure
returned by getpwuid() if $HOME isn't set.  Yes, the returned structure
is static, but since the function is never called again the returned
values stay valid and don't need copying.

5. I removed the test for a NULL pw_dir member in the passwd structure
returned by getpwuid() since that never happens.

6. I removed code that removed a trailing / from mypath if mypath is more
than one character long.  Unnecessary as // is interpreted as / anyway.
mypath never shows up in an error message, so nobody will ever see it.
Besides, this code didn't handle the case of multiple trailing slashes.

7. I removed code that replaced the MH environment variable contents with
an absolute path if it was a relative one.  I'm a bit nervous about this
one, only testing will tell.  Any nmh program that's invoked will use
the same context_read() code to convert a relative path to an absolute
one anyway.  And there's no guarantee (or discussion) of this feature
in any of the documentation.

8. Althout context_read() tested the MH environment variable, install-mh
didn't.  This means that the behavior was not consistent between
install-mh and all other nmh programs.  I changed install-mh to be
consistent with context_read().

9. install-mh has been installed in the lib directory, not bin.  So a
user would be unlikely to find it when prompted to run it.  The
installation has been changed to add a link from bin to lib.

Changed scan to always pass the folder argument. This is in line
with the man page.

New routine that provides interface to external programs.

Added external program hooks.

Moved this file here from the root directory.

Added an improved user interface for sending messages with attachments.

Remove this file on the head as well.

Bring these changes over from the branch.

Remove these undefines (why were they in there?)

Update for new version.

Add/update copyright notice in all source code files.

Remove autoconf-generated files (and note such files in .cvsignore)