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


Reply via email to