On 7/16/07, Jeremy Whitlock <jcscoobyrs@gmail.com> wrote:
> While working on issue 2723, which will update the core diff API
> to have a method argument to specify the format (absolute or relative)
> of the Index path output, has brought up conversation about having the
> "svn diff" subcommand support an argument to override the default
> handling of the Index path output. As of now, here are the tested
> scenarios:
>
> Scenario 1
> ---------------
> svn diff RELATIVE_PATH_TO_WC (Outputs RELATIVE_PATH_TO_WC/MODIFIED_PATH)
>
> Scenario 2
> ---------------
> svn diff ABSOLUTE_PATH_TO_WC (Outputs ABSOLUTE_PATH_TO_WC/MODIFIED_PATH)
>
> Scenario 3
> ---------------
> svn diff -rOLD:NEW URL_TO_PATH (Outputs
> PATH_RELATIVE_TO_REPO_ROOT/MODIFIED_PATH)
>
> Now...why does this matter? Well, all of these scenarios end up with
> the Index path not being -p0 compliant, which is the standard for most
> project patches.
I do not know about this, but I will take your word for it. I do not
recall a lot of complaints about the format of svn diff output though,
so I'd be careful in changing the command line. This request was all
about JavaHL. In that environment we have not control over the CWD so
we always have to use absolute paths. This of course makes the
patches that we produce using Subclipse more difficult for the user to
apply.
Now, I am not against expanding the scope of this issue to include
changing the command line as well. I just think you need to be
careful about doing so unnecessarily. I would rather see the change
happen in the libraries and API and let a change to the command line
UI come out of an actual request for it.
--
Thanks
Mark Phippard
http://markphip.blogspot.com/
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Tue Jul 17 15:11:44 2007