"David Glasser" <glasser@davidglasser.net> writes:
> Yup, we sadly came to that conclusion anew (Issue 1967 etc) on IRC.
Is it time to just rev the editor, and add delete_file() and
delete_directory() calls, and deprecate vanilla delete()?
-Karl
> On 10/15/07, C. Michael Pilato <cmpilato@collab.net> wrote:
>> Besides complexity and performance hits?
>>
>> For 'svn log', the client doesn't know the node kinds of changed files (as
>> the server doesn't report as much). For 'svn diff --summarize' (and similar
>> editor-based operations), the client knows the node kind most of the time,
>> but our editor's delete_entry() call is kind-agnostic, whichs means that any
>> time your deletion isn't of a working copy item (for which you could stat()
>> the kind), you'd have to make more network turnarounds to know whether a
>> given deleted thing was a file or directory.
>>
>>
>> Jeremy Whitlock wrote:
>> > Dave,
>> > Do you know of anything in place that would keep me from
>> > submitting such a patch?
>> >
>> > Take care,
>> >
>> > Jeremy
>> >
>> > On 10/15/07, David Glasser <glasser@davidglasser.net> wrote:
>> >> On 10/15/07, Mark Phippard <markphip@gmail.com> wrote:
>> >>> On 10/15/07, Jeremy Whitlock <jcscoobyrs@gmail.com> wrote:
>> >>>> Mark,
>> >>>> If other subcommands are affected by this, I would propose that we
>> >>>> fix them all then. Not knowing whether an outputted path is a file or
>> >>>> directory can be a pain to find the answer to usually requiring a
>> >>>> server round trip. From a scripting perspective, this is problematic.
>> >>>> I agree with consistency so I understand that if agreed upon, my
>> >>>> patch would have to potentially touch many subcommands but in the end,
>> >>>> I think having a visible way to distinguish file path from directory
>> >>>> path would be a great update to Subversion's output.
>> >>> I don't really disagree, but I do not think we can change the format
>> >>> of our output until 2.0. I believe it is part of the compatibility
>> >>> rules for the API. I could be wrong, but I think I have heard that
>> >>> before.
>> >> At the very least we could add a kind="dir"/"file" attribute to the
>> >> log --xml output. I'd apply a patch for that. (Also, diff
>> >> --summarize --xml would be pretty straightforward to implement.)
>> >>
>> >> --dave
>> >>
>> >> --
>> >> David Glasser | glasser_at_davidglasser.net | http://www.davidglasser.net/
>> >>
>> >
>> > ---------------------------------------------------------------------
>> > To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
>> > For additional commands, e-mail: dev-help@subversion.tigris.org
>> >
>>
>>
>> --
>> C. Michael Pilato <cmpilato@collab.net>
>> CollabNet <> www.collab.net <> Distributed Development On Demand
>>
>>
>>
>
>
> --
> David Glasser | glasser_at_davidglasser.net | http://www.davidglasser.net/
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
> For additional commands, e-mail: dev-help@subversion.tigris.org
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Tue Oct 16 08:32:59 2007