Error Validating Diff with rbt post
I am running ReviewBoard on localhost via BitNami. I am using a local
bazaar repository on the C: Drive. When I try to post using the command
below, I get an error. I discovered this only occurs when the current
revision contains an edit of an exist file. If the revision is just newly
added files, the command works. I am using RBTools 0.7.4. I have tried this
with 0.7.6 and get the same results.
rbt post command:
rbt post --server=http://localhost/reviewboard -p --guess-fields=yes
--repository-type=bazaar --repository-url=file://C:\TheRBRepo\TheBranch -d
13
Output:
>>> RBTools 0.7.4
>>> Python 2.7.10 (default, May 23 2015, 09:40:32) [MSC v.1500 32 bit (Intel
)]
>>> Running on Windows-7-6.1.7601-SP1
>>> Home = C:\Users\gkotas\AppData\Roaming
>>> Current directory = C:\TheRBRepo\TheBranch
>>> Running: git version
>>> Checking for a Bazaar repository...
>>> Running: bzr info
>>> repository info: Path: C:\TheRBRepo\TheBranch, Base path: /, Supports
change
sets: False
>>> Making HTTP GET request to http://localhost/reviewboard/api/
>>> Running: bzr revno -r before:13
>>> Running: bzr revno -r 13
>>> Running: bzr info
>>> repository info: Path: C:\TheRBRepo\TheBranch, Base path: /, Supports
change
sets: False
>>> Running: bzr diff -q -r revno:12..revno:13
>>> Command exited with rc 1: [u'bzr', u'diff', u'-q', u'-r',
u'revno:12..revno:
13']
>>> Making HTTP GET request to http://localhost/reviewboard/api/validation/
diffs
/
>>> Cached response for HTTP GET http:
//localhost/reviewboard/api/validation/dif
fs/ expired and was modified
>>> Making HTTP POST request to
http://localhost/reviewboard/api/validation/diff
s/
>>> Got API Error 224 (HTTP code 400): The specified diff file could not be
pars
ed.
>>> Error data: {u'stat': u'fail', u'reason': u'Not a branch:
"C:/TheRBRepo/.bzr
/branch/": location is a repository.', u'err': {u'msg': u'The specified
diff fil
e could not be parsed.', u'code': 224}}
Traceback (most recent call last):
File "C:\Bitnami\reviewboard-2.5-0\python\Scripts\rbt-script.py", line 9,
in <
module>
load_entry_point('rbtools==0.7.4', 'console_scripts', 'rbt')()
File
"C:\Bitnami\reviewboard-2.5-0\python\lib\site-packages\rbtools-0.7.4-py2.
7.egg\rbtools\commands\main.py", line 133, in main
command.run_from_argv([RB_MAIN, command_name] + args)
File
"C:\Bitnami\reviewboard-2.5-0\python\lib\site-packages\rbtools-0.7.4-py2.
7.egg\rbtools\commands\__init__.py", line 612, in run_from_argv
exit_code = self.main(*args) or 0
File
"C:\Bitnami\reviewboard-2.5-0\python\lib\site-packages\rbtools-0.7.4-py2.
7.egg\rbtools\commands\post.py", line 754, in main
(msg_prefix, e))
rbtools.commands.CommandError: Error validating diff
The specified diff file could not be parsed. (HTTP 400, API Error 224)
--
Supercharge your Review Board with Power Pack:
https://www.reviewboard.org/powerpack/
Want us to host Review Board for you? Check out RBCommons:
https://rbcommons.com/
Happy user? Let us know! https://www.reviewboard.org/users/
---
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 reviewboard+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.
RBTools Ticket #4425: Git submodule modifications not included in diff
--
To reply, visit https://hellosplat.com/s/beanbag/tickets/4425/
--
New ticket #4425 by rhabarbersaft
For Beanbag, Inc. > RBTools
Status: New
Tags: Priority:Medium, Type:Defect
--
Git submodule modifications not included in diff
==
# What version are you running?
0.7.5
# What steps will reproduce the problem?
1. Having a GIT repo with submodules, modify one submodule by checking out
another commit
2. Commit the change in the parent repository
3. rbt post -p
# What is the expected output? What do you see instead?
What I expect:
Changes in submodules have to be included in the diff, even if the diff view in
the web interface doesn't show what changed in the submodule (it could show the
SHA IDs of the submodule versions, though).
We have our repositories set to read-only for everyone except the reviewers,
only they can actually push commits to the repos. We cannot simply use 'rbt
patch -C ...' for applying and pushing a commit, as submodule changes are
completely ignored by RBTools when creating a patch. This means that we have to
ask the developers for every single review request if there are submodule
changes and if they should be included in the commit.
Instead, submodule updates have to be included in the diffs, only then would
'rbt patch -C ...' deliver the expected result.
What I get:
ERROR: There don't seem to be any diffs!
# What operating system are you using?
Windows 10
# Attach the debug out from the command.
```
$ rbt post --debug -p
>>> RBTools 0.7.5
>>> Python 2.7.11 (v2.7.11:6d1b6a68f775, Dec 5 2015, 20:32:19) [MSC v.1500 32
>>> bit (Intel)]
>>> Running on Windows-10-10.0.10586
>>> Home = C:\Users...
>>> Current directory = ...
>>> Checking for a Subversion repository...
>>> Unable to execute "svn help": skipping SVN
>>> Checking for a Git repository...
>>> Running: git rev-parse --git-dir
>>> Running: git config core.bare
>>> Running: git rev-parse --show-toplevel
>>> Running: git symbolic-ref -q HEAD
>>> Running: git config --get branch.master.merge
>>> Running: git config --get branch.master.remote
>>> Running: git config --get remote.origin.url
>>> repository info: Path: ..., Base path: , Supports changesets: False
>>> Running: git config --get reviewboard.url
>>> Making HTTP GET request to ...
>>> Running: git rev-parse refs/heads/master
>>> Running: git merge-base ba6895dab5f871c845befa28e0ee0ff838450d14
>>> origin/master
>>> Running: git rev-parse 3eec4113a5b21c6a20f9bace738940c8698ae028
>>> Running: git status --porcelain --untracked-files=no
>>> Running: git rev-parse --git-dir
>>> Running: git version
>>> Running: git -c core.quotepath=false -c diff.noprefix=false diff --no-color
>>> --full-index --ignore-submodules -M --no-ext-diff
>>> 3eec4113a5b21c6a20f9bace738940c8698ae028..ba6895dab5f871c845befa28e0ee0ff838450d14
Traceback (most recent call last):
File "C:\Python27\Scripts\rbt-script.py", line 9, in
load_entry_point('RBTools==0.7.5', 'console_scripts', 'rbt')()
File
"C:\Python27\lib\site-packages\rbtools-0.7.5-py2.7.egg\rbtools\commands\main.py",
line 133, in main
command.run_from_argv([RB_MAIN, command_name] + args)
File
"C:\Python27\lib\site-packages\rbtools-0.7.5-py2.7.egg\rbtools\commands__init__.py",
line 622, in run_from_argv
exit_code = self.main(*args) or 0
File
"C:\Python27\lib\site-packages\rbtools-0.7.5-py2.7.egg\rbtools\commands\post.py",
line 710, in main
raise CommandError("There don't seem to be any diffs!")
rbtools.commands.CommandError: There don't seem to be any diffs!
```
# Please provide any additional information below.
The problem is the argument '--ignore-submodules' to the 'git diff' command.
--
--
You received this message because you are subscribed to the Google Groups
"reviewboard-issues" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to reviewboard-issues+unsubscr...@googlegroups.com.
To post to this group, send email to reviewboard-issues@googlegroups.com.
Visit this group at https://groups.google.com/group/reviewboard-issues.
For more options, visit https://groups.google.com/d/optout.
Re: RBTools Ticket #4425: Git submodule modifications not included in diff
-- To reply, visit https://hellosplat.com/s/beanbag/tickets/4425/ -- New update by rhabarbersaft For Beanbag, Inc. > RBTools > Ticket #4425 Reply: Review Board doesn't support review requests spanning repositories today, so we can't upload diffs containing submodules. Maybe in the future. Status: - New + Confirmed Tags: - Priority:Medium - Type:Defect + Priority:Low + SCM:Git + Type:Enhancement -- You received this message because you are subscribed to the Google Groups "reviewboard-issues" group. To unsubscribe from this group and stop receiving emails from it, send an email to reviewboard-issues+unsubscr...@googlegroups.com. To post to this group, send email to reviewboard-issues@googlegroups.com. Visit this group at https://groups.google.com/group/reviewboard-issues. For more options, visit https://groups.google.com/d/optout.
Re: Review Board Ticket #4412: WebHook signature specifies incorrect hashing algorithm
-- To reply, visit https://hellosplat.com/s/beanbag/tickets/4412/ -- New update by asaqui For Beanbag, Inc. > Review Board > Ticket #4412 Reply: Fixed in release-2.5.x (52b1b1c). Thanks! Status: - PendingReview + Fixed -- You received this message because you are subscribed to the Google Groups "reviewboard-issues" group. To unsubscribe from this group and stop receiving emails from it, send an email to reviewboard-issues+unsubscr...@googlegroups.com. To post to this group, send email to reviewboard-issues@googlegroups.com. Visit this group at https://groups.google.com/group/reviewboard-issues. For more options, visit https://groups.google.com/d/optout.
Re: Review Board Ticket #4423: Download Diff always downloads latest diff regardless of currently selected revision
-- To reply, visit https://hellosplat.com/s/beanbag/tickets/4423/ -- New update by gmyers For Beanbag, Inc. > Review Board > Ticket #4423 Reply: This is fixed on release-3.0.x as 8708952e04bc1724283c15f460eafa2bc9b39fd7. Thanks! Status: - New + Fixed -- You received this message because you are subscribed to the Google Groups "reviewboard-issues" group. To unsubscribe from this group and stop receiving emails from it, send an email to reviewboard-issues+unsubscr...@googlegroups.com. To post to this group, send email to reviewboard-issues@googlegroups.com. Visit this group at https://groups.google.com/group/reviewboard-issues. For more options, visit https://groups.google.com/d/optout.
