Merge

Include `test' dir in release tarball

The subdirectories `docs' and `test' are managed by the top-level
Makefile now, because there's nothing to do in them but to copy
them for the distribution tarball. (This way, we also don't have
to include each new file in these directories to the file list in
Makefile.in.)

Update lists of files to distribute

Also partly reordered the lists.

initialize mp in ismymbox

Drop the SYNOPSIS section from some man pages

For some man pages this section makes few sense.

Plus some minor formating stuff on the way ...

Minor clarification in mh-alias(5)

Silence the output of `cd -' in the tests

Use cksum(1) instead of md5sum(1) in the tests

cksum(1) is part of POSIX; md5sum(1) is not.

Run the tests in defined (i.e. sorted) order

Merge

consider Default-From as own address

Remove outcommented code (C99 comment)

Our code should be clean C89 (ANSI C). There's no reason to
keep this C99 comment.

Ensure $SHELL is set for the tests

Non-interactive shells might not set it (this is true for dash
and mksh, at least). Currently we use $SHELL in whatnow(1).
Although we likely remove this use, it's rather worthwhile to
have $SHELL available in the test scripts.

Update mmh-intro(7) man page

Remove mh-tailor(5) man page

Split the releasing instructions into own document

This makes them better visible.

Updates README, INSTALL, and docs/MACHINES

Update TODO list

Some comments on the removed items:
- Mmh's show does already invoke a pager
- The reverse action of `rmm' is: refile -src +trash l @
- Disjunct contexts are already possible (just set MMHC accordingly)
- We already generate the mh-chart man page
- Prompter can already be used to input the recipients. Use:
        Editor: prompter
        prompter: -nobody
        prompter-next: vi
- Comp: -nowhatnowproc is obsolete; use `-whatnowproc true' instead.
  (It puts an empty draft into the drafts folder.)
- Forw: Since we use MIME to forward messages, these todos are obsolete
- Mhmail is not considered to be a full-featured interface to MH
- We now do have RFC 2047 encoding of headers

add the mmh mailing list to the doc

remove docs/README.SASL

we don't need it, because smtp support is gone.

add mailsync description

remove TODOs for not supportet programms

send: MIMEify with non-ascii header and ascii body!

This code is not perfect, as it scans body lines twice. We might
want to rework it someday. But, ignoring the minor performance
issue, it ensures, that we MIMEify in the case when some headers
contain non-ASCII text but the body does not.

remove variable mp2 from spost in putadr

This variable is set but not used. Thanks gcc for the warning.

Add DCC support into whom

Plus a test for it.

Merge

DCC contains invisible (not visible) recipients

Added new test for BCC and DCC headers, which discovered the bug.

add config.o and version.o to libmh.a

Now you can easy link against libmh.a with -lmh

Add test for sender header in spost

(Fails currently, because default-from is not part of alternate-mailboxes.)

Adjust coding style from nmh to mmh

fix the prompter test

set psigint befor the setjmp in prompter

this mutes a Compielerwarning.

Rework in the test framework; updated and new tests

Many tests were ported from nmh.

Two tests fail at the moment:
- tests/ali/test-ali:  needs porting the bugfix from nmh
- tests/prompter/test-prompter:  needs further investigation

Merge branch 'master'

Move chdir() call so -recurse option works again

This commit is ported from nmh.

Fix off-by-one error (CPERLIN)

CPERLIN is the chars per line without the line break chars (\r\n),
at least would that match the description in RFC 2822. Actually,
however, the max SHOULD line length (excluding line break chars)
is 78 according to the RFC. Therefore it seems as if we should
set CPERLIN to 78. (This is open to decide and to do.)

Merge

Coding style adjustments for nmh-imported code

- No documentation in h/prototypes.h, but rather in the sbr/*.c files

- Convert coding style from nmh to mmh:
  + indent with tabs only not with a tab-spaces-mixture
  + no space between function name and opening parenthesis
  + no space on the inner side of parens
  + no trailing space
  + different comment style

- Variable definition at the beginning of the block only (ANSI C)

- Choose meaningful identifier name rather than add a comment to a
  meaningless name

Don't remove the draft on SIGQUIT to allow debugging

export MM_CHARSET in mhbuild test

The test has some UTF-8 charracter to test the encoding.

use anno and mhparam in sendfiles

decode recipient-fields by default

add a test for mhbuild rfc2047-support

This is only the first test, more come later.

porte encode_rfc2047 from nmh

Now a user can add any nonascii in header-fields.
mhbuild now encode this acourding to rfc2047.

Thanks Ken Hornstein <kenh@pobox.com>

mhbuild: Avoid a dot on a line on its own

mhbuild's quoted-printable encoder breaks lines to have at most
CPERLIN chars. If the it happens that the last character on the
line is a dot (`.') and the line is broken just before it, then
the following line contains nothing but one single dot. This
currently leads to problems in mmh ... but it may cause problems
to other mail software as well. Hence we avoid it by encoding
any dot at the beginning of a line, just to be sure.

Sleep only 1 sec before retry to lock the file

The previous 5 seconds might have been appropriate on old, heavy
loaded systems, but they will hardly be necessary on modern ones.

Show the date hdr in local time in the default scan listing

Improve wording and more description in man page spost(8)

Break out the unquote functionality to a separate function.

pick(1) decode rfc2047-fields

Attempt to decode each header field as if it were
MIME encoded.

include h/utils.h

I forgot uip/ap.c and uip/burst.c, too

include h/utils.h

I forgot to add it in sbr/addrsbr.c sbr/fmt_compile.c

replaced calloc with mh_xcalloc

This finished the work of David Levine,
which was portet in the last commit.

Added mh_xcalloc().

fix open up to 4 times on mbox_open()

Befor this lkopen was only called ones.

reimplement Dcc-field

Dcc is like Bcc but don't send an extra message
with the orginal message attched. See FAQ for
more details.

find recipients for sendmail

Recipients are given as comandline arguments to sendmail.
Also do aliasing on every header which contain an address.

Revert "Remove stdio internals manipulation in sbr/m_getfld.c"

This only temporaly, to ensure mmh is working correct.

This reverts commit 976303d04d3bc2cad0afd5e3d364264783da56c2.

Remove stdio internals manipulation in sbr/m_getfld.c

Replaced all the stdio buffer access in sbr/m_getfld.c with a single call
to fread() and then some almost straightforward buffer manipulations.

This commit is ported to mmh.

Make msg_delim file static to sbr/m_getfld.c

Thanks to David Levine <levinedl@acm.org> for nmh commit
e69044f7624abe5cb2cb796d528c0cc5f29515f7

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

Merge remote-tracking branch 'origin/master'

Bring back the LINUX-IO check from the last commit

We need them for m_getfld(), too.

This reverts parts of the commit 33516e5b74bb4bb5c626d4ee42bd3a7a9faaabef.

Drop register storage class

Remove discard() total

discard() isn't needed in mmh anymore.

Fix autoconf check for mailspool world write permission

This is a bugfix and improvement for commit b15b7cc (``Use
realpath to find the mailspool'').

We can even do this check much simpler, if we focus on the
problem that

        $lspath -dlL $mailspool

does not check the target of a symlink, although -L is given.
The contradicting -d seems to overrule it.

We can fix this more easily by appending a slash to the path:

        $ ls -ld /var/mail/
        drwxrwxr-t 2 root root 4096 Apr 29 06:36 /var/mail//

Without the slash it was:

        $ ls -ld /var/mail
        lrwxrwxrwx 1 root root 10 Nov 22  2011 /var/mail -> spool/mail/

This small change does it for one level of indirection.
This should be sufficient.

Ignore empty MIME parameter

If a message contains an empty parameter, we only produce a warning.

Ignore wrong encoding in multipart/*

If a multipart message has an encoding other then
7bit or 8bit, 7bit encoding is assumed.

remove dublicated code in uip/spost

To avoid bugs like in commit dc81afc84b4d53177a1c4

fix bug if multible mbox in From-field

Now the first address of the user is used, if multible addresses
are in the From-field. If non of the addresses is an address
of the user, the the behaviour has not changed.

bugfix for a memory-reuse bug spost and whom

spost and whom use add to initialize a temporary char*.
This pointer maybe points to internal data or already freed
memory. Now getcpy is used.

spost: Refactor braces, comments, etc.

gitignore: More specific ignore patterns

I frequently make temporary backups this way:
cp uip/foo.c uip/foo.c.1
These files were ignored, although this ignore filter was meant
for the created man pages in section 1 only.

Add "Default-From" config option

The "Default-From" config option is used, if the Draft
don't contain a From header. If the draft contains a From
header and it's not from the user it's used as the Sender
header.

Bug: If the From-header contains more then one addresse
and one is from the user, no Sender-header is attached.

Use realpath to find the mailspool

inc is istalled with setgid-flag if the mailspool
isn't world writeable and dot-locking is used. This
work now if the mailspool is a sysmlink.

Use sysexits.h for better exit-codes

The sysexits header describe some standard exit-codes, which should
be used. adios has now a status argument so sysexits.h can be used.

reinclude some headers

These includes are necessary to compile on FreeBSD.

Move #include from h/mh.h to source files

Included files should not include further files. Now, only the
necessary files are included and not all of them in every file.

A large bunch of this work is done, but there are some more
nested include files.

Remove h/nmh.h

Move its contents to h/mh.h.

Remove the NLENGTH macro on the way, be cause it is not used.

Minimize the include

It's not necessary to include <h/signals.h>, <signal.h> is enough.

Fix FTBFS bug introduced by 600379c

One #include of h/signals.h is still necessary.

Removed unnecessary #include of h/signals.h.

Fixed the error message in lockit() when it fails to create tmp file.

Removed m_chkids(), per suggestion of Marcin Cieslak.

Removed #include of signal.h because it's in h/signals.h.

Replaced setgid() calls with setegid() so that it works with dot
locking on FreeBSD.  setegid() should be supported on modern POSIX
systems.

remove declaration of child_id in slocal:usr_pipe

Now the global version is used, so the signal handler
works correct.

return -1 if fork(2) fails in execprog

If fork in execprog fails and return 1 there is now
way to different between a proces witch return 1.
Thanks Marcin Cieslak <saper@saper.info> for reporting.

remove longjmp from signal handler in mhl

To avoid undefined behavior set an interrupt flag
in the signal handler.

remove longjmp in signalhandler of slocal

To avoid undefined behavior kill the child process
in the signal handler and set an interrupt flag.

remove longjmp in signalhandler of prompter

to avoid undefined behavior use close(2) in the signal handler

remove longjmp from signalhandler in getans

To avoid undefined behavior use close in the signal handler.

nonstatic function for atexit, check if atexit fails

atexit needs nonstatic functions to work correct. The
error check is importent, because atexit can fail for
some reasons and the cleanup woulden't work.

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

Conflicts:
uip/mhl.c
uip/mhlist.c
uip/mhmail.c
uip/pick.c
uip/send.c

Refactor: Use if for two-way branching; Compare against 0 explicitly

I think that the use of switch for the two-way branches could have
been a performance optimization. I think it's clearer to use if,
especially in the fall-through case.

As setjmp() is known to return with either 0 or something else, we
should compare against 0 and not against some define (for which I
again and again wonder if it stands for 0 or for 1).

Fix minor coding style issues (whitespace)

Remove unnecessary headers

distout and ready_msg return NOTOK on error

If distout or ready_msg fails, the cleanup of send is done.

mhmail: use execprog()

sbr/execprog.c: Fix the return value

Use pidXwait(), which invokes pidstatus(), here for error reporting.
This eases the use of execprog() because we don't have to call it
there everytime. However, it might be unwanted to put the error
printing stuff into execprog() because we might have an invokation
of execprog() that want's to deal with errors itself in a different
way. If this would be the case, then we likely should change
pidXwait() to pidwait() and call pidstatus(execprog()).