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

Re: svn diff --svnpatch & svn patch

From: David Glasser <glasser_at_davidglasser.net>
Date: 2007-09-09 00:39:12 CEST

On 9/8/07, Jay Levitt <lists-svndev@shopwatch.org> wrote:
> On 9/8/2007 4:11 PM, David Glasser wrote:
>
> > Thus there are three types of patches: those which just need an
> > external patch tool to apply (text mods only), those which just need
> > 'svn patch' to apply (no text mods), and those which need both.
> > Perhaps 'svn patch' should include a header saying which one this is?
>
> Even with a header, that's going to be horribly confusing to users.
> Heck, it's confusing to me, and I haven't used since college.
>
> Can't svn patch call patch externally with the unidiff portion if patch
> is present? That makes things better for people who DO have GNU patch,
> without making things worse for anyone who doesn't.

Good call. You should be able to configure a path to GNU patch (or a
similar tool with a similar interface) in the user config directory,
and 'svn patch' should call it (with the right -p argument!); if 'svn
patch' can't find it it can print a message saying "text diffs not
applied, please use an external tool".

--dave

-- 
David Glasser | glasser_at_davidglasser.net | http://www.davidglasser.net/
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Sun Sep 9 00:36:01 2007

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.