You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@subversion.apache.org by gs...@apache.org on 2010/04/21 09:14:21 UTC

svn commit: r936177 - /subversion/trunk/subversion/libsvn_wc/entries.c

Author: gstein
Date: Wed Apr 21 07:14:21 2010
New Revision: 936177

URL: http://svn.apache.org/viewvc?rev=936177&view=rev
Log:
Relax a constraint when reading entries. These are legal states in the DB,
so we shouldn't error on them.

* subversion/libsvn_wc/entries.c:
  (read_one_entry): allow some states to not have a REPOS_RELPATH

Modified:
    subversion/trunk/subversion/libsvn_wc/entries.c

Modified: subversion/trunk/subversion/libsvn_wc/entries.c
URL: http://svn.apache.org/viewvc/subversion/trunk/subversion/libsvn_wc/entries.c?rev=936177&r1=936176&r2=936177&view=diff
==============================================================================
--- subversion/trunk/subversion/libsvn_wc/entries.c (original)
+++ subversion/trunk/subversion/libsvn_wc/entries.c Wed Apr 21 07:14:21 2010
@@ -1041,10 +1041,22 @@ read_one_entry(const svn_wc_entry_t **ne
       else
         entry->kind = svn_node_unknown;
 
+      /* We should always have a REPOS_RELPATH, except for:
+         - deleted nodes
+         - certain obstructed nodes
+         - not-present nodes
+         - absent nodes
+         - excluded nodes
+
+         ### the last three should probably have an "implied" REPOS_RELPATH
+      */
       SVN_ERR_ASSERT(repos_relpath != NULL
                      || entry->schedule == svn_wc_schedule_delete
                      || status == svn_wc__db_status_obstructed
                      || status == svn_wc__db_status_obstructed_delete
+                     || status == svn_wc__db_status_not_present
+                     || status == svn_wc__db_status_absent
+                     || status == svn_wc__db_status_excluded
                      );
       if (repos_relpath)
         entry->url = svn_path_url_add_component2(entry->repos,