On 11/26/2010 01:01 AM, Daniel Shahaf wrote:
> Thanks for this :-). It works as expected when the current directory is
> entered by its name, but not when it's entered through a symlink:
>
> [[[
> % cd /tmp
>
> % ln -s wc1 wcalias
>
> % cd wc1
>
> % $svn up /tmp/wc1/trunk/iota
> Updating 'trunk/iota' ...
>
> % cd /tmp/wcalias
>
> % $svn up /tmp/wc1/trunk/iota11D
> Updating 'trunk/iota' ...
>
> % $svn up /tmp/wcalias/trunk/iota
> Updating '/tmp/wcalias/trunk/iota' ...
> ]]]
>
>
> Ideally, the last output would have used the relative path 'trunk/iota',
> too. However, I'm not sure how to easily solve that --- is it as easy
> as calling some "resolve symlinks" function on the absolute-cwd string?
Do we have a "resolve symlinks" function?
>> I'm wondering what would happen -- that is, how
>> much chaos would ensue -- if we changed subversion/svn/notify.c:notify() to,
>> in the case where n->path is not a URL, always first convert n->path to an
>> absolute path before doing the prefix removal stuff.
>
> What chaos could ensure? Just N+1 test expectations, or something more?
Not necessarily anything more. I mean, test expectations reflect, to some
degree, the expectations of Subversion users wrapping the command-line for
output parsing. I sense we're cool with some sane amount of change in that
output so as to make the tool better without completely breaking the world.
:-) ('Cause if we're cool with breaking the world, I have quite a few
changes in mind that I'd like to make to the output!)
--
C. Michael Pilato <cmpilato_at_collab.net>
CollabNet <> www.collab.net <> Distributed Development On Demand
Received on 2010-11-29 20:17:54 CET