Improve oauth autoconf support.

- By default, enable OAuth if curl is found.

- If OAuth requested (--with-oauth), error if curl not found.

- If OAuth disabled (--without-oauth), don't enable it even if found.

- Sanity-check curl-config --version output so we don't send anything crazy in
  our HTTP requests if curl-config changes.

Rename files to match renamed functions

Follow-up to commit 840b3f0e45a43b8378e88c66f004b2f186662e35

Fix mhlogin(1) title.

Added -user to mhlogin man page.

Added start_test() and finish_test() functions to test/common.sh.in.
See usage information in the (new) cleanup() function in that file.
This allow us to isolate test failures based on naming of individual
tests within a single test file.  As part of this, added set -e to
setup_test().  This required minor updates to test/new/basic and
test/pick/test-stderr to conform.  So, migrated test/new/test-basic
to start_test()/finish_test().

(cherry picked from commit cd6133dd23422f92534328f768a322f63a5c6dfe)

Fix to commit 84932d0a96c79bfb2f0384ebab806dc51a35f600 to
insert blank line between header and content for non-text
content.

(cherry picked from commit fe36a7ee2f4304fd3797443d4f2f950d6dc2475d)

Fix to commit 84932d0a96c79bfb2f0384ebab806dc51a35f600 to
insert blank line between header and content for non-text
content.

Quoted variables in trap so that distcheck doesn't trip over them if
not set.

Moved "const" to after "register" to silence compile warning fron gcc.

check curl_ctx->res_len > 0 not res_body != NULL

res_body is not a pointer and is never NULL

(Thanks clang!)

oops, pass int to m_getfld not size_t

support multiple oauth users; mhlogin now requires -user

plug some memory leaks, in success cases only

(staying clean at least on non-adios paths is not hard)

fix mhlogin -browser test

Rename and document functions that read from stdin.

- gans becomes read_switch
- getans becomes read_switch_multiword
- getans_via_readline becomes read_switch_multiword_via_readline
- getanswer becomes read_yes_or_no_if_tty
- install-mh.c geta becomes read_line

test mhlogin -browser

Don't call sm_auth_sasl() for XAUTH2 with -sasl.  Instead,
call only sm_auth_xoauth2().

Added start_test()/finish_test() to README.developers.

Migrated the oauth tests to start_test()/finish_test() so that
they don't have any output on success, but do report the particular
test on failure.

Added start_test() and finish_test() functions to test/common.sh.in.
See usage information in the (new) cleanup() function in that file.
This allow us to isolate test failures based on naming of individual
tests within a single test file.  As part of this, added set -e to
setup_test().  This required minor updates to test/new/basic and
test/pick/test-stderr to conform.  So, migrated test/new/test-basic
to start_test()/finish_test().

Added -browser switch to mhlogin(1).

Merge remote-tracking branch 'origin' into xoauth

Changed minchars swit values for -[no]concat switches to show(1)
from negative to 0, so that they can be abbreviated and so that they
appear with -help.  Added them to the man page, and fixed text that
was copied from mhshow(1) man page.

Use WIFXEXITED to check return status of system(), because
comparison with NOTOK wasn't valid.

Changed sign of minchars for the SASL-related switches from
negative to positive so the won't be hidden from help messages.
I think.

Merge remote-tracking branch 'origin' into xoauth

 merge is necessary,

Replaced -oauth service switch with -saslmech xoauth2
-authservice service, because OAUTH2 is a SASL mechanism.

Another pass at cleaning up (some of) the manpages.

This continues work I started a couple of years ago, but then got
derailed on.  There are two main goals behind these updates:

1) replace some low-level troff constructs with more modern
   man(7) facilities, and

2) adopt a more consistent visual style to the layout and
   formatting of the manpages, particularly in reference to
   typeset pages.

This time around I have updated README.manpages to describe the new
guidelines.  They are mostly non-intrusive, and serve to clean up
a lot of visual clutter that has crept in over the years.

Added check of return value of write() to quash compiler warning.

Recover properly if the character conversion fails.

Fix substitution code so it uses the right pointer for the utf-8 test.

Merge remote-tracking branch 'origin' into xoauth

Have fix_boundary() return NOTOK if it couldn't fix a boundary,
so that mhfixmsg won't modify the message.

Added oauth support to whatnow.

Added oauth switch to whatnow.

Added use of local variable user to prevent compiler warning
when built without OAuth2 support.

Sort JSON output from oauth tests so that comparisons don't
depend on it being in a specific order.  Also, clean up test
tmp files.

Set MH_VERSION, OAUTH_SUPPORT, and CURL_USER_AGENT in tests/common.sh.in
so that oauth tests can be run individually (not just via make check).

Added OAuth2 support to build_nmh.

Merge branch 'master' into xoauth

Have -nocrlflinebreaks apply to the message parts that -decodetypes
designates.

Added EXAMPLES and BUGS sections.

Implement the -reverse option to pick(1) to change the search order.

Updated date on mh-folders(5) man page.

Added application/ics to default list if types to decode.

mh-folders.man: note that folders should not be named numerically

Added -[no]crlflinebreaks switches to mhfixmsg(1).

Added check for successful decode of a different part with
attempted -decodetext of binary text to test-mhfixmsg.

Don't consider failure to build a text/plain part to be an error.
This allows mhfixmsg to succeed even if there isn't a suitable
mhfixmsg-format-text/subtype profile entry for every text part in
the message.

Added -decodetypes switch to mhfixmsg(1).

Add missing -infile switch name to mhfixmsg-format-text/calendar
rule in mhn.defaults.

mhical enhancement:  If a datetime is missing the time, assume
midnight.  If there is no end datetime (DTEND) and the DTSTART
doesn't contain a time, assume that the event is for the entire day.

Set MH to the resolved, absolute pathname inside context_read() so
that exec'd children can find it.

Added all of mhshow's documented switches to show's handling, on the
notion that nmh's pieces should know about each other.

Added mention of mhfixmsg -fixcte to error message for improper C-T-E in
multipart message.  Thanks to Conrad Hughes for suggesting this.

Moved addition of -ansi -pedantic to CFLAGS with gcc from
configure.ac to docs/contrib/build_nmh, so that they're not
included by default with builds by users.

Allow a RRULE in an iCalendar request to not have an INTERVAL on its
yearly FREQ, because INTERVAL defaults to 1.

Remove any enclosing quotes from a timezone identifier in an
iCalendar event request.  See comment added to format_datetime()
that says that I don't believe that RFC 5545 allows them to be
quoted.  But Oliver Kiddle found them in the wild.

If format_datetime() returns null, store a copy of node->value on c->c_text
to prevent a double free().

If str == buffer, then do NOT do a strncpy(buffer, str).  Many systems
this works, but on some systems it causes a SIGABRT.  POSIX says the
behavior on overlapping copies with str*cpy() is officially undefined.

Add support for setting the environment variable MH_TEST_NOCLEANUP to
prevent a test from cleaning up the test working directory.

Add support for the -sendmail flag to send/post, to support changing the
sendmail executable.

Unquoted addresses in mhshow and scan output.  Suggested by
Anthony Bentley.

Change the reference to the valgrind suppression file from $MH_OBJ_DIR to
$srcdir so "make distcheck" works.

Include a new valgrind warning suppression for MacOS X.

Make sure the valgrind suppression file is honored, and add some extra
suppressions needed when running under the Debian fakeroot environment.

Fix error message to correctly report the program that failed to execute
when doing HTML conversion.
Also, remove perl function prototypes as they are not considered best
practice.

Rearrange all callers of post(8) so that the filename is always last.

mhparse.c: verify all MIME-Version: headers encountered

in addition to verifying, silently ignore duplicate version headers.

Merge remote-tracking branch 'origin/convertargs'

use %l uniformly on all the text-mode browser display strings

otherwise behavior changes depending on which browser was present
when nmh is configured/installed.

Filter converted text/html to remove blank lines and leading space
that lynx inserts.  Also, changed ISO-8859-1 to US-ASCII because it
is, so it doesn't need to be converted.

Removed m_chkids(), per suggestion of Marcin Cieslak.

Removed unnecessary #include of h/signals.h.

Removed #include of signal.h because it's in h/signals.h.

Reset done in inc_done() to prevent endless loop when setegid()
fails (in which case adios() is called, which calls done()).

That didn't work, so removed offset.

Use +0000 isntead of -0000 offset because FreeBSD displays it that way.

Replaced setgid() calls with setegid() so that it works with dot
locking on FreeBSD.  setegid() should be supported on modern POSIX
systems.

Added showmimeproc to test/common.sh.in so that test-show will test
it in the workspace, not the user's installation.  Also added all of
the other procs defined in config/config.c that are currently unused,
just in case they some day are.

Removed a few more msh (MS_MSH and mshproc) and vmh (vmhproc) relics.

Replaced individual assignment of array elements, one of which was
out of range, with array initialization.

Fixed the error message in lockit() when it fails to create tmp file.

Fixed "inc" to be installed setgid mail with dot locking.  MAILGROUP=1
was not set in the config.h because we no longer use "LOCKTYPE"
configure variable.

update tests for new message separator header from mhshow

also added test names to tests, in any file i touched

update man pages for new mhshow message separator header

add new message separator header to mhshow

new switches to mhshow (-headerform and -[no]header) control whether
and what is displayed before the display of every message's content.
the default separator looks like "[ Message folder:msg ]".  the
implementation parallels the -markform feature.

since mhshow is often started by show, show now passes the -[no]header
options through to mhshow.  (also, show no longer attempts to pass
such options through to cat (in the case of -noshowproc)).

fix typo in mh-format.man

Test that show always shows message numbers.

show shows message numbers in two different formats when showing one
non-MIME, multiple non-MIME messages, or one MIME message.

show does not show any message numbers when showing multiple messages
any of which is MIME.

This is not a new bug, just one that has bothered me since I started
using mhshow with 1.6.

test/common.sh: allow giving individual tests a name

man pages -- clarify what -part and -type do when used together

Merge remote-tracking branch 'origin' into convertargs

Added support to mhfixmsg -reformat for multipart/related.  If it has
only a single part that is not text/plain and can be converted to
text/plain, a text/plain part is added and the type of the part is
changed to multipart/alternative.  If the multipart/related has more
than one part but does not have a text/plain part, try to add one.

Fixed test-convert to work on Cygwin and other platforms that
default to UTF-8 locale.

Merge remote-tracking branch 'origin' into convertargs

Removed unused local variable from move_preferred_part().

pending-release-notes: add mhshow's "-prefer", and mh-format's %(kibi/kilo)

test-subpart: add mhlist and mhshow tests for -type/-part/-prefer

update mhshow, mhlist, and mhstore man pages for -prefer switch

implement -prefer, for multipart/alternative preferences