On Thu, May 8, 2008 at 6:17 PM, Karl Fogel <kfogel_at_red-bean.com> wrote:
> kfogel_at_tigris.org writes:
>> * tools/client-side/showchange.pl: Default to repository root for the
>> URL, so people never have to cd to get the result they want.
>
> This was obviously a good change for showchange.pl. I'm wondering if it
> might be a good idea for the command-line client too!
>
> Right now, if you run 'svn log -rNNN' or 'svn diff -cNNN' in a working
> copy directory, you may or may not get all of rNNN, even though rNNN was
> committed in that repository.
>
> 1) If rNNN was committed to some other branch, and never merged to
> this working copy's branch, then you'll get no output (well,
> you'll get a "------------" line in the case of log, but that's
> the same as getting no output).
>
> 2) If rNNN was committed above the current directory, but includes
> some stuff in or under the current directory, then you'll see that
> subset of rNNN, but not the whole thing.
>
> 3) If rNNN was committed to some other branch, but merged to the
> branch for which cwd is a working copy, then you'll see rNNN,
> except perhaps as adjust according to (2).
>
> I think that's all more or less correct. Can you keep it straight? I
> sure can't. I'm not even sure about the last part of (3), but I'm not
> going to bother to test it, because I have a better behavior to propose
> anyway:
>
> If the user explicitly requests a single revision but not a path,
> then show all of that revision, by using cwd's "Repository Root"
> as the URL.
>
> Would that *ever* be the wrong thing to do?
FWIW, I have definitely used this "feature" to create a patch that say
only contained the JavaHL part of a change.
With the new relative URL syntax don't you just need to use ^/ ?
--
Thanks
Mark Phippard
http://markphip.blogspot.com/
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe_at_subversion.tigris.org
For additional commands, e-mail: dev-help_at_subversion.tigris.org
Received on 2008-05-09 00:28:16 CEST