Bert Huijben wrote on Fri, Feb 25, 2011 at 15:58:32 +0100:
> 
> 
> > -----Original Message-----
> > From: Noorul Islam K M [mailto:noo...@collab.net]
> > Sent: vrijdag 25 februari 2011 7:40
> > To: Subversion
> > Subject: [PATCH] New XFail test : 'svn up' fails if external propset is
> not
> > committed.
> > 
> > Log
> > [[[
> > 
> > 'svn up' fails if external propset is not committed.
> 
> I'm not sure if we should call this an error/bug.
> 
> I think externals were designed to operate only on committed nodes. 
> 
> This has the nice side effect that when you remove the externals definition
> it gets removed later on.
> 
> Once we start parsing externals from property definitions that are only
> edited locally we break this nice feature of automatically removing the
> externals.
> 

Fair point.  I don't feel strongly on how things should work here, but
I believe the test has a valid point --- viz., it triggers a bogus warning:
[[[
CMD: /home/daniel/src/svn/t1/subversion/svn/svn update 
svn-test-work/working_copies/externals_tests-29 --config-dir 
/home/daniel/src/svn/t1/subversion/tests/cmdline/svn-test-work/local_tmp/config 
--password rayjandom --no-auth-cache --username jrandom exited with 1
<TIME = 0.236354>
Updating 'svn-test-work/working_copies/externals_tests-29' ...

Fetching external item into 
'svn-test-work/working_copies/externals_tests-29/A2/iota'

At revision 1.
svn: warning: W155010: The node 
'/home/daniel/src/svn/t1/subversion/tests/cmdline/svn-test-work/working_copies/externals_tests-29/A2'
 was not found.
subversion/svn/update-cmd.c:183: (apr_err=205011)
svn: E205011: Failure occurred processing one or more externals definitions
]]]

> 
> Note that we already started breaking this feature a bit before this patch,
> because we now check externals definitions on added nodes.
> (Which now has issues of its own; see issue #3819)
> 
>       Bert
> 

Reply via email to