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

Ignore test-temp-dir.

add one more hint for developers, to explicitly point at docs/README.developers

autogen.sh (new file): add script for running the GNU autotools in the
right order. Mention and use new script in the appropriate places.

* sbr/discard.c, sbr/m_getfld.c, uip/scansbr.c: Add support for
DragonFlyBSD stdio (from NetBSD pkgsrc).

s/ls/cd/ in comment (probably pasted from test-ls)

uip/folder.c: chdir(nmhdir) in main rather than in dodir, which
is called many times.  Drop the chdir in get_folder_info, which
served no purpose at all.  If struct stat has d_type (BSD), use
that to skip processing (strcmp, stat, and so on) of message
files, mostly undoing the slow-down from the last change.
Plus autoconf machinery to detect d_type field. Patch from Eric Gillespie.

Dump hacky overoptimisation in addir -- it doesn't actually get the case
of symlinks to directories right. Patch from Eric Gillespie.

Simplify dodir/addir/addfold (patch from Eric Gillespie)

bug #23319: rework the way we handle the working directory when invoking
the user's shell, so that we don't have issues when the cwd contains a
space or shell special character.

test/setup-test: use 'set -e' so we stop on compile failure.
Configure --enable-debug for convenience in debugging.

Add protective 'do { ... } while (0)' wrappers to multistatement macros

Bug #23436: fix some minor manpage syntax errors and add a test which runs
groff over them and checks that no warnings are emitted.

Allow tests to indicate that they have been skipped by printing their
own message and returning with exit status 120.

Ignore editor backup files

etc/replcomps, etc/replgroupcomps: Trim trailing whitespace from
Subject field (patch from Eric Gillespie)

When doing an if-test on the result of a function which returns a string
result, check whether the string is non-empty (as the documentation says we
do). Previously we were always testing the integer 'value'.

* test/setup-test: Run 'make clean' before building

* test/runtest: cat test-temp-dir only once.  Destroy and create
test Mail hierarchy here, so each test has its own independent
test data.

* test/setup-test: Drop needless pushd/popd, which don't exist in
sh anyway.  Don't use aclocal, which is part of automake.
Don't create Mail hierarchy here (see test/runtest).

* test/tests/folder/test-create, tests/folder/test-total: Drop
unused BINDIR.

* tests/pick/test-pick: Remove failing no-op test.

* test/tests/scan/test-scan: Write expected and actual output to
files and diff them.  This way, if the output differs, diff
provides the FAILing exit code and shows the differences.

* man/repl.man, etc/mhl.reply:
Add an attribution line when replying to messages.

Remove an accidentally committed file

When building on gcc, use noreturn attribute on adios and the various done
functions.

Back out previous, completely broken, change (due to a foulup I was
compiling a different tree to the one I was committing to CVS).

If we're compiling with gcc, mark adios() as never returning (better code
and fewer spurious 'uninitialized variable) warnings)

Update bit of docs about release announcements.

Don't use MB_CUR_MAX if we aren't compiled with multibyte support.

Factor out common code for writing ls shell command. Rework its length
checking so it doesn't rely on the return value of sprintf(), for the
benefit of SunOS 4.

Don't rely on realloc() to follow POSIX in handling a NULL pointer

Add some table size declarations for the benefit of elderly lexes with
small defaults (eg SunOS 4)

Update comment now we don't use lex -o.

* sbr/addrsbr.c: removed "err" from conditional, because gcc
correctly warned that it would always evaluate as true.  It's
the address of a static array.

don't use $< in target rules in makefiles, as POSIX says it's only defined
in inference rules

* configure.in, INSTALL: if --enable-masquerade is not
specified to configure, enable all supported masquerade forms.
This allows users to masquerade with the default
configuration. That seems to be more worthwhile now than
trying to prevent users from using it, especially with
single-user installations or those where a user can edit
etc/mts.conf.

bug #23167: sbr/ruserpass.c (ruserpass): make bad permissions on .netrc be
an instantly fatal error. Previously we returned an error value; however, no
caller was checking it. So now ruserpass() has a void return type.

* bug #23163: various minor fixes for the benefit of older Unixes
(specifically SunOS 4): reintroduce strerror() substitute implementation;
provide memmove() substitute implementation
* bug #23163: fix broken 'build outside source directory' feature
* bug #23162: sbr/dtime.c: fix stray HAVE_TM_GMTOFF that wasn't updated
to the new macro name.

Provide SASL_CB_AUTHNAME callback (avoids problem with wrong username
being sent in some situations)

Correct various places in smtp.c where the reply string might not have been
correctly NUL-terminated. Includes a fix for a particularly nasty and long
standing screwup where the buffer length counting in smhear() was totally
broken for continued lines from the server.

Fix SASL issue properly, by making sm_rrecord() and thus sm_hear() set
the length of the reply string correctly (the SASL libraries now care
if you pass in the wrong length).

Revert previous SASL fix (wrong approach)

Cope with sasl_decode64() returning SASL_CONTINUE as well as SASL_OK.

Add remark about how to update the homepage.

Update some of the 'how to do a release' documentation

Up version string to 1.3-dev for the CVS trunk.

Use hyphen, not plus (corresponding to previous RCx naming convention)

Bump version number, date for release of 1.3 RC1

move most remaining macros out of acconfig.h which is an obsolete feature of
autoconf

* bug #18655: fix use of admonish() for a fatal error (should
be adios(); only actual effect would be wrong exit code).
Thanks to Craig Leres for spotting this.

Bug #20028 (Debian bug 399271): fix code assuming that pointer
differences were 32 bits -- thanks to Dean Gaudet for the patch.

* uip/mhshowsbr.c (show_all_messages): Be more generous when parsing
    multipart messages.

Insert types for args of some function pointers to improve type checking.

Return type of (*done)() changed to void. default_done() replaced by
exit(). All dead code "return 1" lines removed from *_done()s and replaced
by explicit "return 1" following done() calls in main()s (they should
never be reached). This should make cleaning up program termination
structures easier.

Refactor cts pointer and free()ing on abnormal exit of associated list
into mhfree.c, eliminating duplication.

Change link-time overriding of done() to run-time reassigning of equivalent
(*done)(). This should make changing the return type of done and the
rationalisation of exit code easier.

  The MH web site is now rand-mh.sourceforge.net/ and the MH
  book is now at http://rand-mh.sourceforge.net/book/.

  Changed Questions

1.02 ...current version? mailutils 1.2
1.03 Where can I get MH? Debian 4.0 nmh 1.1-RC4, update links, mu 1.2
1.04 ...references for MH? Refer to SourceForge
1.06 How...print a MH manual? Refer to SourceForge
2.17 How...build MH on a Mac? Use fink
4.05 ...tools to archive MH? Mention swish++, mairix, and namazu

* Red Hat Bug #253342: inc.c, utils.c, utils.h: When inc is run with the
-silent flag, don't exit(1) for no apparent reason.

Rename functions to avoid clash with C library exp2().

Suppress 'uninitialised variable' warning.

On Linux, define _GNU_SOURCE so that GNU libc exposes the wcwidth()
prototype in wchar.h.

Change name of internal function to avoid potential conflicts with GNU
libc's getline() function.

Minor rearrangement so variables/functions used only if !HAVE_LIBLOCKFILE
are also declared only in that case.

For gcc, make the default CFLAGS include -Wno-pointer-sign if the compiler
recognises it. gcc 4 now by default produces a large volume of these
warnings, which I believe are probably spurious and definitely a lot of
work to go through and fix. So we suppress them for now so that the other
warnings are more easily visible.

buffer is compared with cp, and so needs to be changed to unsigned too.

Initial checkin of a testsuite

Change all chars being passed to the ctype macros (isalph(), etc.) to
be unsigned chars, following through to change any other types as
necessary (e.g. comparison between two pointers). Not all chars have
been changed to unsigned chars; just those that were in danger of being
sign extended when passed to the int-taking ctype macros.

We're not using the .Bu macro anymore.

Fix for bug #11139: mhl -form mhl.reply omits '> ' on some lines.

putcomp() was assuming only the first call on the body had the buffer
beginning at the start of a line.

Application of patch #5218: patch for bug #9742. Fixes a typo in the
size arg of snprintf(). mhlist output which used to have incorrect lines
such as
     1.2   multipart/mixed            772
     1.21  text/plain                   9
     1.22  multipart/mixed            162
now have the correct
     1.2   multipart/mixed            772
     1.2.1 text/plain                   9
     1.2.2 multipart/mixed            162

bug #18630, #18631, #18632, #18634: various patches from Craig Leres
fixing error message argument problems.

bug #15212: remove configure test for broken AT&T vi. This test was broken
and unfixable. Support for manually defining ATTVIBUG remains.

Add a missing header file

Changed bold "-all" to unbolded "all". It's a reserved msgs word, not a flag.

* uip/sendsbr.c: with attachformat values of 1 or 2, add
name= portion to Content-Type header.  This makes them
consistent with attachformat value 0.  And it allows mhstore
to use that (file)name when extracting attachments.

* configure.in: If we're not using dotlocking, there is no need to set inc
sgid.

Fixed bug introduced when changing from check_folder() to new
create_folder() function. The folder name needed to be expanded to an
absolute path first. Similar things may have happened with other
conversions.

Created pluspath function to check for '+' prefix and change flag for
path() appropriately. This eliminates some duplicate prefix processing,
but not all. The path functions need to be restructed; this is the first
step.

ssequal() does not check for a substring, but for a prefix, so added a
comment to this effect. If this is sufficient, the function body should
remain unchanged but the name and comments should be modified. If not,
then the function needs to be corrected.

fixes case where a very long character-class range (128 or
more characters) causes negative indexing into pattern string.

Fixed a typo.

Created function open_form() to open form file or fallback to default
components file, with error handling. Replaced duplicated code with call
to this function.

Fixed a couple of typos.

Sanitised some dodgy homedir finding code, but it still might not make
sense.

Get rid of some trivial warnings; unused variables, ambiguous if-else,
missing headers.

Make print_aka() and print_usr() static. They're not used outside ali.c.

Created app_msgarg() (append message arg) and a simple resizable array
struct to go with it that do almost exactly what the code they've replaced
was doing.
Replaced this (duplicated) code in many files with a call to app_msgarg().

Completing the .cvsignore list(s).