On Thu, 20 Oct 2005, Mark Phippard wrote:
...
> Does this new method take a path, like "C:\Dev\Project\.svn" or just a
> name like ".svn"?
I had the same question myself while implementing this, and modified the
svn_wc_is_adm_dir() doc string accordingly:
--- svn_wc.h (revision 16862)
+++ svn_wc.h (revision 16863)
@@ -339,7 +339,8 @@
/**
* Return @c TRUE if @a name is the name of the WC administrative
- * directory. Use @a pool for any temporary allocations.
+ * directory. Use @a pool for any temporary allocations. Only works
+ * with base directory names, not paths or URIs.
*
* For compatibility, the default name (.svn) will always be treated
* as an admin dir name, even if the working copy is actually using an
> In the cases we need this information in Subclipse, I think the other
> method will be more desirable.
I assume you mean that the ability to digest a full path would be more
desirable? (This was my thinking as well.) As the JavaHL binding I
added simply wraps svn_wc_is_adm_dir(), a change to that library would
do the trick here, and sounds like a generally more comfortable API.
Maybe something like...?:
--- adm_files.c (revision 16865)
+++ adm_files.c (working copy)
@@ -57,7 +57,7 @@
svn_boolean_t
svn_wc_is_adm_dir (const char *name, apr_pool_t *pool)
{
- (void)pool; /* Silence compiler warnings about unused parameter. */
+ name = svn_path_basename (name, pool);
return (0 == strcmp (name, adm_dir_name)
|| 0 == strcmp (name, default_adm_dir_name));
}
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Thu Oct 20 22:30:36 2005