simple mbox support for rcvstore

Some MTA(opensmtpd) add a ``From '' line ad the head of
mails befor handling the mail to a .forwoad MDA. m_getfld2
don't support such mails, so rcvstore should handle this
mbox-like mails.

check return value of m_getfld2 befor using the field

this bug causes a segfault, because the field.value is a
nullptr if FILEEOF2 is returned.

Use boolean instand of bool

Revert "add free_field as standard for struct field"

This is not necesary, because the compieler have to take care
about NULL != (void*)0

This reverts commit a87df3543d3bc128ba4079d1f95638476ba5ca50.

fix lengthcheck in m_getfld2()

The lenght check akcepted to big field names

fix segfault in prompter

This was necesary because m_getfld2() clears now f.value
if FILEEOF2 is reached

add free_field as standard for struct field

this is a easy way to initializ a struct field. ``{{0}}''
can lead to bugs, because NULL is not necesary ``((void *)0)''

m_getfld2 is more tranceparent on error

now all read content of the file is given to the caller,
so he can possible fix this error.

make m_getfld2() aware of CRLF lineendings

make m_getfld2() called with an error state a noop

This probduce some code dublication, but the will be
reduced, if we make m_getfld2() more transparent on errors.

Finally we get rid of the unmaintainable m_getfld()!

m_getfld() was about 700 lines, 300 of them being comments -- that's 42%!
m_getfld2() is 240 lines, 30 of them being comments. That's only half as
much code. Most important, however, is that this code is understandable
and thus maintainable. We escaped the dead end of the heavily optimized
m_getfld() be re-writing it from scratch. (Thanks to Philipp Tacaks!)

m_getfld2() is line-based and not character-based like m_getfld(). This
change spared us to keep digging into stdio internals or implementing
our own I/O buffering. We are now portable to any standard-conforming
stdio implementation.

Adjust tests to m_getfld2() and write some more

m_getfld2() is more strict than m_getfld() and does less guessing.

Only lines consisting of nothing but dashes are body separators

This is how the documentation specifies it, see mh-mail(5). The old
m_getfld(), however, was more relaxed and treated any header line that
starts with a dash as the body separator. We are more strict here,
because ``-foo: bar'' is a valid header line (according to RFC 2822),
that should thus not be treated as the body separator, whereas a line
``------'' in the header could only be a format error if we don't
take it as the body separator.

Convert inc and scan from m_getfld() to m_getfld2()

The main conversion was done in uip/scansbr.c.

Mbox handling is now completely outside of m_getfld2(), which does not
even know about From_ lines.

No more need to call thisisanmbox() (formerly m_unknown(), ``the magic
invocation'') before calling scan().

The newly introduced gotos should be eliminated ...

Convert rcvdist and repl from m_getfld() to m_getfld2()

m_getfld2: Return line for LENERR2, plus some documentation

m_getfld2: ERR2 means a read error, thus name it IOERR2

m_getfld2: Whitespace in component name is forbidden

Also on style: No function calls should be used to initialize
variables in definition statements. Use separate assignments in
this case.

Use continue instead of one long if for the whole loop body

Plus remove unnecessary comments. This improves the signal-noise ratio.
The identifiers are already telling us enough.

Return invalid line for FMTERR in m_getfld2()

This is necessary for mbox handling outside of m_getfld2().

No error-checking for error-checking alloc functions

The mh_x(m|c|re)alloc function always return successful
... or terminate the program on their own, otherwise.

check for (un)selection befor (un)set

If you select or unselect the same msgnum
numsel won't be increased/decreased twice.

handle hghsel lowsel and numsel in (un)set_selected

Now you can use this funkitions without taking care
of this values.

Thanks c_14 for reporting this bug.

Adjust the tests to less tolerant m_getfld2()

We require the header and body to be separated by an empty line.
We require header lines to end with a newline. m_getfld2() does
not try to guess as much as the old m_getfld() did.

Convert most of the tools from m_getfld() to m_getfld2()

Add new implementation m_getfld2()

This is mainly Philipp Takacs work.

Fix minor typo in man page

Better description of the Sender header in man page

Plus wording improvements and other minor stuff.

Re-add --with-lockdir=DIR configure option

At the same time eliminate acconfig.h (and thus the autoreconf
warning).

Include the lock dir string only if we use dot locking.

More sensible check for 7bit ASCII

Always true condition reported by -Wextra -pendatic.

Remove unused parameter

Reported by -Wextra -pedantic.

Remove dead code: FLDEOF and BODYEOF are never set

Bump version number

This is mmh-0.2

Decode address headers in replcomps

Update README and release howto to new NEWS file

Also reorder the release steps to first check the tarball and
then tag it as final.

Add NEWS file, which summarizes changes per release

Include a static copy of the CVS log in the dist tarball

Teach spost to handle address groups

Plus more tests for address groups.

test Group Addresses in spost

catch unparsable addresses

Now spost don't coredump if an address can't be parsed

test if spost catch bad addresses

test if aliasing work in spost

Fix configure.ac warning

Remove check for not used networking libraries in configure.ac

Mmh does no more contain networking code, hence we don't need this.
This way, we can also get rid of aclocal.m4.

Build the release tarball from git, not manually

Previously the Makefiles contained lists of files to distribute.
They built the release tarball manually. It's much easier to just
use git, which knows which files are the relevant files, because
it's the ones that are under version control. We just need to
add some additional, pre-generated files for convenience, so that
the user does not need to have auto* and lex installed.

The release tarball is now fully assembled in the top-level
Makefile. (The only tricky part is sbr/dtimep.c, because that
needs to be generated in the subdirectory.)

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