Added const to arg of m_backup().

Added const to first arg of adios().

If parse_mime() fails and the -file switch was used, pass the input
message to it unchanged so that pipelines won't break.

(cherry picked from commit 564b5b785c7cc709f92a83afb547267ed7418f18)

If parse_mime() fails and the -file switch was used, pass the input
message to it unchanged so that pipelines won't break.

Plugged leak in mh_oauth_cred_load().

Added free_fs() call to plug leak from fmt_compile().

Added free_fs() call to plug leak from fmt_compile().

spelling-fixes ( code-comments in .c, .h. .y ... )

(cherry picked from commit 942561da2455ee50a1eb82c8c4f2bdb5e25133d1)

spelling-fixes ( docs, man, readme )

(cherry picked from commit fa92642a21119eda8bfc961f8d5a8d3e9ee7d494)

spelling-fixes ( code-comments in .c, .h. .y ... )

spelling-fixes ( docs, man, readme )

Added sendfrom to docs/contrib.

Added sendfrom to docs/contrib.

Added docdir to mhparam.

(cherry picked from commit 89d69fbd2ae1f9c8c989850ec7057814c655a727)

Removed unnecessary memset on fmt_mnull:  it gets initialized to
all 0's.

(cherry picked from commit 56c4ac49d7b8ad674c0a66e9999186bcb98bd048)

Check to see if cross-directory links are supported; if they are not,
then don't run the refile -link test.

(cherry picked from commit 35fce21b692a6259138fc81caf13f4d07bb58da0)

Added docdir to mhparam.

Removed unnecessary memset on fmt_mnull:  it gets initialized to
all 0's.

Check to see if cross-directory links are supported; if they are not,
then don't run the refile -link test.

For slocal, changed default user, if -user switch isn't used, to be
the user part of the Local-Mailbox profile entry, if present, or the
current user.  addr uses the same default.  This allows addr
to contain a '.', which slocal did not allow before.

Added test of negative return code from jsmn_parse() to test-send.

Added test of empty authorization code to test-mhlogin.

Added explicit OATH_SUPPORT=1 to configure.ac when oauth support is
enabled, otherwise it didn't get set to 1.

Fix tests with oauth disabled.

Check for empty code.

Return false from parse_json() if jmsn_parse() returns a negative
(or zero, as it did before) value.

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.