added lint targets for Makefiles and a configure test to find whether lclint or lint...
[mmh] / ChangeLog
index 9caf15e..2b03f0a 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,347 @@
+Wed May 31 07:40:45 2000 Doug Morris <doug@mhost.com>
+
+       * added a lint target to the Makefiles and a check in autoconf
+       to determine whether lint or lclint exists on the system. 
+
+Fri May 30 19:21:48 2000 Dan Harkless <dan-nmh@dilvish.speed.net>
+
+       * etc/Makefile.in was incorrectly installing mts.conf.in and
+       sendfiles.in -- fixed.  Generated sendfiles script was not a
+       dependency of the `all' target, and was incorrectly included in
+       the distribution.  Changed the suffix for the backed-up previous
+       versions of the etc files from the ambiguous .old to .prev.  Added
+       call to diff -- only keep the .prev files around if different from
+       the newly-installed versions (intentionally didn't redirect output
+       to /dev/null so you'll notice when your changed versions are
+       getting moved aside).
+
+       * INSTALL never documented the etc/*.old thing.  Documented the
+       new etc/*.prev thing (including a note to watch for diff output).
+
+       * 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.
+
+Mon May 29 7:48:15 2000 Shantonu Sen <ssen@mit.edu>
+
+       * Moved the date parsing routines from zotnet/tws to sbr/ (and
+       tws.h to h/). Updated all source files to reflect to new location
+       of tws.h.
+
+       * Rewrote dparsetime (dtimep.c) to replace the old
+       zotnet/tws/dtimep.c, lexstring.c, lexedit.c, and
+       dtimep.c-lexed. It should now work with flex (although untested
+       with lex), and requires no sed-ing. For now, I have the lexed
+       version in the distribution, so that end-users don't need to worry
+       about running it through flex/lex. I have not added back support
+       for guessing the time zone when it's not specified.
+       
+Sun May 28 17:44:15 CEST 2000 Ruud de Rooij <ruud@ruud.org>
+
+       * 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.
+
+Sun May 28 14:58:49 CEST 2000 Ruud de Rooij <ruud@ruud.org>
+
+       * 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.
+
+Sun May 28 14:39:31 CEST 2000 Ruud de Rooij <ruud@ruud.org>
+
+       * Fixed m_getfld bug which caused segmentation faults when
+       incorporating messages which ended in multiple linefeeds crossing
+       a buffer boundary.
+
+Fri May 26 13:21:59 2000 Dan Harkless <dan-nmh@dilvish.speed.net>
+
+       * 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.
+
+Fri May 12 02:51:21 2000 Shantonu Sen <ssen@mit.edu>
+
+       * zotnet/bboards is not longer built by default. Goal is to move
+       the assorted functions in zotnet into sbr or some more logical
+       place.
+
+       * Moved zotnet/mf to sbr, and changed mf.h references accordingly,
+       as well as Makefiles.
+       
+Thu May 11 02:21:34 2000 Shantonu Sen <ssen@mit.edu>
+
+       * Simplified sbr/Makefile.in so that both SRCS and OBJS aren't
+       seperately and redundantly defined, but so that OBJS is a
+       pattern-substituted version of SRCS with suffix .c -> .o. This
+       should make maintainability easier.
+
+       * Added section to MACHINES indicating what platforms nmh is known
+       to compile and work on, just to give users peace of mind, or
+       something. This is by no means complete or exhaustive, so add
+       whatever you know works.
+       
+Tue May 09 20:38:04 2000 Dan Harkless <dan-nmh@dilvish.speed.net>
+
+       * 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).
+
+       * Fixed 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.
+
+       * 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.
+
+       * 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()).
+
+       * Prepended "Portions of this code are" to the copyright message
+       in ruserpass.c also.
+
+       * 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.
+       
+Mon May 08 23:51:55 2000 Dan Harkless <dan-nmh@dilvish.speed.net>
+
+       * 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'.
+
+       * 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.
+
+       * Changed DIFFERENCES to say that RPOP is not currently supported
+       rather than implying it by saying that APOP, KPOP, and POP[3] are.
+
+Sun May 07 18:16:43 2000 Shantonu Sen <ssen@mit.edu>
+
+       * Imported NetBSD version of getpass() and made extensive
+       revisions for compatibility with programs that pipe the password
+       to stdin, such as exmh.
+
+       * Removed tests for system ruserpass() which sometimes gave
+       phantom positive results. Also, bext to use internal functions if
+       we ever want to change .netrc format to something else, or access
+       other files.
+
+Sat May 06 08:28:09 2000 Dan Harkless <dan-nmh@dilvish.speed.net>
+
+       * Restored lost version histories for those moved files by doing a
+       manual `mv' in the CVSROOT on mhost.  CVS badly needs a `cvs mv'
+       command so that you can move files (without having physical access
+       to the CVSROOT) without losing versioning.  Put MACHINES back at
+       the top level as it needs to be read before building.  Fixed DIST
+       variable in {.,docs}/Makefile.in to reflect that and to add
+       missing entry for "INSTALL" file.
+
+Sat May 06 13:13:07 2000 Doug Morris <doug@mhost.com>
+
+       * Re-cleaned up nmh documentation (by moving things to docs
+       subdir) and modified Makefile & configure.in to handle the change.
+
+Mon Apr 17 21:28:40 2000 Dan Harkless <dan-nmh@dilvish.speed.net>
+
+       * Scott Blachowicz pointed out that the configure --help output
+       for --enable-masquerade was misleading.  Clarified.
+       
+Mon Apr 17 19:01:00 2000 Shantonu Sen <ssen@mit.edu>
+
+       * APOP support can be compiled in to inc and msgchk using
+       --enable-apop.
+
+       * To access an APOP host, specify -apop on the command line
+       along with any -host or -user option.
+
+Fri Apr 14 23:10:44 2000 Dan Harkless <dan-nmh@dilvish.speed.net>
+
+       * Upped the version number to 1.0.4+dev until the next nmh release.
+
+       * Added a "releasing nmh" section to README.developers, while the
+       process was fresh in my mind.
+       
+Fri Apr 14 18:21:34 2000 Dan Harkless <dan-nmh@dilvish.speed.net>
+
+       * Added new files README.developers, ChangeLog_MH-3_to_MH-6.6, and
+       ChangeLog_MH-6.7.0_to_MH-6.8.4.html to DIST target in Makefile.in.
+       
+       * Released nmh-1.0.4.
+
+Tue Apr 11 21:37:03 2000 Dan Harkless <dan-nmh@dilvish.speed.net>
+
+       * Applied Brian Campbell <bacam@tardis.ed.ac.uk>'s mhn.defaults.sh
+       patch:
+
+           It appears that there shouldn't be quotes around the %s in the
+           iso-8859-1 charset entry; xterm passes the remaining arguments
+           to the program, quoting them means that xterm thinks they're
+           part of the program's name.
+
+       This %s isn't the same as the "Insert content subtype" one from
+       mhshow-show-* -- it doesn't come from MIME headers and is safe not
+       to quote.
+
+Sun Apr 09 13:03:59 2000 Doug Morris <doug@mhost.com>
+
+       * added check in fmt_compile() to handle a single-character
+       format string. fmt_compile() depends on having an array of
+       format characters with an empty item at the end. A
+       single-character format would cause programs using this
+       function to segfault because the algorithm used to decide on
+       the length of the array mistakenly created a single-item array
+       when the format string was one character. This eventually
+       caused problems when the program attempted to test item+1
+       in the array.
+
+Thu Apr 06 21:53:50 2000 Dan Harkless <dan-nmh@dilvish.speed.net>
+
+       * Richard Coleman threw out a lot of old MH-specific files in nmh.
+       Much of the stuff, indeed, is not worth saving, but there are
+       nuggets that are very worthwhile, and should probably be added
+       back in.  Most important, IMHO, are the MH change logs, as they
+       can help answer questions like "Why is this code like this?" or
+       "How long has this been broken?" or "What was this ever used for?"
+
+       I've added a new file to the nmh tree called
+       ChangeLog_MH-3_to_MH-6.6.  It's cobbled together from the
+       mh-6.8.4/papers/mh*/MHCHANGES files.  I've re-ordered the entries
+       to go from newest at the top to oldest at the bottom to match the
+       ChangeLog convention.  Unfortunately there are no change logs for
+       versions of MH prior to 3 in the MH tar files available at
+       <ftp://ftp.ics.uci.edu/pub/mh>.  Also, it appears to me that there
+       are MH-6.6 changes that aren't documented in the logs.
+
+       I've also added ChangeLog_MH-6.7.0_to_MH-6.8.4.html.  This is
+       based on mh-6.8.4/papers/changes/mh-changes.ms.  The nroff format
+       and its "catman"-type output are a pain to deal with, but I was
+       loath to throw away the formatting, so I converted the file to
+       HTML.  The only actual markup in the body are the "<B>" and "<U>"
+       tags, and "&lt;" and "&gt;" instead of '<' and '>', so it's quite
+       doable to view the file in plain ASCII mode as well.  Note that
+       some of the changes this file documents as having been made in
+       MH-6.8.4 may not be present in nmh -- Richard started with 6.8.3
+       and later put in certain 6.8.4 stuff.
+       
+Wed Apr 05 21:09:28 2000 Dan Harkless <dan-nmh@dilvish.speed.net>
+
+       * Applied Eric Schnoebelen <eric@cirr.com>'s mhshowsbr.c patch
+       fixing apparent bugs in Dan Winship's new security quoting code:
+       
+            Since upgrading, I've been getting the following errors
+            while attempting to process some MIME messages:
+
+                (1) Syntax error: Unterminated quoted string
+                    exit 2
+       
+            and:
+
+                (2) line 1/10 (END)Segmentation fault (core dumped)
+
+           (2) appears to be due to the testing of an unset pointer in
+           mhshowsbr.c:show_multi_aux().  (1) appears to be caused by
+           mis-quoting a filename being handed to the shell in
+           mhshowsbr.c:show_content_aux().
+
+           Resolving the pointer reference issue in
+           mhshowsbr.c:show_multi_aux() turned up a similar mis-quoting
+           problem in the routine.
+
+Tue Mar 28 16:17:39 2000 Doug Morris <doug@mhost.com>
+
+       * Applied Todd.Miller@courtesan.com's patch to dropsbr.c to
+       prevent core dumping on packf. Here's the note from his message:
+
+           Since sizeof(buffer) == sizeof(tmpbuffer) packf will dump
+           core on a file w/o a From line with a line >= BUFSIZ.
+           I noticed this because I had a junk file in my mail
+           spool somehow.
+
 Fri Mar 17 11:59:33 2000 Dan Harkless <dan-nmh@dilvish.speed.net>
 
        * wesley.craig@umich.edu did not document his previous KPOP patch,
@@ -227,13 +571,18 @@ Fri Feb 04 12:29:12 2000 Dan Harkless <dan-nmh@dilvish.speed.net>
 
        * 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
+       the -help output.  One theory is that they were making 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.
+       allowed to specify.
+
+       The other theory is that because you can say something like
+       "mhstore -" and get "mhstore: - ambiguous.  It matches" followed
+       by the same list of switches you get with -help, they were saying
+       you can "sorta" abbreviate "-help" as "-".  You don't get the
+       "Usage:" string, though, so it's not really the same thing.
 
 Thu Feb 03 17:52:01 2000 Dan Harkless <dan-nmh@dilvish.speed.net>
 
@@ -244,8 +593,8 @@ Thu Feb 03 17:52:01 2000 Dan Harkless <dan-nmh@dilvish.speed.net>
              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.
+       Did no testing of the new features, as I don't have access to a
+       KPOP server.
 
        * Modified inc.man and msgchk.man to document Wesley's new -kpop.
 
@@ -474,7 +823,7 @@ Sun Sep 12 09:19:27 1999 Doug Morris <doug@mhost.com>
        * Added check for <db1/ndbm.h> which is the new location where
        linux systems appear to be stuffing this header file.
 
-Thu Sep  9 23:15:49 1999 Doug Morris <doug@mhost.com>
+Thu Sep 09 23:15:49 1999 Doug Morris <doug@mhost.com>
 
        * fixed varous mkstmep bugs introduced in 1.0.1 by me. Whups!
 
@@ -508,7 +857,7 @@ Thu Jul 15 18:37:07 1999 Dan Harkless <dan-nmh@dilvish.speed.net>
        * Default rcvdistcomps no longer puts a copy of all outgoing
        messages in outbox.  Added an rcvdistcomps.outbox that does.
 
-Sat Jun  9 12:22:47 1999 Doug Morris <doug@mhost.com>
+Sat Jun 09 12:22:47 1999 Doug Morris <doug@mhost.com>
 
        * Updated configure to check for mkstemp (available on OpenBSD) and 
        substitute it for mktemp if available.