Yes that was my mistake. I configured it as SVN instead of Perforce. Strange that it did accept it
On 16 January 2018 at 22:34, Christian Hammond <christ...@beanbaginc.com> wrote: > Hi, > > Off-hand, I don't see anything wrong with that header. It looks fine for a > Perforce diff. Is there any chance you have a repository added that is not > a Perforce repository, and it's using that one? I saw the other post about > having trouble adding a Perforce repository, and am wondering if it's > really configured right. > > To verify this by hand, I'm actually going to need a non-redacted copy of > the diff attached in a zip file (to prevent re-encoding issues over > e-mail). It's fine to change the depot, but it's absolutely vital that > characters are not removed and that you only change alphanumeric characters > to other alphanumeric characters, but anything like whitespace would need > to be preserved. > > Christian > > On Tue, Jan 16, 2018 at 3:35 AM, Giuseppe Salvatore < > giuseppe.salvat...@gmail.com> wrote: > >> Hi, >> >> I think I hit another nasty one: >> >> rbt post -d 379808 >> >> >>> RBTools 0.7.10 >> >>> Python 2.7.13 (v2.7.13:a06454b1afa1, Dec 17 2016, 20:42:59) [MSC >> v.1500 32 bit (Intel)] >> >>> Running on Windows-10-10.0.16299 >> >>> Home = C:\Users\************************ >> >>> Current directory = C:\Users\************************ >> >>> Command line: rbt post -d 379808 >> >>> Running: tf vc help >> >>> Checking for a Perforce repository... >> >>> Running: p4 info >> >>> Running: diff --version >> >>> repository info: Path: SERVER3.************************:1666, Base >> path: None, Supports changesets: True >> >>> Making HTTP GET request to http://************************.com/api/ >> >>> Running: p4 info >> Generating diff for pending changeset 379808 >> >>> Processing add of //depot************************ >> /Test/reviewboard_test.txt >> >>> Running: diff -urNp c:\users************************\temp\tmpvu_9y8 C >> ************************\Test\reviewboard_test.txt >> >>> Command exited with rc 1: [u'diff', u'-urNp', >> 'c:\\users\\gbsalv~1\\appdata\\local\\temp\\tmpvu_9y8', 'C:\\Users\\ >> ************************\\Test\\reviewboard_test.txt'] >> >>> Making HTTP GET request to http://************************ >> .com/api/validation/diffs/ >> >>> Cached response for HTTP GET >> >>> http://************************.com/api/validation/diffs/ >> expired and was modified >> >>> Making HTTP POST request to http://************************ >> .com/api/validation/diffs/ >> >>> Got API Error 224 (HTTP code 400): Unable to parse diff revision >> header '//depot************************/Test/reviewboard_test.txt#0' >> >>> Error data: {u'stat': u'fail', u'err': {u'msg': u"Unable to parse >> diff revision header '//depot************************ >> /Test/reviewboard_test.txt#0'", u'code': 224}} >> Traceback (most recent call last): >> File "C:\Program Files >> (x86)\RBTools\bin\..\Python27\Scripts\rbt-script.py", >> line 11, in <module> >> load_entry_point('RBTools==0.7.10', 'console_scripts', 'rbt')() >> File "C:\Program Files (x86)\RBTools\Python27\lib\sit >> e-packages\rbtools-0.7.10-py2.7.egg\rbtools\commands\main.py", line 133, >> in main >> command.run_from_argv([RB_MAIN, command_name] + args) >> File "C:\Program Files (x86)\RBTools\Python27\lib\sit >> e-packages\rbtools-0.7.10-py2.7.egg\rbtools\commands\__init__.py", line >> 663, in run_from_argv >> exit_code = self.main(*args) or 0 >> File "C:\Program Files (x86)\RBTools\Python27\lib\sit >> e-packages\rbtools-0.7.10-py2.7.egg\rbtools\commands\post.py", line 812, >> in main >> (msg_prefix, e)) >> rbtools.commands.CommandError: Error validating diff >> >> Unable to parse diff revision header >> '//************************/Test/reviewboard_test.txt#0' >> (HTTP 400, API Error 224) >> >> Sorry I needed to black some fields because of confidentiality, hope you >> don't mind it. >> From what I can see it seems to be the format of the diff, rbt doesn't >> like it. This is the diff file >> >> rbt diff 379808 >> Generating diff for pending changeset 379808 >> --- //depot************************/Test/reviewboard_test.txt //depot >> ************************/Test/reviewboard_test.txt#0 >> +++ //depot************************/Test/reviewboard_test.txt >> 2018-01-15 17:39:10 >> @@ -0,0 +1 @@ >> +This is a dummy test file used to check that reviewboard works correctly >> \ No newline at end of file >> >> I am using rbt 0.7.10 and reviewboard 2.5.15 GNU diff tools 2.8.7. Is >> there any particular configuration to apply to make rbt happy? >> >> Thanks >> >> Giuseppe >> >> -- >> 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 >> "Review Board Community" 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. >> > > > > -- > Christian Hammond > President/CEO of Beanbag <https://www.beanbaginc.com/> > Makers of Review Board <https://www.reviewboard.org/> > > -- > 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 a topic in the > Google Groups "Review Board Community" group. > To unsubscribe from this topic, visit https://groups.google.com/d/ > topic/reviewboard/989C6RHW1XI/unsubscribe. > To unsubscribe from this group and all its topics, send an email to > reviewboard+unsubscr...@googlegroups.com. > For more options, visit https://groups.google.com/d/optout. > -- 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 "Review Board Community" 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.
