It's just possible svn diff no longer supports the --diff option. I'm not at a 
computer right now, but does it appear at all in the help output?

Christian


On Nov 13, 2012, at 11:40, "rb" <reviewbo...@srmstar.net> wrote:

> 
> We have another user with this strange svn diff issue with post-review. I
> somehow got past this issue, completely by accident. Any help on resolution
> would be appreciated.
> 
> Mark
> 
> On Tue, October 30, 2012 9:49 am, rb wrote:
>> On Tue, October 30, 2012 3:40 am, Christian Hammond wrote:
>>> What happens if you run:
>>> 
>>>    svn diff --diff-cmd=diff
>>> 
>>> directly?
>>> 
>>> Christian
>>> 
>>> --
>>> Christian Hammond - chip...@chipx86.com
>>> Review Board - http://www.reviewboard.org
>>> VMware, Inc. - http://www.vmware.com
>> 
>> 
>> Christian,
>> 
>> It is the same as the output from post-review debug. If I remove GNU diff 
>> from
>> first in the path, post-review complains about not having GNU diff.
>> 
>> Mark
>> 
>> +++++++++++++++++++++++++++++++++++++++=
>> 
>> 
>> C:\workareas\svn\test\rb-test > svn up
>> 
>> Updating '.':
>> 
>> At revision 83.
>> 
>> 
>> C:\workareas\svn\test\rb-test > svn stat -u
>> 
>> M               83   test.txt
>> 
>> Status against revision:     83
>> 
>> 
>> C:\workareas\svn\test\rb-test > svn diff test.txt
>> 
>> Index: test.txt
>> 
>> ===================================================================
>> 
>> --- test.txt    (revision 83)
>> 
>> +++ test.txt    (working copy)
>> 
>> @@ -1,3 +1,4 @@
>> 
>> +test test
>> 
>> save this cahnge
>> 
>> nire
>> 
>> owewew
>> 
>> 
>> C:\workareas\svn\test\rb-test > post-review --debug
>> 
>>>>> RBTools 0.4.2
>> 
>>>>> Python 2.7.3 (default, Apr 10 2012, 23:31:26) [MSC v.1500 32 bit (Intel)]
>> 
>>>>> Running on Windows-7-6.1.7601-SP1
>> 
>>>>> Home = C:\Users\user1\AppData\Roaming
>> 
>>>>> Current Directory = C:\workareas\svn\test\rb-test
>> 
>>>>> Checking the repository type. Errors shown below are mostly harmless.
>> 
>> DEBUG:root:Checking for a CVS repository...
>> 
>> DEBUG:root:Checking for a ClearCase repository...
>> 
>> DEBUG:root:Checking for a Git repository...
>> 
>> DEBUG:root:Checking for a Mercurial repository...
>> 
>> DEBUG:root:Checking for a Perforce repository...
>> 
>> DEBUG:root:Checking for a Plastic repository...
>> 
>> DEBUG:root:Checking for a Subversion repository...
>> 
>> DEBUG:root:Running: svn info
>> 
>> DEBUG:root:Running: diff --version
>> 
>> DEBUG:root:repository info: Path:
>> https://svn-srvr/svn/proj_tpc_tca_test-comp-a, Base path:
>> /rel-branches/release_1.0_rc5, Supports changesets: False
>> 
>>>>> Finished checking the repository type.
>> 
>>>>> HTTP GETting api/
>> 
>>>>> HTTP GETting https://rb-srvr/rb/s/test_project_c/api/info/
>> 
>>>>> Using the new web API
>> 
>> DEBUG:root:Running: svn diff --diff-cmd=diff
>> 
>> Failed to execute command: ['svn', 'diff', '--diff-cmd=diff']
>> 
>> ['Index: test.txt\n',
>> '===================================================================\n',
>> 'Usage: diff [OPTIONS] <file1> <file2>\n', '\n', 'Display the differences
>> between <file1> and <file2> in unified diff\n', "format.  OPTIONS are diff
>> extensions as described by 'svn help diff'.\n", "Use '--' alone to indicate
>> that no more options follow.\n", "svn: E200012: 'diff' returned 2\n"]
>> 
>> 
>> 
>> C:\workareas\svn\test\rb-test > diff --version
>> 
>> diff (GNU diffutils) 2.8.7
>> 
>> Written by Paul Eggert, Mike Haertel, David Hayes,
>> 
>> Richard Stallman, and Len Tower.
>> 
>> Copyright (C) 2004 Free Software Foundation, Inc.
>> 
>> This is free software; see the source for copying conditions.  There is NO
>> 
>> warranty; not even for MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.
>> 
>> 
>> 
>> C:\workareas\svn\test\rb-test > svn --version
>> 
>> svn, version 1.7.5 (r1336830)
>> 
>>   compiled May 11 2012, 03:17:57
>> 
>> Copyright (C) 2012 The Apache Software Foundation.
>> 
>> This software consists of contributions made by many people; see the NOTICE
>> 
>> file for more information.
>> 
>> Subversion is open source software, see http://subversion.apache.org/
>> 
>> The following repository access (RA) modules are available:
>> 
>> * ra_neon : Module for accessing a repository via WebDAV protocol using Neon.
>> 
>>  - handles 'http' scheme
>> 
>>  - handles 'https' scheme
>> 
>> * ra_svn : Module for accessing a repository using the svn network protocol.
>> 
>>  - with Cyrus SASL authentication
>> 
>>  - handles 'svn' scheme
>> 
>> * ra_local : Module for accessing a repository on local disk.
>> 
>>  - handles 'file' scheme
>> 
>> * ra_serf : Module for accessing a repository via WebDAV protocol using serf.
>> 
>>  - handles 'http' scheme
>> 
>>  - handles 'https' scheme
>> 
>> 
>> C:\workareas\svn\test\rb-test > svn diff --diff-cmd=diff
>> 
>> Index: test.txt
>> 
>> ===================================================================
>> 
>> Usage: diff [OPTIONS] <file1> <file2>
>> 
>> Display the differences between <file1> and <file2> in unified diff
>> 
>> format.  OPTIONS are diff extensions as described by 'svn help diff'.
>> 
>> Use '--' alone to indicate that no more options follow.
>> 
>> svn: E200012: 'diff' returned 2
>> 
>> 
>> C:\workareas\svn\test\rb-test > svn propget reviewboard:url
>> 
>> https://rb-srvr/rb/s/test_project_c
>> 
>> 
>> 
>> C:\workareas\svn\test\rb-test > svn info
>> 
>> Path: .
>> 
>> Working Copy Root Path: C:\workareas\svn\test\rb-test
>> 
>> URL:
>> https://svn-srvr/svn/proj_tpc_tca_test-comp-a/rel-branches/release_1.0_rc5
>> 
>> Repository Root: https://svn-srvr/svn/proj_tpc_tca_test-comp-a
>> 
>> Revision: 83
>> 
>> Node Kind: directory
>> 
>> Schedule: normal
>> 
>> Last Changed Author: testuser1
>> 
>> Last Changed Rev: 83
>> 
>> Last Changed Date: 2012-10-30 10:19:47 -0400 (Tue, 30 Oct 2012)
>> 
>> 
>> 
>> --
>> Want to help the Review Board project? Donate today at
>> http://www.reviewboard.org/donate/
>> Happy user? Let us know at http://www.reviewboard.org/users/
>> -~----------~----~----~----~------~----~------~--~---
>> To unsubscribe from this group, send email to
>> reviewboard+unsubscr...@googlegroups.com
>> For more options, visit this group at
>> http://groups.google.com/group/reviewboard?hl=en
> 
> 
> -- 
> Want to help the Review Board project? Donate today at 
> http://www.reviewboard.org/donate/
> Happy user? Let us know at http://www.reviewboard.org/users/
> -~----------~----~----~----~------~----~------~--~---
> To unsubscribe from this group, send email to 
> reviewboard+unsubscr...@googlegroups.com
> For more options, visit this group at 
> http://groups.google.com/group/reviewboard?hl=en
> 
> 

-- 
Want to help the Review Board project? Donate today at 
http://www.reviewboard.org/donate/
Happy user? Let us know at http://www.reviewboard.org/users/
-~----------~----~----~----~------~----~------~--~---
To unsubscribe from this group, send email to 
reviewboard+unsubscr...@googlegroups.com
For more options, visit this group at 
http://groups.google.com/group/reviewboard?hl=en


Reply via email to