Applied Kurt J. Lidl <lidl@eng.us.uu.net>'s $MAILHOST patch:

    I have a small patch that would be nice to be included -- basically, it
    allows the usage of the "MAILHOST" environment variable, without having to
    have HESIOD turned on.  I need this functionality for my environment, where
    we have identical /usr/local on all my machines (so I cannot just hardcode
    into the mts.conf file), and I have multiple POP mail servers for my users.

Modified inc.man to reflect that along with "pophost:" and -host, $MAILHOST can
now activate POP mail inclusion as well.

Last pass at README.developers -- Kimmo's 5-step commit was overkill.  You only
need 3 steps, since configure.in is the only autoconf file with the RCS $Id
keyword.

Meaningless modification to test autoconf file commit order stuff.

Meaningless change to test autoconf file commit order stuff.

Meaningless change for testing autoconf file commit order stuff.

Meaningless change to test autoconf commit order stuff.

People did not preserve my alphabetization of the --with options when they added
new ones.  Re-alphabetized.

Ken Hornstein's SASL patch was not integrated properly with Ruud's new merged
mts/sendmail code.  Kimmo has since fixed nmh so it compiles, but according to
Ken, the SASL stuff still does not work.  Integrating a patch from him for this.

One more pass at README.developers now that it's clear that my
previously-suggested one-line autoconf-file commit can cause unnecessary local
makes and an out-of-sync stamp-h.in file, but would not cause problems for other
people using the CVS files.

Moved Kimmo's new "--with-hash-backup" to be output with the rest of the --with
options in the configure --help output, rather than being in between two random
--enable options.

Added answer.

Clarified and made some corrections to Kimmo's README.developers changes (BTW,
if anyone can explain why the RCS Ids are able to cause problems with the
dependencies, please fill in the explanation -- I never encountered a problem
with the old single-commit method).

I don't understand why Kim's split-up of the autoconf-file cvs commits should be
necessary, so I left a little note hoping for someone to explain technically why
that should be the case.  I certainly never ran into any problems with my old
single-commit version (which I added back in along with an explanation of why it
supposedly won't always work).

Also Kim's text implied that the _only_ reason for the given order is the RCS
Ids -- not true -- the timestamps are the main reason a particular order is
necessary.  Made a couple of other clarifications as well.

Iterating the commit a bit, I think this is how it works without extra
runs of autoconf, autoheader and make.

Regen.

If $LIBTOOL is empty we should not try to run it.
This gets rid of the "--version: not found" error.

Got rid of one mktemp.

Complete HAVE_MKSTEMP.

We ignore the fact that the user requested SASL, since the pipe to the
mailer is secure.  This allows for easier sharing of the MH profile
across hosts.

Note replacement of GNU-makeism in sbr/Makefile.in with a common statement.

So I don't think you can commit these all in a single commit.
README.developers recommends that (with precise ordering) but
things still get regenerated when I run "make".

Keep this working with other makes than GNU.

Still missed one SASL stuff thing.

Changed "--with-backup-prefix" to "--with-hash-backup" because # is too
difficult for config.status to get right.

Note "--with-backup-prefix".

Add "--with-backup-prefix" so it can be set easily.

SASL stuff.

SASL stuff.

SASL mods.

Deleted the timezone on Kim's SASL patch entry.  Ruud has taken to adding the
timezone even though all previous entries are missing it, and I don't really
agree with that.  True, on occasion someone in a different timezone will
commit something later in the day than someone else, and even though the second
change happens later, the time may be earlier, but that doesn't happen too often
and isn't really a big deal, IMHO.

Putting textual timezones like "EDT" isn't really the right solution, IMHO,
since some of those tags are ambiguous.  Either an offset-style timezone should
be used or all times should be given as UTC.

I'd just as soon stick with local times, though.

Oh, one thing I forgot to mention in my last CVS entry was that I removed "IMAP"
from the list of protocols supported by Ken's SASL patches -- nmh doesn't have
any IMAP support!

Modified Kimmo's entry for the application of Ken Hornstein's SASL patches to
clarify that it wasn't Ken applying them himself, and clarified what SASL
support actually means.

Kimmo neglected to commit the regenerated versions of these after changing
configure.in, forcing people using the CVS source to have to have the autoconf
tools installed.

Clarified post.man and send.man for those not completely up on SASL terminology.
"SASL encryption layers are not supported for SMTP" means that encryption is
supported for the authentication but not for the subsequent data stream.

Note SASL commit.

SASL support from Ken Hornstein <kenh@cmf.nrl.navy.mil>.

* Merged mts/sendmail functionality into mts/smtp; switching between
smtp and sendmail delivery method is now controlled by mts.conf.
* If tsort cannot deal with loops, in addition to defining tsort as
cat, also define lorder as echo.
* Removed uip/popi.c from list of sources.

New dtimep.lex didn't parse day names properly.  Fixed.  Also clarified
ambiguous comments preceding day_map[] array (from old dtimep.lex) that probably
led to the erroneous cp++ being added.

* Added one more mkstemp invocation to uip/spost.c (which was in a
#if 0 block).
* Applied patch from Peter Maydell to clean up permissions handling
and error handling in uip/inc.c.

Use cat instead of tsort if tsort cannot deal with loops in its
input (which is the case for tsort from GNU textutils).

If liblockfile is present, and its dotlockfile program is setgid,
inc does not need to be setgid.

* Added autoconf test for Miquel van Smoorenburg's liblockfile
library, as found on Debian systems.
* Added liblockfile support to sbr/lock_file.c.

make clean now removes generated bin files.

Made dtimep.lex more lex-friendly.

Added back memory options for AIX to increase available memory.
Took out %option noyywrap, which wasn't understood by AT&T lex, as
well as the -i case-insensitivity flag.

added lint targets for Makefiles and a configure test to find whether lclint or lint is installed on the system.

Applied Alec Wolman <wolman@cs.washington.edu>'s dropsbr.c patch:

  In the map_write routine, a call is made to map_open and this call is supposed
  to set the "clear" variable to 0 or 1, depending on whether the map file is
  empty or not.  In mh6.8.3, this worked because map_open would set "clear" by
  calling the mbx_Xopen routine.  In nmh, the code for mbx_Xopen was merged into
  mbx_open, but the interface for mbx_open doesn't support the clear variable,
  so that functionality was lost.  The map_open interface still contains "int
  *clear" in the prototype, but never sets it.

  My patch eliminates "clear" from the map_open interface (I checked to make
  sure that map_write is the only client of map_open).  Furthermore, my patch
  also sets the "clear" variable properly at the beginning of map_write by
  calling fstat().  This eliminates the bug in that the value of "clear" being
  used later in the routine was just stack garbage.

  Having a bad value of clear causes this next bug to be triggered: The fp file
  pointer was being opened with fdopen, but in two of the three switch cases it
  wasn't being closed.  In certain cases, this was causing packf to run out of
  file descriptors if you attempted to pack a large folder.

A couple more tweaks to etc/Makefile.in:

    Generated sendfiles script was not a dependency of the `all' target, and was
    incorrectly included in the distribution.

INSTALL never documented the etc/*.old thing.  Documented the new etc/*.prev
thing (including a note to watch for diff output).

Reworded my last entry a bit.

etc/Makefile.in was incorrectly installing mts.conf.in and sendfiles.in --
fixed.  Changed the suffix for the backed-up previous versions of the etc files
from the ambiguous .old to .prev.  Use diff and only keep the .prev file around
if different.

ChangeLog:  I think we should use a uniform style in this file.  Reformatted
            Ruud's recent entries.

configure:  Weird.  There's some timestamp screwup somewhere, perhaps in the way
            Makefile.in is written, that caused autoconf to get run after I did
            a `cvs update'.  It regenerated configure, and apparently I'm using
            a different version of autoconf, with a different order of awk
            alternatives.

Added entry for new lexing function and moving from zotnet/tws to sbr

Updating root-level files so that zotnet/tws/Makefile is not generated.

Synchronizing Makefiles that had gotten out of sync with directory
contents.

Updated sbr functions to include <h/tws.h> instead of <zotnet/tws/tws.h>.
Also, update Makefile to include new functions dtime.c and dtimep.c

Updating user programs to use <h/tws> instead of <zotnet/tws/tws.h>

Moved date/time functions from zotnet/tws to sbr. More importantly,
wrote a new lexing function dparsetime (dtimep.lex) which
plays nicely with flex, needs no "sed hackery". It might work with
lex, but I haven't tried.

* Added autoconf check for getutent().
* Changed uip/rcvtty.c and uip/slocal.c to use getutent() and
  friends.  Since I can only check on Linux, please check if
  this works on other systems.

Added autoconf check for getutent().

* Applied patch from Peter Maydell to uip/scansbr.c for more
  checks for write failures.
* Unlink temporary file properly in uip/rcvtty.c.
* Moved viamail from bindir to libdir.
* Changed sendfiles into sendfiles.in, so that path to viamail
  is patched in.
* Added gzip support to sendfiles.
* Added References header to replcomps and replgroupcomps.

Fixed m_getfld bug which caused segmentation faults when
incorporating messages which ended in multiple linefeeds crossing
a buffer boundary.

msh has been unable to show MIME messages ever since 1.0.  Alec Wolman
<wolman@cs.washington.edu> tracked down the problem to the -show flag being
passed to mhshow.  mhshow is equivalent to the old mhn -show, so we don't need
the -show anymore.  Removed it.

Moved zotnet/mf to sbr.

zotnet/bboards no longer built by default.

zotnet/bboards is no longer built by default.

Removed zotnet/bboards from default build.

Change to MACHINES to include what platforms nmh *does* compile on.

Added on whish platforms nmh is known to compile, as polled from
recent mailings to nmh-workers. Please add liberally.

Entered minor change to sbr/Makefile.in

Removed explicit definition of both SRCS *and* OBJS, and made OBJS simple
a pattern substitution of SRCS, with suffix .c -> .o

Added my getpass() addition to ChangeLog (after the fact, but at least
it's in now).

Added mts.conf.5 page per Neil W Rickert <rickert+nmh@cs.niu.edu>'s report:

    This happens on solaris:

     % man mts.conf
     windex entry incorrect:  mts.conf(5) not found.
     No manual entry for mts.conf.

    It is fixed by

     % echo ".so man5/mh-tailor.5" > mts.conf.5

    done in the man5 directory.  We need to add 'mts.conf.5' as a reference
    sourcing mh-tailor.5.

Prepended "Portions of this code are" to the copyright message in ruserpass.c
also.

* Changed configure.in to use gcc -Wall even without --enable-debug, to prevent
  developers compiling optimized from introducing warnings, and to give
  end-users a warm, fuzzy feeling as they (hopefully) see no warnings come out
  (except perhaps on the lex output file) even with -Wall.

* Renamed getpass() to nmh_getpass() since the prototype for getpass() varies
  from OS to OS, and we want to _always_ use our version of the function.  Fixed
  all the callers to use nmh_getpass() and added it to prototypes.h.
  Semi-arbitrarily upped MAX_PASSWORD_LEN from 128 to 256.  buf was being
  calloc()'d and the memory leaked -- should have just been declared as static
  char array.  Prepended "Portions of this code are" to the copyright message,
  as this version has been changed significantly from the BSD version.

* Added "nmh-local functions to use in preference to OS versions" section to
  README.developers (currently just says to use nmh_getpass() instead of system
  getpass()).

Oops.  Forgot to commit this when I made configure changes yesterday.

Added steps to README.developers saying to change the version number to
X.Y.Z+dev.  Did a little rearranging and changed the FTP dir from /home/ftp to
/var/ftp to reflect Doug's new machine.

Got rid of four warnings in Shantonu's new getpass.c.  Needed to #include
<stdlib.h> for calloc(), <unistd.h> for ttyname(), and "h/mh.h" for adios().
Also changed ch from char to int to get rid of "comparison is always 1 due to
limited range of data type" on EOF.

Alphabetized Shantonu's $pop_kinds output on configure's "pop is enabled" line.
If POP3 is the only kind of POP enabled, say so, rather than just saying "yes"
(which is ambiguous).

Shantonu updated the DIFFERENCES file to no longer say that APOP isn't
supported, but in so doing, he removed the note that RPOP isn't supported.  Put
it back.

* I had alphabetized the --configure options in the --help output
awhile back, but Shantonu added --enable-apop just under --enable-pop.  Put it
in alphabetical order and clarified what --enable-apop does vs. --enable-pop and
--with-krb4.  Also changed --with-mts help line from "mail transport agent" to
"mail transport agent/service" so the 's' in "mts" doesn't seem to come out of
nowhere.

* Added two steps to "releasing nmh" in README.developers.  After
making the tarball, it's a good idea to diff the tree vs. the CVS tree to make
sure no files got left out, and then to chown the files so that they're owned by
root, preventing a Trojaning attack by a malicious remote user with a UID
matching yours.

Updated POP capability to include APOP.

Removed unnecessary comments, such as "*This warning no longer applies*"

Fix some compile-time warnings.

Doug informed me that the way I had restored the "lost" version histories was
wrong, because `cvs checkout's of old versions of nmh wouldn't work properly.
It occurs to me that this could be fixed by simply deleting those tags in the
new-location *,v files, but oh well.  I'm putting everything back to the way
Doug originally had it.  To get the old version history for a file that used to
be in the top directory, you'll need to "blindly" do a `cvs log' there (even
though you won't have a local copy of the file in that directory).  `cvs diff'
will no longer be able to diff pre-move versions vs. post-move versions --
you'll have to do a lot of manual gyrations with `cvs checkout' and then use
`diff'.

Okay, now that the old revisions have been recreated, deleting the file.  The
new version is in the 'docs' subdirectory.

This is the nmh-1.0.4 version from 2000-03-15 (I see that I was the one that
originally made this change).

This is the nmh-1.0.3 version from 2000-01-18 (1.0.2 had no changes).

This is the nmh-1.0.1 version from 1999-05-17.

For reasons explained better elsewhere (e.g. Attic/README.developers), the
version history for this file got lost.  Recreating it.  This version is from
1998-05-08 -- it's the original nmh-1.0 version.

After nmh 1.0.4, moved from top-level 'nmh' directory to 'nmh/docs'.

Okay.  Removing it now.  New location is in 'docs' subdirectory.

In the hullaballoo with moving stuff back and forth out of the docs/ directory,
and trying to figure out the right way to preserve the old version history, this
file's version history prior to the first move into docs/ got wiped.  I
introduced this file into the project after 1.0.3 was released, and was the only
one to modify it until after the 1.0.4 release.  The post-1.0.4 changes have
survived in docs/README.developers, so I just need to check in the 1.0.4 version
and tag it as such, then remove it again, to make a `cvs checkout' of nmh 1.0.4
work again.

Man, I'm a retard.  Lemme try that again:

Just a silly note 'cause I'm vain -- below where it currently says:

    Someone ("ssen" -- looks like Doug forgot to update a chroot() passwd file

It originally said " u i d 6 0 0 0 4 " (or a different number), but minus the
spaces.  I guess after Doug added ssen into the chroot() passwd, he did a
search-and-replace for the UID, which changed my comment as well, making it look
like I had no way of determining who ssen was, which is silly.  ;^>

Just a silly note 'cause I'm vain --

After nmh 1.0.4, moved from top-level 'nmh' directory to 'nmh/docs'.

After nmh 1.0.4, moved from top-level 'nmh' directory to 'nmh/docs'.

Updated pop instructions and indicated use of standard "./configure"
instead of "sh configure"

An nmh-specific getpass, to remove dependency on system
getpass()es, which may or may not do what we want (including
flushing input streams, and taking input from stdin in the
absence of a controlling tty).

Added getpass to SRCS and OBJS.

Removed "installing nmh" section.  Moved its note about the DIFFERENCES file up
next to the other MH discussion.  Changed out-of-date "README" title (should
have been "README.history") to "README.about -- about nmh".  After committing,
I'll rename the file to README.about inside the CVSROOT.