3250 lines
127 KiB
Plaintext
3250 lines
127 KiB
Plaintext
1997-12-30 enami tsugutomo <enami@but-b.or.jp>
|
|
|
|
* rcs.c (RCS_checkin): Use gmtime() instead of localtime()
|
|
(restores behavior from RCS 5.x which was broken with RCS library
|
|
-kingdon).
|
|
|
|
Mon Dec 29 12:53:00 1997 Ian Lance Taylor <ian@cygnus.com>
|
|
|
|
* modules.c (do_module): Check for a request for a file within a
|
|
module which is not a directory.
|
|
* sanity.sh (modules): Add test 149b1 for above patch.
|
|
|
|
* client.c (start_tcp_server): Remove useless assignment, left
|
|
behind by Dec 15 patch.
|
|
|
|
Sat Dec 27 17:41:11 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* client.c, options.h.in, history.c, import.c, main.c, rcs.c,
|
|
update.c: Remove !HAVE_RCS5 code. It had bit-rotted a while ago,
|
|
and more to the point is obsolete with the RCS library.
|
|
|
|
27 Dec 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* zlib.c, server.h (gunzip_and_write): New function.
|
|
* client.c (update_entries): Call it instead of a gunzip subprocess.
|
|
* zlib.c, server.h (read_and_gzip): New function.
|
|
* client.c (send_modified): Call it instead of a gzip subprocess.
|
|
|
|
Sat Dec 27 13:07:38 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
Decrease RCS_deltas memory usage to what we need (approximately
|
|
the size of the file we are patching plus the size of the largest
|
|
patch). Previously memory usage had been approximately the size
|
|
of the RCS file because we never freed lines until the end.
|
|
* rcs.c (linevector_free, linevector_copy, linevector_add,
|
|
linevector_delete):
|
|
Instead of having all the lines and struct line's in the alloc_*
|
|
space, have each line and its struct line in its own malloc'd
|
|
space. Use a refcount to deal with curlines vs. headlines
|
|
vs. trunklines in RCS_deltas.
|
|
(struct allocblock, blocks, block_alloc, block_free): Remove; no
|
|
longer used.
|
|
(apply_rcs_changes, RCS_deltas): Don't copy lines into allocated
|
|
space; linevector_add now does that for us.
|
|
(rcs_change_text, RCS_deltas): Don't call block_free.
|
|
|
|
Tue Dec 23 08:28:44 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* cvsbug.sh: Change bug-cvs address from prep.ai.mit.edu to gnu.org
|
|
per email from Martin Hamilton.
|
|
|
|
Sun Dec 21 21:49:50 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* rcs.c (RCS_checkin): Disable keyword expansion when generating
|
|
the change text.
|
|
* sanity.sh: Move tests keyword-24 through keyword-27 into
|
|
new section keywordlog and expand greatly. Note that CVS 1.9.18
|
|
passes the new tests both local and remote but the current
|
|
version failed them both local and remote before this fix.
|
|
|
|
Sat Dec 20 19:56:00 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* rcs.c (RCS_delete_revs): Clean up temporary files even if noexec.
|
|
Without this fix, basica-o5a in sanity.sh would leave files around.
|
|
|
|
Thu Dec 18 13:05:00 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* run.c: Fix typo in declaration (evecvp -> execvp) (credit to
|
|
Erik Walthinsen for reporting this). Only declare it if not
|
|
HAVE_UNISTD_H. Move declaration to before the first use.
|
|
|
|
Tue Dec 16 12:59:00 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* update.c: Collapse two identical declarations for join_file.
|
|
|
|
Mon Dec 15 16:01:49 1997 Ian Lance Taylor <ian@cygnus.com>
|
|
|
|
* client.c (start_tcp_server): Remove calls to htons and add one
|
|
call to ntohs (init_sockaddr calls htons on the port argument).
|
|
|
|
Mon Dec 15 00:07:02 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* client.c (failure_exit): New variable.
|
|
(get_server_responses): If it is set, then return
|
|
failure.
|
|
(updated_seen, updated_fname): New variables.
|
|
(update_entries): Use updated_fname if set. In the "move away
|
|
foo.c; it is in the way" case print "C" not "U", and set
|
|
failure_exit.
|
|
(handle_mt): If we get +updated tagged text, stash it away in
|
|
updated_fname rather than printing it immediately.
|
|
(handle_mt, get_server_responses): If we stashed a filename and
|
|
didn't get around to printing it, go ahead and print it.
|
|
* sanity.sh (conflicts2-142d2): Adjust to test for fix. Remote is
|
|
now like local was in terms of exit status and "C aa.c" message.
|
|
|
|
Sun Dec 14 00:27:26 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
Implement tagged text feature:
|
|
* update.c (write_letter): Take a single finfo argument rather
|
|
than file and update_dir. While we are at it change it to return
|
|
void (since the returned value always had been 0).
|
|
* update.c: Update callers.
|
|
* server.c, cvs.h (cvs_output_tagged): New function.
|
|
* client.c (responses): Add "MT" response.
|
|
(handle_mt): New function.
|
|
* update.c (write_letter): Output via cvs_output_tagged.
|
|
|
|
Sun Dec 14 14:13:05 1997 Ian Lance Taylor <ian@cygnus.com>
|
|
|
|
* recurse.c (do_dir_proc): Only check for CVS/Repository if
|
|
W_LOCAL.
|
|
* sanity.sh (devcom-t2, devcom-t3): New tests for above patch.
|
|
|
|
Sun Dec 14 00:27:26 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* root.c (parse_cvsroot): Initialize check_hostname (fixes thinko
|
|
in GSSAPI changes).
|
|
|
|
Sat Dec 13 13:15:35 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* server.c: Use indentation to indicate nesting of #ifdef's.
|
|
|
|
* client.c (connect_to_gserver): Reindent (in one place).
|
|
|
|
Fri Dec 12 17:38:15 1997 Chris Provenzano <proven@cygnus.com>
|
|
and Ian Lance Taylor <ian@cygnus.com>
|
|
|
|
* cvs.h (CVSmethod): Add gserver_method.
|
|
* root.c (method_names): Add gserver.
|
|
(parse_cvsroot): Handle :gserver:.
|
|
* client.h (cvsauthenticate): Declare.
|
|
(cvs_gssapi_encrypt): Declare if HAVE_GSSAPI and ENCRYPTION.
|
|
(cvs_gssapi_wrap_buffer_initialize): Declare if HAVE_GSSAPI.
|
|
(connect_to_pserver): Update declaration.
|
|
(pserver_authenticate_connection): Declare when HAVE_GSSAPI and
|
|
SERVER_SUPPORT is defined in addition to other case.
|
|
* client.c: If HAVE_GSSAPI, include GSSAPI header files.
|
|
(gcontext): New static variable if HAVE_GSSAPI.
|
|
(connect_to_pserver): Add do_gssapi parameter. Change all
|
|
callers. Move rejection handling to bottom of function.
|
|
(recv_bytes): New static function if HAVE_GSSAPI.
|
|
(connect_to_gserver): Likewise.
|
|
(start_server): Handle gserver_method. Handle GSSAPI encryption
|
|
and authentication.
|
|
* server.c: Include <sys/socket.h> if HAVE_GSSAPI, in addition to
|
|
existing cases. If HAVE_GSSAPI, include GSSAPI header files.
|
|
Include <grp.h> even if AUTH_SERVER_SUPPORT is not defined.
|
|
(gcontext, cvs_gssapi_wrapping): New static variables if
|
|
HAVE_GSSAPI.
|
|
(cvs_gssapi_encrypt): New global variable if HAVE_GSSAPI and
|
|
ENCRYPTION.
|
|
(serve_gssapi_encrypt): New static function if HAVE_GSSAPI and
|
|
ENCRYPTION.
|
|
(serve_gssapi_authenticate): New static function if HAVE_GSSAPI.
|
|
(requests): Add Gssapi-encrypt if HAVE_GSSAPI and ENCRYPTION. Add
|
|
Gssapi-authenticate if HAVE_GSSAPI.
|
|
(switch_to_user): Compile if HAVE_GSSAPI, in addition to existing
|
|
cases.
|
|
(pserver_authenticate_connection): Likewise. Ifdef out part of
|
|
the code for AUTH_SERVER_SUPPORT. Handle a GSSAPI request.
|
|
(gserver_authenticate_connection): New static function if
|
|
HAVE_GSSAPI.
|
|
(cvsauthenticate): New global variable.
|
|
(struct cvs_gssapi_wrap_data): Define if HAVE_GSSAPI.
|
|
(cvs_gssapi_wrap_buffer_initialize): New function if HAVE_GSSAPI.
|
|
(cvs_gssapi_wrap_input): New static function if HAVE_GSSAPI.
|
|
(cvs_gssapi_wrap_output): Likewise.
|
|
* main.c (opt_usage): Mention -a.
|
|
(main): Handle -a. Handle pserver if HAVE_GSSAPI, in addition to
|
|
existing cases.
|
|
* login.c (login): Pass new argument to connect_to_pserver.
|
|
|
|
Fri Dec 12 15:33:19 1997 Ian Lance Taylor <ian@cygnus.com>
|
|
|
|
* buffer.c (PACKET_SLOP): Define.
|
|
(packetizing_buffer_initialize): Use PACKET_SLOP when allocating
|
|
holdbuf.
|
|
(packetizing_buffer_input): Allow up to PACKET_SLOP bytes in
|
|
stackoutbuf.
|
|
(packetizing_buffer_output): Use just BUFFER_DATA_SIZE + 2 for
|
|
inbuf. Allow PACKET_SLOP + 4 extra bytes in stack_outbuf.
|
|
Correct >= to > in test of incoming number of bytes. Use
|
|
PACKET_SLOP in other tests.
|
|
|
|
Fri Dec 12 10:27:08 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* client.c (start_tcp_server): Revise comment to reflect
|
|
SOCK_STRERROR and SOCK_ERRNO now being in use.
|
|
|
|
Thu Dec 11 15:32:31 1997 Ian Lance Taylor <ian@cygnus.com>
|
|
|
|
* client.c (init_sockaddr): Compile if HAVE_KERBEROS, as well as
|
|
if AUTH_CLIENT_SUPPORT. Return a pointer to a struct hostent.
|
|
(start_tcp_server): Clean up. Use init_sockaddr. Use
|
|
SOCK_STRERROR and SOCK_ERRNO. Don't bind the socket.
|
|
|
|
Generalize buffering code used by Kerberos encryption routines
|
|
into a generic packetizing buffer. The new code in buffer.c is a
|
|
modified version of the code removed from server.c.
|
|
* buffer.c (struct packetizing_buffer): Define.
|
|
(packetizing_buffer_initialize): New function.
|
|
(packetizing_buffer_input): New static function.
|
|
(packetizing_buffer_output): New static function.
|
|
(packetizing_buffer_flush): New static function.
|
|
(packetizing_buffer_block): New static function.
|
|
(packetizing_buffer_shutdown): New static function.
|
|
* buffer.h (packetizing_buffer_initialize): Declare.
|
|
* server.c (struct krb_encrypt_data): Rename from
|
|
krb_encrypt_buffer, and remove all fields not related to
|
|
encryption.
|
|
(krb_encrypt_buffer_initialize): Just call
|
|
packetizing_buffer_initialize.
|
|
(krb_encrypt_input): New static function.
|
|
(krb_encrypt_output): New static function.
|
|
(krb_encrypt_buffer_input): Remove.
|
|
(krb_encrypt_buffer_output): Remove.
|
|
(krb_encrypt_buffer_flush): Remove.
|
|
(krb_encrypt_buffer_block): Remove.
|
|
(krb_encrypt_buffer_shutdown): Remove.
|
|
|
|
Wed Dec 10 15:39:44 1997 Ian Lance Taylor <ian@cygnus.com>
|
|
|
|
* buffer.c (stdio_buffer_initialize): Correct formatting.
|
|
|
|
Sun Dec 7 09:37:19 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* sanity.sh (basicb-0d0): New test, for checkout on existing
|
|
directory.
|
|
|
|
Sat Dec 6 00:25:11 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* sanity.sh (binwrap3): Clean up repository too. Clean up working
|
|
directory with "rm -r" not "rm -rf".
|
|
|
|
Thu Dec 4 17:11:18 1997 Larry Jones <larry.jones@sdrc.com>
|
|
|
|
* subr.c (check_numeric): Don't reference argv[1] when argc is 1
|
|
(should be argv[0]).
|
|
|
|
* sanity.sh: Fix lines that look like conflict markers but aren't
|
|
to prevent problems checking in.
|
|
(binwrap3): Remove local CVSROOT when done so that later
|
|
tests that expect to create it don't fail.
|
|
|
|
Thu Dec 4 18:19:21 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* cvs.h: Remove mempcpy definition. I think the polite way to
|
|
describe my feelings about mempcpy is something like "we don't
|
|
have agreement that it is a good idea".
|
|
* rcs.c (truncate_revnum): Don't call it.
|
|
* sanity.sh: Run multibranch2 test by default.
|
|
(multibranch2): Use ${TESTDIR} a few places.
|
|
(multibranch2-9): Accept "P file1" as well as "U file1".
|
|
|
|
* sanity.sh: Don't add 1997 to the copyright notice. Add GPL
|
|
terms. Add discussion of copyright issues.
|
|
* rcs.c (truncate_revnum, truncate_revnum_in_place,
|
|
compare_truncated_revnums): Reindent.
|
|
|
|
1997-12-04 Jim Meyering <meyering@na-net.ornl.gov>
|
|
|
|
* subr.c (xstrdup): Use memcpy rather than strcpy.
|
|
(compare_revnums): Declare parameters to be `const'.
|
|
Remove unnecessary uses of xstrdup and corresponding frees.
|
|
(increment_revnum): Declare parameter to be `const'.
|
|
Use memcpy rather than strcpy.
|
|
(gca): Declare parameters to be `const'.
|
|
(check_numeric): Declare REV parameter to be `const'.
|
|
(file_has_markers): Declare parameter to be `const'.
|
|
(get_file): Declare `char*' parameters to be `const'.
|
|
* run.c (run_exec): Declare `char*' parameters to be `const'.
|
|
* cvs.h (mempcpy) [! HAVE_MEMPCPY]: Define it.
|
|
Add `const' to types in several prototypes.
|
|
|
|
* rcs.c (truncate_revnum): New function.
|
|
(truncate_revnum_in_place): New function.
|
|
(compare_truncated_revnums): New function.
|
|
(max_rev): New function.
|
|
(RCS_addbranch): Make BRANCH parameter `const'.
|
|
Use the above functions rather than open-coding them.
|
|
When BRANCH is a revision number, insert it *in order*
|
|
in the sorted list of branch numbers, not at the end.
|
|
Add assertion that insertion succeeds.
|
|
* sanity.sh (multibranch2): Test for this.
|
|
(Copyright): Add 1997.
|
|
|
|
Dec 1997 Karl Fogel <kfogel@floss.red-bean.com>
|
|
|
|
* wrapper.c (wrap_name_has): loop as far as wrap_count +
|
|
wrap_temp_count, not wrap_count + wrap_saved_count, otherwise
|
|
some wrappers get skipped.
|
|
(wrap_matching_entry): same.
|
|
* sanity.sh (binwrap3): new test, for import with
|
|
CVSROOT/cvswrappers and .cvswrappers specifying -k 'b' options.
|
|
|
|
1997-11-30 Jim Meyering <meyering@na-net.ornl.gov>
|
|
|
|
* client.c (send_a_repository): Strip trailing slashes from the name
|
|
of the update directory. Otherwise, running `cvs update dir/' provokes
|
|
this failure `protocol error: illegal directory syntax in dir/' when
|
|
running in client/server mode.
|
|
|
|
* hash.c (insert_before): New function derived from addnode.
|
|
(addnode): Simply return insert_before.
|
|
(addnode_at_front): Simply return insert_before.
|
|
* hash.h (insert_before): Add prototype.
|
|
|
|
* server.c (dirswitch): Compute `strlen(dir)' once and save it,
|
|
rather than computing it four times. Also do s/illegal/invalid/ to
|
|
this diagnostic: "E protocol error: illegal directory syntax in %s".
|
|
|
|
Sun Nov 30 18:03:02 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* admin.c: Fix comment (no longer a front-end to "rcs").
|
|
|
|
* error.c, error.h (rcserror): Remove.
|
|
* admin.c, rcs.c: Call error instead of rcserror. This changes
|
|
the format of these messages from "rcs: <path>: error" to "cvs
|
|
<command>: <path>: error". The former format wasn't quite what
|
|
RCS printed anyway (because RCS would sometimes print "ci", "co",
|
|
&c, not "rcs"), and preserving RCS's exact output probably is not
|
|
a good idea anyway (because it will make people think that the
|
|
error was caused by an external program). In two cases, I tidied
|
|
up the message in a more drastic fashion ("cannot stat" in
|
|
RCS_checkin and "could not diff" in RCS_delete_revs).
|
|
|
|
* sanity.sh (basica-o2b, binfiles2-o1, admin-18, admin-22-o10,
|
|
admin-22-o17): Look for "cvs <command>" not "rcs".
|
|
|
|
* run.c, cvs.h (run_setup): Replace varargs nonsense with a single
|
|
argument which gets parsed as the result of the vasprintf
|
|
used to.
|
|
* client.c, commit.c, logmsg.c, modules.c, rtag.c, tag.c, update.c,
|
|
wrapper.c: Update callers, either to do the sprintf themself or to
|
|
just call run_arg if it will do the job.
|
|
* rcscmds.c: Likewise for call_diff_setup and callers.
|
|
|
|
* run.c, cvs.h (run_args): Remove; nowhere used.
|
|
|
|
Sat Nov 29 22:15:06 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* options.h.in: Remove declaration of getwd; see lib/ChangeLog for
|
|
rationale.
|
|
|
|
1997-11-29 Jim Meyering <meyering@na-net.ornl.gov>
|
|
|
|
* update.c (checkout_file): Initialize `backup'.
|
|
|
|
* diff.c (diff_fileproc): Initialize `tmp' and `fname'.
|
|
|
|
* modules.c (do_module): Initialize `server_dir_to_restore'.
|
|
(do_module): Initialize `value' in an else clause.
|
|
|
|
* rcs.c (RCS_checkin): Initialize `commitpt'.
|
|
(RCS_delete_revs): Initialize `revp'.
|
|
(RCS_copydeltas): Always initialize `insertbefore'.
|
|
|
|
* run.c (run_print): Define `outfn' even in error case.
|
|
|
|
Mon Nov 24 17:28:50 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* rcs.c (RCS_findlock_or_tip): Prototype.
|
|
(RCS_checkin): Fix call to pass correct number of arguments.
|
|
|
|
Sun Nov 23 10:34:03 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* recurse.c (do_dir_proc): Move check for CVS/Repository and
|
|
CVS/Entries to before where we call the direntproc.
|
|
* client.c (send_dirent_proc): Remove code to check for
|
|
CVS/Repository, now that recurse.c does it.
|
|
* sanity.sh (conflicts3-18 through conflicts3-19): New tests, for this.
|
|
|
|
Sat Nov 22 10:54:16 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* recurse.c (do_dir_proc): Check that CVS/Repository and
|
|
CVS/Entries exist.
|
|
* sanity.sh (conflicts3-14 through conflicts3-17): New tests, for this.
|
|
|
|
* client.c (send_fileproc): Send options field from
|
|
vers->entdata->options not vers->options.
|
|
* cvs.h (struct entnode): Add comment (options and timestamp must
|
|
not be NULL).
|
|
* sanity.sh (binfiles-9 through binfiles-13, binfiles-sticky5,
|
|
keyword-17): Remove kludges for remote; tests for fix.
|
|
|
|
* update.c (update_fileproc): Fix comment; direct checkout is
|
|
still faster than patches for local but not for quite the
|
|
same reasons.
|
|
|
|
* add.c (add): Pass SEND_NO_CONTENTS to send_files.
|
|
|
|
Wed Nov 19 18:25:03 1997 Mike Glendinning <mikeg@sequent.com>
|
|
|
|
* update.c (patch_file_write): Missing cast provided.
|
|
|
|
Wed Nov 19 15:57:59 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* rcs.c (RCS_deltas): Solve trigraph problem (once and for all, I
|
|
hope) with 3 calls to cvs_output.
|
|
|
|
Wed Nov 19 01:52:57 1997 Andy Piper <andyp@parallax.co.uk>
|
|
and Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* classify.c (Classify_File), cvs.h (struct vers_ts), vers_ts.c
|
|
(Version_TS): Clarify NULL versus "" for options in comments.
|
|
* vers_ts.c (Version_TS): Treat "" the same way as NULL in options
|
|
and vers_ts->options.
|
|
* sanity.sh: New tests binfiles-sticky5 through binfiles-17 test
|
|
for this.
|
|
|
|
1997-11-16 Karl Fogel <kfogel@floss.red-bean.com>
|
|
|
|
* client.c (update_entries): parse server-sent entries line even
|
|
in the case of "cvs export", because we need to know if -kb option
|
|
is set.
|
|
Init `options' to NULL like anything else.
|
|
|
|
Tue Nov 18 09:20:29 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* version.c: Change version number to 1.9.21.
|
|
|
|
* Version 1.9.20.
|
|
|
|
Mon Nov 17 14:35:31 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* server.c (server_updated): If scratched_file and noexec are set,
|
|
clean up so we don't get a "duplicate Scratch_Entry" warning
|
|
later.
|
|
* sanity.sh: New tests conflicts3-10 to conflicts3-13, for this.
|
|
|
|
* sanity.sh (conflicts3): Don't allow "file1 was lost" messages
|
|
here; I don't think CVS actually produced them, and they don't
|
|
belong.
|
|
|
|
Sun Nov 16 23:19:41 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* sanity.sh: Accept either "U file1" or "P file1".
|
|
|
|
Fri Nov 14 12:32:05 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* rcs.c: Add comment about cleaning up ,foo, file on ^C.
|
|
|
|
Fri Nov 14 11:56:29 1997 Andy Piper <andyp@parallax.co.uk>
|
|
and Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* filesubr.c (unlink_file_dir): Don't print trace message in the
|
|
server.
|
|
|
|
Fri Nov 14 11:28:55 1997 Jim Meyering and Jim Kingdon
|
|
|
|
* rcs.c (RCS_getdatebranch): If the branch we are looking for
|
|
doesn't exist, return a revision which matches the date, not
|
|
just NULL.
|
|
* sanity.sh (tagdate): New test, for this.
|
|
|
|
Thu Nov 13 10:11:48 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* sanity.sh (basicb-21): Fix comment which described a behavior
|
|
which no longer exists.
|
|
|
|
Wed Nov 12 16:24:45 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
Clean up infrastructure made unnecessary by RCS library:
|
|
* rcscmds.c (diff_exec, diff_execv): Use literal "diff" not DIFF.
|
|
* options.h.in (DIFF), rcs.h (RCS, RCS_CI, RCS_DIFF,
|
|
RCS_RCSMERGE): Removed; no longer used.
|
|
* commit.c (commit), patch.c (patch_fileproc),
|
|
rcscmds.c (RCS_exec_rcsdiff), start of rcscmds.c: Update comments
|
|
to reflect librarification of RCS.
|
|
* options.h.in (RCSBIN_DFLT): Removed.
|
|
* main.c, cvs.h (Rcsbin, free_Rcsbin): Removed.
|
|
* main.c (main): Don't check RCSBIN environment variable. -b
|
|
global option is now a noop.
|
|
* cvs.h (RCSBIN_ENV): Removed.
|
|
* expand_path.c (expand_variable): $RCSBIN is now an error.
|
|
* mkmodules.c (config_contents): Remove RCSBIN.
|
|
* parseinfo.c (parse_config): RCSBIN now a noop.
|
|
* server.c (server): Don't put Rcsbin in PATH.
|
|
|
|
Mon, 10 Nov 1997 Jim Kingdon
|
|
|
|
* rcs.c (RCS_checkin): Actually, when we get a change text
|
|
for a text file using get_file, we want text mode, although
|
|
the reasons are kind of subtle (see comment).
|
|
|
|
* rcs.c (RCS_checkin): Pass correct mode to get_file for
|
|
binary files.
|
|
|
|
* rcscmds.c: Declare vasprintf.
|
|
|
|
Mon Nov 10 11:11:17 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
This fixes problems with windows-NT/run.c improperly quoting, and
|
|
is cleaner anyway.
|
|
* rcscmds.c (call_diff_setup, call_diff_arg, call_diff_add_arg,
|
|
call_diff_argv, call_diff_argc, call_diff_argc_allocated): New
|
|
functions/variables, lightly adapted from src/run.c.
|
|
* cvs.h, run.c (call_diff, call_diff3): Move from here...
|
|
* rcscmds.c: ...to here.
|
|
|
|
Sun, 9 Nov 1997 Jim Kingdon
|
|
|
|
* rcs.c (rcs_internal_unlockfile): Call rename_file not rename.
|
|
This makes it work on NT again.
|
|
|
|
Sun Nov 9 16:54:28 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* sanity.sh (diffmerge2): Protect keywords against unwanted
|
|
expansion. They got clobbered and the testcase stopped working
|
|
when I checked it in.
|
|
|
|
Fri Nov 7 13:23:38 1997 Karl Fogel <kfogel@floss.red-bean.com>
|
|
and Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* sanity.sh (diffmerge1, diffmerge2): new tests, for bugs, or
|
|
potential bugs, in ../diff/analyze.c which were fixed by Paul
|
|
Eggert's patch.
|
|
|
|
Sun Nov 9 10:28:43 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* rcs.c (RCS_settag): Reindent.
|
|
|
|
* rcs.c (rcs_internal_lockfile): Fix typo (thow -> throw).
|
|
|
|
Sat Nov 8 15:58:53 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* sanity.sh (cvsadm): Remove most of the tests which tested
|
|
CVS/Root. This takes the run time for the cvsadm tests from
|
|
about 5 minutes 15 seconds to about 4 minutes 10 seconds with no
|
|
significant loss in coverage.
|
|
|
|
* rcs.c (rcs_internal_lockfile): Check for errors from system
|
|
calls. If open() gives an error, don't muck with stat and errno
|
|
(I don't know what the RCS code that this comes from was trying to
|
|
do, but it clearly isn't accomplishing anything here).
|
|
(RCS_rewrite, RCS_delete_revs): Check for errors from system calls.
|
|
|
|
Sat Nov 1 14:21:29 1997 Michael L.H. Brouwer <michael@thi.nl>
|
|
|
|
* rcs.c (RCS_checkin): Change type of bufsize from int to size_t.
|
|
(RCS_delete_revs): Change type of bufsize and len from int to size_t.
|
|
(RCS_getdeltatext): Change type of textlen from int to size_t.
|
|
* rcs.h (struct deltatext): Change len from int to size_t to keep
|
|
the compiler happy on systems where size_t is unsigned int.
|
|
[This goes well beyond keeping the compiler happy; if sizeof
|
|
(size_t) != sizeof (int), the old code was quite broken -kingdon]
|
|
|
|
Sat Nov 1 14:21:29 1997 Michael L.H. Brouwer <michael@thi.nl>
|
|
and Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* rcs.c (RCS_checkin): When checking if we were holding the lock
|
|
use delta->author instead of user since the latter might have been
|
|
clobbered by a call to getcaller. This resulted in the failure of
|
|
test basica-7.
|
|
[I don't completely follow the scenario where it gets clobbered,
|
|
it but sounds vaguely plausible and the replacement seems
|
|
cleaner, precisely because it avoids allocation issues -kingdon]
|
|
|
|
Wed Nov 5 20:16:12 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* checkin.c, checkout.c, commit.c, cvs.h, import.c, login.c,
|
|
main.c, update.c: Change FALSE to 0 and TRUE to 1 and replace
|
|
monstrosities like "cvswrite == TRUE" with just "cvswrite". FALSE
|
|
and TRUE sometimes conflicted with system headers (NextStep3.3?),
|
|
but more to the point, good old 1 and 0 are fine and were used by
|
|
most of CVS already.
|
|
|
|
Tue Nov 4 12:19:28 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* rcs.c (RCS_checkin, RCS_lock, RCS_unlock): Rename local variable
|
|
quiet to checkin_quiet or some such, to avoid confusion with
|
|
global variable quiet.
|
|
|
|
* lock.c: Update comment to refer to add_rcs_file rather than "rcs
|
|
-i".
|
|
|
|
* rcs.h (struct rcsnode): Add comments for all fields.
|
|
* rcs.c (RCS_delete_revs): Refuse to delete revisions which have
|
|
symbolic names. Fix fencepost bug which caused us to sometimes
|
|
check one more revision than we should for locks, branches, and
|
|
this.
|
|
(findtag): New function, to help above code.
|
|
|
|
* admin.c (admin): Take out writelocks not readlocks. This has
|
|
been a bug "forever", but may become more noticeable with
|
|
rcs_internal_lockfile relying on the writelocks.
|
|
|
|
Mon Nov 3 10:17:19 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* rcs.c (RCS_rewrite): Don't write the file if noexec.
|
|
* sanity.sh (basica, branches): Test for this.
|
|
|
|
Sat Nov 1 10:01:56 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* rcs.h (struct deltatext): Comment text and log fields.
|
|
|
|
* admin.c (admin_fileproc): Call RCS_reparsercsfile not
|
|
RCS_fully_parse. Don't muck with ->other field in RCSVers (it
|
|
doesn't need to be set).
|
|
* rcs.h, rcs.c (RCS_reparsercsfile): No longer static. No point
|
|
in having this static when RCS_rewrite and RCS_fully_parse are not.
|
|
* rcs.c (getdelta): Remove obsolete comment about not storing the
|
|
newphrases from the deltas, since we now do.
|
|
|
|
Sat Nov 1 10:01:56 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
and Paul Eggert
|
|
|
|
* rcs.c (rcs_internal_lockfile): Clarify the comments about O_EXCL
|
|
and such matters.
|
|
|
|
Sat Nov 1 10:01:56 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* rcs.c (RCS_delete_revs): Pass force_tag_match to RCS_settag.
|
|
* sanity.sh (basica-o2a, basica-o2b): Test for this.
|
|
|
|
1997-11-01 Peter Brandstrom <d91-pbr@nada.kth.se>
|
|
|
|
* sanity.sh: Use ${username} more places.
|
|
|
|
Sat Nov 1 00:14:00 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* rcs.c (rcs_internal_lockfile): Add comments about what we are
|
|
trying to accomplish here (versus what RCS tries to accomplish).
|
|
|
|
* rcs.c (RCS_parsercsfile_i): Clarify/expand comment about
|
|
the purpose of having both this and RCS_reparsercsfile.
|
|
(RCS_rewrite): Add comment about how this works.
|
|
|
|
* admin.c (admin_fileproc): Add comment about call to
|
|
RCS_fully_parse not RCS_reparsercsfile.
|
|
* rcs.h: Comment on what delta_pos field of struct rcsnode is.
|
|
|
|
Fri Oct 31 16:38:39 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
and Abe Feldman
|
|
|
|
* client.c (update_entries): If UTIME_EXPECTS_WRITABLE, if
|
|
necessary change the file to be writable temporarily to set its
|
|
modification time.
|
|
|
|
Thu Oct 30 17:42:59 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* admin.c (admin): Deal with new :: syntax for ranges.
|
|
* rcs.c, rcs.h (RCS_delete_revs): New arg inclusive (set for the old
|
|
behavior, clear to enable new code).
|
|
* admin.c (admin_fileproc): Set it if :, clear it if ::.
|
|
* sanity.sh (basica, head, branches, log): Add tests for this feature.
|
|
|
|
* admin.c (admin_fileproc): Clean up the error message which
|
|
happens if one of the RCS_* functions returns an error status; it
|
|
is confusing to say that "rcs" failed now that this is implemented
|
|
internally.
|
|
* sanity.sh (admin): Update accordingly.
|
|
|
|
Wed Oct 29 07:07:36 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* sanity.sh (admin-22-o24): New test, tests that admin -o
|
|
correctly munged the deltatexts on a branch too.
|
|
|
|
* rcs.c (RCS_delete_revs): If we are deleting an entire branch,
|
|
delete the node in ->branches rather than setting the ->key to the
|
|
bogus value NULL.
|
|
* rcs.c (RCS_delete_revs): If "rev1" equals "branchpoint", then set
|
|
"before" to the revision on the trunk that we branch from.
|
|
* rcs.c (RCS_delete_revs): Don't set rev2 to revp->version (the
|
|
code is missing an xstrdup, but it doesn't matter because rev2
|
|
isn't used after this point).
|
|
* sanity.sh (binfiles2-o2 to binfiles2-o4): New tests, for this.
|
|
|
|
Tue Oct 28 19:30:05 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* rcs.c (RCS_delete_revs): Restore code which passes rev2 to
|
|
RCS_getbranchpoint if rev1 is NULL; it still makes sense for the
|
|
non-trunk case. Fixes admin-22-o22 in testsuite.
|
|
|
|
* sanity.sh (admin-18): Adjust to reflect "rcs failed" no longer
|
|
being suppressed by global -q option.
|
|
|
|
* rcs.c (RCS_delete_revs): If rev1 == NULL and rev2 is on the
|
|
trunk, handle it the same way we do everything else--by swapping
|
|
the two. This replaces the code which tried to kludge what we
|
|
passed to RCS_getbranchpoint (which didn't work).
|
|
* sanity.sh (binfiles2-o1 to binfiles2-o4): New tests, for this fix.
|
|
* admin.c (admin_fileproc): Don't have -q global option suppress
|
|
"rcs failed" message.
|
|
|
|
1997-10-28 Jim Kingdon
|
|
|
|
* log.c (printlock_proc), rcs.c (putlock_proc): Prototype.
|
|
* rcs.c (rcs_internal_lockfile): Only try to call fchmod if
|
|
HAVE_FCHMOD is defined.
|
|
|
|
Tue Oct 28 10:27:03 1997 Ian Lance Taylor <ian@cygnus.com>
|
|
|
|
* rcs.c (RCS_deltas): Don't use \? in string if __STDC__ is not
|
|
defined.
|
|
|
|
* rcs.c (make_file_label): Remove extraneous `+'.
|
|
|
|
Mon Oct 27 14:40:15 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* status.c (status): Don't pass SEND_NO_CONTENTS to send_files.
|
|
|
|
Sat Oct 25 00:33:57 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* rcs.c (RCS_delete_revs): Use : not - for range in error message.
|
|
|
|
* rcs.h: Add comment about '\0' in RCS fields.
|
|
|
|
* rcs.c (getdelta): Add comment about branches and next field
|
|
being mandatory.
|
|
|
|
* rcs.c (RCS_reparsercsfile, RCS_deltas), sanity.sh (reserved):
|
|
Reindent sections which were misindented as a result of recent
|
|
changes.
|
|
|
|
Fri Oct 24 10:22:15 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* rcs.c (RCS_findlock_or_tip): Don't worry about file ownership
|
|
and nonstrict locking when returning the default branch or head.
|
|
The most conspicuous problem with the old code is that in the
|
|
error case it would examine rstat.st_uid when it had not been
|
|
set. For a discussion of more fundamental reasons, see comment.
|
|
|
|
* admin.c (admin_fileproc): In handling -A, don't handle relative
|
|
pathnames differently from absolute pathnames. See comment for
|
|
rationale. If problem opening the file, give a nice error not a
|
|
coredump.
|
|
* sanity.sh (admin-19a-admin, admin-19a-log, admin-19a-fix):
|
|
New tests, test for traditional "cvs admin -A" behavior with
|
|
relative pathnames.
|
|
* sanity.sh (admin-19a-nonexist): Test for the core dump fix.
|
|
* sanity.sh (admin-22-o1): Look for ${PROG} not cvs.
|
|
|
|
* sanity.sh (reserved-16): Remove commitinfo change with "cvs
|
|
commit" not "cvs admin -o". In addition to commit being The Right
|
|
Thing on general principles, cvs admin -o doesn't work because it
|
|
doesn't rebuild the administrative file database.
|
|
|
|
* update.c (patch_file): If the first revision does not exist in
|
|
the RCS file, fall back to sending entire file. Fixes
|
|
admin-22-o15 in make remotecheck.
|
|
|
|
1997-10-23 enami tsugutomo <enami@but-b.or.jp>
|
|
|
|
* rcs.c (RCS_checkin): Unlink temporary files stored in variable
|
|
`tmpfile' and `changefile'.
|
|
|
|
Wed Oct 22 12:16:10 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* rcs.c (rcs_lockfilename): Allocate enough memory for terminating
|
|
'\0'.
|
|
|
|
* admin.c (admin_fileproc): Don't support '-' for ranges in "cvs
|
|
admin -o".
|
|
(admin): Adjust comment.
|
|
|
|
* rcs.h (RCSVers): New field other_delta.
|
|
* rcs.c (free_rcsvers_contents): Also free other_delta.
|
|
(getdelta): Read newphrases from deltas into other_delta field.
|
|
(putdelta): Write those newphrases.
|
|
* sanity.sh (rcs-8a): New test, for this fix.
|
|
* admin.c (admin_fileproc): If "-b" without argument, then set
|
|
branch to NULL, not "" (uncovered by rcs-8a test).
|
|
* rcs.c (putrcsfield_proc): Add comment about how we (mis)handle
|
|
values in newphrases.
|
|
|
|
* sanity.sh (reserved): Instead of looking for rcslock.pl in CVS
|
|
distribution, just use our own equivalent.
|
|
|
|
* rcs.c (RCS_rewrite): Call ferror before fclose to avoid "Invalid
|
|
argument" warnings.
|
|
|
|
Mon Oct 20 00:30:16 1997 Tim Pierce <twp@twp.tezcat.com>
|
|
|
|
[I removed a ChangeLog entry for a change to sanity.sh (editor),
|
|
because the actual change was not made. With this change, CVS no
|
|
longer runs RCS. I'll be checking in my cleanups shortly. -kingdon]
|
|
|
|
Librarify `ci'.
|
|
* rcscmds.c, cvs.h (RCS_checkin): Removed.
|
|
* rcs.c, rcs.h (RCS_checkin, RCS_getbranchpoint, RCS_addbranch,
|
|
RCS_findlock_or_tip): New functions. RCS_checkin completely
|
|
rewritten to eliminate RCS 5.7; change `rcs' parameter from string
|
|
to RCSNode, so we can update RCSNode without re-reading from
|
|
disk.
|
|
* checkin.c (Checkin): Updated RCS_checkin caller, moved RCS_parse
|
|
call to before RCS_checkin.
|
|
* import.c (add_rev): Updated caller.
|
|
* commit.c (remove_file): Updated caller.
|
|
(checkaddfile): Updated caller. Parse `rcsfile' after
|
|
calling add_rcs_file. Free `rcsfile' instead of asserting it to
|
|
be NULL.
|
|
|
|
Librarify `rcs'.
|
|
* rcscmds.c, cvs.h (RCS_exec_settag, RCS_exec_deltag,
|
|
RCS_exec_setbranch, RCS_exec_lock, RCS_exec_unlock): Removed.
|
|
|
|
* rcs.c (RCS_settag): Rewritten to eliminate RCS 5.7.
|
|
* commit.c (checkaddfile): Call RCS_rewrite after calling RCS_settag.
|
|
* import.c (add_tags): Same.
|
|
* rtag.c (rtag_fileproc): Same.
|
|
* tag.c (tag_fileproc): Same.
|
|
|
|
* rcs.c (RCS_deltag): Rewritten to eliminate RCS 5.7. Remove
|
|
`quiet' parameter, since this function no longer prints any output.
|
|
* commit.c (remove_file): Update caller. Also call RCS_rewrite
|
|
after RCS_deltag.
|
|
* rtag.c (rtag_delete): Same.
|
|
* tag.c (tag_fileproc): Same.
|
|
|
|
* rcs.c (RCS_setbranch): Rewritten to eliminate RCS 5.7.
|
|
* commit.c (remove_file): Call RCS_rewrite after calling RCS_setbranch.
|
|
(fixbranch): Same.
|
|
(lock_RCS): Same.
|
|
|
|
* rcs.c (RCS_lock): Rewritten to eliminate RCS 5.7. Change third
|
|
arg to mean `quiet' and not `noerr', permitting admin_fileproc to
|
|
run RCS_lock verbosely.
|
|
* commit.c (lock_RCS): Update callers; call
|
|
RCS_rewrite after RCS_lock.
|
|
(remove_file): Same. Call RCS_lock quietly.
|
|
* import.c (add_rev): Same. Do not print `fork failed' error
|
|
message, since we're no longer forking.
|
|
|
|
* rcs.c (RCS_unlock): Rewritten to eliminate RCS 5.7. Change
|
|
`noerr' arg to mean `quiet', permitting admin_fileproc to run
|
|
RCS_unlock verbosely. Use notify_do when breaking another user's
|
|
lock. Include "edit.h" for notify_do prototype.
|
|
* checkin.c (Checkin): Update caller; use RCS_rewrite after RCS_unlock.
|
|
* commit.c (unlockrcs): Same.
|
|
* import.c (add_rev): Same.
|
|
|
|
* rcs.c, rcs.h (RCS_getlocks, RCS_addaccess, RCS_delaccess,
|
|
RCS_getaccess, RCS_delete_revs): New functions.
|
|
(expand_keywords, RCS_lock, RCS_unlock): Use RCS_getlocks.
|
|
* log.c (log_fileproc, log_version): Call RCS_getlocks. Don't add
|
|
bogus ";locker" nodes to RCSVers nodes -- walk lock list with
|
|
printlock_proc.
|
|
(printlock_proc): New function.
|
|
|
|
* admin.c (admin_fileproc): Largely rewritten: call internal RCS
|
|
library functions instead of forking RCS processes.
|
|
(admin, admin_fileproc): Obsolete -V option.
|
|
(struct admin_data): Remove `version' member.
|
|
* sanity.sh (admin-24): Remove -V test case.
|
|
|
|
New functions for reading and writing RCS files.
|
|
* rcs.h (struct deltatext, Deltatext): New types.
|
|
(struct rcsversnode): New members `text' and `outdated'.
|
|
(struct rcsnode): New members `access', `locks', `strict_locks',
|
|
`comment', and `desc'.
|
|
* rcs.c (RCS_reparsercsfile, expand_keywords): Use new RCSNode members.
|
|
(free_rcsnode_contents): Free them.
|
|
* log.c (log_fileproc): Use new RCSNode members instead of ->other.
|
|
|
|
* rcs.c (getdelta, RCS_getdeltatext, freedeltatext, do_locks,
|
|
RCS_putadmin, RCS_putdtree, RCS_putdesc, putdelta,
|
|
putrcsfield_proc, putsymbol_proc, RCS_copydeltas, putdeltatext,
|
|
RCS_rewrite, getrevnum, rcs_internal_lockfile,
|
|
rcs_internal_unlockfile, rcs_lockfilename): New functions.
|
|
|
|
(RCS_reparsercsfile): Use getdelta, making sure fp is positioned
|
|
correctly before calling it. Skip `head' and `branch' nodes: we
|
|
have already parsed them, and they were being added incorrectly to
|
|
rcs->other. Do not signal error if the RCS file has an empty
|
|
delta tree; this made it impossible for RCS_checkin to perform an
|
|
initial checkin. Remove `all' parameter; always store all RCS
|
|
fields.
|
|
(RCS_fully_parse, RCS_gettag, RCS_getbranch, RCS_getdate,
|
|
RCS_getdatebranch, RCS_getrevtime, RCS_symbols, translate_symtag,
|
|
RCS_isdead, RCS_getexpand, RCS_checkout, annotate_fileproc):
|
|
Update all callers to remove `all' parameter.
|
|
|
|
(getrcskey): Do not append trailing whitespace to a value. This
|
|
corrupted some log fields and wrecked some sanity.sh test cases.
|
|
|
|
(free_rcsvers_contents): New function.
|
|
(rcsvers_delproc): Call it.
|
|
|
|
* rcs.h (NODELTA): Removed symbol; now obsolete (since RCSNodes
|
|
do not go stale).
|
|
* import.c (add_rev): Removed NODELTA reference.
|
|
* rcs.c (RCS_reparsercsfile, RCS_checkout, RCS_settag, RCS_deltag,
|
|
RCS_setbranch, RCS_lock, RCS_unlock, RCS_deltas): Removed NODELTA
|
|
references.
|
|
|
|
Miscellaneous changes to support RCS librarification and fix some bugs.
|
|
* subr.c, cvs.h (line2argv): Add `sepchars' argument.
|
|
* modules.c (cat_module, admin_fileproc): Update all callers.
|
|
|
|
* subr.c, cvs.h (compare_revnums, increment_revnum): New functions.
|
|
(make_message_rcslegal): Strip whitespace from end of
|
|
lines and end of string, a la `cleanlogmsg' in RCS 5.7.
|
|
(get_file): Terminate buf with \0, extending it if
|
|
necessary. Read from stdin if `name' arg is NULL.
|
|
* admin.c (admin_fileproc): Call get_file to read -t arg from stdin.
|
|
|
|
* error.c, error.h (rcserror): New function, used everywhere.
|
|
|
|
* hash.c, hash.h (addnode_at_front): New function.
|
|
* rcs.c (RCS_settag, RCS_lock): Call it.
|
|
|
|
* import.c, cvs.h (expand_at_signs): Make extern.
|
|
* rcs.c (putrcsfield_proc, RCS_putadmin, RCS_putdesc,
|
|
putdeltatext): Call it.
|
|
|
|
* rcs.c (make_file_label): Use last_component to get file's basename.
|
|
|
|
* sanity.sh (srcdir): New variable.
|
|
(rcs-7): Remove newphrase warning, no longer produced by CVS.
|
|
(rcs-8): Permit random whitespace around newphrase fields.
|
|
(admin-22): New test cases for -o options: admin-22-o{1..23}.
|
|
(reserved): New test cases for rcslock.pl: reserved-{8..16}.
|
|
|
|
Tue Oct 21 16:48:32 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* tag.c (tag_check_valid): Add comment about locking or lack
|
|
thereof.
|
|
|
|
1997-10-20 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* version.c: Change version number to 1.9.19.
|
|
|
|
1997-10-19 Jim Kingdon
|
|
|
|
* Version 1.9.18.
|
|
|
|
Wed Oct 15 15:21:43 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* sanity.sh (username): Add '-' to characters allowed in user name.
|
|
|
|
* rcscmds.c (diff_exec): Remove item about external diff
|
|
programs. It doesn't really belong here now that diff is
|
|
librarified and TODO #191 now mentions this.
|
|
|
|
* checkout.c (checkout_proc): Add comment about assuming '/' is
|
|
the only path separator.
|
|
* options.h.in: Fix thinko (CVS/Repository -> CVS/Root).
|
|
|
|
Mon Oct 13 22:46:03 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* commit.c (commit): Add comment about CVS_BADROOT and command
|
|
other than "commit".
|
|
|
|
1997-10-11 Noel Cragg <noel@swish.red-bean.com>
|
|
|
|
* options.h.in: RELATIVE_REPOS has been checked for bitrot -- it
|
|
now works again. Change the comment before the #define to say
|
|
that we'll be switching to it soon.
|
|
|
|
* sanity.sh (basicb-1, basicb-1a): update dotest strings to match
|
|
no matter if we're running with RELATIVE_REPOS defined or not.
|
|
(basicb-9b, basicb-9c): same.
|
|
|
|
* sanity.sh (basicb-9b, basicb-9c): modified tests, since the
|
|
checkout.c fix changes the way this test module is checked out.
|
|
(basicb-9d, basicb-9e, basicb-9f): new tests, same.
|
|
(basicb-18): modify test, same.
|
|
(cvsadm): new set of exhaustive tests to check the contents of
|
|
CVS/Root and CVS/Repository files under various conditions. As a
|
|
side effect, it tests the behavior of the "-d" flags (command line
|
|
and modules file).
|
|
(modules3-7e through modules3-7h): removed, since these tests were
|
|
a small subset of what is tested in the new cvsadm section.
|
|
(modules-1b, modules-1c): same.
|
|
(modules-2b, modules-2c): same.
|
|
(modules-3b, modules-3c, modules-3e, modules-3f): same.
|
|
|
|
* create_adm.c (Create_Admin): be a bit more verbose when using
|
|
trace mode.
|
|
|
|
* checkout.c (checkout_proc): rewrote the code that sets the where
|
|
variable and the code that matches directory names with repository
|
|
directories. This fixes a long-standing bug in CVS. (It used to
|
|
be the case that "cvs co -d foo <mod1> <mod2>" would not properly,
|
|
where <mod1> and <mod2> where defined in the modules file. While
|
|
the first module would be checked out correctly, the second would
|
|
be checked out under the name of the directory to which the module
|
|
referred rather than the module name!). This fix also allows us
|
|
to check out things into directories that are more than one deep
|
|
(e.g. "cvs -d foo/bar/baz co blah" will now work).
|
|
(checkout): remove code that performed a CHDIR if the
|
|
number of arguments specified was greater than one, since it's no
|
|
longer necessary. Also remove the code that prevented us from
|
|
doing "cvs co -d <dir1>/<dir2>" without <dir1> existing, since
|
|
checkout_proc handles things correctly now.
|
|
|
|
* cvs.h: fix typo.
|
|
|
|
* rtag.c (rtag): reformat so that we don't run over 80 characters
|
|
per line.
|
|
(rtag_dirproc): same.
|
|
|
|
* sanity.sh: change all old test cases to use pass and fail
|
|
functions rather than doing some combination of echo and exit
|
|
themselves.
|
|
|
|
* commit.c (commit_direntproc): remove the "warm fuzzy" -- this
|
|
code never gets called when running in client/server mode, and we
|
|
should have CVS' output match as much as possible between the two
|
|
modes. Moreover, there is no analogous place to put this same
|
|
message when we're running in c/s mode.
|
|
(find_direntproc): print the same "fuzzy" as in check_direntproc
|
|
so that local and c/s mode have the same messages.
|
|
* sanity.sh (187a3): update test case to reflect the above.
|
|
|
|
Thu Oct 9 10:57:02 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* parseinfo.c (parse_config): Add comment about compatibility
|
|
issues with adding keywords.
|
|
|
|
Wed Oct 8 16:40:37 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* recurse.c (do_dir_proc), commit.c (check_direntproc,
|
|
commit_direntproc, find_dirent_proc): If this
|
|
directory doesn't exist, skip it.
|
|
* diff.c (diff_dirproc): Reindent.
|
|
* sanity.sh (deep-4b0a, deep-4b0b): Check for this fix.
|
|
|
|
Thu Sep 26 16:30:00 1997 Larry Jones <larry.jones@sdrc.com>
|
|
and Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* update.c (checkout_file): Don't set timestamp in noexec mode.
|
|
* vers_ts.c (Version_TS): Add comment about ignoring errors from
|
|
utime.
|
|
* sanity.sh (conflicts3): New tests, for this fix.
|
|
|
|
Fri Oct 3 09:47:04 1997 Noel Cragg <noel@swish.red-bean.com>
|
|
|
|
* sanity.sh (168): use PROG instead of CVSBASE, since they are
|
|
equal.
|
|
(importb-2): refer to PROG instead of "cvs" in error message.
|
|
|
|
* add.c (add): use PROGRAM_NAME in the error message rather than
|
|
"cvs".
|
|
* classify.c (Classify_File): same.
|
|
* commit.c (find_fileproc): same.
|
|
* sanity.sh: change all add notification messages to refer to PROG
|
|
rather than "cvs". Fixed nasty quoting in several places at the
|
|
same time, replacing older "'command'" forms with newer
|
|
".command." for simplicity.
|
|
|
|
Sat Sep 27 01:37:10 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* sanity.sh (rcslib-merge-8): Accept "P file1" as well as "U file1".
|
|
|
|
Fri Sep 26 22:24:10 1997 Noel Cragg <noel@swish.red-bean.com>
|
|
|
|
* create_adm.c (Create_Admin): assign our duplicate pointer `cp'
|
|
after the xrealloc of `reposcopy' because the latter might have
|
|
changed addresses.
|
|
|
|
Fri Sep 26 14:25:59 1997 Tim Pierce <twp@twp.tezcat.com>
|
|
|
|
* run.c (call_diff): Don't reset optind; this is done by diff_run now.
|
|
|
|
Librarify rcsmerge/diff3.
|
|
* rcscmds.c, cvs.h (RCS_merge): Rewritten from scratch: check out
|
|
selected files and diff3 them. Take new `rcs' and `workfile'
|
|
arguments, so we can resolve symbolic tags and manipulate the
|
|
working file.
|
|
* update.c (merge_file, join_file): Update RCS_merge calls.
|
|
* run.c, cvs.h (call_diff3): New function.
|
|
|
|
* sanity.sh (rcslib): New tests, rcslib-merge-{1..13}.
|
|
|
|
Fri Sep 26 22:59:56 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* create_adm.c (Create_Admin): Fix thinko in Noel's change: keep
|
|
track of the originally allocated "cp" and free it, rather than
|
|
calling free on a pointer which may point halfway into the allocation.
|
|
* repos.c (Sanitize_Repository_Name): Per HACKING, assert that
|
|
repository != NULL, rather than just silently returning if NULL.
|
|
Reindent a line.
|
|
|
|
Fri Sep 26 15:40:00 1997 Noel Cragg <noel@swish.red-bean.com>
|
|
|
|
* sanity.sh (modules): add tests modules-[123]* that make sure the
|
|
administrative files get rebuilt in various cases.
|
|
|
|
* add.c (combine_dir): removed function, since we no longer need
|
|
to worry about stripping the "." path element out. Changed the
|
|
two callers to simply concatenate their two arguments.
|
|
|
|
* recurse.c (do_dir_proc): don't bother trying to strip off "." in
|
|
the repository name since the below changes fix that behavior --
|
|
simply concatenate the repository and directory names together.
|
|
|
|
* checkout.c (checkout_proc): sanitize the repository name after
|
|
constructing it (we may create "/path/to/repos/.", but we don't
|
|
want that to be passed around). Remove the code that tacks on
|
|
"/." when constructing top_repository, since the below changes fix
|
|
that behavior. Added comments to the part of this function that
|
|
builds administrative files.
|
|
|
|
* sanity.sh (basicb): now that the below weirdness is fixed, the
|
|
extra "." path element in test basicb-0c doesn't appear when a
|
|
top-level file is checked out. Remove it from the expect string.
|
|
|
|
* create_adm.c (Create_Admin): now that the repository name isn't
|
|
floating around with "." as the last path element, make creation
|
|
of the top-level administrative files a special case -- save the
|
|
repository name as "/path/to/repos/." Why? I considered not
|
|
including the "." but didn't know how it would affect the remote
|
|
protocol when RELATIVE_REPOS was defined (do we have a way of
|
|
sending "" via the protocol?). After I make sure that the
|
|
RELATIVE_REPOS patches still work, I'll check the "" possibility
|
|
so we don't have to have a this special case.
|
|
|
|
* repos.c (Sanitize_Repository_Name): new function that removes
|
|
(if present) the trailing slash and "." component from the
|
|
repository name. Many routines break if we don't guarantee this.
|
|
See the comment before the function for complete information.
|
|
(Name_Repository): call Sanitize_Repository_Name before returning
|
|
the value.
|
|
* cvs.h: add prototype for Sanitize_Repository_Name.
|
|
|
|
Fri Sep 26 14:19:25 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* cvs.h (CVS_CMD_USES_WORK_DIR): Fix comment (the sense of the
|
|
flag is not reversed from what it would seem; when I thought so it
|
|
was because I was misreading the lookup_command_attribute code).
|
|
|
|
Thu Sep 25 23:14:47 1997 Noel Cragg <noel@swish.red-bean.com>
|
|
|
|
* parseinfo.c (Parse_Info): fix typo in the trace message.
|
|
|
|
Thu Sep 25 14:22:54 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* build_src.com: Also link with diff.olb.
|
|
|
|
* main.c (Make_Date): If gmtime returns NULL, try localtime.
|
|
|
|
Wed Sep 24 19:18:40 1997 Noel Cragg <noel@swish.red-bean.com>
|
|
|
|
* checkout.c (checkout): fix typo in comment.
|
|
|
|
Wed Sep 24 08:31:46 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* update.c (patch_file): Revise comments about diff -a now that
|
|
diff is librarified.
|
|
|
|
Sun Sep 21 21:28:26 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* run.c (call_diff): Sleep for a second, in hopes of helping with
|
|
out of order bugs.
|
|
|
|
Sat Sep 20 07:19:18 1997 Tim Pierce <twp@twp.tezcat.com>
|
|
|
|
Integrate diff library into CVS.
|
|
* run.c, cvs.h (call_diff): New function.
|
|
* rcscmds.c (diff_exec, diff_execv): Get diffs from call_diff
|
|
instead of running diff as a subprocess.
|
|
* Makefile.in (cvs): Add ../diff/libdiff.a.
|
|
|
|
* diff.c (longopts, diff): Use 131 for --ifdef, fixing Jim's thinko
|
|
(using 147 for both --side-by-side and --ifdef).
|
|
|
|
* sanity.sh (rcslib): Added tests rcslib-diffrgx-*, to test
|
|
handling of regex diff options.
|
|
|
|
1997-09-21 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
(Note that this requires that DIFF support -L. I'll be checking
|
|
in a fix to that in a moment, but I wanted separate checkins in
|
|
case that helps with clarity).
|
|
Tweaks to rcsdiff librarification:
|
|
* sanity.sh (rcslib): Change "cvs" to "${PROG}" and subcommand
|
|
names (e.g. "add") to "[a-z]*". Former should deal with ${testcvs}
|
|
being "cvs.old" or something; latter fixes make remotecheck.
|
|
* rcs.c (make_file_label): Take into account strlen (rev) when
|
|
allocating space. Removes a FIXME and probably fixes a buffer
|
|
overrun security hole.
|
|
* rcscmds.c (RCS_exec_rcsdiff): Remove #if 0'd code to call
|
|
rcsdiff. Tim says he has compared the new code with rcsdiff code
|
|
and is confident that the behavior is preserved, so we need to
|
|
nuke the comment which says this has not been done. #if 0 isn't
|
|
really a very good way to document the way it used to work anyway;
|
|
the old code is still in CVS.
|
|
* diff.c: Add comment about rcsdiff options that we don't support,
|
|
which Tim had sent in email. Remove -T and -y, as the
|
|
previous meaning had been very confused.
|
|
|
|
1997-09-21 Tim Pierce <twp@xochi.tezcat.com>
|
|
|
|
Librarify rcsdiff.
|
|
|
|
* diff.c (have_rev1_label, have_rev2_label): New variables.
|
|
(diff): Generate file labels with make_file_label if necessary;
|
|
pass labels, revisions and working file name to RCS_exec_rcsdiff.
|
|
* rcscmds.c, cvs.h (RCS_exec_rcsdiff): Completely revised function
|
|
to eliminate rcsdiff dependency, based on patch from JimK.
|
|
(diff_execv): New function, to exec diff with explicit -L args.
|
|
* rcs.c, rcs.h (make_file_label): New function.
|
|
(RCS_output_diff_options): New function.
|
|
* sanity.sh (rcslib): New test.
|
|
|
|
Fri Sep 19 15:08:08 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* rcs.c (RCS_nodeisbranch): Assert that RCS is non-NULL.
|
|
* commit.c (remove_file), rcs.c (RCS_getversion), rtag.c
|
|
(rtag_fileproc), status.c (status_fileproc, tag_list_proc), tag.c
|
|
(tag_fileproc): Call RCS_nodeisbranch not RCS_isbranch
|
|
in contexts where we know the RCS argument is non-NULL.
|
|
|
|
* commit.c (find_fileproc): Pass tag not NULL to Version_TS for
|
|
the tag.
|
|
* vers_ts.c (Version_TS): Improve (somewhat) the introductory
|
|
comment.
|
|
* sanity.sh (editor): New test editor-9 tests for above fix.
|
|
Renumber/tweak surrounding tests to fit.
|
|
|
|
* log.c (log_version): If p->data is NULL, it is an empty log
|
|
message.
|
|
* sanity.sh (rcs-14): New test, tests for above fix (previously
|
|
this was a coredump).
|
|
|
|
Thu Sep 18 08:45:05 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* sanity.sh (rcs-7): Fix stupid ${TESTDIR} omission.
|
|
|
|
Wed Sep 17 16:27:41 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* sanity.sh (rcs): New tests rcs-5 through rcs-13 test for
|
|
getdate.y fix (rcs-12 and rcs-13 both failed with the buggy
|
|
getdate.y).
|
|
|
|
Tue Sep 16 00:07:17 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* sanity.sh (editor): Clean up first-dir at end of test.
|
|
|
|
* sanity.sh (editor): New tests test do_editor.
|
|
|
|
* commit.c (commit): For the client, if we got the log message
|
|
from do_editor and there was an error, don't toss the message.
|
|
|
|
Mon Sep 15 14:27:54 1997 martin.sjoelin@ubs.com
|
|
and Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* rcs.c (RCS_checkout): fwrite in bite-size pieces, not the whole
|
|
file in one fwrite.
|
|
|
|
Sun Sep 14 12:23:15 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* commit.c (check_fileproc): If the file has "conflict
|
|
indicators", spit a warning and proceed with the checkin.
|
|
* sanity.sh (conflicts): Adjust tests conflicts-132,
|
|
conflicts-status-3, conflicts-133, and conflicts-status-4
|
|
for new behavior.
|
|
|
|
Fri Sep 12 11:12:34 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* add.c, admin.c, checkin.c, checkout.c, classify.c, commit.c,
|
|
create_adm.c, cvsrc.c, diff.c, entries.c, find_names.c, hash.c,
|
|
import.c, lock.c, log.c, logmsg.c, main.c, modules.c, myndbm.c,
|
|
no_diff.c, parseinfo.c, patch.c, rcs.c, rcscmds.c, recurse.c,
|
|
remove.c, repos.c, root.c, rtag.c, status.c, subr.c, tag.c,
|
|
update.c, vers_ts.c, hash.h, rcs.h, options.h.in: Change "CVS 1.4
|
|
kit" to "CVS source distribution".
|
|
|
|
* sanity.sh: Comment out call to "whoami".
|
|
|
|
Thu Sep 11 10:09:04 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* commit.c (classify_file_internal): Add comment about saving
|
|
quiet vs. saving really_quiet.
|
|
|
|
* sanity.sh (newb-123j0): Use two regexps instead of assuming that
|
|
expr has "\(", "\|", and "\)". If we want to require the latter,
|
|
we should check for it up front, rather than let people get
|
|
halfway through and wonder why the test failed.
|
|
|
|
Tue Sep 9 19:22:44 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* expand_path.c (expand_path): If GETPWNAM_MISSING is defined,
|
|
just give an error instead of calling getpwnam.
|
|
|
|
* subr.c (getcaller): If SYSTEM_GETCALLER is defined, call it
|
|
instead of all the getlogin/getpwuid/etc.
|
|
* wrapper.c (wrap_setup): Call get_homedir not getpwuid.
|
|
|
|
Mon Sep 8 17:54:14 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* sanity.sh (basicc): Change ls -1 to echo *; according to
|
|
larry.jones@sdrc.com, ls -1 isn't portable.
|
|
|
|
Sun Sep 7 07:45:35 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* sanity.sh (basic2): In test basic2-64, match usernames with
|
|
${username}.
|
|
|
|
* root.c: Reindent a few things.
|
|
* root.c, cvs.h (same_directories): Remove. Never used,
|
|
portability hassle.
|
|
|
|
* add.c (add_directory): When checking for CVSADM, call fncmp not
|
|
strcmp. I actually suspect this code doesn't do much these days,
|
|
but fncmp clearly will make more sense than strcmp.
|
|
|
|
* rtag.c (rtag_usage), tag.c (tag_usage): Reword to hopefully be
|
|
clearer that -r takes either numeric or symbolic revision.
|
|
|
|
* ignore.c (ign_dir_add, ignore_directory): Reindent. Tweaks to
|
|
comments.
|
|
|
|
* update.c (checkout_file): Only ignore existence_error from
|
|
unlink_file_dir, not all errors.
|
|
|
|
* checkout.c (safe_location): Check for errors from xgetwd.
|
|
* create_adm.c (Create_Admin): Remove call to xgetwd; it is just
|
|
debugging code anyway and it wasn't checking for errors.
|
|
|
|
* sanity.sh: Add comment about default value for TESTDIR.
|
|
|
|
* server.c (serve_log): Change "cvslog" to "log". This
|
|
(accidental, I presume) error had made it impossible for anonymous
|
|
users to run "cvs log".
|
|
|
|
* classify.c (sticky_ck): Change to take an finfo argument rather
|
|
than several arguments taken from there. Cleans up the way the
|
|
calling convention had depended on SERVER_SUPPORT.
|
|
(Classify_File): Change callers.
|
|
|
|
* version.c: Change version number to 1.9.17.
|
|
|
|
* Version 1.9.16.
|
|
|
|
* recurse.c (do_dir_proc): In combining repository and dir, omit
|
|
trailing "/." from repository.
|
|
* sanity.sh (modules3): Adjust test modules3-4 so we test for
|
|
this fix (this is not just cosmetic; the bug prevented the
|
|
"Rebuilding administrative file database" from happening).
|
|
modules2 already tests the "co CVSROOT/modules" usage.
|
|
|
|
* checkout.c (checkout_proc): When building top-level CVSADM
|
|
directory, continue the process of walking up the repository one
|
|
more level, rather than putting in the same repository as for the
|
|
first-level directory.
|
|
* sanity.sh: Adjust tests basicb-1b, basicb-9b, modules3-7f,
|
|
toplevel-9, and toplevel-11 to test for this fix.
|
|
|
|
(For reference, this takes CVS's text segment from 344460 to
|
|
344140 bytes. I know, this may seem unimportant, but it is so
|
|
unusual for programs to shrink and I think it is so cool when they
|
|
do without losing functionality/clarity/etc).
|
|
* checkout.c, cvs.h (emptydir_name): New function.
|
|
* checkout.c (checkout, checkout_proc), modules.c (do_module):
|
|
Call it instead of duplicating the code to do that.
|
|
|
|
* sanity.sh (basicc): Clean up first-dir at end of test.
|
|
|
|
Sat Sep 6 09:48:39 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* sanity.sh (join): Fix cut and paste error in join-28
|
|
(/home/kingdon/... -> ${TESTDIR}).
|
|
|
|
* recurse.c (start_recursion): If there is no CVSADM and no
|
|
subdirectories, give an error.
|
|
* sanity.sh (basicc): New tests, test for this fix.
|
|
|
|
* sanity.sh (join): New tests join-25 through join-29 test merging
|
|
from one branch to a different branch.
|
|
|
|
* release.c: In comment about CVSROOTADM_IGNORE, also mention
|
|
comment just added to entries.c.
|
|
* entries.c: Expand this comment, especially the part about
|
|
CVS/Template.
|
|
|
|
Keep track of what revisions CVS/Base correspond to:
|
|
* cvs.h (CVSADM_BASEREV, CVSADM_BASEREVTMP): Added.
|
|
* entries.c, cvs.h (base_register, base_deregister, base_get,
|
|
base_walk): New functions.
|
|
* edit.c (edit_fileproc): Call base_register when setting up CVS/Base.
|
|
(unedit_fileproc): When taking a file out of CVS/Base, put its
|
|
revision back into entries, and base_deregister it.
|
|
* sanity.sh (watch4): New tests watch4-10 through watch4-18 test
|
|
for above fix.
|
|
|
|
Fri Sep 5 09:14:10 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* client.c: Only declare start_rsh_server if we are going to
|
|
define it (!NO_EXT_METHOD).
|
|
|
|
* subr.c, cvs.h (check_numeric): New function.
|
|
* admin.c (admin): Call it.
|
|
* sanity.sh (admin): New tests admin-10a and admin-10b test for fix.
|
|
|
|
Thu Sep 4 15:55:39 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* sanity.sh (binwrap2): New tests, test for the ability to specify
|
|
all files are binary except certain patterns.
|
|
|
|
* sanity.sh (modules3): New tests modules3-16 and modules3-17 test
|
|
for another behavior involving '/' in a module name.
|
|
|
|
Sun Aug 31 12:03:15 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* add.c (add): Remove checks for '/' in pathnames.
|
|
(add): Move code which handles entries and repository inside loop,
|
|
since these now might be different for each argument. Add code to
|
|
set finfo.update_dir, finfo.file, and finfo.fullname appropriately
|
|
even if pathname contains '/'. Replace user variable with
|
|
finfo.file or finfo.fullname, depending on which is meant. chdir
|
|
into update_dir for each argument. Likewise for the client code
|
|
which creates directories.
|
|
(add_directory): Replace arguments with a single finfo argument.
|
|
Replace dir with finfo->fullname as needed.
|
|
(add): Update call to add_directory.
|
|
* client.c, client.h (send_a_repository): No longer static.
|
|
* sanity.sh (errmsg2): Adjust tests to test for '/' in pathname.
|
|
|
|
* sanity.sh (errmsg2): New tests errmsg2-13 through errmsg2-16
|
|
test the status quo with respect to '/' in cvs add argument.
|
|
|
|
Sat Aug 30 17:37:29 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* run.c (run_popen): Add comment on return value.
|
|
* release.c (release): Check for NULL return from popen.
|
|
|
|
Fri Aug 29 17:49:20 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* client.c (get_server_responses): Add comment about "ok^M".
|
|
|
|
Thu Aug 28 13:35:12 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* edit.c (edit_fileproc): If file doesn't exist, give an error.
|
|
* sanity.sh (devcom2): Tests devcom2-12 through devcom2-17 test
|
|
for above fix.
|
|
|
|
Tue Aug 26 16:42:28 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* subr.c (xmalloc): Reword error message to clarify that memory,
|
|
not disk space or some other resource, is in question.
|
|
|
|
Tue Aug 26 01:04:48 1997 Steve Ralston <sralston@ppdpost.ks.symbios.com>
|
|
and Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* add.c (add_directory): In allocating message, also allocate
|
|
enough for tag and date related text.
|
|
|
|
Tue Aug 26 01:04:48 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* update.c (update_dirent_proc): Use update_dir not dir in "new
|
|
directory" message.
|
|
* find_names.c (find_dirs): Skip CVSNULLREPOS.
|
|
(Find_Directories): Add comment about find_dirs skipping CVSATTIC
|
|
and CVSLCK in working directories.
|
|
* sanity.sh (basicb): New tests basicb-edir-* test for find_dirs
|
|
fix. Change other tests to test that Emptydir is not special in
|
|
non-CVSNULLREPOS contexts.
|
|
|
|
Sun Aug 17 14:44:57 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* import.c (add_rcs_file): Add comment about -k overriding wrappers.
|
|
|
|
Sat Aug 16 18:09:05 1997 Martin Sjoelin <martin.sjoelin@ubs.ch>
|
|
and Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* import.c (add_rcs_file): Before opening the input file
|
|
when importing, if options is binary, open the file in
|
|
binary mode.
|
|
|
|
1997-08-16 enami tsugutomo <enami@ba2.so-net.or.jp>
|
|
|
|
* sanity.sh (mcopy): Unset CVSWRAPPERS last of all.
|
|
|
|
Fri Aug 15 11:11:44 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* add.c (add_directory): Copy default file attributes from the
|
|
parent directory to the directory we are creating.
|
|
* fileattr.h, fileattr.c (fileattr_getall, fileattr_setall):
|
|
New functions, in support of above.
|
|
* fileattr.c (fileattr_free): Add comment about fileattr_write
|
|
maybe not clearing attrs_modified.
|
|
* sanity.sh (watch4): New test, tests for above fix.
|
|
|
|
Thu Aug 14 11:08:40 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* update.c (merge_file, join_file): If wrap_merge_is_copy, treat
|
|
files as nonmergeable (as we had been treating binary files).
|
|
(join_file, update_fileproc): Remove previous wrap_merge_is_copy
|
|
cruft.
|
|
* sanity.sh (mcopy): New tests, test for above fix.
|
|
(binfiles2): New tests binfiles2-9a-* correct an oversight.
|
|
(binfiles, binfiles2): Adjust to reflect wording change from
|
|
"binary file" to "nonmergeable file".
|
|
(mwrap): Adjust tests mwrap-8 through mwrap-10 for new behavior.
|
|
|
|
* main.c (main): Reword copyright notices to include the latest
|
|
year, to refer to "other authors" in addition to the ones listed,
|
|
and to be more concisely formatted.
|
|
|
|
Wed Aug 13 13:50:00 1997 Larry Jones <larry.jones@sdrc.com>
|
|
|
|
* sanity.sh: Replace hard-coded directory with ${TESTDIR}, add
|
|
join3 to default tests
|
|
|
|
Wed Aug 13 11:42:24 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* rcscmds.c: Adjust comment to reflect progress on removing RCS
|
|
execs outside this file.
|
|
|
|
Mon Aug 11 10:14:47 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* vers_ts.c (Version_TS): If vers_ts->vn_rcs == NULL, skip setting
|
|
modification time in server case as well as local case.
|
|
* server.c (server_modtime): Add assertion to clarify that caller
|
|
must assure that vers_ts->vn_rcs != NULL.
|
|
* sanity.sh (join3): Add file "file2" to test for above fix.
|
|
|
|
* modules.c (save_d): When parsing -s option, don't assume that
|
|
we will hit a space before we hit the '\0'.
|
|
(struct sortrec): Document allocation policies (status quo except
|
|
status field is now malloc'd).
|
|
(cat_module): No longer need to set the '\0' at the end of the
|
|
status field back to ' ', as it no longer shares storage with the
|
|
rest field.
|
|
* sanity.sh (modules): Add "statusmod" to test for above fix.
|
|
|
|
Sun Aug 10 12:18:31 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* sanity.sh: Remove TODO item about more keyword expansion tests.
|
|
The keyword test and others cover it pretty well, and such an item
|
|
isn't useful unless it is specific.
|
|
|
|
* sanity.sh (importb): New tests test "cvs import -b".
|
|
|
|
* mkmodules.c: Update comment with more reasons why having
|
|
CVSROOT/passwd be a regular administrative file would be a Bad
|
|
Idea.
|
|
|
|
* server.c (switch_to_user): Add comment about checking for errors
|
|
from setuid and friends.
|
|
|
|
Wed Aug 6 13:48:29 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* main.c (main): Add comment about errors writing CVS/Root in
|
|
need_to_create_root code.
|
|
|
|
Tue Aug 5 22:05:20 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* entries.c (write_entries): If trouble writing Entries.Backup,
|
|
make it a warning not an error.
|
|
|
|
Wed Jul 30 08:42:04 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* parseinfo.c (parse_config): If AUTH_SERVER_SUPPORT is not
|
|
defined, don't set system_auth.
|
|
|
|
* version.c: Change version number to 1.9.15.
|
|
|
|
* Version 1.9.14.
|
|
|
|
* create_adm.c, cvs.h (Create_Admin): If new argument WARN is set,
|
|
then make creating the CVS directory itself a warning not a fatal
|
|
error. New return value indicates whether we did this.
|
|
* checkout.c (build_one_dir), client.c (call_in_directory):
|
|
Pass WARN as one.
|
|
* add.c, client.c, checkout.c, modules.c, update.c: Pass WARN as
|
|
zero for all other Create_Admin callers.
|
|
* sanity.sh (toplevel): New test toplevel-12 tests for this fix.
|
|
* filesubr.c (mkdir_if_needed): Also check EACCES/isdir. Needed
|
|
to make toplevel-12 test work.
|
|
|
|
* sanity.sh (toplevel): New test toplevel-11 and friends test for
|
|
another variation of the toplevel-9 bug.
|
|
|
|
Tue Jul 29 12:11:16 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* login.c (construct_cvspass_filename): Revert this change. The
|
|
main reason is procedural; Karl is not a current CVS developer.
|
|
The other thing is that the new text doesn't say anything about
|
|
HOMEDRIVE and HOMEPATH.
|
|
|
|
Tue Jul 29 11:36:22 1997 Karl Fogel <kfogel@harvey.cyclic.com>
|
|
|
|
* login.c (construct_cvspass_filename): error message informs user
|
|
she may need to set HOME environment variable by hand.
|
|
|
|
Sun Jul 27 15:36:44 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* admin.c (admin): Remove comment "XXX send -ko too with i = 0".
|
|
It turns out to be a description of a bugfix which was applied on
|
|
8 Oct 1995, and never should have been in a comment in the first
|
|
place.
|
|
|
|
* admin.c (admin, admin_fileproc): Parse options ourself rather
|
|
than blindly passing them to RCS.
|
|
Accordingly, add struct admin_data and function arg_add, and delete
|
|
global variables ac and av.
|
|
* sanity.sh (admin): Change admin-3 test to reflect cvs admin -i
|
|
now being an error.
|
|
(basicb): Change basicb-21 test to relect the improved error
|
|
message here.
|
|
|
|
Sat Jul 26 11:34:51 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* sanity.sh (join3): New tests, test a new branch topology and
|
|
greatest common ancestor.
|
|
|
|
Fri Jul 25 09:51:49 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* server.c (serve_directory): Repository must start with
|
|
CVSroot_directory rather than some random pathname.
|
|
|
|
* remove.c (remove_fileproc): If there is a numeric sticky tag,
|
|
don't allow the remove.
|
|
* commit.c (check_fileproc): Add comment about this case.
|
|
* sanity.sh (sticky): New tests sticky-15 through sticky-23
|
|
test for this behavior and the analogous behavior with
|
|
non-branch sticky tags (which is unchanged).
|
|
|
|
* client.c (update_entries): Clear the stored mode, modtime, and
|
|
checksum even on an error.
|
|
* sanity.sh (contents2-142d*): Also test cvs status. Also test
|
|
the case in which the contents of the file are unchanged. Also
|
|
test running diff to see the conflict, and resolving the conflict.
|
|
Without the fix above, the new contents2-142d2 test would get a
|
|
"duplicate Mod-time" warning.
|
|
|
|
Thu Jul 24 13:29:15 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* server.c (pserver_authenticate_connection): Call parse_config
|
|
here too.
|
|
* parseinfo.c (parse_config): If we are called several times, the
|
|
times beyond the first do nothing.
|
|
* cvs.h, parseinfo.c (parse_config): New argument cvsroot.
|
|
* server.c, main.c: Update callers.
|
|
* server.c, server.h (system_auth): New variable.
|
|
* parseinfo.c (parse_config): Parse new keyword SystemAuth.
|
|
* mkmodules.c (config_contents): Add comments for SystemAuth.
|
|
* server.c (check_password): If !system_auth, then skip the check
|
|
for a system username/password.
|
|
|
|
* main.c (main): No fatal error if parse_config returned an error.
|
|
* server.c (serve_root): Likewise.
|
|
* error.c (error): Add comment about calling from the server.
|
|
* parseinfo.c, cvs.h (parse_config): Remove NOERR crock.
|
|
Closer reading of server.c makes it seem like calling error
|
|
here is OK after all.
|
|
* sanity.sh (config): New test, tests for above fix.
|
|
|
|
* server.c (serve_root): Fix typo ("doign" -> "doing").
|
|
|
|
Wed Jul 23 13:55:09 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* vers_ts.c (Version_TS): If entdata is for a directory, don't set
|
|
vn_user and friends.
|
|
* server.c (dirswitch): Add comment about why Subdir_Register is
|
|
sometimes a noop here.
|
|
* cvs.h (struct vers_ts): Fix comments about NULL vs. "" in vn_user.
|
|
* sanity.sh (errmsg2): New tests errmsg2-10 through errmsg2-12 test
|
|
for above fix.
|
|
* add.c (add_directory): Call cvs_output not printf. This fixes
|
|
an out-of-order bug which was showing up in the testcase.
|
|
|
|
21 Jul 1997 Jim Kingdon
|
|
|
|
* subr.c (get_file): Put st_size into an unsigned variable to
|
|
avoid signed/unsigned warnings.
|
|
|
|
Mon Jul 21 00:19:30 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* version.c: Change version number to 1.9.13.
|
|
|
|
* Version 1.9.12.
|
|
|
|
* sanity.sh (toplevel, head): Delete our files from the repository
|
|
when done with them.
|
|
|
|
Sun Jul 20 15:53:08 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* sanity.sh (admin, reserved): New tests, test most cvs admin
|
|
behaviors.
|
|
(log2): New tests log2-5 through log2-10 test setting the
|
|
description via cvs admin.
|
|
|
|
Thu Jul 17 12:39:27 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* client.c (socket_buffer_input, socket_buffer_output): Add
|
|
comment regarding size of of buffer we pass to send and recv.
|
|
|
|
Sat Jul 12 15:21:24 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* import.c, rcs.h (add_rcs_file): New argument key_opt replaces
|
|
access to global variable keyword_opt. New arguments desctext and
|
|
desclen allow one to set the description. If add_vhead is NULL,
|
|
then omit a revision, like rcs -i.
|
|
* import.c (process_import_file), mkmodules.c (init): Change
|
|
callers.
|
|
* subr.c, cvs.h (get_file): New function, adapted from code in
|
|
update_entries.
|
|
* client.c (update_entries): Call it.
|
|
* commit.c (checkaddfile): Create new RCS files with add_rcs_file
|
|
rather than rcs -i.
|
|
|
|
Fri Jul 11 12:14:54 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* update.c (join_file): Handle binary files ourself rather than
|
|
passing them to RCS_merge (which sort of tries to handle binary
|
|
files, but does so badly).
|
|
* sanity.sh (binfile2): New "brmod", "brmod-trmod", and
|
|
"brmod-wdmod" tests test for this fix.
|
|
|
|
* add.c (add): Exit status is now nonzero if any of the arguments
|
|
failed (already was mostly true, make it true for the new sanity
|
|
check). Move check for '/' (now ISDIRSEP) up to sanity check, so
|
|
the client does it too.
|
|
* sanity.sh (errmsg2): Test for this fix.
|
|
|
|
Thu Jul 10 00:02:54 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* add.c (add): Check for CVSADM, ".", and "..", and skip any
|
|
attempt to add them.
|
|
* sanity.sh (errmsg2): New tests, tests for above fix.
|
|
|
|
* main.c (main): In text printed upon --version, also refer
|
|
people to --help.
|
|
|
|
* rcscmds.c (RCS_exec_rcsdiff): Add comment about timezones.
|
|
|
|
* server.c, cvs.h (cvs_output_binary): New function.
|
|
* rcs.c (RCS_checkout): For a binary file, call cvs_output_binary
|
|
rather than cvs_output.
|
|
* client.c (handle_mbinary): New function.
|
|
(responses): Add "Mbinary".
|
|
|
|
Tue Jul 8 12:18:16 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* filesubr.c (get_homedir): Add comment about root vs. user
|
|
directory in pserver server.
|
|
|
|
Mon Jul 7 14:39:33 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* sanity.sh (big): Also test Rcs-diff case.
|
|
|
|
* client.c (update_entries): Reindent a line.
|
|
|
|
* sanity.sh (death2): Add comment about Sun diff. Thanks to
|
|
Warren Jones <wjones@TC.FLUKE.COM> for reporting this.
|
|
|
|
* client.c (update_entries): If DONT_USE_PATCH, then just treat a
|
|
Patched response as an indication to try again with complete files.
|
|
* update.c (update): Remove DONT_USE_PATCH ifdefs, since a CVS
|
|
with DONT_USE_PATCH defined can still handle Rcs-diff.
|
|
* sanity.sh (serverpatch): Add comment about this coming up in
|
|
real life if the user modifies the file while CVS is running.
|
|
|
|
* client.c (start_server): Add comment about logfiles if there are
|
|
several connections to the server.
|
|
(log_buffer_shutdown): Close the logfile after shutting down the
|
|
underlying buffer. This way at least we get the last set of
|
|
logfiles rather than a bizarre mishmash.
|
|
|
|
1997-07-06 enami tsugutomo <enami@but-b.or.jp>
|
|
|
|
* logmsg.c (do_verify): Unlink temporary file before call error().
|
|
Remove unneeded `return' statement. Fix comment.
|
|
|
|
Sun Jul 6 13:36:32 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* diff.c (diff_fileproc): Change message from "during rcsdiff of"
|
|
to "while diffing". For a while now, that message might be
|
|
printed after a call to DIFF instead of rcsdiff.
|
|
|
|
* diff.c (diff_fileproc): Call cvs_output not printf. Remove
|
|
calls to fflush (should be handled now by call to cvs_outflush in
|
|
recurse.c).
|
|
* patch.c (patch_fileproc): Likewise.
|
|
|
|
* rcscmds.c, cvs.h (diff_exec): New function.
|
|
* diff.c (diff_fileproc), patch.c (patch_fileproc),
|
|
update.c (patch_file): Call it.
|
|
|
|
* rcscmds.c: Adjust comments concerning diff library to point to
|
|
new, expanded comments at diff_exec. Remove comments concerning
|
|
patch library; Rcs-diff should be adequate.
|
|
|
|
* client.c (update_entries): Add comment about GNU patch usage (-b
|
|
option and so on).
|
|
|
|
Sat Jul 5 04:13:28 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* rcscmds.c, cvs.h (RCS_exec_rcsdiff): New function.
|
|
* diff.c (diff_fileproc): Call it.
|
|
|
|
Thu Jul 3 09:50:07 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* main.c (main): Add comment about how long we should keep the
|
|
deprecated "cvs rlog" alias.
|
|
|
|
* server.c (cvs_output): Add comment about whether to fflush.
|
|
|
|
Wed Jul 2 18:57:29 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* sanity.sh (join2): New tests join2-19 and friends test for a
|
|
case that we can't readily get right (see comments).
|
|
|
|
Tue Jul 1 09:52:09 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* main.c (cmd_usage): Say "specify" --help rather than the vague
|
|
"use".
|
|
(opt_usage, cmd_synonyms): Mention --help here too.
|
|
* add.c, admin.c, checkout.c, commit.c, diff.c, edit.c, import.c,
|
|
log.c, login.c, mkmodules.c, patch.c, rcs.c, release.c, remove.c,
|
|
rtag.c, status.c, tag.c, update.c, watch.c: Likewise, for all the
|
|
other help messages.
|
|
|
|
* sanity.sh (join2): New tests.
|
|
|
|
* repos.c (Name_Repository): Check for errors from fclose.
|
|
|
|
Fri Jun 27 10:27:48 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* scramble.c, login.c: Reindent.
|
|
|
|
* client.c (connect_to_pserver): Check for errors from send().
|
|
If socket() fails, include SOCK_STRERROR (SOCK_ERRNO) in message.
|
|
|
|
Wed Jun 25 11:21:52 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* main.c: New option --help-options, with much of the text
|
|
from --help. Fix a few capitalization and punctuation problems.
|
|
Rewrite text for --help to be an intro to all the --help-*
|
|
options.
|
|
|
|
* sanity.sh (head): New tests, concerning meaning of HEAD.
|
|
|
|
Tue Jun 24 10:14:18 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* client.c (recv_line): New function.
|
|
(connect_to_pserver): Call it, and rewrite accordingly. Also
|
|
accept new "E" and "error" responses. Change \n to \012.
|
|
* server.c (pserver_authenticate_connection): Adjust comment
|
|
accordingly, concerning sending I HATE YOU not "error".
|
|
|
|
Sun, 22 Jun 1997 Jim Kingdon
|
|
|
|
* main.c (main): Move setting of server_active inside #ifdef
|
|
SERVER_SUPPORT; otherwise the variable doesn't exist.
|
|
* main.c (main): Call return after exit to shut up warning.
|
|
|
|
Fri Jun 20 22:56:34 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* client.c (connect_to_pserver): On "I HATE YOU", give
|
|
"authorization failed" fatal error regardless of verify_only.
|
|
* login.c (login): Don't print that "incorrect password" message;
|
|
it now is possible with an --allow-root failure as well as an
|
|
incorrect password. cvsclient.texi doesn't really specify what I
|
|
HATE YOU means in any detail, and it seems a little silly for
|
|
login to give different messages than the other commands.
|
|
* client.c, client.h (connect_to_pserver): Return type now void.
|
|
|
|
Thu Jun 19 12:16:10 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* server.c (serve_root): Give error on duplicate Root request.
|
|
Call parse_config.
|
|
* parseinfo.c, cvs.h (parse_config): New function.
|
|
* cvs.h (CVSROOTADM_CONFIG): Added.
|
|
* main.c (main): Set server_active here...
|
|
* server.c (server): ...not here. That seems cleaner than
|
|
strcmp's between command_name and "server" in main.c.
|
|
* main.c (main): Call parse_config.
|
|
* main.c, cvs.h (free_Rcsbin): Make global.
|
|
* mkmodules.c (filelist): Add CVSROOTADM_CONFIG.
|
|
|
|
Wed Jun 18 11:24:31 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* version.c: Change version number to 1.9.11.
|
|
|
|
* Version 1.9.10.
|
|
|
|
Tue Jun 17 22:48:00 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* main.c (main): Add --allow-root=ROOT argument; call
|
|
root_allow_add for each time it is specified. Call
|
|
root_allow_free before exiting.
|
|
* root.c, cvs.h (root_allow_add, root_allow_free, root_allow_ok):
|
|
New function.
|
|
* server.c (pserver_authenticate_connection): If root_allow_ok
|
|
doesn't like the CVSROOT directory, don't allow access.
|
|
|
|
Tue Jun 17 14:30:14 1997 Jim Kingdon (unknown@beezley)
|
|
|
|
* client.c: Add "copyright" notice. If NO_EXT_METHOD, omit
|
|
start_rsh_method.
|
|
* client.c (update_entries): Cast argument to MD5Update from
|
|
char * to unsigned char *.
|
|
|
|
Mon Jun 16 16:46:28 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* run.c (piped_child, filter_stream_through_program):
|
|
If USE_SETMODE_BINARY, then put the pipes into binary mode.
|
|
* find_names.c, ignore.c, lock.c, wrapper.c: Change fnmatch to
|
|
CVS_FNMATCH.
|
|
* client.c (start_server): If NO_EXT_METHOD, then give a fatal
|
|
error on any use of :ext:.
|
|
|
|
Sun Jun 15 22:30:27 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* sanity.sh (toplevel): Match U CVSROOT/* lines with DOTSTAR in
|
|
test toplevel-9.
|
|
|
|
Thu Jun 12 10:27:51 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* sanity.sh (toplevel): Remove Emptydir before starting.
|
|
|
|
* sanity.sh: Change "rm -rf" to "rm -r" when deleting working
|
|
directories (except a few watches cases). Helps detect cases
|
|
where the testsuite has cd'd to somewhere other than where we
|
|
think it has.
|
|
(basic2): Remove "rm -r first-dir" between tests 49 and 50. The
|
|
directory was already deleted in test 45.5.
|
|
(rcs): Add "cd .." at end of tests.
|
|
(stamps): No longer cd to TESTDIR; shouldn't be necessary with
|
|
fix to "rcs" test.
|
|
|
|
Wed Jun 11 22:28:38 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* sanity.sh (basicb): Also remove CVSROOT/Emptydir at end of
|
|
test. Otherwise it affects the toplevel-9 test for remote.
|
|
|
|
Tue Jun 10 14:03:32 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* sanity.sh (toplevel): Change "update" and "checkout" to "[a-z]*"
|
|
as these read "server" instead for "make remotecheck". Change
|
|
expect strings for toplevel-9 to accept the behavior of remote CVS
|
|
(see comments for more discussion).
|
|
|
|
* sanity.sh: New tests stamps-9 through stamps-11 test timestamp
|
|
behavior on cvs update.
|
|
|
|
Mon Jun 9 22:42:50 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* sanity.sh: Remove "#! /bin/zsh" line at end. I assume it was
|
|
added accidentally.
|
|
|
|
Tue Jun 10 03:08:46 1997 Norbert Kiesel <nk@psycho.de>
|
|
|
|
* sanity.sh: new tests "toplevel" for the new toplevel CVS
|
|
directory creation (including one test which shows an error in
|
|
this area).
|
|
|
|
Sun Jun 8 20:52:00 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* rcs.c (getrcsrev): Before printing error, check whether it was
|
|
feof or ferror.
|
|
|
|
* rcs.h, import.c (add_rcs_file): No longer static. New arguments
|
|
add_vbranch, add_vhead, and add_logfp replace access to static
|
|
variables vbranch, vhead, and logfp.
|
|
* mkmodules.c: Call it instead of RCS_CI.
|
|
* import.c (process_import_file): Adjust call to add_rcs_file.
|
|
|
|
Tue Jun 3 10:18:33 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* sanity.sh (basicb): Match "." with "\." not ".".
|
|
|
|
Tue Jun 3 13:02:37 1997 Norbert Kiesel <nk@cosa.de>
|
|
|
|
* checkout.c (checkout): Removed restriction of not sending -k in
|
|
remote export (I think this was introduced while the -k handling
|
|
was still broken in remote mode). Give better error texts
|
|
regarding -c and -s options. Use error() instead of usage() for
|
|
reporting errors in all places. Reindented some lines. Free
|
|
xmalloc'd space of options.
|
|
|
|
Thu May 29 16:32:47 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* rcscmds.c (RCS_checkin), mkmodules.c (init): Pass -w option to
|
|
"ci", specifying getcaller ().
|
|
* server.h, server.c (CVS_Username): Now extern.
|
|
* subr.c (getcaller): Return CVS_Username if it is set.
|
|
|
|
Wed May 28 22:31:38 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* update.c (update_fileproc): If wrap_merge_is_copy and we would
|
|
like to do a merge, give a fatal error. See comment for why.
|
|
* sanity.sh (mwrap): New tests, tests for above fix.
|
|
|
|
Tue May 27 21:59:32 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* sanity.sh (stamps): cd to ${TESTDIR} before starting.
|
|
|
|
Mon May 26 15:31:30 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* client.c (handle_mod_time): New function.
|
|
(responses): Add "Mod-time".
|
|
(stored_modtime_valid, stored_modtime): New variables.
|
|
(update_entries): If it is set, change the file's modtime.
|
|
* server.c, server.h (server_modtime): New function.
|
|
* vers_ts.c (Version_TS): Call it.
|
|
* patch.c (patch_fileproc): Add comment about why we don't.
|
|
* sanity.sh (stamps): Added, tests for above fix.
|
|
|
|
Fri May 16 13:14:30 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* subr.c (free_names): Update documentation to reflect fact that
|
|
free_names is now called to free vectors allocated by expand_wild
|
|
as well as by line2argv.
|
|
|
|
* main.c (main): Use "xstrdup (foo)" not "xstrdup(foo)" as
|
|
specified in HACKING.
|
|
|
|
Fri May 16 15:10:37 1997 Norbert Kiesel <nk@cosa.de>
|
|
|
|
* modules.c (do_module): initialize optind to 0. use local copies
|
|
of optarg's (because they might me freed within free_names).
|
|
|
|
Thu May 15 11:50:15 1997 Norbert Kiesel <nk@cosa.de>
|
|
|
|
* main.c (main): initialize optind to 0. use local copies of
|
|
optarg's (because they might me freed within read_cvsrc).
|
|
|
|
* cvsrc.c (read_cvsrc): free old argv after constructing a new
|
|
one. This fixes a memory leak.
|
|
|
|
* recurse.c (start_recursion): use free_names() instead of
|
|
reimplementing it
|
|
|
|
* rcs.c (RCS_deltas): free branchversion (memory leak).
|
|
|
|
* parseinfo.c (Parse_Info): free some vars (3 memory leaks).
|
|
|
|
* logmsg.c (logfile_write): free str_list_format (memory leak).
|
|
|
|
* watch.c (watch_addremove), (watchers), update.c (update), tag.c
|
|
(cvstag), status.c (status), rtag.c (rtag), remove.c (cvsremove),
|
|
release.c (release), patch.c (patch), log.c (cvslog), import.c
|
|
(import), history.c (history), edit.c (watch_onoff), (edit),
|
|
(unedit), (editors), diff.c (diff), commit.c (commit), checkout.c
|
|
(checkout), add.c (add): initialize optind to 0
|
|
|
|
* diff.c (diff_fileproc): cosmetic change (whitespace added).
|
|
|
|
* checkout.c (checkout): move local variable definition into the
|
|
block where the variable is used.
|
|
|
|
* client.c (update_entries): initialize some local variables to shut up
|
|
gcc -O -Wall.
|
|
|
|
* buffer.c (buf_read_line): initialize a local variable to shut up
|
|
gcc -O -Wall.
|
|
|
|
|
|
Wed May 14 16:29:50 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* admin.c (admin): When sending options to server, don't try to
|
|
send av[ac]. It may contain one of the names that we'll send in
|
|
send_file_names (which caused tests like keyword-6 to work,
|
|
sort of accidentally), or it may contain NULL (which would tend to
|
|
cause a coredump).
|
|
* sanity.sh (basicb): New test basicb-21 tests for above fix.
|
|
|
|
Mon May 12 16:22:00 1997 Larry Jones <larry.jones@sdrc.com>
|
|
|
|
* add.c (add): Free message and repository in client code.
|
|
* checkout.c (checkout): Don't free repository unless allocated.
|
|
* client.c (start_rsh_server): Free command.
|
|
|
|
Sun May 11 11:43:54 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* client.c: Remove all references to USE_DIRECT_TCP; see
|
|
../ChangeLog for rationale.
|
|
|
|
Fri May 9 22:19:36 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* main.c (main): Add comment explaining why we call exit. Pass 0
|
|
not EXIT_SUCCESS, because lib/system.h has portability cruft for
|
|
EXIT_FAILURE but not EXIT_SUCCESS.
|
|
|
|
Fri May 9 17:25:00 1997 Larry Jones <larry.jones@sdrc.com>
|
|
|
|
Fix miscellaneous memory allocation problems:
|
|
* add.c (add): Free repository.
|
|
* client.c (notified_a_file): Free getline buffer.
|
|
* edit.c (notify_check): Free getline buffer.
|
|
* hash.c (dellist): Free header node when not caching.
|
|
* login.c (login): Don't continually free & allocate getline
|
|
buffer, use xstrdup instead of xmalloc/strcpy, free getline
|
|
buffer before returning.
|
|
* main.c (main): Call exit instead of returning so tools like
|
|
Purify won't consider permanently allocated memory as leaks.
|
|
* mkmodules.c (mkmodules): Free getline buffer.
|
|
* modules.c (cat_module): Call close_module.
|
|
* rcs.c (rcsvers_delproc): Free state.
|
|
* recurse.c (start_recursion): Free files_by_dir.
|
|
(unroll_files_proc): NULL out p->data after using it to set
|
|
filelist to avoid multiple frees.
|
|
* server.c (check_command_legal_p): Don't continually free &
|
|
allocate getline buffer, free getline buffer before returning.
|
|
(check_repository_password): Ditto, use xstrdup instead of
|
|
xmalloc/strcpy.
|
|
* wrapper.c (wrap_add_file): Free getline buffer.
|
|
|
|
Thu May 8 14:21:00 1997 Larry Jones <larry.jones@sdrc.com>
|
|
and Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* checkout.c (checkout_proc): Free finfo.rcs (memory leak).
|
|
|
|
8 May 1997 Larry Jones <larry.jones@sdrc.com>
|
|
and Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* hash.c: Add #ifdef's to disable caching. This makes it easier
|
|
to track down memory allocation problems.
|
|
|
|
Thu May 8 11:40:39 1997 Larry Jones <larry.jones@sdrc.com>
|
|
|
|
* sanity.sh: In setting "tests" use a number of statements rather
|
|
than one very long line.
|
|
|
|
Thu May 8 11:40:39 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* cvsbug.sh: Remove $Id; we decided to get rid of these some time
|
|
ago.
|
|
|
|
Thu May 8 11:34:02 1997 Larry Jones <larry.jones@sdrc.com>
|
|
|
|
* cvsbug.sh: Put separate statements on separate lines, so it
|
|
works if awk is AT&T nawk.
|
|
|
|
Tue May 6 16:56:00 1997 Larry Jones <larry.jones@sdrc.com>
|
|
and Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* cvsrc.c (read_cvsrc): Fix various memory allocation problems:
|
|
rearrange code to avoid leaks, use xrealloc instead of xmalloc/
|
|
copy/free, make sure there's room for the remaining args before
|
|
appending them.
|
|
|
|
Tue May 6 14:20:00 1997 Larry Jones <larry.jones@sdrc.com>
|
|
|
|
* edit.c (watch_onoff, edit, unedit, editors): Add -R like
|
|
other things with -l.
|
|
* watch.c (watch_addremove, watchers): Ditto.
|
|
|
|
Mon May 5 18:10:37 1997 larry.jones@sdrc.com
|
|
and Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* sanity.sh: Change all /tmp/cvs-sanity to TESTDIR. If TESTDIR
|
|
environment variable is set, use it instead of /tmp/cvs-sanity.
|
|
* sanity.sh: Make TMPPWD the pwd equivalent of TESTDIR, not of /tmp.
|
|
|
|
4 May 1997 Larry jones <larry.jones@sdrc.com>
|
|
and Jim Kingdon
|
|
|
|
* checkout.c, diff.c, patch.c, rcs.c: Update usage messages.
|
|
* rcs.c (annotate): Add -R like other things with -l.
|
|
|
|
Sat May 3 14:57:40 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* sanity.sh (basic1): Rewrite test (use dotest, unroll the loops
|
|
which IMHO makes the test a zillion times more understandable, and
|
|
only do the variant which tests for 4 files at a time--we test one
|
|
file at a time lots of places).
|
|
|
|
2 May 1997 Ziv Gigus <ziv@rest.home.net>
|
|
and Jim Kingdon
|
|
|
|
* client.c, client.h (client_process_import_file): New argument
|
|
all_files_binary means treat all files as binary.
|
|
* import.c (import_descend): Pass it if -kb is specified.
|
|
* client.c (client_process_import_file): In the
|
|
non-all_files_binary case, call wrap_rcsoption to determine
|
|
whether the file is binary.
|
|
|
|
Thu May 1 13:44:51 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* sanity.sh (binfiles2): New tests, for update -j and binary files.
|
|
|
|
Wed Apr 30 11:18:36 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* recurse.c (start_recursion): Also free reposfile.
|
|
Don't look in repository if client_active (latter bug reported by Paul
|
|
Sanders <p.sanders@dial.pipex.com>).
|
|
|
|
Mon Apr 28 22:36:39 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* diff.c (diff_file_nodiff): Remove SERVER_SUPPORT ifdefs. They
|
|
were not based on server_active, which doesn't really make any
|
|
sense (it meant that compiling --disable-server could affect the
|
|
behavior of the non-client/server CVS). This affected the output
|
|
in tests death2-diff-11 and death2-diff-12 in the testsuite.
|
|
* sanity.sh (newb-123j0): Also accept "Needs Checkout", for a
|
|
--disable-server CVS.
|
|
|
|
* main.c (cmd_usage): Change "run diffs" to "show differences";
|
|
the former is jargon.
|
|
* edit.c (edit_usage): Fix typo ("." -> ",").
|
|
* edit.c (editors_usage), watch.c (watchers_usage): Mention -l.
|
|
* checkout.c (export_usage): Say what -P does.
|
|
* history.c (history_usg): Add comment about message wording.
|
|
|
|
Mon Apr 28 14:47:45 1997 Norbert Kiesel <nk@cosa.de>
|
|
|
|
* checkin.c (Checkin): use filename without path when calling
|
|
wrapper (bug found by Michal Schmitz <ms@cosa.de>).
|
|
|
|
Fri Apr 25 13:28:55 1997 Ian Lance Taylor <ian@cygnus.com>
|
|
|
|
* client.c (update_entries): In UPDATE_ENTRIES_RCS_DIFF case,
|
|
write to a temporary file and then rename it.
|
|
|
|
Thu Apr 24 11:35:40 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* subr.c, cvs.h (pathname_levels): New function, from a piece of
|
|
send_file_names.
|
|
* client.c (send_file_names): Call pathname_levels in place of the
|
|
code which was moved there.
|
|
* server.c, server.h (server_pathname_check): New function.
|
|
* recurse.c (start_recursion): Call it.
|
|
* sanity.sh (modules3): New test modules3-11b tests for above fix.
|
|
|
|
* filesubr.c: Do not define L_tmpnam. It is in ANSI and SunOS4,
|
|
so I don't think there will be a problem with it being missing.
|
|
Defining it too small can cause memory corruption.
|
|
(cvs_temp_name): Do not use L_tmpnam in the mktemp code; this
|
|
could cause a buffer overflow if the -T global option was in use.
|
|
|
|
Thu Apr 24 13:21:15 1997 Norbert Kiesel <nk@cosa.de>
|
|
|
|
* filesubr.c (cvs_temp_name): Use tempnam if available, else
|
|
mktemp, else tmpnam. See the comment for rationale.
|
|
|
|
* sanity.sh: use "tar cf - ." instead of "tar cf - *" for
|
|
directory copies.
|
|
|
|
Wed Apr 23 23:41:47 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* server.c (serve_update_prog): If in readonly mode, give an error.
|
|
|
|
Wed Apr 23 19:07:41 1997 Norbert Kiesel <nk@cosa.de>
|
|
|
|
* subr.c (line2argv): Allocate at least 4 slots for argv.
|
|
|
|
* checkout.c (checkout_proc): Add a comment which says why the
|
|
above change was necessary to avoid writing to unallocated memory.
|
|
|
|
Wed Apr 23 11:20:40 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* entries.c (ParseTag): Always set *NONBRANCHP.
|
|
|
|
21 Apr 1997 Jim Kingdon
|
|
|
|
* client.c (update_entries), rcs.c (expand_keywords): Rewrite
|
|
test to avoid signed/unsigned warning.
|
|
|
|
Mon Apr 21 09:02:22 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* update.c (patch_file): Add comment about whether auto-detecting
|
|
features of the DIFF program is a good idea.
|
|
|
|
Mon Apr 21 00:03:34 1997 Ian Lance Taylor <ian@cygnus.com>
|
|
|
|
Don't require the patch program:
|
|
* client.c (struct update_entries_data): Add
|
|
UPDATE_ENTRIES_RCS_DIFF to contents enum.
|
|
(update_entries): Handle UPDATE_ENTRIES_RCS_DIFF.
|
|
(handle_rcs_diff): New static function.
|
|
(responses): Add "Rcs-diff".
|
|
* server.c (server_updated): Handle SERVER_RCS_DIFF.
|
|
(server_use_rcs_diff): New function.
|
|
* server.h (enum server_updated_arg4): Add SERVER_RCS_DIFF.
|
|
(server_use_rcs_diff): Declare.
|
|
* update.c (rcs_diff_patches): New static variable.
|
|
(update): Set rcs_diff_patches.
|
|
(update_fileproc): If rcs_diff_patches, pass SERVER_RCS_DIFF
|
|
rather than SERVER_PATCHED to server_updated.
|
|
(patch_file): Correct initial comment to say diff rather than
|
|
rcsdiff. If rcs_diff_options, pass -n to diff rather than -c.
|
|
* rcs.c (rcs_change_text): New function.
|
|
* rcs.h (rcs_change_text): Declare.
|
|
|
|
Mon Apr 21 00:08:59 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* diff.c (diff_fileproc): Add comment concerning updating the
|
|
client timestamp.
|
|
|
|
Sun Apr 20 23:20:37 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* commit.c (commit): Add comment regarding SEND_FORCE rationale.
|
|
|
|
Sat Apr 19 17:10:36 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* server.c (dirswitch): If directory ends in '/', complain.
|
|
|
|
Fri Apr 18 18:09:57 1997 Ian Lance Taylor <ian@cygnus.com>
|
|
|
|
* rcs.c (apply_rcs_changes): New static function, broken out of
|
|
RCS_deltas.
|
|
(RCS_deltas): Call it.
|
|
(linevector_add): Change return type to int. Return an indication
|
|
of an error for an invalid add, rather than calling error.
|
|
|
|
Fri Apr 18 11:24:26 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* version.c: Change version number to 1.9.9.
|
|
|
|
* version.c: Version 1.9.8.
|
|
|
|
* commit.c (struct find_data): Add field force.
|
|
(find_fileproc, commit): Use it instead of force_ci to decide
|
|
whether to send files to server.
|
|
(commit): Set it if either -f or -r is specified.
|
|
* sanity.sh (basica): Add tests basica-8a0, basica-8a1, and
|
|
basica-8a2; tests for above fix.
|
|
|
|
Wed Apr 16 11:50:59 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* zlib.c: Remove paragraph with Free Software Foundation address.
|
|
See 2 Jan 1997 entry in ../ChangeLog for rationale.
|
|
|
|
Tue Apr 15 00:36:23 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* update.c (patch_file_write): Always assign to final_nl, so that
|
|
it ends up reflecting whether the data from the last call had a
|
|
newline, not whether the data from any of the calls ended in a
|
|
newline. Doesn't matter with the current RCS_checkout
|
|
implementation, but it will if RCS_checkout is changed to pass
|
|
less than the entire file.
|
|
|
|
* rcs.c (RCS_cmp_file): Change NULL to RUN_TTY in passing sout to
|
|
RCS_checkout, for clarity.
|
|
|
|
* import.c (update_rcs_file): Remove unused variable ierrno.
|
|
|
|
* add.c, checkout.c, commit.c, diff.c, edit.c, import.c,
|
|
history.c, log.c, main.c, patch.c, release.c, remove.c, rtag.c,
|
|
status.c, tag.c, update.c, watch.c: Pass "+" to all calls to
|
|
getopt. This ensures that we maintain existing behavior even with
|
|
glibc2.
|
|
|
|
* filesubr.c (fopen_case): Don't set *PATHP if we return an
|
|
error. Since the 9 Apr 1997 change, the behavior has been to
|
|
sometimes set it and sometimes not.
|
|
* rcs.c (RCS_parse): Adjust callers to not free it. Without this
|
|
change, they could call free() on an uninitialized variable.
|
|
|
|
* checkout.c (checkout): Add comment about export -k.
|
|
|
|
* root.c (check_root_consistent): Add comment about wording of
|
|
message.
|
|
|
|
Mon Apr 14 11:51:49 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* client.c (call_in_directory): If rdirp reaches the end of
|
|
reposdirname, then just set it to NULL. If server does not create
|
|
directories one at a time, give a warning.
|
|
* sanity.sh (modules3): Enable tests modules3-8 through
|
|
modules3-11 for remote; tests for above fix.
|
|
|
|
* client.c (call_in_directory): Don't set short_pathname to
|
|
pathname for a while; just use pathname itself (cleans up a relic
|
|
of the old "Repository" (not "Directory") code). Add comment
|
|
explaining short_pathname.
|
|
|
|
Sun Apr 13 18:07:50 1997 Ian Lance Taylor <ian@cygnus.com>
|
|
|
|
* rcs.c (RCS_checkout): Add pfn and callerdat parameters. Change
|
|
all callers. Move setting of expand after retrieval of file
|
|
data.
|
|
(struct cmp_file_data): Define.
|
|
(RCS_cmp_file): New function.
|
|
(cmp_file_buffer): New static function.
|
|
* rcs.h (RCSCHECKOUTPROC): Define type.
|
|
(RCS_checkout): Update declaration.
|
|
(RCS_cmp_file): Define.
|
|
* diff.c (diff_file_nodiff): Call RCS_cmp_file rather than
|
|
RCS_checkout and xcmp.
|
|
* import.c (update_rcs_file): Likewise.
|
|
* no_diff.c (No_Difference): Likewise.
|
|
* update.c (struct patch_file_data): Define.
|
|
(patch_file): Just return if noexec, or if binary file. Pass
|
|
patch_file_write to RCS_checkout. Don't check for newlines or
|
|
compute checksums here. Stat RCS file to set modes.
|
|
(patch_file_write): New static function.
|
|
|
|
* update.c (patch_file): Checkout directly to file2, rather than
|
|
to finfo->file followed by rename. Remove check for whether
|
|
result of checkout is readable; that was for an old, obsolete,
|
|
form of death support.
|
|
|
|
Sun Apr 13 13:16:40 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* checkout.c (build_one_dir): New function.
|
|
(struct dir_to_build): New structure.
|
|
(build_dirs_and_chir): Rewritten to accept a linked list of struct
|
|
dir_to_build rather than the silly string processing we had been
|
|
doing before.
|
|
(checkout_proc): Rewrite code that calls build_dirs_and_chdir
|
|
accordingly.
|
|
* sanity.sh: Enable tests modules3-10 and modules3-11 for local CVS;
|
|
tests for above fix.
|
|
|
|
* rcs.h (RCS_CO): Removed; no longer used.
|
|
|
|
Sun Apr 13 00:04:34 1997 Ian Lance Taylor <ian@cygnus.com>
|
|
|
|
Expand RCS keywords internally; never call co:
|
|
* rcs.h (struct rcsversnode): Add state field.
|
|
* rcs.c (kflags): Move out of RCS_check_kflag, and make file
|
|
static.
|
|
(enum kflag): Define.
|
|
(RCS_reparsercsfile): Always save lock information. Save state in
|
|
new state field, rather than other field.
|
|
(struct rcs_keyword): Define.
|
|
(keywords): New static variable.
|
|
(enum keyword): Define.
|
|
(printable_date, escape_keyword_value): New static functions.
|
|
(expand_keywords): New static function.
|
|
(RCS_checkout): Call expand_keywords. Don't call
|
|
RCS_exec_checkout.
|
|
(RCS_deltas): Add log and loglen parameters. Change all callers.
|
|
* log.c (log_version_requested): Use new state field.
|
|
(log_version): Likewise.
|
|
* cvs.h (RCS_exec_checkout): Don't declare.
|
|
* rcscmds.c (RCS_exec_checkout): Remove.
|
|
|
|
Sat Apr 12 17:32:59 1997 Ian Lance Taylor <ian@cygnus.com>
|
|
|
|
* sanity.sh (modules3): Remove second-dir at end of tests.
|
|
(sticky): Correct removal of directories at end of tests.
|
|
|
|
* sanity.sh (keyword): New tests for RCS keyword expansion.
|
|
|
|
Sat Apr 12 16:47:13 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* sanity.sh (basicb): New tests basicb-1b, basicb-1c, basicb-9b,
|
|
basic-9c test current build_dirs_and_chdir behavior.
|
|
|
|
Fri Apr 11 23:54:56 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* sanity.sh (modules3): New tests modules3-7* test for ability to
|
|
supply a path in -d in modules. Similar to modules3-8 through
|
|
modules3-11 except because the nesting is different, these ones
|
|
work.
|
|
|
|
Thu Apr 10 00:14:47 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* sanity.sh (modules3): New tests modules3-12 through modules3-15
|
|
test use of a module name which contains a slash.
|
|
|
|
* sanity.sh (basicb): New tests basicb-14 to basicb-20 test use of
|
|
co -d with two or more arguments.
|
|
|
|
* rcscmds.c: Refer to doc/RCSFILES in comment.
|
|
|
|
Wed Apr 9 09:49:17 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* sanity.sh (basicb): New tests basicb-11 through basicb-13 test
|
|
ability to specify several directory levels in co -d (commented
|
|
out).
|
|
|
|
* filesubr.c (fopen_case): If CVS_OPENDIR gives an
|
|
existence_error, return it to the caller instead of giving a fatal
|
|
error.
|
|
|
|
* client.c (update_entries): Fix typo in call to error (1 -> errno).
|
|
|
|
Tue Apr 8 23:02:22 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* error.h, error.c: Test for #ifdef __STDC__, not #if __STDC__.
|
|
This is consistent with other parts of CVS; it means that the
|
|
declaration for fperror will match the definition even if __STDC__
|
|
is defined to 0 as the SunPro 4.0 compiler does. Reported by
|
|
Richard Smith <rjsmith@cisco.com>.
|
|
|
|
2 Apr 1997 Jim Kingdon
|
|
|
|
* entries.c (ParseTag): Add "break;" after "default:" to avoid
|
|
error from Visual C++.
|
|
|
|
Wed Apr 2 12:06:44 1997 Vince Del Vecchio <vdelvecc@spd.analog.com>
|
|
and Jim Kingdon
|
|
|
|
* client.c: In reporting errors from socket calls, use
|
|
SOCK_STRERROR and SOCK_ERRNO since strerror(errno) doesn't work
|
|
for Win32.
|
|
|
|
Tue Apr 8 10:45:32 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* sanity.sh (modules3): Add tests modules3-8 to modules3-11, to
|
|
test for ability to supply a path to -d in modules. Mostly
|
|
commented out as CVS is buggy in this area.
|
|
|
|
Mon Apr 7 12:41:44 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* add.c (add): Add comment about SEND_NO_CONTENTS.
|
|
|
|
Sun Apr 6 21:46:32 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* update.c (update): Add comment about noexec and SEND_NO_CONTENTS.
|
|
|
|
Sun Apr 6 17:34:08 1997 Robert Bihlmeyer <robbe@orcus.priv.at>
|
|
|
|
* Pass +f not f to getopt_long to prevent options from being
|
|
permuted with glibc 2.0.1.
|
|
|
|
Sun Mar 30 00:07:05 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* cvs.h (struct vers_ts): Adjust comment regarding ts_user.
|
|
* server.c (serve_is_modified): New function. Set entries to show
|
|
that the file is modified but we don't know the contents.
|
|
* server.c (requests): Add "Is-modified" request.
|
|
* vers_ts.c (time_stamp_server): If the timestamp in entdata is
|
|
"M" or "D", just copy that over into ts_user.
|
|
* vers_ts.c (Version_TS): If timestamp is "D", use the entries
|
|
line for the sole purpose of passing it to time_stamp_server.
|
|
* no_diff.c (No_Difference): If ts_user is "M", conclude the files
|
|
are different.
|
|
* client.h, client.c (send_files): Replace arguments build_dirs
|
|
and force with argument flags. Add flag SEND_NO_CONTENTS and add
|
|
to struct send_data.
|
|
(send_fileproc): If no_contents, then send Is-modified instead of
|
|
Modified.
|
|
* add.c, admin.c, client.c, commit.c, diff.c, edit.c, log.c,
|
|
rcs.c, remove.c, status.c, tag.c, update.c, watch.c: Change all
|
|
send_files callers.
|
|
|
|
Fri Mar 28 22:32:25 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* server.c (requests): Change "Repository" to rq_optional. I'm
|
|
not sure whether I overlooked this when I removed support for
|
|
Repository, or whether I was thinking that servers would need to
|
|
support it anyway, for CVS 1.5 to 1.9 clients, but making it
|
|
optional doesn't prevent the server from supporting it and it
|
|
seems silly for the client to complain about absence of a request
|
|
that it never will use.
|
|
|
|
Fri Mar 28 10:06:59 1997 Steven Miller <Miller@wingra.com>
|
|
|
|
* entries.c (Subdirs_Known): Don't create Entries.Log if noexec.
|
|
|
|
Thu Mar 27 18:14:12 1997 Ian Lance Taylor <ian@cygnus.com>
|
|
|
|
* sanity.sh (death2): Remove commented out test death2-21. It
|
|
would now pass, but it duplicates the new test sticky-11.
|
|
|
|
Thu Mar 27 10:21:19 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* sanity.sh (dotest_internal): Write test output to logfile even
|
|
if test succeeds. This was the behavior prior to 30 Sep 1996.
|
|
See the comment for rationale.
|
|
|
|
Tue Mar 25 13:26:52 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* cvs.h, entries.c (WriteTag): Add arguments nonbranch,
|
|
update_dir, and repository. Move the server_set_sticky call from
|
|
callers to here.
|
|
* cvs.h, create_adm.c (Create_Admin): New argument nonbranch.
|
|
* cvs.h, entries.c (ParseTag): Add argument nonbranchp.
|
|
* cvs.h (struct stickydirtag): Add field nonbranch.
|
|
* entries.c (Entries_Open): Set it.
|
|
* cvs.h (Vers_TS): Add field nonbranch.
|
|
* vers_ts.c (Version_TS): Copy it from struct stickydirtag.
|
|
* server.c, server.h (server_set_sticky): Add argument nonbranch.
|
|
* add.c, client.c, checkout.c, modules.c, update.c, create_adm.c,
|
|
commit.c: Update callers.
|
|
* add.c (add): If nonbranch, don't add the file on that "branch".
|
|
* commit.c (write_dirnonbranch): New variable.
|
|
(commit_fileproc, commit): Set it.
|
|
(commit_dirleaveproc): Pass it to WriteTag.
|
|
* update.c (rewrite_tag, nonbranch): New variables.
|
|
(update, update_dirent_proc, update_fileproc): Set them.
|
|
(update_filesdoneproc): If rewrite_tag, call WriteTag.
|
|
* sanity.sh (sticky): New tests, test for above fix.
|
|
|
|
* version.c: Change version number to 1.9.7.
|
|
|
|
* version.c: Version 1.9.6.
|
|
|
|
Mon Mar 24 13:02:04 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* entries.c (ParseTag): Add comment about unrecognized characters
|
|
in CVS/Tag file.
|
|
|
|
* classify.c (Classify_File): Add comment about how specifying a
|
|
tag (bogusly?) suppresses certain messages.
|
|
|
|
Fri Mar 21 13:37:46 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* rcs.h (struct rcsnode): Add comment about case of PATH.
|
|
* rcs.c (RCS_parse): If ign_case, then try opening the file with
|
|
fopen_case.
|
|
* ignore.c (ign_case): Adjust comment.
|
|
* cvs.h, filesubr.c (cvs_casecmp, fopen_case): New functions.
|
|
|
|
20 Mar 1997 Jim Kingdon
|
|
|
|
* client.c (send_repository): When sending Directory request,
|
|
send any ISDIRSEP character as '/'. Fixes
|
|
"cvs log foo\bar\baz.c" on NT & friends.
|
|
|
|
* client.c (send_file_names): Don't try to read Entries file if
|
|
CVSADM directory does not exist. Fixes fairly serious regression
|
|
(warning on all fresh checkouts) introduced by 1997-01-08 change.
|
|
|
|
Tue Mar 18 13:03:33 1997 Jim Meyering <meyering@totoro.cyclic.com>
|
|
|
|
* sanity.sh (RCSINIT): Define to be empty and export, to hide any
|
|
existing value that might cause spurious failures.
|
|
|
|
* Makefile.in: (install): Depend on installdirs.
|
|
Remove `CYGNUS LOCAL' comment saying not to.
|
|
|
|
Tue Mar 18 09:36:26 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* recurse.c (struct recursion_frame): Reindent.
|
|
(do_dir_proc): Print message if we try to recurse into a CVSADM
|
|
directory.
|
|
* sanity.sh (basicb): New test basicb-4a tests for above fix.
|
|
|
|
Sun Mar 16 10:18:28 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* sanity.sh (death2): Replace regexp matching temporary file name
|
|
with new variable ${tempname}. For most of the tests this is a
|
|
cosmetic change, but death2-diff-6 had been missing _ which caused
|
|
it to fail on Solaris (at least sometimes).
|
|
|
|
* sanity.sh (modes): Don't use export -n; it doesn't seem
|
|
to be sufficiently portable.
|
|
|
|
* version.c: Change version number to 1.9.5.
|
|
|
|
* version.c: Version 1.9.4.
|
|
|
|
* rcscmds.c (RCS_checkin): Preserve the mode of the rcsfile.
|
|
RCS_CI usually, but not always, does this for us.
|
|
* commit.c (fix_rcs_modes): Replace algorithm with a more
|
|
CVSUMASK-friendly one.
|
|
* sanity.sh (modes): Update tests modes-5, modes-7, modes-10, and
|
|
modes-15 so they test that CVSUMASK is honored.
|
|
|
|
Sun Mar 16 10:18:28 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* sanity.sh (modes): New tests modes-7a and modes-7b test behavior
|
|
if one manually changes the modes in the repository.
|
|
|
|
* server.c (server): Revise code which checks for errors creating
|
|
temporary directory. This won't solve the intermittent
|
|
can't create temporary directory
|
|
Unknown error -1
|
|
but it will mean (a) the right message based on errno gets
|
|
printed, instead of "unknown error -1", and (b) the message says
|
|
that it happened in chmod instead of mkdir_p.
|
|
|
|
Sat Mar 15 16:47:12 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* sanity.sh (modes): New tests. Note that (for now) these are
|
|
just testing how CVS already behaves; I want to record that before
|
|
I move on to changing CVS's behavior with modes of RCS files.
|
|
|
|
13 Mar 1997 Jim Kingdon
|
|
|
|
* subr.c (line2argv): Change argv_allocated from size_t to int.
|
|
|
|
Wed Mar 12 22:16:44 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* add.c (add_directory): If repository has an extraneous "."
|
|
directory at the end, strip it off. This fixes a bug which was
|
|
introduced when strip_path was nuked (this fix is much more
|
|
limited in scope than strip_path was; I _think_ that is a good
|
|
thing).
|
|
(add): Likewise, for client.
|
|
(combine_dir): New function, helps with above.
|
|
* sanity.sh (modules3): Reenable tests for this behavior.
|
|
(basica-0b, basicb-0e): Adjust test to reflect "foo/bar" instead
|
|
of "foo/./bar" in message. As with the rest of this, I believe
|
|
this is just restoring the behavior prior to the strip_path nuking
|
|
(I tried it with CVS 1.9).
|
|
|
|
Sun Mar 9 10:06:29 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* root.c (parse_cvsroot), server.c (serve_root, serve_init):
|
|
If CVSroot_directory is not an absolute pathname, give a fatal error.
|
|
* sanity.sh (crerepos): New tests crerepos-6* test for above fixes.
|
|
|
|
Sat Mar 8 22:06:17 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
This cleans up the last known code which can overflow its buffers
|
|
(except the Mac client). I've skimmed through much of CVS looking
|
|
for other such places; but I didn't read everything. If I missed
|
|
any please report it to bug-cvs.
|
|
* logmsg.c (logfile_write, title_proc): Realloc str_list as
|
|
needed; don't assume MAXLISTLEN is enough.
|
|
* cvs.h (MAXLISTLEN, MAXFILEPERDIR): Removed; no longer used.
|
|
* add.c, myndbm.c, parseinfo.c, update.c: Nuke MAXLINELEN limit.
|
|
* parseinfo.c, update.c, mkmodules.c: Check for errors reading file.
|
|
* cvs.h (MAXLINELEN): Removed; no longer used.
|
|
* logmsg (MAXHOSTNAMELEN): Removed; not used.
|
|
* main.c (cmd_synonyms): Allocate based on fullname, nick1, and
|
|
nick2, just in case someone makes those big enough so that 100
|
|
bytes is not enough.
|
|
(Make_Date): Use MAXDATELEN rather than our own fixed size.
|
|
* mkmodules.c (mkmodules): Nuke arbitrary limit on line length.
|
|
* rcs.c (ALLOCINCR): Remove; not used.
|
|
(RCS_check_kflag): Add comment concerning karg size.
|
|
* run.c: Allocate run_prog to the needed size, rather than
|
|
allocating a fixed size buffer.
|
|
|
|
Fri Mar 7 22:39:08 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* logmsg.c (logfile_write): Allocate prog to needed length rather
|
|
than assuming MAXPROGLEN is enough.
|
|
* cvs.h (MAXPROGLEN): Removed; no longer used.
|
|
* subr.c (MIN_INCR): Update comment to reflect MAXPROGLEN's demise.
|
|
|
|
* subr.c (free_names): Fix comment: this function is not used to
|
|
free memory allocated by Find_Names (at least it hasn't for a long
|
|
time).
|
|
* subr.c, cvs.h (line2argv): Change calling convention so that we
|
|
allocate argv array rather than the caller. The previous one had
|
|
no way of checking whether we overflowed the passed-in buffer.
|
|
* subr.c (free_names): Free the argv array too.
|
|
* modules.c (do_module, cat_module): Update callers.
|
|
|
|
Thu Mar 6 12:44:42 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* import.c: Allocate vhead and vbranch dynamically; removes
|
|
arbitrary limit.
|
|
* history.c: Likewise (since_rev, since_tag, backto, rec_types).
|
|
* ignore.c: Likewise (line). Also check for errors from getline
|
|
and add 'copyright' notice to top of file.
|
|
* wrapper.c (wrap_add_file): Likewise (line). Also check for
|
|
errors from various calls and add 'copyright' notice to top of file.
|
|
|
|
Tue Mar 4 17:39:15 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* client.c (update_entries): Add comment about "move away <file>"
|
|
message.
|
|
|
|
Mon Mar 3 21:51:40 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* sanity.sh (basicb): Clean up topfile,v at end of test. Fixes
|
|
failure in modules-155b.
|
|
|
|
Sun Mar 2 18:11:09 1997 Dan Wilder <dan@gasboy.com>
|
|
and Jim Kingdon
|
|
|
|
* admin.c (admin): Arrange to perform recursion if "cvs admin"
|
|
is passed only options.
|
|
|
|
Sun Mar 2 18:11:09 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* sanity.sh (basicb): New tests basicb-0* test for files at top
|
|
level.
|
|
|
|
* error.c (error): Add newline to "out of memory" message. I think
|
|
that its omission probably could cause the message to be lost in
|
|
the bowels of server.c and never passed to the user.
|
|
|
|
* client.c (start_rsh_server): Add comment about "remsh" vs. "rsh".
|
|
|
|
* cvs.h: Move copyright notice to top of file.
|
|
|
|
Sun Mar 2 13:44:36 1997 Ian Lance Taylor <ian@cygnus.com>
|
|
|
|
* sanity.sh: Use -n when testing whether rsh works.
|
|
|
|
* server.c (serve_root): Free path.
|
|
|
|
Sun Mar 2 13:12:46 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
The following are things that I noticed in the process of trying
|
|
to track down:
|
|
can't create temporary directory
|
|
Unknown error -1
|
|
FAIL: test 28
|
|
from nightly testing. I'm not sure that either item explains that
|
|
message however.
|
|
* server.c (server): Allocate pending_error_text;
|
|
print_pending_error will try to free it so
|
|
pending_error_text = "foo"
|
|
won't work.
|
|
(mkdir_p): Don't assume that isdir will leave errno unmolested.
|
|
|
|
Thu Feb 27 15:29:58 1997 Ian Lance Taylor <ian@cygnus.com>
|
|
|
|
* remove.c (cvsremove): When forcing removal in client mode, use
|
|
start_recursion rather than calling CVS_UNLINK on each argument.
|
|
(remove_force_fileproc): New static function.
|
|
* sanity.sh (deep): Add tests deep-rm7 through deep-rm10 for above
|
|
patch.
|
|
|
|
* sanity.sh (death): Enable death-76a0 and death-76a1 tests for
|
|
remote, since they now work.
|
|
|
|
Wed Feb 26 16:13:26 1997 Ian Lance Taylor <ian@cygnus.com>
|
|
|
|
* client.c (add_prune_candidate): Skip adding this directory if
|
|
it is the same as the first directory already on the list.
|
|
|
|
Mon Feb 24 21:36:43 1997 Noel Cragg <noel@gargle.rain.org>
|
|
|
|
* main.c (lookup_command_attribute): Add the "init" command to the
|
|
list of commands that don't use the current working directory.
|
|
|
|
Sun Feb 23 09:54:49 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* sanity.sh (devcom3): Clean up at end of test.
|
|
|
|
* sanity.sh (basicb): Add commented out test basicb-8a0, for
|
|
whether CVS can print an error on bad numeric revision to diff.
|
|
Commented out until we get around to fixing CVS.
|
|
* diff.c (diff_file_nodiff): Add comment about this case.
|
|
|
|
* fileattr.c (fileattr_read): If a filename is duplicated,
|
|
continue to ignore subsequent lines but free the node so that we
|
|
don't leak memory.
|
|
* sanity.sh (devcom3): New tests devcom3-8 and devcom3-9 test for
|
|
behavior on duplicated filenames.
|
|
|
|
* fileattr.h: Add comment about unrecognized ENT-TYPE and order of
|
|
lines in fileattr file.
|
|
* fileattr.c (struct unrecog, unrecog_head): New variables, to
|
|
record unrecognized lines.
|
|
(fileattr_startdir): Assert that unrecog_head == NULL.
|
|
(fileattr_read): Record unrecognized lines in unrecog_head linked
|
|
list rather than ignoring them.
|
|
(fileattr_write): Also write out unrecognized lines, if any.
|
|
* sanity.sh (devcom3): New tests, test for above fix.
|
|
|
|
* fileattr.h (fileattr_modify): Fix example in comment.
|
|
|
|
Sat Feb 22 08:30:27 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* sanity.sh: Add variable username.
|
|
(basica rdiff multibranch log log2): Use it instead of our own
|
|
(inconsistent) ways of matching an author name.
|
|
|
|
* filesubr.c, root.c, rtag.c, server.c, subr.c, update.c,
|
|
wrapper.c: Nuke PATH_MAX.
|
|
* cvs.h, wrapper.c (wrap_fromcvs_process_file): Now returns void
|
|
(return value had been unused).
|
|
* cvs.h: Adjust comment to reflect the fact that PATH_MAX is
|
|
gone, at least from src/*.c (except safe_location, as noted).
|
|
|
|
22 Feb 1997 patch by Tom Hageman <tom@basil.icce.rug.nl> (4 Jun 1996)
|
|
updated and commented by Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* update.c (checkout_file): Call unlink_file_dir on backup, not
|
|
unlink_file.
|
|
|
|
Fri Feb 21 16:40:03 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* Makefile.in (DISTFILES): Remove NOTES.
|
|
|
|
* NOTES: Removed. bcopy->memcpy is done. "static buffers" I
|
|
assume refers to what is covered by reentrancy text in HACKING.
|
|
Obstack idea moved to comment in hash.c (at nodecache). Checking
|
|
system calls for error returns largely done, and isn't a very
|
|
helpful suggestion unless you know where the bogus calls are
|
|
anyway. Sizing limits--we're in the progress of removing them
|
|
(assuming it meant things like PATH_MAX and earlier, already
|
|
nuked, limits). Removed various items about changes which were
|
|
done a long time ago (I realize that the ChangeLog's probably
|
|
aren't reliable that far back, but I'm not convinced anyone cares
|
|
anymore). CONFIRM_DIRECTORY_ADDS: I assume this is a
|
|
reference to the #if 0'd code in add_directory which asks for
|
|
confirmation--a better way of making it harder to accidentally add
|
|
directories would be to have to add and commit directories like
|
|
for files. I don't know what FORCE_MESSAGE_ON_ADD meant.
|
|
|
|
* rcs.c (RCS_getrevtime): Fix documentation (in particular, the
|
|
size of the array that DATE must point to, but many other things
|
|
too).
|
|
* patch.c, recurse.c, release.c, remove.c, repos.c: Nuke PATH_MAX.
|
|
(patch_fileproc): Use MAXDATELEN not hardcoded 50.
|
|
|
|
Sun Feb 16 12:00:44 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* client.c (client_process_import_file): New variable fullname;
|
|
pass it to send_modified. This finishes the job of untangling the
|
|
old short_pathname variable into update_dir vs. fullname.
|
|
|
|
* client.c (client_process_import_file): Nuke first_time. If
|
|
toplevel_repos were ever NULL here, the code would dump core in
|
|
strncmp a few lines down. And client_import_setup ensures
|
|
toplevel_repos is not NULL.
|
|
|
|
Sun Feb 16 08:16:48 1997 Ian Lance Taylor <ian@cygnus.com>
|
|
|
|
* client.c (client_process_import_file): Rename short_pathname to
|
|
update_dir (to reflect its function) and make sure that it doesn't
|
|
point to uninitialized memory if repository and toplevel_repos
|
|
contain the same string.
|
|
|
|
Sun Feb 16 08:16:48 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* client.c (start_rsh_server): Nuke comment about weirdnesses with
|
|
pre-1.5 versions of CVS and .bashrc/.cshrc. The remote protocol
|
|
is interoperable only back to 1.5, and people who need to know are
|
|
unlikely to see this comment anyway.
|
|
|
|
Sun Dec 15 13:12:30 1996 Michael Douglass <mikedoug@texas.net>
|
|
and Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* main.c (cmds): Added an entry for new logout command.
|
|
(cmd_usage): Added an entry for new logout command.
|
|
(lookup_command_attribute): Added 'logout' to list of commands
|
|
that set need_to_crate_root to 1.
|
|
* login.c, cvs.h (logout): New command for removing entries from
|
|
the .cvspass file.
|
|
(logout_usage): Usage information on the logout command.
|
|
|
|
Wed Feb 12 11:19:42 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* client.c (struct send_data): Fix indentation.
|
|
|
|
Wed Feb 12 08:48:04 1997 Greg A. Woods <woods@most.weird.com>
|
|
|
|
* mkmodules.c (loginfo_contents): add missing comma in
|
|
initializer statement (caused syntax error on SunOS-4).
|
|
|
|
Tue Feb 11 21:14:28 1997 Ian Lance Taylor <ian@cygnus.com>
|
|
|
|
* commit.c (find_fileproc): If force_ci is set, set the status to
|
|
T_MODIFIED even if the file hasn't changed.
|
|
(commit): Pass force_ci to send_files as new force argument.
|
|
* client.c (struct send_data): Define.
|
|
(send_fileproc): The callerdat parameter now points to a send_data
|
|
struct. If force is set, always call send_modified.
|
|
(send_dirent_proc): The callerdat parameter now points to a
|
|
send_data struct.
|
|
(send_files): Add force parameter. Change all callers. Set up a
|
|
send_data struct and pass it to start_recursion as callerdat.
|
|
* client.h (send_files): Update declaration.
|
|
* sanity.sh (basica): Add a simple test for the above patch.
|
|
|
|
Sun Feb 9 12:58:59 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* tag.c (cvstag), rtag.c (rtag): Pass -f to server if specified in
|
|
the client. I haven't tried to come up with a test case because
|
|
the fix seems obvious.
|
|
|
|
* import.c (add_rcs_file): Change size of altdate1 and altdate2 to
|
|
MAXDATELEN.
|
|
* cvs.h (MAXDATELEN): Fix comments; describe what this is for.
|
|
|
|
* diff.c (diff_usage): Document --ifdef and try to briefly say
|
|
what "rcsdiff-options" means.
|
|
|
|
* update.c (update): If update had a nonzero status and we haven't
|
|
yet tried to fetch unpatchable files, go ahead and try it again.
|
|
The previous behavior was to quit, which meant that updates would
|
|
keep failing until you hacked around the problem. Patch and bug
|
|
report by Ian; comment, ChangeLog entry, and willingness to take
|
|
the flak if checking it is premature by Jim.
|
|
|
|
* server.c (alloc_pending): New function.
|
|
* server.c: Call it. Fixes places where we had neglected to
|
|
check for NULL return from malloc.
|
|
|
|
* sanity.sh (binwrap): Add test binwrap-0, tests for import.c fix
|
|
below.
|
|
|
|
Sun, 9 Feb 1997 (submitted 19 Jul 1996) John Polstra <jdp@polstra.com>
|
|
|
|
* import.c (import): Give error if the same tag is specified more
|
|
than once. The previous behavior was to write an RCS file which
|
|
had the same tag listed twice, once pointing to each revision,
|
|
which is not legal.
|
|
|
|
Sun Feb 9 12:37:09 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* checkin.c (Checkin): Use cvs_output to print message (should
|
|
make out of order bugs no worse, as it merely substitues a
|
|
protocol_pipe vs. stderr_pipe race instead of a stdout_pipe
|
|
vs. stderr_pipe race). Add comment about stdout vs. stderr.
|
|
|
|
Fri Feb 7 08:29:52 1997 Josef Nelissen <josef.nelissen@munich.ixos.de>
|
|
|
|
* server.c (check_command_legal_p): Don't use ANSI-style definition.
|
|
|
|
Thu Feb 6 10:55:37 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* patch.c (patch): Give a fatal error for -V option (see comment
|
|
for rationale).
|
|
|
|
* diff.c (diff): Also send "options" to server. Pretty much the
|
|
patch submitted independently by josef.nelissen@munich.ixos.de and
|
|
Ronald Khoo <ronald@demon.net>.
|
|
|
|
Wed Feb 5 18:57:14 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* modules.c (do_module): Fix typo in 30 Jan 97 PATH_MAX nuking
|
|
(free -> free_cwd). Testsuite test 151 gets credit for catching
|
|
this one.
|
|
|
|
Mon Feb 3 16:14:54 1997 Ian Lance Taylor <ian@cygnus.com>
|
|
|
|
* main.c (lookup_command_attribute): Don't use an ANSI prototype
|
|
when defining the function.
|
|
|
|
Fri Jan 31 12:49:02 1997 Ian Lance Taylor <ian@cygnus.com>
|
|
|
|
* modules.c (do_module): Actually goto found if is_found is set
|
|
(fixes thinko in PATH_MAX nuking of 30 Jan 97).
|
|
|
|
Fri Jan 31 12:49:02 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* sanity.sh: Add modules3 and big to list of tests to run
|
|
by default; they were omitted by accident.
|
|
|
|
Thu Jan 30 11:46:33 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* logmsg.c, main.c, mkmodules.c, modules.c, parseinfo.c, patch.c:
|
|
Nuke more PATH_MAX.
|
|
|
|
* server.c (server_updated): After we send Created or
|
|
Update-existing for a file, mark it as unchanged, in case we
|
|
process it again.
|
|
* sanity.sh (modules3): New tests, test for above fix.
|
|
|
|
* logmsg.c (do_verify): Error return from fopen is NULL, not -1.
|
|
Pass errno to error().
|
|
|
|
* login.c [_CRAY]: Don't declare getpass.
|
|
|
|
Mon Jan 27 17:25:27 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* import.c (process_import_file): Fix freeing of rcs (Don't free
|
|
it before we are done using it, and don't free it twice).
|
|
|
|
* modules.c (cat_module): Allocate line right before we use
|
|
it. The previous code was wrong because the length of the
|
|
s_h->rest changes between the time we allocate line and the time we
|
|
sprintf s_h->rest into it.
|
|
|
|
Sun Jan 26 21:58:16 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* expand_path.c (expand_path): Revise to call expand_string as
|
|
needed. Nuke PATH_MAX.
|
|
* find_names.c (find_dirs): Likewise.
|
|
* import.c, lock.c: Nuke more PATH_MAX.
|
|
|
|
* server.c (mkdir_p): Set retval to 0 at start of function.
|
|
Previously it had been uninitialized for some cases. Thanks are
|
|
due to nightly testing for catching this one.
|
|
|
|
Sat Jan 25 21:34:19 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* subr.c, cvs.h (expand_string): New function.
|
|
* rcs.c (getrcskey, getrcsrev): Call it. This greatly reduces the
|
|
number of calls to realloc if there is a very large file in the
|
|
RCS file. Credit goes to Mike Heath <mike@pswtech.com> for
|
|
pointing out the problem and the basic solution (MIN_INCR,
|
|
MAX_INCR); I adapted it into the separate function expand_string.
|
|
* sanity.sh (big): New test helps insure this hasn't broken
|
|
anything obvious.
|
|
|
|
Wed Jan 22 10:06:13 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* status.c (status_fileproc): Change message which is printed for
|
|
T_MODIFIED and ts_conflict set, so that it doesn't say "unresolved
|
|
confict". This message occurs whether the conflict is resolved or
|
|
not.
|
|
* sanity.sh (conflicts): Add tests conflicts-status-* to test
|
|
output of "cvs status" in the context of conflicts. Tests for
|
|
above fix.
|
|
|
|
* rtag.c (rtag): Send -n if run_module_prog is NOT true.
|
|
|
|
Thu Jan 16 00:06:00 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* version.c: Change version number to 1.9.3.
|
|
|
|
* version.c: Version 1.9.2.
|
|
|
|
Wed Jan 15 09:14:38 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* client.c (call_in_directory): Take code that creates CVSADM at
|
|
top level, move it before the CVS_CHDIR (dir_name) call, and do it
|
|
regardless of whether dir_name is ".". Pass "." not dir_name to
|
|
Create_Admin (when the code was written they were always the
|
|
same). Don't add reposdirname to the repository we pass to
|
|
Create_Admin (when the code was written, I think reposdirname
|
|
probably would always be "."). Don't create CVSADM if
|
|
reposdirname_absolute.
|
|
* sanity.sh (basicb): Enable tests basicb-1a and basicb-9a for
|
|
remote; tests for above fix.
|
|
(basic1): Do entire test within a "1" directory to deal with
|
|
creation of CVS directories at top level. Support --keep.
|
|
(conflicts): In test conflicts-136, only update first-dir.
|
|
(basica): Uncomment the part that tests "cvs co -l .". That tests
|
|
the existing functionality which I might have (but hopefully did not)
|
|
perturbed with the call_in_directory changes.
|
|
|
|
Mon Jan 13 11:04:32 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* server.c (check_command_legal_p): Do not call error (1, ...)
|
|
here; that will always cause a protocol violation by shutting down
|
|
the connection prematurely. Remove croak_on_illegal arg.
|
|
(do_cvs_command): Move call to check_command_legal_p until after
|
|
the call to print_pending_error. Print the error message ourself.
|
|
|
|
* mkmodules.c (filelist): Add readers and writers. Add comment
|
|
about why passwd is not included. Add comment about meaning of
|
|
NULL contents field.
|
|
|
|
Fri Jan 10 13:23:09 1997 Norbert Kiesel <nk@col.sw-ley.de>
|
|
|
|
* release.c (release): Initialize delete_flag before reading it
|
|
(found by running purify)
|
|
|
|
* logmsg.c (do_verify): Fix reading unallocated memory (found by
|
|
running purify)
|
|
|
|
Thu Jan 9 16:32:47 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* checkout.c (build_dirs_and_chdir): Partially revert 3 Jan
|
|
change--move call to Subdir_Register back above the CVS_CHDIR call
|
|
(we need to register in the old, not the new, directory). Instead
|
|
of calling CVS_MKDIR and ignoring errors, call mkdir_if_needed;
|
|
this is an effort to catch errors there rather than catching them
|
|
in the CVS_CHDIR. This makes test 27-add-add in sanity.sh work
|
|
again.
|
|
|
|
* find_names.c (Find_Directories): Remove code inside
|
|
#ifdef ATTIC_DIR_SUPPORT and replace it with a comment explaining
|
|
why we don't look in the attic. ATTIC_DIR_SUPPORT was never defined.
|
|
|
|
* find_names.c (find_dirs): Add comment about tmp being unset.
|
|
|
|
* commit.c (checkaddfile): Report errors with errno and specific
|
|
error messages.
|
|
|
|
* rcs.c, commit.c, create_adm.c, entries.c, find_names.c,
|
|
ignore.c, history.c: Nuke PATH_MAX arbitrary limits.
|
|
|
|
Wed Jan 8 23:07:41 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* add.c (add): Reindent a portion which needed it.
|
|
|
|
1997-01-08 Jim Kingdon
|
|
|
|
* client.c (send_file_names): When looking name up in Entries,
|
|
call Entries_Open and Entries_Close. This has two effects:
|
|
(1) we look at Entries.Log, and (2) we don't skip 'D' entries,
|
|
both of which are needed to make us get the right (command
|
|
line) name for a directory we are adding.
|
|
|
|
Wed Jan 8 14:50:47 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* Makefile.in, cvs.h, hash.h, myndbm.h, rcs.h: Remove CVSid; we
|
|
decided to get rid of these some time ago.
|
|
|
|
Tue Jan 7 12:56:10 1997 Karl Fogel <kfogel@ynu38.ynu.edu.cn>
|
|
|
|
* root.c (check_root_consistent): new func, compares below new
|
|
global var with CVSroot_directory, assuming both set.
|
|
(set_local_cvsroot): use above new func for security check.
|
|
(parse_cvsroot): same.
|
|
But do all of above only #ifdef AUTH_SERVER_SUPPORT.
|
|
|
|
* server.c: (Pserver_Repos): new global var, init to NULL.
|
|
(pserver_authenticate_connection): set above new global.
|
|
(check_repository_password): be a good scout and use
|
|
CVSROOTADM and CVSROOTADM_PASSWD, now that they are the standard.
|
|
Make sure all of above is in #ifdef AUTH_SERVER_SUPPORT.
|
|
(check_command_legal_p): wrap most of body in #ifdef
|
|
AUTH_SERVER_SUPPORT.
|
|
Everywhere: wrap all references to CVS_Username in #ifdef
|
|
AUTH_SERVER_SUPPORT.
|
|
|
|
* cvs.h (Pserver_Repos): new var, used in consistency [security]
|
|
check. Defined only #ifdef AUTH_SERVER_SUPPORT.
|
|
(CVSROOTADM_PASSWD): new #define, trying to get with the program.
|
|
|
|
Fri Jan 3 18:10:39 1997 Ian Lance Taylor <ian@cygnus.com>
|
|
|
|
* checkout.c (build_dirs_and_chdir): Move call to Subdir_Register
|
|
until after we know that the directory exists.
|
|
|
|
Thu Jan 2 13:30:56 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* Makefile.in, cvsbug.sh, edit.c, edit.h, error.c, error.h,
|
|
fileattr.c, fileattr.h, filesubr.c, run.c, update.h, watch.c,
|
|
watch.h: Remove "675" paragraph; see ../ChangeLog for rationale.
|
|
|
|
Thu Jan 2 12:27:46 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* sanity.sh (info): New test info-cleanup-verifymsg gets rid of
|
|
verifymsg when we are done with it.
|
|
|
|
* sanity.sh (basicb): Skip tests basicb-1a and basicb-9a for remote.
|
|
* sanity.sh (modules-155a4): It is OK if a CVS directory exists.
|
|
* sanity.sh (ignore): Do everything inside a "1" directory. The
|
|
change to create CVS directories at top-level causes messages such as
|
|
"? home" otherwise. In test 191, specify -I CVS so that new CVS
|
|
directory is ignored.
|
|
* sanity.sh (crerepos): Manually remove CVS directory which had not
|
|
existed before.
|
|
|
|
Thu Jan 2 09:06:20 1997 Karl Fogel <kfogel@ynu38.ynu.edu.cn>
|
|
|
|
* server.c: Changes for pserver read-only repository access:
|
|
(check_command_legal_p): new func. Right now, just checks if
|
|
repository modification is permitted, and that only if pserver is
|
|
active.
|
|
(do_cvs_command): take new parameter `cmd_name', a string. All
|
|
callers changed. Pass it on to new func check_command_legal_p()
|
|
before executing the command.
|
|
(CVS_Username): new global, init to NULL. Used by
|
|
check_command_legal_p(), set in check_password().
|
|
(check_password): set above new global CVS_Username; reorganized a
|
|
bit to facilitate this.
|
|
(check_repository_password): give *host_user_ptr permanent
|
|
storage iff success.
|
|
|
|
* main.c: Changes for pserver read-only repository access:
|
|
(lookup_command_attribute): new func.
|
|
(main): use new func lookup_command_attribute() to establish if
|
|
CVS_CMD_IGNORE_ADMROOT and CVS_CMD_USES_WORK_DIR.
|
|
|
|
* cvs.h: Changes for pserver read-only repository access:
|
|
(CVSROOTADM_READERS, CVSROOTADM_WRITERS): new #defines.
|
|
Prototype lookup_command_attribute().
|
|
(CVS_CMD_IGNORE_ADMROOT, CVS_CMD_USES_WORK_DIR,
|
|
CVS_CMD_MODIFIES_REPOSITORY): new #defines for
|
|
lookup_command_attribute() and its callers.
|
|
|
|
Wed Jan 1 19:50:38 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* options.h.in: Reword comment for TMPDIR_DFLT to make it clear
|
|
that this isn't specific to the pserver server.
|
|
|
|
* modules.c (do_module): Give an error message if one tries to
|
|
specify -a together with another option.
|
|
* sanity.sh (modules2): New tests modules2-a* test for above fix.
|
|
|
|
* sanity.sh (devcom): Add tests devcom-a-nonexist and
|
|
devcom-t-nonexist for "cvs watchers" on nonexistent argument.
|
|
|
|
1997-01-01 Fred Fish <fnf@ninemoons.com>
|
|
|
|
* run.c (piped_child, filter_stream_through_program): Actually
|
|
install these HAVE_VFORK patches that got missed.
|
|
(There was a log entry for these changes for 29 Nov 1996 but it
|
|
seems I accidentally forgot to actually check them in -kingdon).
|
|
|
|
Wed Jan 1 18:32:44 1997 Jim Kingdon <kingdon@harvey.cyclic.com>
|
|
|
|
* Makefile.in: Add ChangeLog-96.
|
|
* ChangeLog-96: New file, contains former contents of ChangeLog.
|
|
* ChangeLog: Now just contains 1997 changes.
|
|
|
|
|
|
For older changes see ChangeLog-96.
|