Oops.  My last group commit comment failed to mention what I was doing in this
file.  I was adding:

* Change all man pages to group all the commandline options together in one
  section with each as a separate mini-heading.  Having to dig through prose to
  find what a particular option does is a pain in the nads.

This time, moved it out of the [POSSIBILITIES] section and put it in the [TODO]
section, as I feel it's very important (and wouldn't be that difficult to do --
somewhat time-consuming, yes, but not difficult).

Changed the GECOS-field '&' translation behavior to be controlled by the BSD42
#define rather than GCOS_HACK, since it's apparently always appropriate on OSes
where BSD42 is #defined, and never appropriate on any other OSes.  Thanks to
Kimmo Suominen for responding to my "What is this code here for?" comment in
mts.c and explaining the feature.

Also added ULTRIX 4.2A to the list of OSes that have an initgroups() function
but no prototype in the system headers.

Made a goofy change so that I could produce a new configure because when I
checked in the last version I did it with "cvs commit ... configure configure.in
..." and that apparently gives configure.in a newer timestamp than configure,
which results in spurious autoconf runs when people download the source.  We
need to write a little note to developers somewhere saying what in order you
need to check in those autoconf files and how long you need to wait in between
each one and such...

Changed "echo > stamp-h.in" in Makefile.in to "date > stamp-h.in" so that
stamp-h.in will be different each time configure.in and related files are
changed, making it easier to check it in (which is necessary to prevent
unnecessary autoconf calls).

My declaration of initgroups() in slocal.c to eliminate the "no prototype"
warning wasn't portable (FreeBSD 3.[23] choked).  Now use AC_EGREP_HEADER to see
where initgroups() is declared, if anywhere.

Upped the version number to 1.0.3+dev (ideally this should be done by whoever
makes a release tar file, immediately after doing so).

Applied Paul Fox <pgf-spam@foxharp.boston.ma.us>'s scansbr.c patch, posted to
comp.mail.mh, which he says prevents loss of mail when inc'ing into a full
filesystem.

* Fix security hole in mhshowsbr.c which allowed untrusted shell
  code to be executed.
* Released nmh 1.0.3.

Oops.  %-escapes on mhstore lines in mhn.defaults.sh should not be surrounded by
single quotes, as a shell is not spawned when just saving files, and the
filenames will end up with literal quotes embedded in them.

Removed an item that Richard did without removing its TODO, and removed an item
that was duplicated in two different priority sections.

Whoever originally added the -help switch to all the commands got too cute and
had the option itself print out as "-(help)" in the -help output.  I guess the
idea was to make reference to the fact that clearly you know about the -help
option since you're currently looking at its output.  I think it's a bad idea to
overload the meaning of the parentheses, however -- they're supposed to indicate
what abbreviated prefix of the switch you're allowed to specify.  It doesn't
make sense to show that you're allowed to "abbreviate" the switch to its entire
length.

Whoever originally added the -help switch to all the commands got too cute and
had the option itself print out as "-(help)" in the -help output.  I guess the
idea was to make reference to the fact that clearly you know about the -help
option since you're currently looking at its output.  I think it's a bad idea to
overload the meaning of the parentheses, however -- they're supposed to indicate
what abbreviated prefix of the switch you're allowed to specify.  It doesn't
make sense to show that you're allowed to "abbreviate" the switch to its entire
length.

Modified INSTALL and config.h.in to reflect the new -kpop feature.

* Applied wesley.craig@umich.edu's KPOP patches.  According to him:

      The following patch fixes a problem with requesting a service key for a
      machine that has multiple 'A' records.  It also makes "-kpop" a command
      line option, for users who would like to use both "kpop" and "pop".

  Did no testing of the new features, as I don't have access to a KPOP server.

  One tweak I made to the patch he sent to nmh-workers was to move the
  declaration of his new hp2 variable into the #ifdef KPOP section so you don't
  get an "unused variable" warning when compiling without KPOP.

* Modified inc.man and msgchk.man to document Wesley's new -kpop.

Added check for lynx to write mhshow-show-text/html line in mhn.defaults.sh.

Made a couple of minor whitespace changes to test if the latest beta version of
CVS fixes this bug:

    dilvish-dan> cvs commit ../ChangeLog mhn.defaults.sh
    danh@mhost.com's password:
    Warning: Remote host denied X11 forwarding, perhaps xauth program could not be run on the server side.
    cvs: lock.c:179: lock_name: Assertion `*short_repos++ == '/'' failed.
    Terminated with fatal signal 6
    cvs commit: saving log message in /tmp/cvslIx_yb

All %-escapes in mhn.defaults that actually expand to something should be
surrounded by single quotes.  Added quotes to the ones in mhn.defaults.sh that
were missing them.

makedir() had multiple bugs dating back to MH.  It passed an octal constant to
atoi(), which interpreted it as decimal, resulting in directories with no user
read or execute permissions, making nested directory creation fail.  Also, when
creating a nested directory, correct permissions were only set on the inner one.

A second `make install' would always fail because the check for whether
mh_profile.5 existed was written incorrectly.

Doug's check for whether mh_profile.5 exists was bad -- it looked in the local
directory rather than the installation directory, so the check would always
return false and re-installation when mh_profile.5 did exist would bomb out.

Fixed multiple bugs in makedir().  First off, when creating nested folders, it
only set permissions properly on the innermost one.  Secondly, it passwd an
octal constant to atoi(), which only works on decimal numbers, resulting in
directories with no user read or execute permission, making creation of nested
dirs fail.

Also added a comment wondering why we do special processing when euid != uid.
If no one disagrees with my comments, I'll remove that weird code in the future.

Dan Winship informed me that the right fix when gcc -Wall complains about char
indexes to arrays is to cast to unsigned char, not int.  You want to prevent
negative indexes into the array.

Added documentation on both types of masquerading to post's man page.

Also clarified some wording and added a note that the new type of masquerading
has been implemented only for post, not for the undocumented spost backend.

Expanded the mention of the $SIGNATURE environment variable to talk about the
three different ways to override "From:" header construction.  Also added
mh-tailor(5) to the "SEE ALSO" section.

Just added a comment explaining that akvalue() does mh alias substitution.

I had my envelope-From:-setting in the wrong place in the code, before alias
processing was done.  Put it in the right place, and properly implemented
Resent-From:-handling, which I had neglected before.

Added a comment bemoaning the lack of aliasing on the From: header and
explaining why I wasn't going to bother implementing my new draft-From:-based
email address masquerading right now.

Doug's portability fix of my setgid inc autodetection had a caching bug -- if
you re-ran configure, uip/Makefile would be corrupted, and installation would
fail on OSes where inc needs to be setgid.

Implemented a new kind of email address masquerading.  When a user writes a
custom "From:" header in a draft, nmh uses it rather than generating one.
However, the user's true address is used in the SMTP envelope "From:" and is
revealed in the "Sender:" header.  Now, when mmailid is set to non-zero, the
envelope "From:" uses the address specified in the draft "From:" header, and
there is no "Sender:" header.  This is useful when sending on behalf of a remote
POP3 account or when remote mail robots incorrectly use the envelope "From:" in
preference to the body "From:".

I'm surprised this didn't hit anyone sooner.  Doug's portability fix of my
setgid inc autodetection didn't do its caching properly.  It cached the variable
$nmh_cv_ls_mail_grp (previously $nmh_group_mailspool), but did substitution on
$MAIL_SPOOL_GRP (previously $MAILGROUP) without setting the latter to the
former.  That meant that when re-running configure, $MAIL_SPOOL_GRP would be
NULL, so the $(MAIL_SPOOL_GRP) Make variable would be NULL, and installation
would fail with the error "install:  2755 does not exist" (since the argument to
-g was missing).

Implemented a new type of username masquerading, which is also activated by
setting mmailid to non-zero in mts.conf.  The new type is based on the From:
header in the message draft rather than the GECOS field of the passwd file like
the old type.

With mmailid turned off, when a user specifies a custom From: header in a draft,
it'll be used, but the user's real address will be revealed in the SMTP envelope
From: and in the Sender: header.  With mmailid on, this revelation is disabled.

Also copied to here my new comment from mh.h explaining (if I understand it
correctly) the functionality of struct swit's minchars field.  Also, changed the
minchars for "help" from 4 to 0.  It doesn't make sense to use a positive
integer that's the exact length of the option.  That'll make it print out like
"-(help)".  The parentheses are supposed to go around the minimum part of the
option you need to specify when abbreviating it.  Perhaps someone thought it
would be cute to put "help" in parentheses to imply that you know about it
already as you just specified it be seeing the current output, but I think this
"overloading" of the meaning of the parentheses is awkward.  I need to go around
and change "help" to 0 everywhere, but it's getting too late to do it tonight.

Also added some other explanatory comments to existing code.

Updated to reflect that there are now two types of username masquerading
(GECOS-based, the old one, and draft-From:-based, the new one).  Also clarified
existing wording.

Documented the minchars field of struct swit to the best of my understanding.

Modified the mmailid comment to reflect that there are now two types of
masquerading (wouldn't want sysadmins to not realize that by allowing one,
they're allowing the other).  I also prefer my new wording of "allow" vs. the
old "turn on" because the latter implied that masquerading would take effect
just by setting that variable (not the case unless /etc/passwd had been
previously modified to have the <fakeusername>s).

Whoever changed all the references to nmh-workers@math.gatech.edu to
nmh-workers@mhost.com missed this file.

Changed Mmailids to an extern so that we can access it from post.c to see if we
should do the new draft-From:-based masquerading.  Also clarified some comments
now that there are two different types of masquerading.

Just clarified a couple of points in my last change description.

The prototype I gave for initgroups() which was documented in the AIX 4.1 info
page conflicted with the real one that comes from grp.h on Solaris 2.6.
Adjusted it so the local prototype is correct on both (hopefully all)
platforms.  Also added an include of <unistd.h>, which is where initgroups()
lives on HP-UX 10.20.

I kept going back and forth checking in new pointless revisions of
configure[.in], but it now looks like the problem was that stamp-h.in was older
than configure.in (or maybe there were two problems).  Hopefully this'll get rid
of the needless auto* calls.

Shoot.  Apparently it gets the timestamp from the check-in time rather than
preserving the actual timestamp on the file when you check it in, so the
timestamp was the same for configure.in and configure, so the Makefile was doing
an autoconf just in case.  This time I'm checking in configure.in first and
configure at least a second later.  What a pain.

Accidentally checked in configure with older date than configure.in, but I
hadn't made changes to configure.in that would result in a different configure,
so I can't check one in with the right timestamp.  Therefore, I made a
meaningless change to configure.in (adding a "dnl" to the end of line that was
like others that had "dnl" at the end but didn't have one itself) and
regenerated configure.

The conditional definition of the "np" variable didn't make any sense.  It was
only defined if HAVE_GETHOSTBYNAME was defined and only used when
HAVE_GETHOSTBYNAME wasn't defined.  I'm kind of surprised that never blew up
anywhere.  Changed the definition conditional to match the usage conditional.

Got rid of the rest of the warnings that I didn't have time for on 1999-07-15
(and, it would seem, some new ones people introduced since then).  The primary
ones were the warnings that default prototypes were being used for [v]snprintf()
and str[n]casecmp().  As of right now, there are _no_ compilation warnings
except on the lex output file (at least under AIX 4.1.5 and Solaris 2.6).

On AIX 4.1, snprintf() is defined in libc.a but there's no prototype in
<stdio.h> or elsewhere.  Apparently it's not officially supported (though it
seems to work perfectly and IBM apparently uses it in internal code).  Anyhow,
if we omit our own snprintf() and vsnprintf() prototypes when we HAVE_SNPRINTF,
we get a billion warnings at compile time.  Use the C preprocessor to preprocess
stdio.h and make sure that there's actually a prototype.  Define
HAVE_SNPRINTF_PROTOTYPE if so, and use that to control our local prototype
definition.

Also, define strcasecmp() and strncasecmp() here all the time since we define
our own versions and always use them.  The only way we could get into trouble
doing this would be if the vendor's versions didn't have the same parameters,
but I don't see that happening.  If we don't define them here, we get warnings
all over the place that default prototypes are being used for 'em, and the
system header to include varies from OS to OS.

On AIX 4.1, snprintf() is defined in libc.a but there's no prototype in
<stdio.h> or elsewhere.  Apparently it's not officially supported (though it
seems to work perfectly and IBM apparently uses it in internal code).  Anyhow,
if we omit our own snprintf() and vsnprintf() prototypes when we HAVE_SNPRINTF,
we get a billion warnings at compile time.  Use the C preprocessor to preprocess
stdio.h and make sure that there's actually a prototype.  Define
HAVE_SNPRINTF_PROTOTYPE if so, and use that to control our local prototype
definition.

Added include of signals.h for SIGNAL() prototype.

Added include of mh.h for snprintf() prototype.

Added include of mh.h to get snprintf() prototype and changed all references to
getcpy() to our_getcpy() to prevent conflict with the external one.

Put in a prototype for stdio internal function _filbuf() to get rid of the
"default prototype" warning (only do this when we didn't #define it to something
specific due to another #define telling us to do so).

Put in a prototype for client() to get rid of the "default prototype" warning.
The prototype should really go in a new client.h file or somewhere, but I don't
have time right now for that.

Typecast to get rid of warnings that index into cc[] is a char.

Put in prototype for initgroups() to get rid of "default prototype" warning.  On
AIX 4.1, at least, the function exists in libc.a but doesn't exist in any .h.

Typecast to get rid of warnings that array index into types[] is a char.

* Move mhtest from bindir to libdir.

* Move sendfiles from libdir to bindir.

* Updated sendfiles manpage to reflect this change.

* Added documentation for -build and -file switches to repl and
  forw manpages (patch from Peter Maydell).

* Fixed interaction between specifying -cc in profile and -group on
  command-line.

* Add command to reverse the action of rmm [-nounlink] -- "urmm"?  "umm"?  "um"?
* Allow user customization of which kinds of text parts in multipart/alternative
  messages are preferred.  It would be nice if one selectable behavior would be
  to be prompted at runtime for which version to show.
* Add a new %-escape that makes temp files more permanent than %f, so that you
  can use gnuclient -q or netscape -remote on message parts.  Put netscape
  -remote in mhn.defaults for text/html once this is done.

Changed the version number from 1.0.3 to 1.0.2+dev.  There was not unanimous
support for my proposed even/odd release/developer version number dichotomy.
1.0.2+dev implies release 1.0.2 plus some development.

Upped the version number to 1.0.3.  If we don't do this, then when people report
bugs against 1.0.2, we won't know "which" 1.0.2 they're talking about (since the
development source is publically available via CVS).  I think the Linux kernel
version numbers are a good model, so the next time we roll a tarball, it should
be version 1.0.4 (or higher -- anyhow, an even-numbered version).

updated INSTALL directions

release nmh-1.0.2

updated the TODO list and bumped the revision to 1.0.2.

fixed test in Makefiles for LIBTOOL for the case where it doesn't exist

fixed libtool check, sort of...

changes to allow building under Mac OS X/Rhapsody

* Changed repl defaults to partly revert to MH behaviour:
  "-cc all" is now only implied with -group.
* The replcomps template includes cc: header again (as in MH).
* Updated repl man page to reflect these changes.

sunos/solaris modifications

replaced the BIND guess (based on BSDishness) with checks for gethostbyname and sethostent

fixed bug in mh-profile/mh_profile manpage link - it did not check first

updated manpage's Makefile and modified the way configure tests to see whether
it should define LINUX_STDIO.

Added '-L' to the calls of 'ls' in configure.in -- I have seen multiple machines
in the past where the mail spool was a symlink to a directory on another device
with more free space.

* Fixed a portability problem in Doug's fix of a portability
problem in my MAILGROUP autoconf support ('ls -l' vs. 'ls -lg').

I didn't like the double use of "MAILGROUP" for two different variables used in
different languages, with different values.  I renamed Doug's new Makefile
"MAILGROUP" to MAIL_SPOOL_GRP.

Touched because I changed configure.in.

Doug's fix for my MAILGROUP autoconf support on his platform (yep, I was
ignorant about that "wheel" group) broke it on my platforms (AIX 4.1.5.0.01 and
Solaris 2.6).  `ls -lg` doesn't do what Doug was expecting on these platforms --
it displays _only_ the group rather than the user and the group (thus awk
'{print $4}' did the wrong thing).

Also did some miscellaneous cleanup, like making variables that were intended to
be cached actually be cached (renamed to nmh_cv_*, etc.), and using the $AWK
that we've auto-detected rather than relying on the $PATH to find one.

* Added config.sub and config.guess to the list of files to be
distributed.

* Fixed bug in sbr/fmt_scan.c where an extra newline would be
added if a list of addresses was split over several header lines.

* In mts/smtp/smtp.c, undefine strlen and strcpy if they are
macros, regardless of platform.

* Allow q to quit mhshow, and n to skip to next part.  Patch from
Kimmo Suominen <kim@tac.nyc.ny.us>.

* Modified mhstore to recognize attachments created by sendfiles
with x-conversions=gzip.

moved canonical guesses back into user config section of config.h

updates to post.c and termsbr.c that prevent warnings on OpenBSD

Lots of little code cleanups to prevent warnings - mostly making sure
some function prototypes are available at compile time. Also fixed
Makefile.in so that make nmhdist will work -- it just blew up after
COMPLETION-TCSH was added and ZSH-COMPLETION was changed to
COMPLETION-ZSH.

updating changelog, should have been done with last commit

fix for configure to find db1/ndbm.h in new linux systems

configure, configure.in stamp-h.in, and uip/Makefile.in modified to fix
  MAILGROUP test.
m_getfld.c - added the patch submitted by Richard Cohen to fix
    crashing when mh_profile has no trailing newline.
smtp.c, lock_file.c, m_scratch.c, m_tmpfil.c, mhscmds.c, rcvtty.c, and
  spost.c modified to fix mkstemp bugs. Some I've simply commented out
  because the repair requries either creating another new tempfile
  function (there are 3 already) or cleaning up the existing tmpfile
  functions and all the calls to them. I'm opting for the latter but
  it will take a while.

Renamed ZSH.COMPLETION to COMPLETION-ZSH and added COMPLETION-TCSH.

New file along the lines of the old ZSH.COMPLETION, but for tcsh.  Pretty bare
right now.

Renamed to COMPLETION-ZSH so it'll alphabetize next to new COMPLETION-TCSH.

Renamed from ZSH.COMPLETION so it'll alphabetize next to new COMPLETION-TCSH.

Crap.  That's what I get for using vi instead of xemacs for a change.  Inserted
missing newline.

Just adding some text to this file so that cvs will see a difference and let me check it in.  Every time acconfig.h or configure.in are changed (which I just
did), the timestamp on this file needs to be updated or when people do a make
an unnecessary `cd . && autoheader` will be done, and not everybody has the
autoconf tools installed.  The next person to change those two files (or
aclocal.m4) will need to change this text, maybe just to a message saying which
dependency files were changed and by whom.

Automated #define of MAILGROUP and installation of inc as setgid mail when the
mail spool directory isn't world-writable.

Automated #define of MAILGROUP and installation of inc as setgid mail when the
mail spool directory isn't world-writable.

Just removed some useless whitespace that [X]Emacs' Make mode highlights as
being semi-bogus.

Automated #define of MAILGROUP and installation of inc as setgid mail when the
mail spool directory isn't world-writable.

* slocal -debug used to leave a copy of each message processed in /tmp.
* Got rid of a ton of compilation warnings output by gcc -Wall.
* rcvdistcomps no longer has "Resent-Fcc: outbox"; new rcvdistcomps.outbox does.

Eliminated compilation warnings.

Eliminated unsued sigset_t variables set and oset.

Previously contained the code:

if (numburst == 0)
    if (!quietsw)
admonish (NULL, "message %d not in digest format",
  msgnum);
else
    adios (NULL, "burst() botch -- you lose big");

If the indentation is to be believed, this is a bug.  I trusted the indentation
and added {}s around the inner if().

Eliminated compilation warnings.  pop_action() and pop_pack() used to return
garbage.  They now (arbitrarily) return 0.  Dunno if their return values are
ever used...

Eliminated compilation warnings.

donote() previously was written such that it could return garbage.

err2peer() previously returned garbage.  Now it returns result of callee
verr2peer().

Eliminated compilation warnings.

Eliminated compilation warnings.

Eliminated compilation warnings.

slocal -debug left a file in /tmp for every single message it processed!
Totally unacceptable since some people like to have slocal -debug in their
.forward files to get better logging.

Changed done() from returning void to returning int so that main()s who call
done() at the end can instead return done() at the end to eliminate the
compilation warning about falling off the end of a non-void function.

Changed from returning void to returning int so that main()s who call done() at
the end can instead return done() at the end to eliminate the compilation
warning about falling off the end of a non-void function.

Changed call of done() to return of done() to eliminate compilation warning.