Mon Dec 30 15:43:48 1996  Abe Feldman  <feldman@harvey.cyclic.com>

	* checkout.c (build_dirs_and_chdir): Reproduced block containing
	Create_Admin, placing it before Subdir_Register.
	* sanity.sh (basicb): Added tests 1a and 9a to test above changes
	to the checkout command.

Mon Dec 30 13:29:14 1996  uz@wuschel.ibb.schwaben.com (Ullrich von Bassewitz)
			  and Jim Kingdon  <kingdon@harvey.cyclic.com>

	* cvs.h (CVSEDITPREFIXLEN): New define.
	* logmsg.c (do_editor): Use CVSEDITPREFIXLEN when deciding whether
	to strip off CVSEDITPREFIX and when telling the user what we will
	strip off.

Sun Dec 22 22:06:49 1996  Jim Kingdon  <kingdon@harvey.cyclic.com>

	* logmsg.c (do_verify): If noexec, skip the verification *without*
	printing a message.  Use cvs_output not printf.  Skip verification
	for client_active.

Wed Dec 18 12:27:35 1996  Jim Kingdon  <kingdon@harvey.cyclic.com>

	* repos.c (Name_Repository): Add comment regarding wording of
	"*PANIC*" error message.

1996-12-18  Jim Kingdon

	* client.c (call_in_directory): If the directory we are about
	to create is the same as CVSADM as seen by fncmp (for example,
	it is "cvs" and filenames are case-insensitive), then give a
	fatal error.

Tue Dec 17 13:14:22 1996  Jim Kingdon  <kingdon@harvey.cyclic.com>

	* options.h.in: Add comments about SETXID security holes.

	* logmsg.c (do_verify): Reindent comments.  Check errno if return
	code from run_exec is -1, not if it is 1.
	* sanity.sh (info): Move tests info-4 and info-8 to end and rename
	them.  Add verifymsg tests.  Instead of forcibly removing loginfo,
	remove it nicely (test info-11).

Tue Dec 17 12:45:32 1996  Abe Feldman  <feldman@cyclic.com>

	* commit.c, import.c: Call do_verify as well as do_editor.
	* cvs.h (CVSROOTADM_VERIFYMSG): Define.
	* logmsg.c, cvs.h (do_verify, verifymsg_proc): New functions.
	(verifymsg_script): New variable.
	* mkmodules.c (filelist): Add CVSROOTADM_VERIFYMSG.

Mon Dec 16 13:24:47 1996  Ian Lance Taylor  <ian@cygnus.com>

	* lock.c (remove_locks): New static function, copied from part of
	Lock_Cleanup.
	(Lock_Cleanup): Call remove_locks.
	(Writer_Lock): Call remove_locks rather than Lock_Cleanup when
	waiting for a lock.

Thu Dec 12 10:36:37 1996  Jim Kingdon  <kingdon@harvey.cyclic.com>

	* login.c (get_cvs_password): If CVS_PASSWORD is set, print a
	warning (and then proceed to ignore it).  It was a documented
	feature, so we should point people who were using it to the
	replacement.

Mon Dec  9 12:35:43 1996  Ian Lance Taylor  <ian@cygnus.com>

	* server.c (server_updated): Change comment to only worry about
	umask in the rsh case.
	(server): Create the temporary directory, and change the mode to
	S_IRWXU.
	(switch_to_user): Set the umask to 0, not 077.

Mon Dec  9 10:58:28 1996  Jim Blandy  <jimb@floss.cyclic.com>

	* login.c (get_cvs_password): Remove code to check for value of
	CVS_PASSWORD.  Keeping cleartext passwords in environment
	variables is a really bad idea on Unix, since anyone can print
	out a processes' environment using 'ps' (on BSD variants
	anyway).  Update help message.

Fri Dec  6 15:59:40 1996  Jim Kingdon  <kingdon@harvey.cyclic.com>

	* sanity.sh: When matching "use .cvs commit. to remove this file
	permanently" messages, change "cvs" to "${PROG}".
	(rdiff, binfiles): Likewise.
	This fixes testing a program named something other than "cvs", e.g.
	$ cp cvs cvs-test
	$ /bin/sh <srcdir>/sanity.sh `pwd`/cvs-test

1996-12-02  Jim Kingdon

	* client.c: In comment saying that socket buffers don't
	implement the blocking routine, say they are blocking.
	* buffer.h (struct buffer): In description of input function,
	describe blocking, non-blocking, and NEED more fully.  Say
	what happens if we read a nonzero amount less than NEED and
	then get end of file.
	* client.c (socket_buffer_input): If NEED == 0, still call
	recv (once).  Handle the case where recv returns 0.

Sat Nov 30 15:10:07 1996  Jim Kingdon  <kingdon@harvey.cyclic.com>

	* subr.c, cvs.h (file_has_markers): New function.
	* rcs.h (RCS_MERGE_PAT): Now a fixed string not a regexp.
	* options.h.in (GREP): Removed; no longer used.
	* update.c (update_fileproc), commit.c (check_fileproc): Call
	file_has_markers rather than GREP.
	* rcscmds.c (RCS_merge): Just give a fatal error in the case where
	we had been calling GREP.  I suspect noone is using this code
	any more.
	* sanity.sh (conflicts): Rewrite tests 131, 132, and 133 to use
	dotest; tests that the above changes didn't break anything.

Fri Nov 29 09:06:41 1996  fnf@ninemoons.com (Fred Fish)
			  and Jim Kingdon  <kingdon@harvey.cyclic.com>

	* checkout.c (safe_location): Only call readlink if HAVE_READLINK.

	* run.c (piped_child, filter_stream_through_program): If
	HAVE_VFORK, call vfork not fork.
	* run.c (run_exec): Add comment about why we use vfork.

Mon Nov 25 12:48:31 1996  Jim Kingdon  <kingdon@harvey.cyclic.com>

	* release.c (release): Don't return after processing the first
	argument; that kind of defeats the purpose of having a loop, eh?
	For client, close the connection after we've processed them all.
	* sanity.sh: Remove workaround for modules2-8 test; tests for
	above fix.  Adjust modules2-6 test to answer both questions.

	* login.c: Reindent (all of get_cvs_password, a handful of lines
	elsewhere).

	Cleanups to release, mostly cosmetic:
	* release.c (release_server): New function; breaks out server code
	from the release function.
	* release.c: Move delete_flag inside the release function.
	* release.c (release): Reindent.  Rewrite comments about how the
	implementation could be improved.  Don't declare variables as
	"register".  Include errno in error message.  Don't cast result of
	printf to void.  Remove unused variable srepos.
	* release.c: Remove comments at top of file about what it does.
	They were not particularly coherent and they were also out of date
	(I think).  Likewise for comment in release function about "if we
	are in a repository".
	* release.c: Change "module" to "directory" in a few messages
	since that is what is meant.
	* sanity.sh: In tests ignore-195 and ignore-193, change expected
	message accordingly.

Sun Nov 24 11:30:55 1996  Jim Kingdon  <kingdon@harvey.cyclic.com>

	* sanity.sh: Clarify a few items in the todo list.

	* log.c (log_parse_date): Use the "end" of the epoch not "next
	week" as the time which means "no end time".
	* sanity.sh (rcs): New test, tests dates and importing RCS files.

1996-11-19  Jim Kingdon

	Visual C++ lint:
	* hash.c: Declare qsort_comp.
	* update.c: Declare isremoved.
	
1996-11-19  Jim Kingdon  <kingdon@harvey.cyclic.com>

	* root.c, repos.c, modules.c, create_adm.c: Change all calls to
	strip_path to strip_trailing_slashes.  Basically strip_path is
	just an unneeded complication (we should keep the pathname the way
	the user specifies it, and the system can worry about things like
	consecutive /'s if it wants to).  Stripping trailing slashes is
	potentially dubious for the same reason, but it is a somewhat
	different case which I won't try to tackle now.
	* cvs.h (strip_path): Remove declaration.

Tue Nov 19 15:18:13 1996  Jim Kingdon  <kingdon@harvey.cyclic.com>

	There are a lot of details to this change, but the concept is
	relatively simple: make it so that for every CVSLCK lock that we
	might take out, there is a flag which is set iff we have created
	the CVSLCK directory.
	* lock.c (struct lock): New structure.
	* lock.c: Remove static variables repository and cleanup_lckdir.
 	They are replaced by global_readlock.repository and
	global_readlock.have_lckdir, respectively (except insofar as the
	rest of these changes change the concept of cleanup_lckdir).
	New static variable global_readlock.
	(Reader_Lock, Lock_Cleanup): Use global_readlock in place of
	repository.
	(lock_simple_remove, set_lock, clear_lock, write_lock): Take a
	struct lock * instead of just a repository.  Set/clear
	lock->have_lckdir instead of cleanup_lckdir.
	(set_writelock_proc, unlock_proc): Pass ->data, not ->key, to
	write_lock or lock_simple_remove.
	(lock_filesdoneproc,lock_dir_for_write): Allocate a struct lock,
	put it in the ->data field, and fill in its fields.
	(lock_simple_remove): Use lock->have_lckdir as the sole test for
	whether the CVSLCK directory needs to be removed.  Add
	comments about why readlock and writelock variables don't tell us
	for sure whether locks exist.
	(lock_simple_remove, clear_lock): Use SIG_beginCrSect and
	SIG_endCrSect to ensure that ->have_lckdir is set to 0 iff the
	CVSLCK directory was really removed.
	(lock_simple_remove): Check for errors removing CVSLCK directory.
	(lock_simple_remove, Check_Owner, set_lock): Remove all code which
	checks userids (including all of Check_Owner and all the AFCVS
	code).  It was bogus if several CVS processes with the same userid
	were running (common if several users share a userid; a common
	practice with remote CVS), and with the rest of the changes here
	should not be needed.

1996-11-16  Paul Eggert  <eggert@twinsun.com>

	* rcs.c (RCS_deltas): Fix unintended trigraphs.

Fri Nov 15 13:06:03 1996  Tom Hageman  <tom@basil.icce.rug.nl>

	* diff.c (diff_fileproc): In printing error messages, use the
	correct filename for which the error occurred.

Sun Nov 10 21:13:38 1996  Paul Sanders <p.sanders@dial.pipex.com>
			  and Jim Kingdon  <kingdon@harvey.cyclic.com>

	* server.c: Use all the right cruft which goes along with
	including sys/time.h.

	* server.c: Include a "copyright" notice.

	* server.c: If HAVE_WINSOCK_H, include winsock.h.

	* server.c (server): Only set a handler for SIGHUP if it is
	defined.  Likewise for all the other signals.

	* server.c (do_cvs_command): Use DEVNULL not /dev/null.

Fri Nov 08 12:14:20 1996  Jim Kingdon

	IBM ICC (OS/2) lint:
	* add.c (add): Only declare begin_added_files if
	SERVER_SUPPORT.
	* client.c (init_sockaddr): Change port argument from
	unsigned short to unsigned int.  Change hostname
	argument from const char * to char *.

Sun Nov  3 18:24:28 1996  Noel Cragg  <noel@gargle.rain.org>

	* sanity.sh (info): add new tests that check behavior of format
 	string substitution in loginfo file.

Sat Nov  2 09:39:09 1996  Jim Kingdon  <kingdon@harvey.cyclic.com>

	* client.c (do_deferred_progs): Don't access memory once it is
	freed (we already did it right for checkin_progs; do the same
	thing for update_progs).

	* update.c, client.c, classify.c, client.h, diff.c, commit.c,
	create_adm.c: Nuke more PATH_MAX.

Fri Nov  1 18:22:32 1996  Jim Kingdon  <kingdon@harvey.cyclic.com>

	* error.h: Define PROTO if it is not defined.

Wed Oct 30 08:53:20 1996  jalving@ibm.net

	* patch.c (patch_fileproc): Set line1 and line2 to NULL up-front
	(before the first "goto out") so we don't try to free them.

Wed Oct 30 08:53:20 1996  Jim Kingdon  <kingdon@harvey.cyclic.com>

	* commit.c (remove_file, commit_filesdoneproc), run.c (run_print,
	run_exec), modules.c (open_module, cat_module, do_module), update.c
	(update_dirleave_proc), tag.c (tag_fileproc): Call cvs_out* rather
	than stdio.
	* server.c (serve_expand_modules): Remove comment about do_module
	writing to stdout/stderr; above changes should fix this.

Tue Oct 29 17:23:59 1996  Ian Lance Taylor  <ian@cygnus.com>

	* status.c (tag_list_proc): When printing the tag name, don't
	truncate it to 25 characters.

Tue Oct 29 12:49:07 1996  Jim Kingdon  <kingdon@harvey.cyclic.com>

	* add.c, checkin.c, checkout.c, filesubr.c: Nuke arbitrary limit
	of PATH_MAX.  Many more such limits surely remain.

	* fileattr.c (fileattr_set): Set attrs_modified *after* we might
	call fileattr_read, because fileattr_read clears it.
	* sanity.sh (devcom2): New tests, test for above fix and other
	behaviors I discovered in the process of looking into it.

Mon Oct 28 08:55:57 1996  Jim Kingdon  <kingdon@harvey.cyclic.com>

	The following changes are to ensure that SYSTEM_CLEANUP is always
	called.
	* error.c, cvs.h, main.c: Remove error_set_cleanup and related
	machinery.  It was for a time when error.c was intended to be
	shared with other programs, but that is no longer true.
	* error.c, error.h (error_exit): New function; like error_cleanup
	from main.c but also calls SYSTEM_CLEANUP and exit (EXIT_FAILURE).
	* error.c (error, fperror): Call error_exit instead of doing it
	ourself.
	* server.c (server, serve_valid_responses, switch_to_user,
	check_password, pserver_authenticate_connection, 
	kserver_authenticate_connection): Call SYSTEM_CLEANUP before exit.
	* add.c, client.c, import.c, main.c, mkmodules.c, modules.c,
	recurse.c, server.c, tag.c, update.c: Call error_exit ()
	instead of exit (EXIT_FAILURE).

Sun Oct 27 08:34:16 1996  Jim Kingdon  <kingdon@harvey.cyclic.com>

	* sanity.sh (conflicts): New test 128.5 tests "cvs co -p" in an empty
	directory (like 126.5), but when the file has nonempty contents.
	* rcs.c (RCS_checkout): If writing to stdout, use cvs_output
	rather than fwrite.
	* update.c (checkout_file): Call cvs_stderr not fprintf.
	These changes should fix some out-of-order bugs which show up in
	situations like conflicts-126.5 and conflicts-128.5.

	* mkmodules.c (checkout_file): Call RCS_checkout rather than
	run_exec on RCS_CO.

Sat Oct 26 18:29:46 1996  Jim Kingdon  <kingdon@harvey.cyclic.com>

	* sanity.sh (rdiff): cd out of testimport directory and remove it
	when done.

	* sanity.sh (info): Adjust tests to reflect fact that loginfo was
	created by cvs init.

	* sanity.sh (ignore): Change test 187a1 to allow any number of
	files in CVSROOT, not just modules.

	* sanity.sh (modules): In tests 148a0 and 148a1, don't expect a
	module which defines CVSROOT to itself, since we don't define one
	any more.  Also change test to rewrite modules rather than append
	to it (in case any previous tests are changed to do something with
	modules).  Change test 155b to allow any number of files in
	CVSROOT, not just modules.

	* add.c (add_directory): Set rev_old and rev_new fields of struct
	logfile_info to NULL (prevents us from trying to free them later).
	* commit.c (find_fileproc), import.c (import): Likewise.

	* sanity.sh (crerepos): New tests, to test alternate ways of
	creating a repository and related matters.
	* sanity.sh: Remove tests 1 through 3 and related cruft; replace
	them with a new test 1 which merely tests "cvs init".  By doing
	the obscure stuff in crerepos we avoid having to do all this stuff
	any time we run any single test.

Sat Oct 26 16:19:48 1996  Jim Blandy  <jimb@totoro.cyclic.com>

	* main.c (main): If HAVE_TZSET is #defined, call tzset.  This is
	harmless on all systems I know of, and required on some.

Fri Oct 25 13:20:44 1996  Ian Lance Taylor  <ian@cygnus.com>

	* diff.c (diff_file_nodiff): When setting use_rev1, only return
	DIFF_SAME if empty_file is DIFF_DIFFERENT and ts_user is not
	NULL.  Don't get confused by a vn_user field of "0" or one
	starting with '-'.
	* sanity.sh (death2): Add new death2-diff-{1,2,7,8} tests for
	above patch.  Renumber existing death2-diff tests to make room.

Fri Oct 25 12:38:29 1996  Jim Wilson  <wilson@cygnus.com>

	* sanity.sh (death2): In tests death2-diff-{2,4,6,8,10}, allow "_"
	in temp file names.  The system (tmpnam or whatever) generates
	these names so they vary from system to system.

Fri Oct 25 07:52:44 1996  Jim Kingdon  <kingdon@harvey.cyclic.com>

	* logmsg.c (logfile_write): Give an error for several cases which
	should not be legal.  Adjust comments accordingly.
	* mkmodules.c (loginfo_contents): Make description of loginfo much
	more concise.  This should be a reminder, not full documentation.

Tue Oct 22 10:37:37 1996  Noel Cragg  <noel@gargle.rain.org>

	* commit.c (update_delproc): free structure members rev_old and
 	rev_new if they have been allocated.

	* mkmodules.c: change loginfo_contents to include a description of
 	the new format string.

	* logmsg.c (logfile_write): change syntax of format string.

Sat Oct 19 16:09:55 1996  Jim Kingdon  <kingdon@harvey.cyclic.com>

	For reference, this takes CVS's text segment from 348348 bytes to
	347420 bytes.
	* server.c (requests): Change Directory to rq_essential
	per change in doc/cvsclient.texi.
	* client.c: Remove use_directory and all code which executed if
	it wasn't set.  This includes the get_short_pathname function.
	* server.c: Likewise, for use_dir_and_repos.
	(serve_repository): Give a fatal error.
	* server.c (requests): Remove Lost.  Change Unchanged to rq_essential.
	(serve_lost): Removed.
	* server.c, server.h, client.c, vers_ts.c: Remove use_unchanged,
	code to set it, and all code which executed if it wasn't set.

Sat Oct 19 12:44:08 1996  J. Richard Sladkey  <jrs@world.std.com>
			  and Jim Kingdon  <kingdon@harvey.cyclic.com>

	* hash.c (sortlist, new function qsort_comp): Rewrite to use qsort
	instead of insert sort.  Changes algorithm from n^2 to n log n
	(assuming qsort is implemented with quicksort or similar).

Sat Oct 19 12:44:08 1996  Jim Kingdon  <kingdon@harvey.cyclic.com>

	* sanity.sh (basic2): In test basic2-64, use -x and specify types
	which exclude E; the test is not built to deal with E (or any
	other new types).

Sat Oct 19 12:00:00 1996  Mark Mitchell <mmitchell@usa.net>
			  and Jim Kingdon  <kingdon@harvey.cyclic.com>

	* update.c (isemptydir, new function isremoved): Avoid pruning
	directories that contain files marked for removal but not
	comitted.
	* update.c, update.h (isemptydir): Now extern, not static.
	* update.c (isemptydir): New parameter might_not_exist handles
	difference in functionality from old client_isemptydir.  Bring
	over the improved error checking from client_isemptydir.
	* client.c (client_isemptydir): Removed; isemptydir now suffices.
	* update.c (update_dirleave_proc), client.c
	(process_prune_candidates): Update callers.
	* sanity.sh (deep): Add tests deep-rm* for above fix.

Fri Oct 18 15:53:41 1996  Jim Kingdon  <kingdon@harvey.cyclic.com>

	* sanity.sh (devcom): Add tests devcom-some* to test watching just
	a single file.

	* root.c (Name_Root): Use isabsolute to test whether a pathname is
	absolute instead of checking for the first character being '/'.
	(Reported by Antoine P. Brusseau <brusseau@jprc.com>).

	* commit.c (checkaddfile): Free rev only if it is non-NULL (thanks
	to cwong@world.std.com (Christopher Wong) for diagnosing this; the
	death2-15 test in sanity.sh hits it).

Thu Oct 17 15:21:56 1996  Jim Kingdon  <kingdon@harvey.cyclic.com>

	* sanity.sh: Reenable rdiff tests.  Delete rdiff-9 test to reflect
	the fact that the change to add a -K option has not been
	incorporated.  Adjust rdiff-8 test to reflect the fact that the
	change to change the default keyword expansion for the first
	revision has not been incorporated.
	* patch.c (patch_fileproc): Pass the symbolic revision to
	RCS_checkout so that Name can be expanded correctly.  Reinstates
	one of the 30 Sep 96 changes and fixes a bug which the sanity.sh
	rdiff test tests for.

	Reinstate change from 30 Sep 96:
	* patch.c (patch): CLIENT_SUPPORT: send '-f' if NOT force_tag_match

	* client.c (process_prune_candidates): Do not ignore errors from
	unlink_file_dir.

	* filesubr.c (deep_remove_dir): If rmdir returns an error other
	than ENOTEMPTY or EEXIST, return -1 not 0.  Add workaround for AIX
	header bug.

Tue Apr 30 08:21:27 1996  Mike Sutton <mike_sutton@dayton.saic.com>

	* checkout.c, history.c: added logging/reporting of cvs export
	command

Wed Oct 16 10:16:57 1996  Jim Kingdon  <kingdon@harvey.cyclic.com>

	* sanity.sh: Remove tests 4, 4.5, and 4.75; this functionality is
	already tested by 45, 45.5 and other tests.
	(ignore): New tests ignore-192, ignore-193, ignore-194, and
	ignore-195 test output from "cvs release".
	(modules2): New tests modules2-6, modules2-7, and modules2-8 test
	ability of cvs release to handle multiple arguments.  Since it
	currently doesn't, the tests are kludged.

	* server.c, cvs.h (cvs_flushout): New function.
	* recurse.c (do_file_proc): Call it.
	* server.c (cvs_outerr): Call fflush (stdout) in non-server case.
	* main.c (main): Don't call setvbuf.  The code was incorrectly
	checking for "patch" (it really is "rdiff"); the concern about
	slowing down large amounts of output is not specific to rdiff
	(it applies to "log" for example); and the above changes should
	meet the need.

Tue Oct 15 10:22:10 1996  Jim Kingdon  <kingdon@harvey.cyclic.com>

	This is intended to facilitate some future cleanups to the
	locking, but by itself it is a simple, conversative rearrangement:
	* tag.c (locked_dir, locked_list): Move from here...
	* lock.c: ...to here.
	* lock.c (Lock_Cleanup): If locked_dir is set clean it up too.
	* tag.c (tag_unlockdir): Removed; with the above change
	Lock_Cleanup suffices.
	* tag.c (tag_lockdir): Move from here...
	* lock.c (lock_dir_for_write): ...to here.
	* tag.c (tag_fileproc), rtag.c (rtag_fileproc): Update callers.
	Move comments concerning why we are locking what we are from
	tag_lockdir to here.
	* tag.c (tag_filesdoneproc), rtag.c (rtag_filesdoneproc):
	Update callers.
	* lock.c (Writer_Lock): Made static.
	* cvs.h: Update declarations.
	* server.c (server_notify): Call lock_dir_for_write rather than
	calling Writer_Lock ourselves.

	This is intended to facilitate some future cleanups to the
	locking, but by itself it is a simple, conversative rearrangement:
	* lock.c (Lock_Cleanup): Also dellist (lock_tree_list).
	* lock.c, cvs.h (lock_tree_cleanup): Removed; with the above change
	Lock_Cleanup suffices.
	* commit.c, edit.c, watch.c: Change callers.

Sat Oct 12 21:41:46 1996  Jim Kingdon  <kingdon@harvey.cyclic.com>

	* sanity.sh (deep): Add comment about whether the deep-4b behavior
	is considered desirable.

Sat Oct 12 20:36:36 1996  Ian Lance Taylor  <ian@cygnus.com>

	* client.c (client_send_expansions): Add build_dirs parameter.
	Change all callers.
	(send_dirent_proc): Get build_dirs from callerdat; if it is
	zero, don't send a nonexistent directory to the server.
	(send_files): Add build_dirs parameter.  Change all callers.
	* client.h (send_files): Update prototype.
	(send_files_contents): Remove prototype for nonexistent function.
	(client_send_expansions): Update prototype.
	* sanity.sh (deep): Add deep-4b test for above patch.

Fri Oct 11 14:07:12 1996  Jim Kingdon  <kingdon@harvey.cyclic.com>

	gcc -Wall lint:
	* logmsg.c (title_proc): Remove unused variables title and comma.

	* sanity.sh (modules2): Don't be picky about whether we are
	checking in 1.3 or 1.2 of modules; it depends on whether we are
	running all the tests or just some.

Thu Oct 10 14:52:06 1996  Jim Kingdon  <kingdon@harvey.cyclic.com>

	* server.c, server.h (server_dir): New variable.
	(output_dir): If it is set, send it before the directory name.
	* modules.c (do_module): Set it, in the case of & modules, and
	restore it when done.
	* sanity.sh (modules): Don't clean up first-dir before starting;
	tests now clean up for themselves at the end.
	(modules2): New tests, for above fix.

Wed Oct  9 15:52:34 1996  Jim Blandy  <jimb@totoro.cyclic.com>

	* sanity.sh: Barf immediately if run as root. 

	* rcs.c (RCS_getrevtime): When giving a date to get_date, use the full
	year, not the year - 1900, so that dates after 1999 are parsed
	correctly.  (Change thanks to Paul Eggert.)

Wed Oct  9 10:59:11 1996  Jim Kingdon  <kingdon@harvey.cyclic.com>

	Clean up gcc -Wmissing-prototypes lint:
	* cvs.h (admin, add, checkout, commit, diff, history, import,
	cvslog, login, patch, release, cvsremove, rtag, status, tag):
	Declare.
	* server.c, main.c: Don't declare them here.  Don't declare update
	either (which is already declared in cvs.h).
	* tag.c, cvs.h, main.c, server.c: Rename tag to cvstag to avoid
	name conflicts.
	* client.c (init_sockaddr, auth_server_port_number), entries.c
	(Entnode_Create, Entnode_Destroy), hash.c (nodetypestring),
	login.c (construct_cvspass_filename), server.c
	(supported_response), wrapper.c (wrap_matching_entry): Make static;
	prototype.
	* hash.c (printlist): Prototype.
	* myndbm.c (mydbm_load_file): Change declaration to prototype.

Tue Oct  8 22:35:34 1996  Jim Kingdon  <kingdon@harvey.cyclic.com>

	* sanity.sh (log2): Re-add these tests; they were deleted for 1.9
	(because they were thought to be destabilizing and/or due to
	confusion/accident), but they can be put back now.

	* sanity.sh (death2): In tests death2-diff-{2,4,6,8,10}, allow "-"
	or "%" in temp file names.  The system (tmpnam or whatever)
	generates these names so they vary from system to system.

Tue Oct  8 12:37:09 1996  Ian Lance Taylor  <ian@cygnus.com>

	* options.h.in (HAD_RCS4): Remove; no longer used.

Sun Oct  6 15:58:11 1996  Noel Cragg  <noel@gargle.rain.org>

	* The following changes address problem #56 in the GNATS database
 	on harvey.cyclic.com:

	* logmsg.c (str_list_format): new global -- contains the format
 	for items to be placed in str_list.
	(Update_Logfile): move code that creates the "title" string...
	(logfile_write): ...to here.  Pull apart the filter program and
 	look for a format string, extracting it if there is one.
	(title_proc): write a given filename/value based on the format
 	string.

	* commit.c (classify_file_internal): new routine, old code (needed
 	to use the code in more than one place).  Determines the status
 	and version information about a file.
	(check_fileproc): use classify_file_internal.  Fill in the rev_old
 	field for the struct logfile_info.
	(commit_fileproc): Fill in the rev_new field.

	* cvs.h (struct logfile_info): add two new fields -- rev_old and
 	rev_new -- that keep track of revision numbers across commits.

Fri Sep 27 15:21:47 1996  Peter Wemm  <peter@spinner.dialix.com>

	* logmsg.c (do_editor): Do not use editinfo if running on the client.

Fri Oct  4 15:11:46 1996  Jim Kingdon  <kingdon@harvey.cyclic.com>

	* server.c (server_cleanup): Temporarily clear noexec when calling
	unlink_file_dir.  This is so we clean up the temp directory even
	when the -n global option is specified.

Wed Oct  2 10:47:33 1996  Norbert Kiesel  <nk@col.sw-ley.de>

	* client.c (send_repository): initialize some variables before
 	first usage

Tue Oct  1 13:01:24 1996  Jim Blandy  <jimb@floss.cyclic.com>

	Revert some of Greg's changes; they're welcome later, but we're
	trying to keep CVS stable for pre-release testing at the moment. 
	* checkin.c, commit.c, cvs.h, diff.c, import.c, main.c, no_diff.c,
	options.h.in, patch.c, rcs.c, rcs.h, rcscmds.c, sanity.sh, update.c:
	Revert changes of Sep 29 and 30.

Tue Oct  1 13:17:31 1996  Ian Lance Taylor  <ian@cygnus.com>

	Make sure the server temporary directory is removed even if
	Max-dotdot is used.
	* server.c (orig_server_temp_dir): New static variable.
	(serve_max_dotdot): Don't free server_temp_dir if it is the same
	as orig_server_temp_dir.
	(do_cvs_command): Use orig_server_temp_dir in error message.
	(server_cleanup): Remove orig_server_temp_dir.
	(server): Set orig_server_temp_dir.  Remove incorrect indentation
	of error message.

	* import.c (update_rcs_file): Restore new argument to
	RCS_checkout, removed in last patch.

Tue Oct  1 00:32:55 1996  Jim Blandy  <jimb@floss.cyclic.com>

	* import.c: Revert Greg Woods' changes of Sep 30.  We may want
	them later, but not before 1.9.

Mon Sep 30 23:31:01 1996  Jim Blandy  <jimb@floss.cyclic.com>

	* log.c (log_fileproc): Now that we might actually find a "desc"
 	node in rcsfile->other, thanks to Ian's change below, we had
 	better deal correctly if we find a null pointer in it.

Mon Sep 30 13:55:03 1996  Greg A. Woods  <woods@most.weird.com>

	* main.c (main): don't set need_to_create_root for "cvs init"
	either, just in case it's run from within a valid working
	directory.

	* sanity.sh (testcvs): oops, forgot to comment out test version I
 	was using...

	* diff.c (diff_fileproc): use Diffbin instead of DIFF (3).
	* patch.c (patch_fileproc): use Diffbin instead of DIFF.
	* commit.c (check_fileproc): use Grepbin instead of GREP.
	* rcscmds.c (RCS_merge): use Grepbin instead of GREP.
	* update.c (patch_file): use Diffbin instead of DIFF.
	(update_fileproc): use Grepbin instead of GREP.
	* cvs.h (Diffbin): new declaration.
	(Grepbin): new declaration.
	(DIFFBIN_ENV): new manifest to name DIFFBIN environ var.
	(GREPBIN_ENV): new manifest to name GREPBIN environ var.
	* option.h.in (DIFFBIN_DFLT): renamed from DIFF.
	(GREPBIN_DFLT): renamed from GREP.
	* main.c (main): new variables diffbin_update_env and
	grepbin_update_inv, ala rcsbin_update_env.
	(main): new options -D diffbin and -g grepbin
	(usg): describe new options -D diffbin and -g grepbin.
	(Diffbin): new global variable for DIFF path.
	(Grepfbin): new global variable for GREP path.

	* options.h.in (RCSBIN_DFLT): mention this needs to be set if
	your PATH isn't set properly by rshd.

	* sanity.sh (rdiff): re-do Jim's change, but with the original
	keywords I had intended (should be a bit more like real life), and
	Jim's better RCS date and user matching form.
	[I guess that's what I get for checking things in at 3am! ;-)]

Mon Sep 30 17:00:20 1996  Ian Lance Taylor  <ian@cygnus.com>

	* rcs.c (RCS_reparsercsfile): Store desc field value in main RCS
	node data, not in version specific data.
	* sanity.sh: Enable log2 test (for local CVS only).

Mon Sep 30 13:01:45 1996  Jim Kingdon  <kingdon@harvey.cyclic.com>

	* sanity.sh (log2): New test, tests cvs add -m.  Not yet enabled
	in "tests" because CVS currently flunks this test.

	* sanity.sh (rdiff, basic2): Allow "cvs server" as well as "cvs
	checkout" and friends in messages.  In testing output of cvs
	status, don't require a tab which isn't there for remote.  Skip
	test rdiff-9 for remote.  In test basic2-64, add missing slash in
	the pattern which gets used for remote.

	* sanity.sh (rdiff): Fix strings we were matching against which
	got keyword-expanded in checking in sanity.sh.

Mon Sep 30 03:21:37 1996  Greg A. Woods  <woods@most.weird.com>

	* sanity.sh: change all regexpr literal '.' to '\.'
 	(basic2): why are tests 34 & 42 commented out (because
	of 'diff -u'?)?
	add tests 56[abc], 57a, and 58a to test import to the main
	branch (i.e. branch '1').
	(rdiff): new test section for rdiff -K, etc.
	(dotest): remove dotest.ex? before running a new test.
	(dotest_fail): remove dotest.ex? before running a new test.
	(dotest_internal): write expected output to dotest.exp, or if $4
	also used, to dotest.ex1 and dotest.ex2.
	(patch): renamed this test to 'serverpatch'.
	(dotest_lit): rename dotest.res to dotest.exp ala dotest().
	remove dotest.ex? before running a new test.
	(DOTSTAR): mention the bug exists up to 1.12
	(ENDANCHOR): mention the bug exists up to 1.12
	(dotest_all_in_one): new function for debugging.
	(dotest_line_by_line): new function for debugging.
	(dotest_internal_debug): new function for debugging.
	(dotest_internal): stop emulating the ancient tests and don't spew
	the dotest.tmp contents onto $LOGFILE -- it's just too much
	meaningless noise.  Only do this if the test fails.  Many tests
	don't use dotest() yet, so this isn't quite so helpful as it might
	otherwise be.
	(TODO): mention CVS/* files, especially CVS/Root.
	
	* main.c (main): add a commented out piece of code to suggest that
	there should be a function lookup_command_attribute() that could
	tell us various things about internal commands, such as whether
	they use CVS/Root, or if they're repository-only, or if they need
	a working directory, etc....
	(main): don't set need_to_create_root if command doesn't use a
	local working directory.

	* patch.c (patch): CLIENT_SUPPORT: send '-f' if NOT force_tag_match

	* error.c (fperror): protect declaration for un-defined __STDC__

	* import.c (import): permit imports to a branch with zero dots,
	i.e. the trunk.
	(update_rcs_file): don't detect conflicts if importing to the
	trunk.
	(import): add hint that we should allow a module name, instead of
	just a pathname relative to $CVSROOT.
	(add_rcs_file): if importing to trunk, do it with ci(1).

	* import.c: XXX the following are all #if 0'ed out until a full
	implementation can be designed....
	(cbranch): new variable to support conflict detection on another
	branch set by -c.
	(import): truncate -b and -c optarg if to fit in static storage.
	(import_usage): describe -c

	* rcscmds.c (RCS_checkout): add new argument 'rcsver'.  If rcsver
	is set, turn on 'keywords' to force call to RCS_exec_checkout.
	* rcs.c (RCS_exec_checkout): add new argument 'rcsver'.  Pass
	'rcsver' to "co" with run_arg().
	* cvs.h: (RCS_checkout): add new argument 'rcsver' to prototype.
	(RCS_exec_checkout): add new argument 'rcsver' to prototype.
	* commit.c (remove_file): supply new argument to RCS_checkout.
	* checkin.c (Checkin): supply new argument to RCS_checkout.
	* diff.c (diff_fileproc): supply new argument to RCS_checkout.
	(diff_file_nodiff): supply new argument to RCS_checkout.
	* no_diff.c (No_Difference): supply new argument to RCS_checkout.
	* update.c (checkout_file): supply new argument to RCS_checkout.
	(patch_file): supply new argument to RCS_checkout.
	(join_file): supply new argument to RCS_checkout.

	* patch.c: (o_options): new variable for -K
	(rcsver): new variable for -V.
	(patch): add -K flag which sets o_options, change -V to set
	rcsver, send o_options and rcsver if in client mode.
	(patch_fileproc): use RCS_checkout instead of RCS_fast_checkout in
	order to ensure $Name is expanded, use o_options if set, or
	options if set, or by default "-ko" when getting "old" file.

Sun Sep 29 16:43:28 1996  Jim Kingdon  <kingdon@harvey.cyclic.com>

	* rcscmds.c: Replace comment at top of file concerning RCS library
	with a reworded version based on discussion between me, Ian, Paul
	Eggert, and JimB.

Sun Sep 29 13:09:45 1996  Noel Cragg  <noel@kiva.rain.org>

	* main.c (main): don't create/update CVS/Root when doing the "cvs
 	login" command.  Consider: if the user executes "cvs login" with
 	the working directory inside an already checked out module, we'd
 	incorrectly change the CVS/Root file to reflect the CVSROOT of the
 	"cvs login" command.

	* login.c (login): if we're re-logging into a server for which a
 	.cvspass entry already exists, copy the temporary file to its home
 	location rather than renaming.  Renaming doesn't work between
 	filesystems.  After copying, unlink the temporary file.

Fri Sep 27 05:24:56 1996  Jim Kingdon  <kingdon@harvey.cyclic.com>

	* diff.c: Add comment about --brief option.

	* README-rm-add: Removed; the information which was here is now in
	cvs.texinfo.
	* Makefile.in (DISTFILES): Remove README-rm-add.

Wed Sep 25 10:00:00 1996  Larry Jones  <larry.jones@sdrc.com>

	* Makefile.in (cvsbug): Add dependency on version.c.

Wed Sep 25 09:01:48 1996  Jim Kingdon  <kingdon@harvey.cyclic.com>

	* filesubr.c (get_homedir), update.c (update): Reindent.

Wed Sep 25 04:44:54 1996  Jim Blandy  <jimb@totoro.cyclic.com>

	* version.c (version_string): Bump to 1.8.86.

Wed Sep 25 05:17:50 1996  Jim Blandy  <jimb@floss.cyclic.com>

	* update.c (update): Don't neglect to pass the -kmumble options
	to the server.
	* sanity.sh (binfiles-sticky): New tests for above.

	* cvsrc.c (read_cvsrc): Deal correctly with lines that specify a
	command, but no options; don't corrupt argv.

	* sanity.sh: When testing rsh, use the program specified by
 	the CVS_RSH environment variable, if it's set.  Move test to top
	of file, so it runs before all other tests (it's really a
	meta-test).

	* filesubr.c (get_homedir): Use getpwuid to find the home
	directory, if the HOME environment variable isn't set.
	* ignore.c (ign_add_file): Call get_homedir to find the user's
	home directory; this is more portable than calling getpwuid.

Tue Sep 24 09:08:17 1996  Jim Kingdon  <kingdon@harvey.cyclic.com>

	* log.c (log_tree): When walking through branches, follow the
	->prev field rather than following ->next which insures that the
	loop only executes once and we only see the last branch.
	* sanity.sh (multibranch): Test "cvs log" too; tests for above fix.

Mon Sep 23 09:55:22 1996  Norbert Kiesel  <nk@col.sw-ley.de>

	* options.h.in: Fixed some typos in the comments and reindented
 	them.

Sat Sep 21 02:33:26 1996  Jim Blandy  <jimb@totoro.cyclic.com>

	* sanity.sh: If we're testing remote CVS, make sure rsh itself is
	working before running any tests.  It's confusing when basica-1
	fails just because you don't have the local host in your .rhosts
	file.

	* version.c (version_string): Bump to 1.8.85.

Thu Sep 19 09:15:41 1996  Jim Kingdon  <kingdon@harvey.cyclic.com>

	* options.h.in: Define SERVER_FLOWCONTROL, SERVER_HI_WATER,
	SERVER_LO_WATER.  Several large sites (FreeBSD, Cygnus) have been
	pounding on this code without problems, and it doesn't seem to
	have any significant downsides.

Tue Sep 17 01:13:41 1996  Jim Kingdon  <kingdon@harvey.cyclic.com>

	* status.c (status_fileproc): Instead of a default case, set sstat
	before the switch.  This way gcc -Wall can detect a missed case.
	Add explicit T_TITLE case.

Tue Sep 17 00:09:44 1996  Assar Westerlund  <assar@pdc.kth.se>

	* login.c (login): Print usage if argc < 0.

Tue Sep 17 00:09:44 1996  Jim Kingdon  <kingdon@harvey.cyclic.com>

	* lock.c: In comment, mention one more function of readlocks
	(fileattr not updated atomically).  Note similarity between
	solutions #2 and #5.

	* checkout.c (safe_location): Do not reject a location merely
	because it textually starts with hardpath; insist that it be
	hardpath or a subdirectory thereof.

Mon Sep 16 11:46:36 1996  Jim Kingdon  <kingdon@harvey.cyclic.com>

	* server.c (server_cleanup): Add comment about ignoring errors
	from unlink_file_dir.

Mon Sep 16 10:31:48 1996  Norbert Kiesel  <nk@col.sw-ley.de>

	* main.c: Add support for -T <tmpdir> command line option.  This
 	is needed for servers started via inetd.
	(usg): Added line for -T.  Improved -z documentation.
	(main): Read default for tmpdir from the environment. Test for 'T'
	in getopt loop.  Use '/tmp' as ultimative fallback.  Update
	environment if possible.

	* cvs.h (TMPDIR_ENV): Added for -T <tmpdir> command line option.

	* options.h.in: Add TMPDIR_DFLT

	* import.c (update_rcs_file): Use global variable Tmpdir instead
 	of reading the environment.

	* server.c (server_cleanup): Use global variable Tmpdir instead of
 	reading the environment.  Also, replace system("rm -rf") with
 	unlink_file_dir.
	(server): Use global variable Tmpdir instead of reading the
 	environment.

Thu Sep 12 1996  Jim Kingdon  <kingdon@cyclic.com>

	* main.c (main): If ARGV0_NOT_PROGRAM_NAME, then just set
	program_name to "cvs" rather than argv[0].

Thu Sep 12 12:06:56 1996  Jim Kingdon  <kingdon@harvey.cyclic.com>

	* client.c (update_entries): If we can't write the file, don't
	make it a fatal error.

Wed Sep 11 12:46:23 1996  Jim Kingdon  <kingdon@harvey.cyclic.com>

	* client.c (start_server): Move START_SERVER_RETURNS_SOCKET code
	so that it is only run for server_method.  It is wrong for
	pserver_method (in which connect_to_pserver sets server_sock).

	* login.c (construct_cvspass_filename): If NO_SLASH_AFTER_HOME,
	don't put a '/' between $HOME and .cvspass.  Reindent function.
	* build_src.com: Add zlib.c, login.c, and scramble.c.

	* rcs.c (RCS_deltas): When looking for our branch in ->branches,
	check the branch number.
	* sanity.sh (multibranch): New tests test for above fix.

	* commit.c (precommit_list_proc): Fix typo in last change
	(->status to ->type).

Tue Sep 10 23:05:41 1996  Jim Kingdon  <kingdon@harvey.cyclic.com>

	* Makefile.in (DISTFILES): Add build_src.com.
	* build_src.com: Add buffer.c, buffer.obj, and zlib.olb.

Tue Sep 10 20:35:23 1996  Juergen Renz  <renz@conware.de>
			  and Jim Kingdon  <kingdon@harvey.cyclic.com>

	* commit.c (precommit_list_proc): Update to reflect Jul 22 change
	in which p->data was changed from a Ctype to a struct
	logfile_info *.  This means that commitinfo scripts again get
	passed the file	list like they should.

Tue Sep 10 20:35:23 1996  Jim Kingdon  <kingdon@harvey.cyclic.com>

	* client.c (auth_server_port_number): Change name of service from
	"cvs" to "cvspserver".  The latter is what the manual has always
	recommended, and it is also officially registered with IANA.

Tue Sep 10 11:12:42 1996  Mark A. Solinski <markso@mcs.com>
			  and Jim Kingdon  <kingdon@harvey.cyclic.com>

	* client.c (socket_buffer_output): Change ifdef VMS to ifdef
	SEND_NEVER_PARTIAL. 
	(start_server): Change ifdef VMS to ifdef START_SERVER_RETURNS_SOCKET.

Tue Sep 10 17:15:21 1996  Jim Blandy  <jimb@totoro.cyclic.com>

	* client.c (auth_server_port_number): Look up "cvs" in the
 	services database, and use the value it returns; fall back to
	CVS_AUTH_PORT if no entry is present.
	(connect_to_pserver): Use the correct port number in any error
	messages.

Tue Sep 10 11:12:42 1996  Jim Kingdon  <kingdon@harvey.cyclic.com>

	* sanity.sh (newb): New test newb-123j0 tests for another "cvs
	status" case.

Sun Sep  8 15:20:37 1996  Ian Lance Taylor  <ian@cygnus.com>

	* rcs.c (RCS_checkout): Clarify handling of options parameter.

	* rcs.c (RCS_checkout): Free buffer allocated by RCS_deltas.

Sat Sep  7 21:28:27 1996  Jim Kingdon  <kingdon@harvey.cyclic.com>

	* main.c (struct cmd): Add comment concerning recognizing unique
	abbreviations.

Fri Sep  6 22:31:52 1996  Jim Kingdon  <kingdon@harvey.cyclic.com>

	* rcs.c (RCS_checkout): Fix indentation.

Fri Sep  6 11:48:08 1996  Ian Lance Taylor  <ian@cygnus.com>

	* rcs.c (RCS_checkout): Replace tag parameter with rev and nametag
	parameters.  Change all callers.
	* rcs.h (RCS_checkout): Update declaration.

	* rcs.c (RCS_getversion): Replace return_both parameter with
	simple_tag.  Change all callers.
	(RCS_gettag): Likewise.
	* rcs.h (RCS_getversion, RCS_gettag): Update declarations.
	* vers_ts.c (Version_TS): Simplify vn_tag initialization using new
	simple_tag rather than old return_both.
	* cvs.h (struct vers_ts): Clarify vn_tag comment a bit.

	* main.c (usg): Only mention -x if ENCRYPTION is defined.
	(main): Mention ENCRYPTION define in comment for -x.
	* client.h (krb_encrypt_buffer_initialize): Only declare if
	ENCRYPTION is defined.
	* client.c (start_server): Only encrypt if ENCRYPTION is defined.
	* server.c (serve_kerberos_encrypt): Only define if ENCRYPTION is
	defined.
	(requests): Only include Kerberos-encrypt is ENCRYPTION is
	defined.
	(krb_encrypt_*): Only define if ENCRYPTION is defined.

Thu Sep  5 17:32:39 1996  Ian Lance Taylor  <ian@cygnus.com>

	* sanity.sh: When testing remote, use :ext: instead of :server: to
 	match change made earlier today.

Thu Sep  5 13:57:47 1996  Jim Kingdon  <kingdon@harvey.cyclic.com>

	* client.c (start_tcp_server): Don't allow :kserver: to mean
	"direct tcp" (root.c already takes care of this, but I want to
	make it clear what is intended, and not intended, here).
	(start_server): Handle ext_method (external rsh program) and
	server_method (internal rsh client) separately.
	* client.c: Take rsh_pid and start_rsh_server out of
	RSH_NOT_TRANSPARENT ifdefs.  It is useful for things like SSH on NT.
	* cvs.h (CVSmethod), root.c (method_names): Add ext_method.
	* root.c (parse_cvsroot): Recognize "ext" access method.
	If access method is not specified and CVSROOT contains a colon,
	use either ext_method or server_method depending on
	RSH_NOT_TRANSPARENT.

Thu Sep  5 00:09:49 1996  Ian Lance Taylor  <ian@cygnus.com>

	* rcs.c (RCS_checkout): Remove flags parameter, which was not
	serving any useful purpose.  Change all callers.
	* rcscmds.c (RCS_exec_checkout): Likewise.

	* rcscmds.c (RCS_exec_checkout): Rename from RCS_checkout.  Change
	all callers.
	* rcs.c (RCS_checkout): Rename from RCS_fast_checkout.  Change all
	callers.

Wed Sep  4 14:42:28 1996  Ian Lance Taylor  <ian@cygnus.com>

	* rcs.c (RCS_fast_checkout): If tracing, output a message.  If
	noexec, and workfile is not NULL, just return immediately.  Assert
	that sout is RUN_TTY or workfile is NULL, rather than using it as
	a conditional.  Replace found variable with two variables--gothead
	and keywords--reflecting what it actually means.

	* rcs.c (RCS_fast_checkout): Don't handle the case of workfile set
	to "".
	* rcscmds.c (RCS_checkout): Likewise.
	* checkin.c (Checkin): Pass explicit file name, not "", to
	RCS_fast_checkout.
	* update.c (join_file): Likewise.
	* commit.c (remove_file): Pass explicit file name to
	RCS_fast_checkout and RCS_checkin.

	* rcs.c (RCS_reparsercsfile): Always continue after seeing
	RCSSYMBOLS, even if the value is NULL.  Clear the NODELTA flag
	after setting delta_pos.
	(free_rcsnode_contents): New static function.
	(freercsnode): Call free_rcsnode_contents.
	(RCS_fast_checkout): If NODELTA is set, reparse the RCS file.
	(RCS_settag): New function.  Change all callers to old function.
	(RCS_deltag, RCS_setbranch): Likewise.
	(RCS_lock, RCS_unlock): Likewise.
	(RCS_deltas): If NODELTA is set, reparse the RCS file.
	* rcs.h (NODELTA): Define.
	(RCS_settag, RCS_deltag, RCS_setbranch): Declare.
	(RCS_lock, RCS_unlock): Declare.
	* rcscmds.c (RCS_exec_settag): Rename from RCS_settag.  Don't
	check tag against BASE or HEAD (now done in new RCS_settag).
	(RCS_exec_deltag): Rename from RCS_deltag.
	(RCS_exec_setbranch): Rename from RCS_setbranch.
	(RCS_exec_lock): Rename from RCS_lock.
	(RCS_exec_unlock): Rename from RCS_unlock.
	* cvs.h: Update declarations of renamed functions.
	* checkin.c (Checkin): Remove rcscopy variable (no longer needed
	because of change in RCS_unlock call).
	* commit.c: Include <assert.h>.
	(remove_file): Update RCSNode path if the file is renamed.
	(unblockrcs): Change rcs parameter to RCSNode.  Change all
	callers.
	(fixbranch): Likewise.
	(lock_RCS): Likewise.  Don't call RCS_parsercsfile.
	(checkaddfile): Update RCSNode path if the file is renamed.  After
	creating a new file, call RCS_parse.  When stubbing a branch, use
	the passed in RCSNode if there is one, rather than calling
	RCS_Parse.  Don't call RCS_Parse again after calling RCS_settag.
	Free head and magicrev even if RCS_settag fails.
	* import.c (add_rev): Change rcs parameter to RCSNode.  Change all
	callers.
	(add_tag): Likewise.

	* rcs.c (RCS_fast_checkout): Amend last patch: if workfile is
	NULL, but sout is not NULL, use sout in error message.

Wed Sep  4 13:35:09 1996  Jim Kingdon  <kingdon@harvey.cyclic.com>

	* version.c: Increment version number to 1.8.8.

	* Version 1.8.7.

Wed Sep  4 1996  Jim Kingdon  <kingdon@cyclic.com>

	* client.c (send_file_names): Look for the name to send in
	Entries even if the file doesn't exist; we should send the
	name as it appears in Entries in the "rm foo; cvs update FOO"
	case.

Tue Sep  3 20:50:11 1996  William A. Hoffman  <hoffman@albirio.crd.ge.com>

	* rcs.c (RCS_fast_checkout): If workfile is NULL, don't try to
	include it in error message.

Mon Aug 26 12:27:38 1996  Jim Kingdon  <kingdon@harvey.cyclic.com>

	* mkmodules.c (mkdir_if_needed): Move from here ...
	* filesubr.c, cvs.h (mkdir_if_needed): ... to here.  Have it
	return a value saying whether the directory was created.
	* client.c (call_in_directory), edit.c (edit_fileproc): Call it.

Fri Aug 23 19:19:44 1996  Ian Lance Taylor  <ian@cygnus.com>

	* checkin.c (Checkin): Copy rcs parameter in case it is freed when
	finfo->rcs is freed.

Fri Aug 23 14:55:41 1996  Jim Kingdon  <kingdon@harvey.cyclic.com>

	* remove.c (remove_fileproc): Revert change of 23 Aug to print
	getwd and finfo->file in message.  The latter is redundant with
	fullname and the former is redundant with fullname and the working
	directory when CVS was invoked.  The implementation was also
	lacking as the getwd call could overflow the buffer.

Fri Aug 23 18:40:35 1996  Norbert Kiesel  <nk@col.sw-ley.de>

	* remove.c (cvsremove): fix remove -f for client/server

Fri Aug 23 11:28:27 1996  Jim Kingdon  <kingdon@harvey.cyclic.com>

	* wrapper.c, cvs.h: Remove conflictHook field of WrapperEntry,
	WRAP_CONFLICT in WrapMergeHas, and 'c' option in wrap_add; they
	are never used.

Fri Aug 23 11:41:46 1996  Norbert Kiesel  <nk@col.sw-ley.de>

	* server.c (switch_to_user): use #ifdef SETXID_SUPPORT instead of
 	#if SETXID_SUPPORT

Thu Aug 22 14:18:43 1996  Ian Lance Taylor  <ian@cygnus.com>

	* checkin.c (Checkin): Remove local variable xfinfo.  Reparse the
	RCS file after the checkin.  Call RCS_fast_checkout rather than
	RCS_checkout.

	* cvs.h (RCS_FLAGS_LOCK): Don't define.
	(RCS_FLAGS_*): Adjust values to fill in hole left by removal of
	RCS_FLAGS_LOCK.
	* rcs.c (RCS_fast_checkout): Don't check for RCS_FLAGS_LOCK.
	* rcscmds.c (RCS_checkout): Likewise.
	* commit.c (commit_fileproc): Remove rcs local variable.  If
	status is T_MODIFIED, require that finfo->rcs be set, call
	Lock_RCS directly, and don't call locate_rcs.  If adding to a tag,
	require that finfo->rcs be set, and don't call locate_rcs.
	(remove_file): Remove rcs local variable.  Require that finfo->rcs
	be set.  Don't call locate_rcs.  Don't pass RCS_FLAGS_LOCK to
	RCS_checkout; use RCS_lock instead.  Call RCS_fast_checkout rather
	than RCS_checkout.
	(unlockrcs): Use a single rcs parameter rather than two parameters
	for file and repository.  Change all callers.  Don't call
	locate_rcs.
	(fixbranch): Likewise.
	(lockrcsfile): Remove; no more callers.

Tue Aug 20 10:13:59 1996  Jim Kingdon  <kingdon@harvey.cyclic.com>

	* buffer.c, rcs.c: Don't use inline.  It wasn't being used in a
	loop or any such place where it would matter for performance, and
	it was a (minor) portability hassle.

	* server.c (server): Change "Dummy argument 0" to "cvs server" and
	add comment explaining why.

	* rcs.c (linevector_add): Add comment regarding changing \n to \0.

Tue Aug 20 09:19:19 1996  Norbert Kiesel  <nk@col.sw-ley.de>

	* checkout.c (checkout_proc): Call RCS_parse to get the default
 	options from the RCS file.

	* sanity.sh (binfiles): Add tests 5.5b0 and 5.5b1 for the above fix

Mon Aug 19 18:13:32 1996  Ian Lance Taylor  <ian@cygnus.com>

	* rcs.c (linevector_init): Make inline.  Set lines_alloced to 0,
	not 10.  Set vector to NULL.
	(linevector_add): Remove assertion that lines_alloced is greater
	than zero…