You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@subversion.apache.org by Ben Collins-Sussman <su...@collab.net> on 2003/11/12 23:44:14 UTC

Re: svn commit: rev 7711 - in trunk/subversion: include libsvn_wc

On Wed, 2003-11-12 at 01:41, brane@tigris.org wrote:
> Author: brane
> Date: Wed Nov 12 01:41:50 2003
> New Revision: 7711
> 
> Modified:
>    trunk/subversion/include/svn_wc.h
>    trunk/subversion/libsvn_wc/entries.c
> Log:
> Use "" instead of "svn:this_dir" in the entries file.
> 
> * subversion/include/svn_wc.h (SVN_WC_ENTRIES_THIS_DIR): Define to "".
> * subversion/libsvn_wc/entries.c (svn_wc__atts_to_entry): It the
>   entry name is "svn:this_dir", change it to SVN_WC_ENTRIES_THIS_DIR.

Brane, this is really dangerous.  You made this change without bumping
the working-copy 'format' version number.  Old clients are going to
choke on new working copies:

$ svn st
subversion/libsvn_wc/entries.c:567: (apr_err=150000)
svn: Can't find an entry
svn: missing default entry

I was just burned by this.



---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org

Re: svn commit: rev 7711 - in trunk/subversion: include libsvn_wc

Posted by Branko Čibej <br...@xbc.nu>.
Ben Collins-Sussman wrote:

>On Wed, 2003-11-12 at 01:41, brane@tigris.org wrote:
>  
>
>>Author: brane
>>Date: Wed Nov 12 01:41:50 2003
>>New Revision: 7711
>>
>>Modified:
>>   trunk/subversion/include/svn_wc.h
>>   trunk/subversion/libsvn_wc/entries.c
>>Log:
>>Use "" instead of "svn:this_dir" in the entries file.
>>
>>* subversion/include/svn_wc.h (SVN_WC_ENTRIES_THIS_DIR): Define to "".
>>* subversion/libsvn_wc/entries.c (svn_wc__atts_to_entry): It the
>>  entry name is "svn:this_dir", change it to SVN_WC_ENTRIES_THIS_DIR.
>>    
>>
>
>Brane, this is really dangerous.  You made this change without bumping
>the working-copy 'format' version number.  Old clients are going to
>choke on new working copies:
>
>$ svn st
>subversion/libsvn_wc/entries.c:567: (apr_err=150000)
>svn: Can't find an entry
>svn: missing default entry
>
>I was just burned by this.
>  
>
Ouch. Shows what I know about the WC innards. Should be fixed in r7725.

-- 
Brane Čibej   <br...@xbc.nu>   http://www.xbc.nu/brane/


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org