Enable %a format string for all pgp command configs. (closes #3699)

Currently, the documentation states that, with the exception of
pgp_getkeys_command, all the PGP format commands should be available to
the pgp_*_command configuration variables.

However, the %a format is actually only enabled for
pgp_clearsign_command, pgp_encrypt_sign_command, and pgp_sign_command.

Philip Rinn discovered this problem when he was trying to add a
"--encrypt-to %a" to pgp_encrypt_only_command.  Rather than just fix
the one configuration he was having the problem with, this patch
enables %a usage for all of them. (With the exception of the documented
pgp_getkeys_command.)

Note correct usage of $edit_headers. (closes #3673)

Ticket 3673 shows that it isn't necessarily obvious a blank
line is needed between the headers and body of a message when using
$edit_headers.  This patch adds a small note to the documentation,
indicating the blank line should be preserved between the headers and
body.

Thank you Sinan Kaan Yerli for the patch.

Clarify PGPPATH usage in manpages.

Make more clear that, when used with the original PGP program, mutt and
pgpring rely on this being set.

Fix a memory leak in mutt_query_complete.

When a single result was returned, it was written to the buffer and
returned, but the query result was never freed.

This patch creates a free_query function and changes the code to use
that everywhere.

Remove unused variable t in start_debug().

t was set but never used.  This was generating a compiler warning.

Fix variable name in configure.ac.

$ac_cv_search_STRINGPREP_CHECK_VERSION should be
$ac_cv_search_stringprep_check_version, to match the first parameter of
the AC_SEARCH_LIBS([stringprep_check_version] above.

Running configure was giving a "test: =: unary operator expected" error.

Remove duplicate SmimeCryptAlg declaration in globals.h

Thank you Elias Diem for the patch.

Disable SSLv3 by default.

Since Oct. 14 2014 SSLv3 is no longer considered safe. See POODLE:
https://en.wikipedia.org/wiki/Transport_Layer_Security#POODLE_attack

hcache tcdb debug logging

add check-syntax target for flymake

ignore generated pgpewrap/pgpring man pages

ignore gnu global tags

merge stable

mutt_sasl: fix double negative in iptostring result check (fixes #3705)

mutt_sasl: whitespace only

merge stable

Fix deprecated gnutls_protocol_set_priority.  (closes #3515)

This patch is based off of the patch at
http://dev.mutt.org/trac/attachment/ticket/3515/624058-gnutls-deprecated-set-priority.patch

The mutt source has changed since then, with more TLS options being
added.  This patch therefore uses a slightly different strategy for generating the
priority string.

The patch also adds autoconf support, as requested by brendan at
http://dev.mutt.org/trac/ticket/3515#comment:1

Updated French translation.

Updated Catalan translation.

Updated Ukrainian translation.

Updated Russian translation.

fix typo in man page.

Updated Czech translation

mutt-1.5.23 signed

Added tag mutt-1-5-23-rel for changeset b8a63730c848

automatic post-release commit for mutt-1.5.23

build-release: make sure version is picked up by autoconf

mkchangelog: only show log entries on current branch

No more devel

add user visible changes since 1.5.22 release.

Fix buffer overrun caused by not updating a string length after address expansion.

Updated Dutch translation

Merge stable into default

remove trailing period from documentation for $pgp_list_secring_command

remove trailing period from documentation for $pgp_list_pubring_command

merge changesets from stable branch

Merge stable into default

Move HEAD to default

create stable branch

add note about use of ! ~Q in send-hook when paired with reply-hook.

remove <stdbool.h> and use "1" instead of "true".  closes #3657

see #3642 for the change that caused this bug.

update UPDATING with 1.5.22 release date

mutt-1.5.22 signed

Added tag mutt-1-5-22-rel for changeset 8c5ea668e7e0

automatic post-release commit for mutt-1.5.22

Rework logic to avoid advancing list pointer inside the for loop.

closes #3531

updated Catalan translation

updated Ukranian translation

updated Japanese translation

add missing files that were supposed to be included with [76f8df0ac80396559aa80e7f145be6f235a6db51]

adds some missing options in doc and adds man pages for binaries pgpewrap and pgpring.

closes #3643

Problems with muttrc.5:

Ambiguous or invalid backslash.  This doesn't cause groff a problem.
but it confuses doclifter and may confuse older troff implementations.

Unknown or invalid macro.  That is, one that does not fit in the
macro set that the man page seems to be using.  This is a serious
error; it often means part of your text is being lost or rendered
incorrectly.

Use local definitions of .EX/.EE or .DS/.DE to avoid low-level troff
requests in the page body.  There are plans to add these to groff man;
in the interim, this patch adds a compatible definition to your page.

Problems with mutt.1:

Ambiguous or invalid backslash.  This doesn't cause groff a problem.
but it confuses doclifter and may confuse older troff implementations.

Spelling error or typo.

include <stdbool.h> for definition 'true'

closes #3642

Fix segfault when viewing text attachments in compose menu.  (closes #3644)

The segfault was introduced in changeset b9f9e3147eb4.  Since decoding
and charset conversion aren't needed for attachments when composing a
message, this patch reverts to just using mutt_save_attachment() to view
"raw data" for text attachments in the compose/send case.

This patch is based on Michael Elkins' patch at
http://dev.mutt.org/trac/attachment/ticket/3644/view_attach_compose_segfault
with just a missing return value check added.

Ensure that BUFFER variables are initialized to prevent later attempts
to traverse an uninitialized pointer.

Consistently use mutt_buffer_init function

Despite the existence of the mutt_buffer_init function, most
initializations were instead done with memset.  Use the specific
function instead to make it easier if later changes are made to how the
initialization should be done.

When recalling a postponed message that does not have a x-mutt-fcc header
field, generate a default FCC just as we do for new messages.  If x-mutt-fcc is
present but empty, we assume that means the user does not want a copy.

closes #3653

Ensure that at least one signature was generated to catch the case where $pgp_sign_as is unset and no default key is specified in ~/.gnupg/gpg.conf

closes #3658

use LOFF_T as the type for variables used in fseeko()

closes #3487

Display the KeyID when the pubkey is not found in the user's keyring during signature verification.

closes #3654

return 0 if import of cert failed to match other error conditions.

only terminate the write-half of the TLS connection to avoid hanging if the remote doesn't respond.

RFC5246 section 7.2.4 states that the initiator of the closure need not wait for the remote to response before closing the read-half.

closes #3633

fix more comment typos

fix typos in comments

Fix postpone/resume to not remove a Mail-Followup-To header  (closes #3070)

This solution changes mutt_prepare_template() to check whether the
message_id field is NULL to decide whether to wipe the message-id and
mail-followup-to headers when instantiating the message.

If we are resending a message, we don't want the previous message-id
and mail-followup-to headers.  If we are resuming a postponed message,
however, we want to keep the mail-followup-to header if any was set
before the postpone.
(grafted from d7d83298011a6e7aa31bde49e0b6b21209a85fd2)

version.sh: step back from $() to ``.  closes #3661
(grafted from 01cf5814dfe5e79b65b1d950aab1276d19a9f031)

Allow filename prompt when saving multiple attachments to a directory. (closes #3083)

Currently, if you specify a directory when saving multiple attachments
and choose (a)ll, the first attachment is saved without confirming the
filename.  Subsequent attachments prompt for the filename.

Fix postpone/resume to not remove a Mail-Followup-To header  (closes #3070)

This solution changes mutt_prepare_template() to check whether the
message_id field is NULL to decide whether to wipe the message-id and
mail-followup-to headers when instantiating the message.

If we are resending a message, we don't want the previous message-id
and mail-followup-to headers.  If we are resuming a postponed message,
however, we want to keep the mail-followup-to header if any was set
before the postpone.

UPDATING blurb for terminal status features

version.sh: step back from $() to ``.  closes #3661

Terminal status line support, based on the xterm title patch.

Christoph Berg took the xterm title patch from the mutt mailing list and
maintained it for Debian:
* Changes made:
  - 2007-01-27 myon: using %P caused a segfault, updated status.c to catch
    menu==NULL.
  - 2007-02-20 myon: make the note about the xterm_set_titles defaults a
    comment.
  - 2008-08-02 myon: move set_xterm_* prototypes into the proper header file
    (cleaner code, no functional change, evades conflict with sidebar patch)

This update reworks the Debian version as generic support for tsl/fsl
terminfo capability.  It no longer depends on a static list of supported
terminal types, although it will use default support for these terminal
types *if* they do not advertise tsl/fsl or XT in terminfo.

The generic terminal status line can be implemented in any fashion, and
on hardware terminals it can be a special out-of-screen region of the
display.  Most modern terminal emulators appear to emulate the status
line in the window title; thus the notion of the tsl escape as an xterm
window title escape.

Configuration variables have been renamed:
$ts_status_format
    format string (using $status_format codes) for the terminal status line
$ts_icon_format
    format string (using $status_format codes) for the xterm icon name
$ts_enabled
    toggle for whether to issue status updates

The old configuration names exist as synonyms.

Logic flow:
* If tsl (string) is set in terminfo, mutt will use that escape to switch
  to the status area, and will use fsl to switch back to the regular
  display area.
* Otherwise, if XT (boolean) is set in terminfo, mutt will use the
  standard xterm-compatible escapes.
* Otherwise, if $TERM matches a known terminal type, mutt will use the
  standard xterm-compatible escapes.
* Otherwise, this feature is disabled.

= NOTE =

The XTerm icon escape sequence is not formalized in terminfo, so the
egregious kludge of hardcoding the escape is necessary in this case.
However, some terminal emulators (e.g. MacOS X Terminal.app) will set
the status line (window title) when the icon sequence is received.
Since there is no way to detect this behavior, the only solution is to
write your configuration to take it into account.  Rather than add a
variable to control icon escapes separately, we just provide the advice:
if you use such a terminal, you may wish to set ts_icon_format to an
empty string.  This will cause it not to be used.

Don't exit pager if quit=ask is answered "no".

This patch addresses ticket 2788.

Currently if you quit ('Q') from inside the pager, with quit=ask-yes
or quit=ask-no, and you respond "no", mutt leaves you back in the index
menu.

This patch changes it so that mutt does not exit the pager.

backout c1371176ea45

Backed out changeset 1142ed8974fa

use gethostname() to determine the system host name

use getaddrinfo() to look up canonical DNS name, and fall back to hinting from /etc/resolv.conf

see #3298

fix various compiler warnings; most were due to unchecked return values from system calls.

use mkdtemp() to create temporary directory rather than mktemp() followed by mkdir()

closes #3637

use of sscanf() had undefined behavior, replace with simple parsing instead

see #3636

increase string size by one byte to accomodate the terminating null character in fscanf() since the maximum field width does not include it

closes #3636

hat tip: Will Fiveash

fix potential buffer overflow in off-by-one bounds checking error

closes #3635

explicitly call mutt_refresh() prior to endwin() to force a screen redraw.

[4903e80356d3] removed the call, but some environments (screen+xterm under
SuSE11/12) don't properly flush without it prior to the endwin() call

do not use mutt_message() to display prompt in mutt_yesorno().

properly calculate screen width of prompt string by using mutt_strwidth() rather than strlen()

use safe_asprintf() return the formatted yes/no string since specifying COLS+1 is wrong when dealing with multibyte characters

closes #3412
closes #3352

add support for $mail_check_recent to mh mailbox driver

closes #3629

do not explicitly reset the terminal attributes and move the cursor in mutt_endwin().  endwin() itself should handle this.

closes #3627

remove PKG_CHECK_MODULE() check and always use AC_CHECK_HEADERS()

only check for <idn/idna.h> when <idna.h> isn't found

closes #3626

use pkg-config to look for libidn if available.

checking <idn/idna.h> rather than including -I/usr/include/idn for Solaris 11.

closes #3624

backout [392e945dfba7]

closes #3293
see #3234

when falling back to viewing an attachment as text in the internal pager, perform charset conversion

this change is meant to replace [392e945dfba7]

see #3234
see #3293

use a separate flag for mutt_hcache_store() to signal that the gettimeofday() value should be substituted, so that a 0 uidvalidity can be used by the imap driver.

closes #3501

change semantics of mutt_is_autoview() to return 1 if there is a matching mailcap entry and either $implicit_autoview is set or the MIME type matches the user's auto_view list.

closes #3496
closes #3426

detect pipermail-style obscured return path in mbox From lines

see #1200

allow use of sync-mailbox when the current limit has no visible messages

closes #3538

avoid debug warning about trailing whitespace in a base64 encoded attachment

closes #3558

When "make install" fails to chgrp or chmod the mutt_dotlock binary, add the reason why we are doing this so the user knows what to look for.

Fix regression introduced by [10cc06eff52b] where the CLEARLINE() removal was not replaced by a move() to position the cursor at the correct location to draw the status bar.

closes #3619

Remove unused variable `matched'.

Fix memory leak when multiple x-label or supersedes header fields exist in the header.

In order to keep the current behavior where the value of the last header field present is the one used, free any previous value.

Backed out changeset bf84f24e512d

display +/-NCURSES_WIDECHAR when displaying ncurses version to show whether we are linked against -lncurses or -lncursesw