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()).

Minor clarifications in mhmail(1)

Update the man pages to the shortened reserved sequence names

These reserved sequence names were shortened to one character
long ago in the code. Now the docs are adjusted to that change.

Thanks to Marcin for spotting it.

Initialize status in uip/send.c

Thanks Marcin Cieslak <saper@saper.info> for reporting.

Merge ../mmh

Conflicts:
uip/rmm.c
uip/whatnow.c

remove done finaly

removed the used of done in sbr and the dafinition
in the header. Also the source file is removed.

remove last rests of done in uip

At some points done was used as callback funktion.

Remove done from mhl pick and send

Second part of removing done, to make the code
better to read. This tools needed some refactoring
before done could removed. Next step is removing
done from sbr.

Initialize vecp in send anno()

Access to an uninialized variable can cause
a a buffer overflow or underrun.
(Thanks to the compiler for reporting.)

Add tests for pick

Add two patches for pick. One to test the normale behavior,
and one to test if a ``0'' is print to stdout when no message is
find.

Bugfix in pick.c

Undo last last commit in uip/pick.c. pick shoud only print "0" if
it exit with a non-zero exit.

Replace done with exit at uip

Replace done with exit in most sourcefiles of uip. This makes the
code better to read. If the done was overwritten by an externel function
atexit() is used instand.

send.c and mhl.c are excluded, because need a bit rewrite first.

new: Fix a pointer problem

I should learn to get such stuff right without the need to draw
the memory layout with boxes and lines on paper ...

Thanks to the test suite for catching this one!

send: Fix uninitialied variable

whatnow: Use execprog() ... one more time

whatnow: No need to fork

Plus refactoring.

sbr/ext_hook.c: Use execprog(); Refactoring

rmm: Use execprog() instead of own fork-exec

Remove unused variable

Convert pidXwait() from macro to function

Plus some refactoring and syntactic sugar in pidstatus().

Don't save the context within execprog()

This context_save() call was copied there accidently.

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.