On 9/12/07, Philip Martin <philip@codematters.co.uk> wrote:
> "Erik Huelsmann" <ehuels@gmail.com> writes:
>
> > There's one problem: APR isn't really clear on which errors should be
> > expected when incorrectly removing a dir with a remove_file() call.
> > That's why I'm submitting the patch below for review. What it does is:
> > try to remove the file, if there's an error (any error) it proceeds to
> > try directory removal. This may lead to strange error reports if the
> > node at hand *was* a file, but coludn't be removed for some reason.
>
> You could use an algorithm that tries three times, then the final
> error would be "correct":
>
> try to remove file
> if that fails
> try to remove directory
> if that fails
> check kind
> if file
> try to remove file
> else
> try to remove directory
>
> The final try might even succeed!
Thanks! Committed in r26561.
bye,
Erik.
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Wed Sep 12 23:28:46 2007