send: Convert system() to execprog() ... second half

Now only uip/spost.c has one left.

send: Convert system() to execprog() ... partly

forw: convert system() to execprog(); fix buffer check

Cleanup of code layout only

Refactoring in the small

Out-sorting failure cases instead of deep-nesting the success case.

Remove verbosity for -noverbose; cleanup in code layout and message texts

We shouldn't print information messages if we run non-verbosely.
In this case, the message appeared not to be helpful because it
suggests to re-run the command with `-verbose' ... but sending the
message again is usually not possible, not wanted or would produce
different results.

This is a cleanup of Philipp's patch.

Use execprog() instead of system() ... partly transition

There are several more places where system() should be replaced.

As execprog() does not split strings into multiple arguments,
`listproc' could no longer contain ``show -file''. We need to
rework the concept of these `...proc' entries.

This commit fixes tests/whatnow/test-attach-detach.

Added execprog() and execprogl(), which are safer alternatives to system(3)

This is motivated by the currently failing test case
tests/whatnow/test-attach-detach.

mhsign: use the strongest, newest not-expired secret key

Is there no GnuPG option to exclude expired keys from the listing?

Fix manpages (nroff code), as reported by tests/manpages/test-manpages

Partly fix tests/whatnow/test-attach-detach: changes in detach and alist

Whatnow's detach and alist commands are different today:
- detach does only take number arguments
- alist prints full path names

Fix tests/mhbuild/test-forw: rfc934 mode is no longer supported

Check with witch exit-code sendmail/spost exit and give an better error
message. So the User sees better wat's wrong. At the moment only
exit-code 75 is handlel extra.

Adjusted tests/new/test-basic to mmh.

mhparam: Renamed some sequences
We should fix these names once and for all!

tests: Adjusted most of the tests to mmh.

tests: Supress progress meter if non-interactive.

Added more include directories to fix building in another directory.
I discovered these problems when I tried to build from a git clone
(which has no config.h in the source directory).

Fixed the test framework by adjusting it to mmh.
In the same go: Cleaned up its code a bit and added a summary to
test/runalltests.

Adjusting the individual tests to mmh still needs to be done. ... and
afterwards we can ;dd a lot more of them. :-) (But before we write our
own tests, we first incorporate all the ones they've added to nmh in
the meanwhile. ;-) )

gitignore: ignore test dir name file

uip/mmh.sh: minor cleanup

Fixed building in different build directory, as described in INSTALL.

Merge

mhsign: Ignore expired, revoked, invalid, and similar public keys.

mhparam: Added more items for direct query and -debug listing.
Most new values are not included in the -all listing because they
are not part of the profile or context but are mainly internal
configuration values.

The -debug listing has now section headings (pipe through
`grep -v ^#' to remove them).

The motivation for this change was querying the default inbox
name. If `Inbox' wasn't set in the profile, `mhparam inbox' had
failed. Now it prints `+inbox' (if the defaultfolder has not been
changed on your installation).

mhpgp: Bugfix for 2932d136debcf53a45c44891ee80e6d34e61f109
(I should do better testing ...)

mhpgp: remove unused sed label
OpenBSD complained about it.
Reported by Dominik Lang <phil@phildom.de>. Thanks.

pick: Clarified the man page with respect to -ddd.
-ddd is an argument to e.g. -after, not a separate switch.

mhsign: Bugfix: Don't ignore MMHPGPKEY and the Pgpkey profile entry.
Bugreport and patch by Philipp Takacs <philipp@bureaucracy.de>. Thanks.
Additionally: use `test -z'.

Merge branch 'master' of ssh://marmaro.de:443/var/git/mmh

spost: If we add a From header, we add a qualified email address.
Also clarified a comment and changed the description for blind
distribution lists to the one that is most common today.

TODO: removed the double-dash TODO item, because it will not be done
The GNU style double-dashes collide with MH's command line concepts.
It makes no sense to simply allow a second dash, because all the
rest is different. (Think of `pick --component'.) It's more important
to keep the concepts clean than trying to imitate different concepts.

rmm: Use `refile -src' instead of a folder changing hack.
This is a cleanup for a ``hackish'' solution.
Thanks to Philipp Takacs <philipp@bureaucracy.de> for discovering this
improvement possibility and for preparing a patch.

Fixed a possible uninitialized variable access.
(Thanks to the compiler for reporting.)

coding style: Added braces even for one-line blocks
This commit changes only uip/rmm.c. It would be tedious to
change it everywhere. We can do this bit by bit as we go ...

Converted msgstats macros & Co. to functions and added range checks
This change is a reaction to bug, fixed in changeset 5b093c3. Although
the bug was fixed on a higher level, it would have been detected much
earlier if we would have had the range checks that we now have.

This patch was prepared by Philipp Takacs <philipp@bureaucracy.de>.

send: Check the draft folder only if we send msgs, not when sneding files.
Send can send files (specified by Unix path names) or messages (specified
by MH message names/sequences. (NB: It cannot send msgs from other folders
yet.) In case we've only specified files (this is done by whatnow, for
instance) we don't need to care if there is a draft folder and if it
contains any msgs.
This commit probably fixes a problem discovered by <philipp@bureaucracy.de>.

stringdex(): Clarified what it does
`stringdex' stands for `string index'. (See Changlog for why the `g'
was added.) This function is different to strcasestr() ... in the
order of the arguments and in the return value.

uprf: We don't have to implement it ourselves but can use strncasecmp()
Also clarified what an ``unsigned prefix'' really is.
I haven't changed all uses of uprf() in the code to strncasecmp(). This
could be done. I'm not sure if it would enhance the clarity of the code.
We don't need a prototype for strncasecmp(), because it's declared in
strings.h.

Use trim() to strip whitespace from the end of header fields
The main motivation was to ease adding attachments. You'll easy happen
to have trailing whitespace there, whereas you'll probably never want
to send files that have a name with whitespace at the end ... at least
you should never want to have or even send such files.
I don't know if it is a good idea to strip the trailing whitespace
in `anno -list' as well. I couldn't come up with an example when I
would want to keep it, hence I decided to remove it. This can be
changed back, if needed.

sbr/trim.c: New helper function trim(). Strips whitespace from a string.
There is already trimcpy(), which does a bit more work, but adding trim()
appears to be a useful investment.
The static function trim() in slocal needed to be renamed to not collide.

mhbuild: Transfer-encode MIME parts of type application always with base64
Application files likely contain NUL bytes. scan_content(), however, uses
stdio to check the files to attach. NUL bytes break these checks. Instead
of improving the checks to allow e.g. application/x-tex files to be sent
using 7bit or QP encoding, it was chosen to take the easy way and send
application files base64-encoded.

This solves a bug when attaching tar files. (tar files begin with a
header mostly filled with NUL bytes). Thanks to <philipp@bureaucracy.de>
for discovering and reporting the bug.

print-mimetype: fixed a syntax error

repl: use different format as the lead-in to quotes in replies
This format is prefered because it does not include words and is
thus independent of the language of the user.

mhbuild: Don't fail if one attaches an rfc822 message as a file
Although the form
attach: +folder 1234
is preferred, one may as well use
attach: /path/to/message.rfc822
A warning is printed though.

Bump version number.
This is version 0.1 plus some development changes.

This is mmh-0.1.
Eventually, I managed to release it ...

docs/README.developers: minor cleanups.

Added docs/DIFFERENCES_mmh_nmh.
The list is not perfect, but better than nothing.
Also added docs/schnalke-mmh.pdf to the distributed files.

Added my master's thesis, which explains mmh.
See also: http://marmaro.de/docs/master

README.developers: Various minor updates.
I also dropped the ugly double-spacing.

sbr/getpass.c: Removed nmh_getpass() for it is not used anymore.

Updated the list of documentation files to be distributed.

Makefile.in: Removed mmhrelease target because it's not used for mmh.

Reworked docs/README.developes to match mmh.

Delete docs/pending-release-notes as this is without value for mmh.

COPYRIGHT: Add a note to include mmh into nmh.
This appears to be most simple.
The remaining question: How to handle the year of the copyright notice?

Updated the installation instructions in INSTALL.

Included the relevant parts of docs/README.about into README.
Also updated the information to mmh.

Updated man page mh-profile(5): file(1) is not from GNU.
btw: Thanks to Dennis Herrmann (dhn) for pointing me to the whole issue.

Report useful error message on broken file(1).
GNU file-4.26, for instance, is known to be broken.
file(1) is the default Mime-Type-Query program.

Updated man page mmh-intro(7): Replaced obsolete sequence names.
Plus further minor cleanups.

Updated DATE, finally.

mhl forms: Display the sender header before the recipient headers.
This is the more natural order.

Minor man page fix.

Added Neil Rickert's mhpgp, the companion to mhsign.
Currently it is only usable on the command line directly.
In the future, it should be invoked by show(1) as needed.

Improved portability of mhsend(1).
Thanks to Dennis Herrmann for the bug report and patch.

show: Print type of MIME part instead of error message if not displayable.
A further step in the general rework of show/mhshow. Print less error
messages. Not being able to directly display some attachment is no
exceptional event, but quite normal. Therefore, no error message should
be printed but an information message that there is a MIME part.
Printing to stdout makes the message to be integrated into the normal
output.
There's more rework of show needed.

spost: Don't send Sign: or Enc: headers out.
Also, fixed the format of header fields with empty values. The trailing
newline is no longer stripped.

send: Sign and encrypt messages automatically, based on Sign: and Enc: Hdrs.
Currently, the value of the header fields is ignored. The sole presence
of such a header caused signing or encryption.

Removed old man page of whom.

whom: Suppress the separator if no hidden rcpts are present.

Added Neil Rickert's mhsign.
It is adjusted to mmh and thus somehow different to the original version.
mhpgp will follow soon.

whatnow: Changes for consistency.
Removed comments and arg names in prototypes.

whatnow: Re-integrated whom(1); cleanups.
Removed reference to `send -watch', which was removed in
9677896bbb11f7b49ca3e1665601407b6c7a459f
Reordered the options.

Re-added whom(1). This implementation is written from scratch.
This new version of whom(1) is quite different to the old one,
yet it serves the same purpose: listing the recipients of a message.
It still misses the ability to process +folder and msgs arguments.
As this implementation is new and not much practically tested,
it may change in the future, if flaws appear.

Minor wording change in an error message.

Completely removed the backup-prefix (,). We move to +trash instead.
mhbuild does not create `,foo.orig' but `foo.orig' now. Maybe it should
move the file to +trash as well ... this is likely to be changed.
Temporary files of bcc mails get removed now.

spost.man8: spost does read the profile now.

mhbuild: Fixed order of file names in error messages.
(The argument order is: rename(src, dest) and admonish("foo", "%s", "bar")
produces: ``bar foo''.)

Removed context_foil(). We don't need it anymore.
To not read the profile and context, don't call context_read().
Anyway, all mmh tools should read the context. slocal, which is
not truly part of mmh, does not.

ali: Either read default alias files or the ones on command line.
Renamed -alias to -file because the semantic changed and the new
name appears to be better suited.

send,spost: No more use of -alias, spost reads Aliasfile from the profile.
The profile entry Aliasfile was introduced in MH-6.7 to replace the
-alias switches. Later, the profile entry was enhanced to support multiple
files. Now that spost reads the profile itself, there is no more need
to pass the file with switches. Operating with varying sets of alias
files is now only possible with different profiles, but that should be
a rare setup.

spost: Read profile and context now. Removed -library switch.
spost is a full part of the mmh toolchest, hence, it shall read the
profile/context. This will remove the need to pass profile information
from send to spost via command line switches.
In January 2012, there had been a discussion on the nmh-workers ML
whether post should read the profile/context. There wasn't a clear
answer. It behavior was mainly motivated by the historic situation,
it seems. My opinion on the topic goes into the direction that every
tool that is part of the mmh toolchest should read the profile. That
is a clear and simple concept. Using MH tools without wanting to
interact with MH (like mhmail had been) is no more a practical problem.

mhmail: Read the context!
mhmail will change from a mailx-replacment to an alternative to
`comp -ed prompter', thus being a send front-end. Hence, mhmail
should not stay outside the profile/context respecting mmh toolchest.

slocal: Do not at all interact with the context or profile.
Slocal is not a true part of mmh, it is an MDA that is (currently)
shipped with the MUA mmh.

Merge `spost -watch' rework.

Added missing header. Reported by Dennis Herrmann.
It had worked on GNU/Linux, but not on FreeBSD.

Removed the -[no]watch flags of send and post.
Without direct SMTP delivery, they are useless.

whatnow: Improved the listing of available commands in the WhatNow shell.

mhl: Removed remaining code of obsolete -sleep switch.
The switch was used for the faceproc, for Face: header field handling.

Updated man page send(1) for mime-type-query instead of mhshow-suffix.
This aligns the man page to the code.

whatnow: Call send(1) as `send', not as `whatnow'.

mmh.sh: Changed from -version to -Version, here too.

spost: Just a move of some code.

Renamed switches variable to ease automatic text extraction.
Now, it's easier to get a list of all switches of a program from the
source code. The change improves the homogeneity, too.

Whatnow prompt: From now on, detaching only by numbers.
-n flag of `detach' is now default and thus removed.
(`alist' prints the numbers by default.)
If by mistake a whole bunch of of files was added (e.g. `attach *'),
these attachments should be removed with the editor in the draft.

Fixed `alist' command at whatnow prompt.

Removed configure flag --disable-locale and have it always enabled.
Todays system are assumed to have locale support.

show: Don't display strange headers by default.
This follows a suggestion by steve on the nmh-workers mailing list.