| Commit message (Collapse) | Author | Age |
|
|
|
|
| |
Drop it also from the corresponding UI functions, where it does not do
anything.
|
|
|
|
|
| |
The value from the hook would currently get overridden by the value
provided to the ExternalCommand() constructor.
|
|
|
|
|
|
|
|
|
| |
This is not just "being extra safe" as the comment says, it actually
prevents using shell constructs in the commandline, thus nullifying the
main reason to use the shell in the first place.
If the shell is requested, assume the command list is already properly
prepared for the shell and just join the list elements with spaces.
|
| |
|
|
|
|
|
|
| |
It is unnecessary extra complexity. If the user set a signature, then it
must exist. If it does not exist, the user should unset the signature
path.
|
| |
|
| |
|
| |
|
|
|
|
|
| |
Make it a plain container around raw data and a few bits of metadata,
rather than around a whole MIME part.
|
|
|
|
| |
It has nothing to do with the database.
|
|
|
|
|
|
| |
We should not try to guess anything, just use the platform encoding.
Remove helper.try_decode(), as it has no more callers.
|
|
|
|
|
|
|
| |
It does not actually save any code. The new code also uses a shell as is
documented.
Remove call_cmd(), as it no longer has any callers.
|
|
|
|
|
|
|
| |
It does not actually save any code. The new code also uses a shell as is
documented.
Remove call_cmd_async(), as it no longer has any callers.
|
| |
|
| |
|
| |
|
| |
|
| |
|
| |
|
|
|
|
|
| |
This is not implemented in notmuch2 and does not really belong in alot.
It can be done better through the notmuch utility.
|
| |
|
| |
|
|
|
|
|
|
|
|
| |
Bypasses the inspection of editor_cmdstring if it is still None after
all attempts to fill it. This will eventually lead to EditCommand.apply
erroring with 'no editor set' instead of alot crashing.
Fixes #1438
|
|
|
|
|
| |
Implements the suggestion in #1427 to display the actual error code, and
substitutes an empty stderr with "No stderr output".
|
|
|
|
| |
... instead of from alot.completion
|
|
|
|
| |
The local variable proc was not defined in these branches.
|
| |
|
|
|
|
|
|
|
|
| |
Since this branch was created, master has diverged in that
ComposeCommand now stores the determined sender account in the envelope.
(see #117).
This commit makes sure that the newly split methods read the account from
there.
|
| |
|
|
|
|
| |
The last few functions are very small so just doe them in one go.
|
| |
|
| |
|
| |
|
| |
|
| |
|
|
|
|
| |
account and From header
|
| |
|
|
|
|
|
|
|
|
| |
As we split the apply method up into separate methods we need a way for
those helpers to stop execution of the method. This wraps the real apply
method inside of a try/except block with a custom exception. Any helper
that is intended to run in the apply method simply needs to raise this
exception, and apply will return.
|
|
|
|
| |
Fixes #1351.
|
|
|
|
|
| |
... since accounts' addresses can overlap and get_account_by_address
promises too much. Also remove now-obsolete get_addresses.
|
|
|
|
| |
simplifies a few tests, fixes #1177
|
|
|
|
| |
Since that's what they're for.
|
|
|
|
|
|
| |
This allows it to easily wrap calls that are themselves coroutines,
while it's still able to wrap synchronous calls. This fixes the async
test and allows hooks that are coroutines to work.
|
|
|
|
|
|
| |
This is a pretty invasive patch, since the ui code is used so
extensively, it requires going into a lot of other code and converting
those to coroutines, since before they returned deferred's.
|
|
|
|
|
|
| |
Rather than returning a deferred in some cases, this makes the function
a coroutine, in some cases it calls regular subprocess, in other cases
it uses asyncio subprocess.
|
| |
|
|\
| |
| | |
Copyedit documentation
|
| | |
|
|/
|
|
|
|
|
|
| |
Email utils apparently assumes it's getting a string, and calls encode
directly. This can only be hit if you have a single account configured,
not with multiple accounts.
Fixes #1277
|
|
|
|
| |
that removes a named query string from the notmuch index
|