On 6/23/06, Jelmer Vernooij <jelmer@samba.org> wrote:
> On Fri, 2006-06-23 at 16:55 +0200, Jelmer Vernooij wrote:
> > On Fri, 2006-06-23 at 10:47 -0400, Garrett Rooney wrote:
> > > On 6/23/06, Jelmer Vernooij <jelmer@samba.org> wrote:
> > > > Hi,
> > > >
> > > > [[[
> > > > Return error code for unversioned files in svn_wc_crawl_revisions2()
> > > > rather than crashing.
> > > >
> > > > * subversion/libsvn_wc/adm_crawler.c
> > > > (svn_wc_crawl_revisions2): Return error for unversioned files.
> > > > ]]]
> > >
> > > Do you have a way to reproduce this problem? I'm just curious how it
> > > can actually happen...
> > Yep, just pass an unversioned path to svn_wc_crawl_revisions2().
> >
> > This was biting us in the Python bindings.
> To be more specific: This problem occurs because svn_wc_entry() returns
> a successful error code at subversion/libsvn_wc/adm_crawler.c:449 but
> parent_entry is set to NULL (indicating a unversioned file according to
> svn_wc.h).
>
> Other places in the code (such as svn_wc_merge_props) explicitly check
> for this already.
Works for me. Committed with minor tweaks in r20227.
-garrett
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Fri Jun 23 17:10:56 2006