Login | Register
My pages Projects Community openCollabNet

Discussions > dev [DISABLED] > Re: [PATCH] Extend 'svn diff URL URL' to 'svn diff URL URL [WC]'

subversion
Discussion topic

There will be a brief maintenance window every Friday at 17:00 Pacific.
For further details, see CollabNet's maintenance and upgrade policy.

Back to topic list

Re: [PATCH] Extend 'svn diff URL URL' to 'svn diff URL URL [WC]'

Author philip
Full name Philip Martin
Date 2009-12-03 06:00:54 PST
Message Julian Foad <julianfoad@btope​nworld.com> writes:

> Philip Martin wrote:
>> Julian Foad <julian.foad@wand​isco.com> writes:
>>
>> >> > > $ svn diff ^/repo/branches/foo ^/repo/branches/bar --repos=[URL|WC]
>> >> >
>> >> > But that's much harder to type.
>> >
>> > Yup. :-) But not if you configure it somewhere so you don't have to type
>> > it.
>>
>> So your solution is I should define an alias that allows me to type
>> 'svndiff ^/foo ^/bar wc' to run 'svn diff ^/foo ^/bar --repos=wc'.
>
> I didn't say that.

Then I'm not sure what you did mean.

>> I'd prefer to fix the client.
>>
>> >> Also the new syntax is consistent
>> >> > with most of the other commands that use '.' as a default, if one
>> >> > doesn't want to use '.' one specifies the path explicitly:
>> >
>> > No, hardly any commands take a WC argument which they only use to
>> > identify the repository. They nearly all focus their behaviour onto the
>> > particular path you specify.
>>
>> Off the top of my head:
>>
>> $ svn log -rNNN
>> $ svn list ^/branches
>> $ svn diff -cNNN
>>
>> all will fail if wc is not a working copy and all will succeed if I
>> add a wc path.
>
> Sure, but they will produce output that relates to the WC path you
> specify, not just use it to find a repository and ignore it otherwise.

Perhaps. However

$ svn list ^/subversion/branches ../src/COMMITTERS

appears to use it just find a repository, and

$ svn merge ^/foo ^/bar wc

behaves in almost exactly the way I would like diff to behave. I
agree it does use the wc path beyond merely constructing URLs, but the
way it constructs URLs is the way I would like diff to behave.

>> Comparing two branches in a repository is not an obscure operation.
>> We should make it easy to invoke.
>
> Sure. I agree.
>
> Can you come on IRC #svn-dev where we can discuss more easily? (Actually
> I must go get some lunch and put dinner in the oven now, but in an hour
> or so?)

OK.

--
Philip

« Previous message in topic | 10 of 12 | Next message in topic »

Messages

Show all messages in topic

Subject
Author
Full name
Date posted
+
[PATCH] Extend 'svn diff URL URL' to 'svn diff URL URL [WC]' philip Philip Martin 2009-12-03 03:00:57 PST
     Re: [PATCH] Extend 'svn diff URL URL' to 'svn diff URL URL [WC]' Julian Foad <julian dot foad at wandisco dot com> Julian Foad <julian dot foad at wandisco dot com> 2009-12-03 03:39:11 PST
         Re: [PATCH] Extend 'svn diff URL URL' to 'svn diff URL URL [WC]' philip Philip Martin 2009-12-03 03:56:44 PST
             Re: [PATCH] Extend 'svn diff URL URL' to 'svn diff URL URL [WC]' philip2 Philip Martin 2009-12-03 03:57:55 PST
             RE: [PATCH] Extend 'svn diff URL URL' to 'svn diff URL URL [WC]' rhuijben Bert Huijben 2009-12-03 04:37:26 PST
                 Re: [PATCH] Extend 'svn diff URL URL' to 'svn diff URL URL [WC]' philip Philip Martin 2009-12-03 04:56:47 PST
                 RE: [PATCH] Extend 'svn diff URL URL' to 'svn diff URL URL [WC]' Julian Foad <julian dot foad at wandisco dot com> Julian Foad <julian dot foad at wandisco dot com> 2009-12-03 05:04:55 PST
                     Re: [PATCH] Extend 'svn diff URL URL' to 'svn diff URL URL [WC]' philip Philip Martin 2009-12-03 05:20:52 PST
                         Re: [PATCH] Extend 'svn diff URL URL' to 'svn diff URL URL [WC]' julianfoad Julian Foad 2009-12-03 05:37:35 PST
                             Re: [PATCH] Extend 'svn diff URL URL' to 'svn diff URL URL [WC]' philip Philip Martin 2009-12-03 06:00:54 PST
     Re: [PATCH] Extend 'svn diff URL URL' to 'svn diff URL URL [WC]' cmpilato C. Michael Pilato 2009-12-03 07:01:47 PST
         Re: [PATCH] Extend 'svn diff URL URL' to 'svn diff URL URL [WC]' cmpilato C. Michael Pilato 2009-12-03 07:06:53 PST
Messages per page: