Hi, I've upgraded to RBTools 0.5.2 and the release notes say... *"Fixed a breakage where the git-svn support attempted to access the value for --parent-branch, which isn’t available for git-svn. Patch by Hanno S"*
Is that a fix for this issue or something else? As I just tried this again and this problem is still happening for me. Here is the debug output... DEBUG:root:Checking for a Subversion repository... DEBUG:root:Running: svn info --non-interactive DEBUG:root:Command exited with rc 1: ['svn', 'info', '--non-interactive'] svn: E155007: '/path/to/dev/dir' is not a working copy --- DEBUG:root:Checking for a Git repository... DEBUG:root:Running: git rev-parse --git-dir DEBUG:root:Running: git config core.bare DEBUG:root:Running: git rev-parse --show-toplevel DEBUG:root:Running: git symbolic-ref -q HEAD DEBUG:root:Running: git svn info DEBUG:root:repository info: Path: http://someserver/svn/repo, Base path: /trunk, Supports changesets: False DEBUG:root:Making HTTP GET request to https://internal.server.com/api/ DEBUG:root:Making HTTP GET request to https://internal.server.com/api/info/ DEBUG:root:Running: git merge-base develop refs/heads/feature-x DEBUG:root:Running: git diff --no-color --no-prefix --no-ext-diff -r -u develop DEBUG:root:Running: git svn find-rev develop DEBUG:root:Running: git svn find-rev 64378f1345f488e34d5752e49891d5986d07d00b DEBUG:root:Running: git diff --no-color --no-prefix --no-ext-diff -r -u 64378f1345f488e34d5752e49891d5986d07d00b..develop DEBUG:root:Running: git svn find-rev 64378f1345f488e34d5752e49891d5986d07d00b DEBUG:root:Running: git svn find-rev 64378f1345f488e34d5752e49891d5986d07d00b Traceback (most recent call last): File "/usr/local/bin/rbt", line 9, in <module> load_entry_point('RBTools==0.5.2', 'console_scripts', 'rbt')() File "/usr/local/lib/python2.7/dist-packages/RBTools-0.5.2-py2.7.egg/rbtools/commands/main.py", line 106, in main command.run_from_argv([RB_MAIN] + args) File "/usr/local/lib/python2.7/dist-packages/RBTools-0.5.2-py2.7.egg/rbtools/commands/__init__.py", line 157, in run_from_argv exit_code = self.main(*args) or 0 File "/usr/local/lib/python2.7/dist-packages/RBTools-0.5.2-py2.7.egg/rbtools/commands/post.py", line 494, in main if len(diff) == 0: TypeError: object of type 'NoneType' has no len() Output of some of the commands... git diff --no-color --no-prefix --no-ext-diff -r -u develop - Diff of the specific changes in the feature-x branch. git svn find-rev develop - No output, to be expected as develop is not in svn. git svn find-rev 64378f1345f488e34d5752e49891d5986d07d00b - No output. git diff --no-color --no-prefix --no-ext-diff -r -u 64378f1345f488e34d5752e49891d5986d07d00b..develop - No output. git svn find-rev 64378f1345f488e34d5752e49891d5986d07d00b - This line appeared twice for some reason, both times had no output (so three times in total including the time above). As I've found the source for RBTools on GitHub I'll see if I can identify the problem/fix. HTH Cheers, Alex On Monday, 29 July 2013 16:48:36 UTC+1, Alex Lewis wrote: > > Hi, > Please forgive me for raising this issue as I believe it has been raised > a couple of times over the years but I didn't see a resolution to those > issues and I'm seeing it with post-review 0.5.1. I hope you don't mind. > > I have the following branch setup... > > |- xxx (feature-x) > |- xxx > |- xxx > |- yyy (develop) > |- yyy > |- zzz (master) (git-svn) > > ...where xxx, yyy and zzz are commits. > > As far as I understand it, to post a review of just the changes introduced > by feature-x excluding the changes introduced by develop I use the --parent > flag set to "develop". If I do this the command fails with an error about a > NoneType (see debug output below). I believe the cause of the issue is that > post-review tries to do a git svn find-rev on the yyy commits which aren't > in SVN, so the find-rev returns nothing and post-review fails. > > Am I doing the right with the parent flag and if I am is there any chance > that post-review could support this case in the future? > > I can work around this issue by creating a new branch from feature-x and > rebasing that branch directly on master and raise the review from that > branch without supplying the parent flag. Although this workaround exists > the workflow is not ideal. Apologies if it's just me doing something stupid. > > Thanks for your help. > > Cheers, > Alex > > (Please note I've changed some of the path values in the debug below to > obfuscate private info) > >>> RBTools 0.5.1 > >>> Python 2.7.4 (default, Apr 19 2013, 18:28:01) > [GCC 4.7.3] > >>> Running on Linux-3.8.0-26-generic-x86_64-with-Ubuntu-13.04-raring > >>> Home = /home/alex > >>> Current Directory = /path/to/dev/dir > >>> Checking the repository type. Errors shown below are mostly harmless. > DEBUG:root:Checking for a Subversion repository... > DEBUG:root:Running: svn info --non-interactive > DEBUG:root:Command exited with rc 1: ['svn', 'info', '--non-interactive'] > svn: E155007: '/path/to/dev/dir' is not a working copy > --- > DEBUG:root:Checking for a Git repository... > DEBUG:root:Running: git rev-parse --git-dir > DEBUG:root:Running: git config core.bare > DEBUG:root:Running: git rev-parse --show-toplevel > DEBUG:root:Running: git symbolic-ref -q HEAD > DEBUG:root:Running: git svn info > DEBUG:root:repository info: Path: http://someserver/svn/repo, Base path: > /trunk, Supports changesets: False > >>> Finished checking the repository type. > >>> HTTP GETting api/info/ > DEBUG:root:Running: git merge-base develop refs/heads/feature-x > DEBUG:root:Running: git diff --no-color --no-prefix --no-ext-diff -r -u > develop > DEBUG:root:Running: git svn find-rev develop > DEBUG:root:Running: git diff --no-color --no-prefix --no-ext-diff -r -u > 5fe45c363bc8600ef727d0b4a441c8ed30e76c84..develop > DEBUG:root:Running: git svn find-rev > 5fe45c363bc8600ef727d0b4a441c8ed30e76c84 > Traceback (most recent call last): > File "/usr/local/bin/post-review", line 9, in <module> > load_entry_point('RBTools==0.5.1', 'console_scripts', 'post-review')() > File > "/usr/local/lib/python2.7/dist-packages/RBTools-0.5.1-py2.7.egg/rbtools/postreview.py", > > line 1334, in main > if len(diff) == 0: > TypeError: object of type 'NoneType' has no len() > -- 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 [email protected] For more options, visit this group at http://groups.google.com/group/reviewboard?hl=en --- You received this message because you are subscribed to the Google Groups "reviewboard" group. To unsubscribe from this group and stop receiving emails from it, send an email to [email protected]. For more options, visit https://groups.google.com/groups/opt_out.
