Re: Disable diff-cmd

2010-08-24 Thread anatoly techtonik
On Tue, Aug 24, 2010 at 3:48 PM, Philip Martin wrote: > >> Not really. In Rietveld project we call "svn diff" by upload.py script >> used to send diffs for review to remote server. We can't instruct >> users to copy config dir, so we need to do this automatically. That >> means copying the whole c

Re: Disable diff-cmd

2010-08-24 Thread Stefan Sperling
On Tue, Aug 24, 2010 at 01:48:43PM +0100, Julian Foad wrote: > anatoly techtonik wrote: > > On Tue, Aug 24, 2010 at 2:29 PM, Stefan Sperling wrote: > > >> > > > >> > --internal-diff seems to be new in 1.7 so i guess the only option is > > >> > using > > >> > --config-dir to override it. > > >> >

Re: Disable diff-cmd

2010-08-24 Thread Philip Martin
Philip Martin writes: > anatoly techtonik writes: > >> Not really. In Rietveld project we call "svn diff" by upload.py script >> used to send diffs for review to remote server. We can't instruct >> users to copy config dir, so we need to do this automatically. That >> means copying the whole con

Re: Disable diff-cmd

2010-08-24 Thread Julian Foad
anatoly techtonik wrote: > On Tue, Aug 24, 2010 at 2:29 PM, Stefan Sperling wrote: > >> > > >> > --internal-diff seems to be new in 1.7 so i guess the only option is > >> > using > >> > --config-dir to override it. > >> > >> Why not to stop proliferation of options by allowing an empty value > >>

Re: Disable diff-cmd

2010-08-24 Thread Philip Martin
anatoly techtonik writes: > Not really. In Rietveld project we call "svn diff" by upload.py script > used to send diffs for review to remote server. We can't instruct > users to copy config dir, so we need to do this automatically. That > means copying the whole config dir and comment only one op

Re: Disable diff-cmd

2010-08-24 Thread anatoly techtonik
On Tue, Aug 24, 2010 at 2:29 PM, Stefan Sperling wrote: >> > >> > --internal-diff seems to be new in 1.7 so i guess the only option is using >> > --config-dir to override it. >> >> Why not to stop proliferation of options by allowing an empty value >> for --diff-cmd to be treated as an instruction

Re: Disable diff-cmd

2010-08-24 Thread Stefan Sperling
On Tue, Aug 24, 2010 at 02:24:25PM +0300, anatoly techtonik wrote: > On Tue, Aug 24, 2010 at 11:35 AM, Martin Furter wrote: > > > > --internal-diff seems to be new in 1.7 so i guess the only option is using > > --config-dir to override it. > > Why not to stop proliferation of options by allowing

Re: Disable diff-cmd

2010-08-24 Thread anatoly techtonik
On Tue, Aug 24, 2010 at 11:35 AM, Martin Furter wrote: > > --internal-diff seems to be new in 1.7 so i guess the only option is using > --config-dir to override it. Why not to stop proliferation of options by allowing an empty value for --diff-cmd to be treated as an instructions to use internal

Re: Disable diff-cmd

2010-08-24 Thread Martin Furter
On Tue, 24 Aug 2010, anatoly techtonik wrote: Hello, Is there any way to disable "diff-cmd" setting for just one run of "svn diff"? I need it for code review upload script, which fails when users have configured graphical diff. Is there any way to force svn use internal

Disable diff-cmd

2010-08-23 Thread anatoly techtonik
Hello, Is there any way to disable "diff-cmd" setting for just one run of "svn diff"? I need it for code review upload script, which fails when users have configured graphical diff. Is there any way to force svn use internal diff implemantation? I've tried: