Chris Foote <>

Patch Suggested
r22932, r20927, r18874, r16803, r16708, r7320, r7245, r6443, r6324, r4577 r24289, r9596

r24289 | djh | 2007-03-30 21:46:58 -0500 (Fri, 30 Mar 2007)

Replace os.rmtree with the safe_rmtree implementation from the test
suite so that we can remove read-only files on Windows.

Suggested by: Chris Foote <>
              Jens Peters <>

* tools/backup/
  (chmod_tree): New method for setting permissions on a tree.
  (safe_rmtree): New method for dropping possibly read-only directory trees.

r22932 | lgo | 2007-01-08 13:55:26 -0600 (Mon, 08 Jan 2007)

Ensure that the repos path is passed to the hook scripts in local style.

Patch by: Chris Foote <>

* subversion/mod_dav_svn/mod_dav_svn.c
  (dav_svn_path_cmd, dav_svn_parent_path_cmd): Convert the repos path to
    internal style.

* subversion/libsvn_repos/hooks.c
  (svn_repos__hooks_start_commit, svn_repos__hooks_pre_commit,
  svn_repos__hooks_post_commit, svn_repos__hooks_pre_revprop_change,
  svn_repos__hooks_post_revprop_change, svn_repos__hooks_pre_lock,
  svn_repos__hooks_post_lock, svn_repos__hooks_pre_unlock,
  svn_repos__hooks_post_unlock): Convert the repos path to local style.

r20927 | dlr | 2006-08-01 14:48:55 -0500 (Tue, 01 Aug 2006)

Show locking notifications using local path style.

* subversion/svn/notify.c
  (notify): For svn_wc_notify_locked and svn_wc_notify_unlocked, print
   out path_local in the message.

Patch by: Chris Foote <>

r18874 | dlr | 2006-03-13 13:41:28 -0600 (Mon, 13 Mar 2006)

Add option to archive the backup using gz, bz2 or zip to

* tools/backup/
  Update copyright year, and add imports.

  (archive_map): New dictionary mapping archive types to file

  (usage): New. Print out a usage message.

  Redo command line processing using getopt and add --archive-type
  option to specify the archive type.

  (comparator): If archiving, add the archive extenstion to the regexp
   to find archives as well.

  Step 2, also find archives.
  Step 3, print the error message to stderr.
  Step 4, archive backup.
  Step 5, look for archives to remove as well.

Patch by: Chris Foote <>
(Tweaked by me.)
Review by: me

r16803 | rooneg | 2005-10-18 16:49:03 -0500 (Tue, 18 Oct 2005)

Fix neon warning messages.

Patch by: Chris Foote <>

* build/generator/
  (_find_neon): add newlines to messages that lacked them.

r16708 | djames | 2005-10-14 07:21:01 -0500 (Fri, 14 Oct 2005)

Add a function to duplicate the svn_info_t structure.

* subversion/include/svn_client.h,
  (svn_info_dup): New function to duplicate an svn_info_t.

Patch by: Chris Foote <>
(davidjames made a few minor tweaks)

r9596 | jpieper | 2004-05-01 11:25:43 -0500 (Sat, 01 May 2004)

Fix two compiler warnings.

Suggested by: Chris Foote <>

* subversion/libsvn_fs_fs/fs_fs.c
  (hash_read): Remove buf as it is not used in the stream version of

* subversion/libsvn_client/blame.c
  (log_message_receiver): Remove the now unused variable change.

r7320 | cmpilato | 2003-10-06 17:47:55 -0500 (Mon, 06 Oct 2003)

* doc/book/book/ch05.xml:
  Fix some sample usages of the 'svnadmin dump' commandline.  

Patch by: Chris Foote <>
(Tweaked by me.)

r7245 | rey4 | 2003-09-29 15:43:59 -0500 (Mon, 29 Sep 2003)

Add a windows specific option to gen-make so that the user can specify
a path to the Berkely DB headers and libs.

Patch by: Chris Foote <>

Review by: Branko ─îibej <>
(Pointed out problem using new option on unix.)

  (_usage_exit): Add --with-berkely-db option.
  (__main__): ditto.

* build/generator/
  (parse_options): Add --with-berkely-db option.
  (__init__): Search for libdb4? in the specified dir and then
  the default dirs.
  (map_rootpath): If the path is absolute then use it, else
  prepend the root path.

r6443 | philip | 2003-07-10 17:15:51 -0500 (Thu, 10 Jul 2003)

Fix an 'svn cat' bug where keyword expansion substituted values from
the wrong revision.

Patch by: Chris Foote <>
(Regression test by me.)

* subversion/libsvn_client/cat.c (svn_client_cat): Expand keywords
  to the values of the committed revision.

* subversion/tests/clients/cmdline/
  (cat_keyword_expand): New test.

r6324 | sussman | 2003-06-23 20:06:50 -0500 (Mon, 23 Jun 2003)

Fix keyword expansion buglet.

Patch by: Chris Foote <>

* subversion/libsvn_wc/translate.c
  (svn_wc__get_keywords): Use the entry->cmt_rev, not entry->revision.

r4577 | kevin | 2003-01-24 17:21:42 -0600 (Fri, 24 Jan 2003)

Run hook programs on WIN32. This doesn't quite work right on my XP system,
since I don't actually get the output of a failed pre-commit script.  Instaed I
see "The handle is invalid." Several times.  I thought this was better than the
status quo however, so I'm committing it.

* subversion/libsvn_repos/hooks.c:
  (check_hook_cmd): New function to check for the hook program.
  (svn_repos__hooks_start_commit, svn_repos__hooks_pre_commit,
   svn_repos__hooks_post_commit, svn_repos__hooks_pre_revprop_change,
   svn_repos__hooks_post_revprop_change) : use check_hook_cmd.

Patch by: Chris Foote <>
(Tweaked by me to remove the [admittedly already present] memory leak
if svn_io_check_resolved_path fails.)