Fixed chmod of temp file in prompter.  Noticed by valgrind.

Null-terminate buffer in mbx_copy() in dropsbr.c.  Noticed by
valgrind.

Added support for running the test suite with valgrind by setting
the NMH_VALGRIND environment variable.  To support that, nmh
programs that are invoked in the test suite should be run via
(the new function) run_prog or run_test.

Have configure print out what it detects for the program to detect
the MIME type string for a file.

Added "true" to end of flex fixups in configure.ac, in case the
user has hacked flex to not produce the errant code.

Removed flex fixup for version 2.5.37, it's not necessary with
flex 2.5.37 on Fedora 20.

Use --brief option, if supported, with file --mime.

In test-mhmail, insert semicolon in output of "file --mime" if needed.

file --mime on OpenBSD returns strings such as "text/plain
charset=us-ascii", without an intervening semilcolon.  Insert one if
there isn't one.

Removed support for file -i, it just spits out "regular file"
on Solaris.

Removed temporary printouts.

More debugging statements for OpenBSD.

Added debuging printouts.

Whooops, gcc on OpenBSD noticed a nasty in the last commit.

With whatnow attach, determine the content MIME type that's inserted
into the Content-Type header using file --mime or file -i, if
available on the platform at configuration time.  If not, continue to
use the mhshow-suffix- method.  Added a mimetypeproc component to
mhparam so that a user can query the configuration.

Updated date on a few man pages.

Removed temporary probes added in commit
3441b5648a0c21e067c05f428a51bf96352735e7.

Added temporary probes to see what file --mime and file -i do on
OpenBSD and Solaris.

Increased size of file command buffer in make_mime_composition_file_entry()
because it was a bit too small.

Cleaned up setting of CFLAGS in configure.ac.

Added POSTLINK to remainder of executables in Makefile.am.
Replaced hard-coded sbr/libmh.a with $(LDADD) in LDADD
macro for test/getfullname.

The buildbot was reporting "warning" on stdin, not stderr.
Escape "warning" so that it doesn't snag it.

The build failed mysteriously on the OpenBSD buildbot host ("Provide
an AUTOCONF_VERSION environment variable, please").  This was in the
compile, not autogen, phase.  Committing trivial change to kick off
another build.

That didn't quite work, try reducing size of warning filter
temporarily.

Filter out "often/almost always misused" warnings from OpenBSD linker.
Added POSTLINK macro to configure/Makefile for this purpose.

Added mention of filename completion at whatnow prompt if nmh
was configured with readline enabled.

Moved the set -x after the uname and string test in autogen.sh
to reduce build log clutter.

Generalized -Wel fix in test-manpages of commit
27659f0aa30c07081f7b8647c51a6ca82837e5d1 to apply on
any platform.

rand()/srand() are not cryptographically secure PRNGs.  Some systems
have the much better suited arc4random() family of functions; there's
no reason to not use it if it is available.  Make m_rand() just a
wrapper around arc4random_buf() in that case.  (There's no need to
ever seed it manually.)

As a bonus, silences some warnings on OpenBSD.

groff -man silence the test-manpages warnings.  One more try: -Wel.

Try using -man instead of -mandoc in test-manpages on OpenBSD
to see if it silences the warnings.

Disable assertions by default.

Minor autoconf style cleanup.

Minor changes to configure.ac:
1) Replaced direct manipulation of NDEBUG with AC_HEADER_ASSERT.
2) Removed unnecessary level of quoting around LFLAGS assignments.

Moved all LDFLAGS manipulations in configure.ac to one place.

Tweaked rcvtty.c to squelch warnings on OpenBSD.  Three of the
four were due to unused artifacts with HAVE_GETUTXENT not defined.
The fourth was due to control reaching the end of non-void
function message_id(), which had an _exit() call at the end.

Initialize local child_id in main() of whom.c to squelch warning
from cc on OpenBSD.

Squelch warning from cc on OpenBSD that fd argument of usr_pipe() in
slocal.c might get clobbered by vfork() by copying it to a volatile
local.

Removed --posix from awk invocation in test-anno.

It looks like the sed "a" in test-anno didn't emit a trailing
newline on OpenBSD.  It does on other platforms.  Try using
awk instead.

Add buildbot hacks for OpenBSD.

OpenBSD auto{conf,make} want a version specified in the environment
for each of the tools.  For now, hardwire this into autogen.sh until
we can figure out a better way of dealing with this.

Sun Studio cc -V must output to stderr.

Try 9efa994ecac2790dbc56113f78c464eed1371590 again.

Undid last commit of temporary printouts because buildbot is down.

Temporarily add some printouts to configure.ac to see why -v
isn't added to Sun Studio cc on Solaris build host.

Prefer cc over gcc.

One more attempt at getting the buildbot $PATH correct on Solaris.

OK, this should work to detect Sun Studio cc.

Oops, that didn't work.  Disable -v with SunStudio cc for now.

OK, use SunStudio cc, if present, on Solaris if user didn't set CC.
And with it, use -v and suppress the statement not reached warning.

Suppress E_STATEMENT_NOT_REACHED warning from SunStudio cc.

Added -errtags=yes to SunStudio cc options.

Removed a bunch of unreachable break statements found by
SunStudio cc -v.  Also fixed an "end-of-loop code not
reached" condition in uip/dropsbr.c.  That one is significant
because it meant that retries were not attempted when attempting
to open and lock a file.

Need full path to cc on Solaris buildbot host, at this point.

Try -v with SunStudio cc.  Also, try setting CC instead of modifying
AC_PROC_CC to pick up cc on the buildbot Solaris host.

Remove the Solaris cc hack; the buildbot should have the correct
cc in its $PATH now.

Removed +w from SunStudio cc, it caused compilation failure.
Maybe it can only be used with CC.

That worked.  Now try +w2 with SunStudio cc.

Of course that didn't work.  Try specifying the SunStudio
compiler in CC.

That didn't work.  Set PATH to pick up SunStudio cc.

Use SolarisStudio cc if available.

Added temporary probe to find cc on Solaris.

Removed temporary favor of cc over gcc.

Temporarily favor cc over gcc to try it out on Solaris.

Commit 9c663383ab54e0290781ac1c25d790ce03160e08 introduced
a backward incompatibility to build_nmh.  Refuse to support
-y if the current nmh installation has an mhparam that doesn't
support the sasl and tls components.

Clean up comments, and update information on file locking.

Removed dependence of build_nmh on ldd by adding sasl and tls
components to mhparam(1).

Fix one of the POP tests to work on wide terminals.

Removed temporary diagnostic to determine gcc version on Solaris
buildbot host.  It's 4.5.2.  4.6.0 and later return an error
status with an unrecognized option, which we rely on for the test
for -Qunused-arguments.  4.5.2 doesn't.  But that option doesn't
produce any errors, just warnings that don't contain "warning".
So ignore it.

Added temporary diagnostic to see what version of gcc is
used on the buildbot Solaris host.

With the fix of 4929cad323afc87404752a1e88664a7f42cbb737,
prepare_space() no longer needs to insert initial space.

Try this sed command for squeeze_lines().

Added -eval 'set document.browse.margin_width = 0' to elinks
command in mhfixmsg-format-text/html in mhn.defaults, to disable
indentation of converted text.  Thanks to Rickard Carlsson for
this suggestion.

That didn't work, so try this awk command from Robert Elz.

Fixed squeeze_lines() test function to be compatible with POSIX awk.

Fixed test-mhfixmsg to work with elinks, which inserted leading
space on a line.  Thanks to Valdis for reporting that and to
Ralph for the awk and sed magic to emulate diff -w (and cat -s).

Added notes to README.developers about write access to
the git repo and git rebase.

Replaced hard-coded use of "," with `mhparam sbackup` in
test-header-encode.

Disable clang complaint about unused -ansi when linking by
adding -Qunused-arguments to LDFLAGS, if supported.

Removed bash-isms from inc/test-deb359167.

Skip inc/test-deb359167 on FreeBSD >= 10 because its valgrind
throws a strange error.  Code provided by Lyndon.

test/common.sh.in: use awk to pick off first output field.

md5 and md5sum are inconsistent in their use of SP vs HT as a field separator.
cut(1) can only handle a single field delimiter character; awk is more forgiving.

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

Renamed test/getfqdn to test/getcanon.

I give up for now; revert these debugging commits.

One final check to make sure the binary isn't stripped

Sigh. Make sure the binaries aren't stripped as well.

Switch to default of -g so hopefully we can get something useful from the
FreeBSD buildbot valgrind output.

Sigh, I always forget this is errno.h, not sys/errno.h.

Have getfqdn fall back to printing out the given hostname if the DNS lookup
fails, to replicate the behavior of LocalName().

That didn't work, either.  Restoring configure.ac.

That didn't work, try adding -pedantic-errors.

The linker on FreeBSD 10 doesn't like -ansi.  So added a
configure check to not put -ansi in CFLAGS if the linker
doesn't like it.  Though may need to add -pedantic-errors.
There doesn't seem to be an easy way to get automake to
not put CFLAGS in the link command.

Clean up mhbuild backup files left by test-header-encode.

Better fix than commit 605d80e7cf0d02d4bab9387f03b4e1d43177e33e
for test-mhfixmsg:  always overwrite HOME environment variable
if mhfixmsg uses w3m.

Fix to commit 7e40862e6a4da7810a46aa1e28b05e527fdc208a:
only one of the variables should have ICONV_CONST.

In test-mhfixmsg, make sure that $HOME is writeable, in case
nmh has been configured to use w3m.

Really fixed mhmail.

Removed temporary hack.

Removed temporary hack to enable debug from configure.ac.