Author: dannas
Date: Fri Mar 12 08:21:45 2010
New Revision: 922176
URL: http://svn.apache.org/viewvc?rev=922176&view=rev
Log:
As part of WC-NG, remove some uses of svn_wc_entry_t.
* subversion/include/svn_wc.h
(svn_wc_revision_status2): Add note about us returning
SVN_ERR_WC_PATH_NOT_FOUND.
* subversion/libsvn_wc/revision_status.c
(status_baton): Rename this...
(walk_baton): .. to this.
(analyze_status): Change this to implement svn_wc__node_func_t and use
WC-NG funcs instead of information in svn_wc_entry_t.
(svn_wc_revision_status2): Use svn_wc__node_walk_children() instead of
svn_wc_walk_status() to spare us from the overhead of invoking an
editor.
* subversion/svnversion/main.c
(main): Determine if a path is unversioned by checking for
SVN_ERR_WC_PATH_NOT_FOUND.
Approved by: gstein
Modified:
subversion/trunk/subversion/include/svn_wc.h
subversion/trunk/subversion/libsvn_wc/revision_status.c
subversion/trunk/subversion/svnversion/main.c
Modified: subversion/trunk/subversion/include/svn_wc.h
URL:
http://svn.apache.org/viewvc/subversion/trunk/subversion/include/svn_wc.h?rev=922176&r1=922175&r2=922176&view=diff
==============================================================================
--- subversion/trunk/subversion/include/svn_wc.h (original)
+++ subversion/trunk/subversion/include/svn_wc.h Fri Mar 12 08:21:45 2010
@@ -7192,7 +7192,8 @@ typedef struct svn_wc_revision_status_t
/** Set @a *result_p to point to a new #svn_wc_revision_status_t structure
* containing a summary of the revision range and status of the working copy
* at @a local_abspath (not including "externals"). @a local_abspath must
- * be absolute.
+ * be absolute. Return SVN_ERR_WC_PATH_NOT_FOUND if @a local_abspath is not
+ * a working copy path.
*
* Set @a (*result_p)->min_rev and @a (*result_p)->max_rev respectively to the
* lowest and highest revision numbers in the working copy. If @a committed
Modified: subversion/trunk/subversion/libsvn_wc/revision_status.c
URL:
http://svn.apache.org/viewvc/subversion/trunk/subversion/libsvn_wc/revision_status.c?rev=922176&r1=922175&r2=922176&view=diff
==============================================================================
--- subversion/trunk/subversion/libsvn_wc/revision_status.c (original)
+++ subversion/trunk/subversion/libsvn_wc/revision_status.c Fri Mar 12 08:21:45
2010
@@ -23,65 +23,100 @@
#include "svn_wc.h"
#include "svn_dirent_uri.h"
+#include "wc_db.h"
+#include "wc.h"
+#include "props.h"
#include "private/svn_wc_private.h"
#include "svn_private_config.h"
/* A baton for analyze_status(). */
-struct status_baton
+struct walk_baton
{
svn_wc_revision_status_t *result; /* where to put the result */
svn_boolean_t committed; /* examine last committed revisions */
const char *local_abspath; /* path whose URL we're looking for */
- const char *wc_url; /* URL for the path whose URL we're looking for */
- apr_pool_t *pool; /* pool in which to store alloc-needy things */
+ svn_wc__db_t *db;
};
-/* An svn_wc_status_func4_t callback function for analyzing status
- structures. */
+/* An svn_wc__node_found_func_t callback function for analyzing the status
+ * of nodes */
static svn_error_t *
-analyze_status(void *baton,
- const char *local_abspath,
- const svn_wc_status2_t *status,
- apr_pool_t *pool)
+analyze_status(const char *local_abspath,
+ void *baton,
+ apr_pool_t *scratch_pool)
{
- struct status_baton *sb = baton;
+ struct walk_baton *wb = baton;
+ svn_revnum_t changed_rev;
+ svn_revnum_t revision;
+ svn_depth_t depth;
+ svn_wc__db_status_t status;
+ svn_boolean_t wc_root;
+ svn_boolean_t switched;
+
+ SVN_ERR(svn_wc__db_read_info(&status, NULL, &revision, NULL,
+ NULL, NULL, &changed_rev,
+ NULL, NULL, NULL, &depth, NULL, NULL, NULL,
+ NULL, NULL, NULL, NULL, NULL, NULL,
+ NULL, NULL, NULL, NULL, wb->db,
+ local_abspath, scratch_pool, scratch_pool));
+
+ /* We need the excluded and absent paths when checking for sparse
+ * checkouts. But only for that. To collect those we're walking all the
+ * hidden nodes. */
+ if (status == svn_wc__db_status_excluded
+ || status == svn_wc__db_status_absent)
+ {
+ wb->result->sparse_checkout = TRUE;
+ return SVN_NO_ERROR;
+ }
- if (! status->entry)
+ if (status == svn_wc__db_status_not_present)
return SVN_NO_ERROR;
- /* Added files have a revision of no interest */
- if (status->text_status != svn_wc_status_added)
+ if (! wb->result->switched)
{
- svn_revnum_t item_rev = (sb->committed
- ? status->entry->cmt_rev
- : status->entry->revision);
-
- if (sb->result->min_rev == SVN_INVALID_REVNUM
- || item_rev < sb->result->min_rev)
- sb->result->min_rev = item_rev;
-
- if (sb->result->max_rev == SVN_INVALID_REVNUM
- || item_rev > sb->result->max_rev)
- sb->result->max_rev = item_rev;
+ SVN_ERR(svn_wc__check_wc_root(&wc_root, NULL, &switched, wb->db,
+ local_abspath, scratch_pool));
+
+ wb->result->switched |= switched;
}
- if (status->entry->depth != svn_depth_exclude)
+ /* Added files have a revision of no interest */
+ if (revision != SVN_INVALID_REVNUM)
{
- sb->result->switched |= status->switched;
- sb->result->modified |= (status->text_status != svn_wc_status_normal);
- sb->result->modified |= (status->prop_status != svn_wc_status_normal
- && status->prop_status != svn_wc_status_none);
+ svn_revnum_t item_rev = (wb->committed
+ ? changed_rev
+ : revision);
+
+ if (wb->result->min_rev == SVN_INVALID_REVNUM
+ || item_rev < wb->result->min_rev)
+ wb->result->min_rev = item_rev;
+
+ if (wb->result->max_rev == SVN_INVALID_REVNUM
+ || item_rev > wb->result->max_rev)
+ wb->result->max_rev = item_rev;
}
- sb->result->sparse_checkout |= (status->entry->depth != svn_depth_infinity);
- if (sb->local_abspath
- && (! sb->wc_url)
- && (strcmp(local_abspath, sb->local_abspath) == 0)
- && (status->entry))
- sb->wc_url = apr_pstrdup(sb->pool, status->entry->url);
+ if (! wb->result->modified)
+ {
+ svn_boolean_t text_mod;
+ svn_boolean_t props_mod;
+ SVN_ERR(svn_wc__props_modified(&props_mod, wb->db, local_abspath,
+ scratch_pool));
+
+ SVN_ERR(svn_wc__internal_text_modified_p(&text_mod, wb->db,
+ local_abspath,
+ FALSE,
+ TRUE,
+ scratch_pool));
+ wb->result->modified |= (text_mod || props_mod);
+ }
+
+ wb->result->sparse_checkout |= ((depth != svn_depth_infinity
+ && depth != svn_depth_unknown));
return SVN_NO_ERROR;
}
@@ -96,7 +131,8 @@ svn_wc_revision_status2(svn_wc_revision_
apr_pool_t *result_pool,
apr_pool_t *scratch_pool)
{
- struct status_baton sb;
+ struct walk_baton wb;
+ const char *url;
svn_wc_revision_status_t *result = apr_palloc(result_pool, sizeof(*result));
*result_p = result;
@@ -110,50 +146,39 @@ svn_wc_revision_status2(svn_wc_revision_
result->sparse_checkout = FALSE;
/* initialize walking baton */
- sb.result = result;
- sb.committed = committed;
- sb.local_abspath = local_abspath;
- sb.wc_url = NULL;
- sb.pool = scratch_pool;
-
- SVN_ERR(svn_wc_walk_status(wc_ctx,
- local_abspath,
- svn_depth_infinity,
- TRUE /* get_all */,
- FALSE /* no_ignore */,
- TRUE, /* get_excluded */
- NULL /* ignore_patterns */,
- analyze_status, &sb,
- NULL, NULL,
- cancel_func, cancel_baton,
- scratch_pool));
+ wb.result = result;
+ wb.committed = committed;
+ wb.local_abspath = local_abspath;
+ wb.db = wc_ctx->db;
+ SVN_ERR(svn_wc__internal_node_get_url(&url, wc_ctx->db, local_abspath,
+ result_pool, scratch_pool));
- if ((! result->switched) && (trail_url != NULL))
+ if (trail_url != NULL)
{
/* If the trailing part of the URL of the working copy directory
does not match the given trailing URL then the whole working
copy is switched. */
- if (! sb.wc_url)
+ if (! url)
{
result->switched = TRUE;
}
else
{
apr_size_t len1 = strlen(trail_url);
- apr_size_t len2 = strlen(sb.wc_url);
- if ((len1 > len2) || strcmp(sb.wc_url + len2 - len1, trail_url))
+ apr_size_t len2 = strlen(url);
+ if ((len1 > len2) || strcmp(url + len2 - len1, trail_url))
result->switched = TRUE;
}
}
- /* ### 1.6+: If result->sparse_checkout is FALSE the answer is not final
- We can still have excluded nodes!
-
- ### TODO: Check every node below local_abspath for excluded
-
- ### BH: What about absent? You don't know if you have a complete tree
- without checking for absent too */
+ SVN_ERR(svn_wc__node_walk_children(wc_ctx,
+ local_abspath,
+ TRUE /* show_hidden */,
+ analyze_status, &wb,
+ svn_depth_infinity,
+ cancel_func, cancel_baton,
+ scratch_pool));
return SVN_NO_ERROR;
}
Modified: subversion/trunk/subversion/svnversion/main.c
URL:
http://svn.apache.org/viewvc/subversion/trunk/subversion/svnversion/main.c?rev=922176&r1=922175&r2=922176&view=diff
==============================================================================
--- subversion/trunk/subversion/svnversion/main.c (original)
+++ subversion/trunk/subversion/svnversion/main.c Fri Mar 12 08:21:45 2010
@@ -253,18 +253,25 @@ main(int argc, const char *argv[])
return EXIT_SUCCESS;
}
- SVN_INT_ERR(svn_wc_revision_status2(&res, wc_ctx, local_abspath,
- trail_url, committed, NULL, NULL,
- pool, pool));
+ err = svn_wc_revision_status2(&res, wc_ctx, local_abspath,
+ trail_url, committed, NULL, NULL,
+ pool, pool);
- /* Unversioned file in versioned directory */
- if (res->min_rev == -1)
+ if (err)
{
- SVN_INT_ERR(svn_cmdline_printf(pool, _("Unversioned file%s"),
- no_newline ? "" : "\n"));
- svn_pool_destroy(pool);
- return EXIT_SUCCESS;
+ /* Unversioned file in versioned directory */
+ if (err->apr_err == SVN_ERR_WC_PATH_NOT_FOUND)
+ {
+ svn_error_clear(err);
+ SVN_INT_ERR(svn_cmdline_printf(pool, _("Unversioned file%s"),
+ no_newline ? "" : "\n"));
+ svn_pool_destroy(pool);
+ return EXIT_SUCCESS;
+ }
+ else
+ SVN_INT_ERR(err);
}
+
}
else if (kind == svn_node_none)
{
