configure: drop --enable-nntp

Make NNTP feature built-in.

configure: drop --enable-pop

Make pop feature built-in.

configure: drop need_socket

Socket files are now always integrated in the build.

configure: drop need_imap

Imap being built-in, body cache is too.

configure: need_md5 is now always built-in

md5 was dependent on other features to be built or not. Now that imap is
built-in, we will always need md5 as built-in as well.

configure: drop --enable-imap

Make imap feature built-in.

configure: drop --enable-compressed

Make compression feature built-in.

build: default to silent make rules

build: add separate hcache dir

build: make dotlock optional

add %z format strings to index_format

The %Z format string consists of three columns.
Split those up into individual columns to allow more fine grained
control over which flags should be displayed.
%zs message status flags
%zc message crypto flags
%zt message tag flags

There's a small difference compared to %Z which is that the delete flags
have been moved from the crypto column to the status column. %Z will not
be removed for backwards compatibility reasons.

Signed-off-by: Stefan Assmann <sassmann@kpanic.de>

Add debug_level/debug_file options

This change allows to change debug_level and debug_file on the fly.

It adds the -l mutt arguments to pass the logging file on startup.

debug options passed as mutt arguments always override the configuration
file.

Closes #445

merge: Warn on bindkey aliasing

 * Warn on bindkey aliasing
 * Create source warnings for binding aliases
 * Detect missing function mapping
 * fix bind error messages

fix bind error messages

Some messages were displayed that shouldn't have been (when bind
succeeded).  Some errors weren't getting reported (they were overwritten
by other messages).

Detect missing function mapping

Move the missing function detection code into the `try_bind` function.
Improve the error message to include the menu name.

Create source warnings for binding aliases

This is somewhat hack-y, but it makes it possible to have the binding
alias trigger source warnings with file name and location.

Warn on bindkey aliasing

On binding a key sequence that alias an existing mapping, warn the user.

merge: configure.ac improvements

 * remove check on unix.h
 * remove check on stdarg.h
 * remove check on inline and const
 * remove check on wide chars
 * remove check on fgetpos
 * remove check on memmove
 * remove check on strerror
 * remove check on srand48
 * remove check on long long int
 * signal: remove test on SYS_SIGLIST_DECLARED
 * simplify sig_atomic_t check
 * remove check for <dirent.h> d_ino
 * drop LANGINFO checks
 * remove stuff defined in POSIX:2001
 * use basic getopt from unistd.h
 * drop HAVE_UNISTD_H
 * unused prototypes - unsupported systems
 * sys/types.h
 * sys/select.h
 * inttypes.h includes stdint.h
 * Drop obsolete iconv check
 * Drop another obsolete iconv check
 * tidy md5

tidy md5

No code changes.

This code is imported from the GNU Library.
It contained a lot of definitions and compatability code that we don't
need.  Also, tidy some comment blocks.

@gahr
Leverage on AC_C_BIGENDIAN instead of glibc's endian.h

Drop another obsolete iconv check

Don't check for a bug in 18 year-old packages

Drop obsolete iconv check

Don't check for a bug in 18 year-old packages

inttypes.h includes stdint.h

<inttypes.h> is part of posix:2001
It includes <stdint.h>

sys/select.h

<sys/select.h> is part of posix:2001

sys/types.h

<sys/types.h> is part of posix:2001

unused prototypes - unsupported systems

Drop compatibility for unsupported platforms

drop HAVE_UNISTD_H

<unistd.h> is part of posix:2001

use basic getopt from unistd.h

getopt() is defined in <unistd.h>
<getopt.h> would only be needed if we used getopt_long()

remove stuff defined in POSIX:2001

Headers:
    locale.h
    sys/time.h
    sys/resource.h
    sys/select.h

Header checks:
    HAVE_FTRUNCATE
    HAVE_INTTYPES_H
    HAVE_ISWBLANK
    HAVE_LOCALE_H
    HAVE_SETRLIMIT
    HAVE_SYS_RESOURCE_H
    HAVE_SYS_SELECT_H
    HAVE_SYS_TIME_H

Functions:
    getopt
    getsid
    setegid
    setenv
    setrlimit
    strcasecmp
    strdup
    strftime
    strncasecmp
    strtok_r
    fseeko

Types:
    pid_t
    ssize_t
    socklen_t

drop LANGINFO checks

posix:2001 defines nl_langinfo in <langinfo.h>

All the constants we use are also part of posix:2001
CODESET, D_T_FMT, YESEXPR, NOEXPR

remove check for <dirent.h> d_ino

posix:2001 declares 'd_ino' in <dirent.h>

simplify sig_atomic_t check

posix:2001 declares 'sig_atomic_t' in signal.h
We only need to check if it's declared with the 'volatile' symbol.
Streamline the autoconf checking.

signal: remove test on SYS_SIGLIST_DECLARED

AC_DECL_SYS_SIGLIST check was removed in:

  7ce41d94e: update configure.ac with autoupdate

So testing SYS_SIGLIST_DECLARED doesn't make sense any more, it has been
replaced by HAVE_DECL_SYS_SIGLIST.

remove check on long long int

This is C99 standard, and nobody relies on the LONGLONG symbol anyway

remove check on srand48

Code doesn't refer to srand48

remove check on strerror

Part of C99

remove check on memmove

Part of C99

remove check on fgetpos

Part of C99

remove check on wide chars

These types (wchar_t, wint_t, mbstate_t) and functions (isw* and tow*)
are part of C99 standard.

remove check on inline and const

AC_C_CONST is marked obsolescent and inline is already checked by
AC_PROG_CC_C99, so it's redundant.

remove check on stdarg.h

As stated in Autotools Manual, AC_HEADER_STDC already checks for
stdarg.h presence, so double checking is useless.

remove check on unix.h

This header is not used anymore, this check can be dropped.

build: fix dist, distclean targets

'make dist' didn't install the 'lua' and 'hcache-bench' dirs.
'make distclean' deleted 'po/mutt.pot' and 'doc/neomutt-syntax.vim'

Fixes #566

build: change gpgme requirement back to 1.1.0

RHEL6 is still using GPGME 1.1.8, which doesn't have the function
`gpgme_op_export_keys`.  Revert the GPGME requirement to 1.1.0 and put
back the #ifdef's around the missing function.

build: remove check_sec.sh

This script looked for mistakes when using mutt functions such as
safe_free().  It was crude.  Coverity can do a better job without us
having to litter the code with /* __FREE_CHECKED__ */ comments.

configure: make it more consistent

Most of the diff is reworking how notmuch is built. It used an extra
AM_CONDITIONAL but was the only component to do so, so change that to
make it consistent with the rest of the configure.

Lua picked the same thing but BUILD_LUA is actually never used, so it
could be dropped too.

Rest of the commit cleans up some pollution between MUTTLIBS and LIBS.

remove some obsolete HAVE_ tests

Delete tests against some defines because there is nothing in configure
to generate those defines:

     HAVE_ALLOCA_H
     HAVE_GNUTLS_OPENSSL_H
     HAVE_PGP
     HAVE_REGCOMP
     HAVE_SMIME

They are like `#if 0`, only harder to spot!

clang-format: Insert spaces inside braces

Adding the option 'Cpp11BracedListStyle: false' means that struct data
in braces has some whitespace padding.

const char *missing[3] = {"brown", "lightgray", "default"};

const char *missing[3] = { "brown", "lightgray", "default" };

clang-format: adjust layout of structs

Some structs are packed by clang-format, leaving the closing brace
n the same line as the data:

char *known[] = {"color-xterm", "cygwin", "eterm",  "kterm", "nxterm",
                 "putty",       "rxvt",   "screen", "xterm", NULL};

By inserting a comma after the last data item (allowed by C99),
clang-format produces a more pleasant-looking layout:

char *known[] = {
    "color-xterm", "cygwin", "eterm",  "kterm", "nxterm",
    "putty",       "rxvt",   "screen", "xterm", NULL,
};

translations: remove unused (f)orget option

This undocumented option interfered with the order of the translations.

merge: cleanup of lots of docs

doc: remove feature muttrc files

are duplicates of manual muttrc sections

merge README.notmuch into manual

remove unneded scripts

* the functionality of the mutt_xtitle script already covered by
  https://www.neomutt.org/guide/reference#ts-status-format (according to
  @flatcap)

[ci skip]

remove README.SECURITY

content is already covered inside
https://www.neomutt.org/guide/mimesupport.html#secure-mailcap

[ci skip]

remove BEWARE and devel-notes.txt

* For the inclusion of the BEWARE/devel-notes.txt file on the website,
  please see: neomutt/neomutt.github.io#33/

[ci skip]

update Makefiles

delete TODO files

see https://github.com/neomutt/neomutt/milestone/9

remove legacy files

* README (infos for mutt 1.5) doesn't contain useful infos
* NEWS (infos about mutt 1.2) is too legacy, neomutt started with mutt 1.5

don't generate vim-neomutt syntax file

doc: remove LaTeX/pdf manual generation

remove --enable-sidebar and make it persistent

It was decided to keep the USE_SIDEBAR tests in the codebase to make it
easier to find code related to that feature, but that requires
USE_SIDEBAR to be defined, or it breaks compilation because some symbol
definitions depend on that config option.

github: improve PR template and CONTRIBUTING.md

PR template:

* made structural questions bold to contrast it from the actual
  PR description.

* add note that acceptance criteria can be removed when the PR is
  submitted. Its only meant as a reminder.

* fixed manual link.

CONTRIBUTING.md:

* wrap text at 80 characters.

* add note to mention specific methods or files if the commit only
  changes things there.

[ci skip]

NeoMutt Release 2017-04-28

Fix km_error_key() infinite loop and unget buffer pollution.

'bind pager \Ch help' produces an infinite loop when an unbound key is
pressed in the pager.  The reason is because km_error_key() tries to
verify that the key sequence is really bound to the OP_HELP operation.
It does this by using km_expand_key(), tokenize_unget_string() on the
resulting buffer, then checking if the next km_dokey() returns OP_HELP.

The problem is that km_expand_key() does not always produce a string
that is properly reparsed by tokenize_unget_string().  Control-h
sequences are expanded to ^H.  tokenize_unget_string() recognizes this
as two characters '^' and 'H'.  km_error_key() checks the OP returned,
which is OP_PAGER_TOP for the '^'.  This is not OP_HELP, so it prints
a generic error and returns.  This leaves the 'H' in the input buffer!
Since 'H' (by default) is unbound in the pager, it retriggers
km_error_key(), resulting in an infinite loop.

The same issues can occur without control sequences:
  bind generic ? noop
  bind generic dq help
In the index, hitting an unbound key will end up leaving 'q' in the unget
buffer, because 'd' is bound in the index menu and will be read by km_dokey().

A simple approach to fix this would be to just use the same code as in
mutt_make_help(), which has no double-check.  This would be no worse
than the help menu, but can generate an inaccurate error message (e.g
if '?' were bound to noop)

This patch instead uses OP_END_COND as a barrier in the unget buffer.
It directly inserts the keys in the OP_HELP keymap, instead of using
km_expand_key() + tokenize_unget_string().  After calling km_dokey()
it flushes the unget buffer to the OP_END_COND barrier.

Thanks to Walter Alejandro Iglesias for reporting the bug.

Fix error message when opening a mailbox with no read permission. (closes #3934)

ctx->mx_ops ends up being NULL for both the case that ctx->magic is 0
and -1.  This meant the mutt_perror() error message was never being
printed, because the check for ctx->mx_ops == NULL was taking place
first.

Move the "ctx->magic == -1" check first, so mutt will print out an
appropriate perror message in that case.

docs: fix remaining TLS typos

Fix typo in README.neomutt (#548)

Fix and simplify handling of GPGME in configure.ac (#545)

* Fix
When GPGME is not installed, the m4 macro AM_PATH_GPGME is not defined,
which causes autoconf to fail. The fix adds an `ifdef` around the block
to skip expanding the macro, if it's not defined.

* Enhancement
The enhancement is to assume that we have at leat GPGME 1.2.0 (released
in 2009). This allows us to simplify the configure.ac code block by
avoiding checks for functionalities we know to be there.

Fixes #542

NeoMutt Release 2017-04-21

sync translation to upstream changes

merge: upstream release 1.8.2

 * automatic post-release commit for mutt-1.8.2
 * Added tag mutt-1-8-2-rel for changeset c6ea4aed6bec
 * mutt-1.8.2 signed

automatic post-release commit for mutt-1.8.2

fix: various minor fixes

- nroff typo in docs
- fix function prototype
- add spaces inside comments
- add missing name to mailmap

build: add keymap_alldefs.h to BUILT_SOURCES

keymap_alldefs.h needs to be generated by the main build so that
targets, "update-po", "dist" and "distcheck" succeed.

build: lower the gettext requirement 0.18 -> 0.17

RHEL6 only has gettext 0.17.

fix broken from_chars behaviour

Upstream adopted and extended the from_chars feature.
A couple of changes to the behaviour were introduced by the change:
- '\r' wasn't being recognised
- An unwanted space was being displayed

Fixes #533

mutt_envlist_set: change 1/0 int parameter to boolean (#537)

Remove -Iimap from CFLAGS and include imap/imap.h explicitely (#536)

fix decode of encrypted mail

This one-char typo caused mutt to lock up when reading encrypted email.

merge stable

mutt-1.8.2 signed

Added tag mutt-1-8-2-rel for changeset c6ea4aed6bec

automatic post-release commit for mutt-1.8.2

Fix GPG_TTY to be added to envlist. (closes #3931)

Changeset 37209157e33c converted filters to use the envlist.
Unfortunately, I missed that pgp.c sets GPG_TTY when using the GnuPG
agent.  Convert to add GPG_TTY to the envlist too.

Revert sort prompt labels. (see #3930)

Using the (s)ort style may look a bit better, but it makes the prompt
over 80 columns.  The multichoice prompt supports multiple lines now,
but it is better to have it fit on one line if possible.

Revert back to the "capital letter" method, but leave the ticket open,
to explore other ideas that Vincent and chdiza have for better ways to
present a long choice like this.

merge stable

Fix GPG_TTY to be added to envlist. (closes #3931)

Changeset 37209157e33c converted filters to use the envlist.
Unfortunately, I missed that pgp.c sets GPG_TTY when using the GnuPG
agent.  Convert to add GPG_TTY to the envlist too.

merge: upstream fixes (mutt/default)

 * Add ifdefs around new mutt_resize_screen calls.
 * Add multiline and sigwinch handling to mutt_multi_choice. (see #3877)
 * Set pager's REDRAW_SIGWINCH when reflowing windows.
 * Add multiline and sigwinch handling to mutt_yesorno. (closes #3877)
 * Change the sort prompt to use (s)ort style prompts.
 * Handle the pager sort prompt inside the pager.

Handle the pager sort prompt inside the pager.

Display the prompt in the pager, so a multiline prompt or resize
doesn't refresh the index menu.  Then, bounce back through the index
to handle resorting and status line updates.

This also fixes cancelling to stay in the pager.

Change the sort prompt to use (s)ort style prompts.

This is consistent with the rest of mutt, and in my opinion, more readable.

Add multiline and sigwinch handling to mutt_yesorno. (closes #3877)

Most of the yes/no and query_quadoption prompts are pretty short, but
for completeness add handling for those too.

Set pager's REDRAW_SIGWINCH when reflowing windows.

So that all external reflow handling functions don't have to remember
to set the flag too.

Add multiline and sigwinch handling to mutt_multi_choice. (see #3877)

Resize the message window up to three lines to fix wide prompts.

Enable sigwinch processing and redraw the current menu as needed.

Add ifdefs around new mutt_resize_screen calls.

Changeset 231fa2eff206 added sigwinch handling to _mutt_enter_string()
but neglected to add an ifdef check.

build: fix distcheck

Fix the location of keymap_alldefs.h so the translations can be created.

Handle the pager sort prompt inside the pager.

Display the prompt in the pager, so a multiline prompt or resize
doesn't refresh the index menu.  Then, bounce back through the index
to handle resorting and status line updates.

This also fixes cancelling to stay in the pager.

Change the sort prompt to use (s)ort style prompts.

This is consistent with the rest of mutt, and in my opinion, more readable.

resync translations

Some translations got lost in the move.
This commit restores them and updates all the file references.

build: restore po/mutt.pot

This file is autogenerated from the the files listed in po/POTFILES.in
However, if it's missing it causes the build to regenerate ALL the .po
files every time there's a change to the code.  This is annoying.

If you want to update the .po files, try:
    make -C po update-po

fix: build of imap/auth_sasl.c

Header file <sasl/sasl.h> doesn't seem to include a header defining
'size_t'.  Make sure we include <stddef.h> before it.

Closes #525