[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

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

From: Gavin 'Beau' Baumanis <gavinb_at_thespidernet.com>
Date: Tue, 15 Dec 2009 20:50:01 +1100

Hi Philip / Julian,

Just thought I would send this thread back to you both and see if there was going to be anything further happening with it, or can I drop it from my todo list?

Gavin.

On 04/12/2009, at 01:00 , Philip Martin wrote:

> Julian Foad <julianfoad_at_btopenworld.com> writes:
>
>> Philip Martin wrote:
>>> Julian Foad <julian.foad_at_wandisco.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
>
> ------------------------------------------------------
> http://subversion.tigris.org/ds/viewMessage.do?dsForumId=462&dsMessageId=2426707
>
> Please start new threads on the <dev_at_subversion.apache.org> mailing list.
> To subscribe to the new list, send an empty e-mail to <dev-subscribe_at_subversion.apache.org>.
Received on 2009-12-15 11:20:20 CET

This is an archived mail posted to the Subversion Dev mailing list.

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.