Re: RBtools 0.7 - issue posting a review

2015-04-08 Thread Daniel Laird
Ok - Will post the raw contents and other stuff I think is useful to you at the email address. Much appreciated Dan On Tuesday, 7 April 2015 23:53:24 UTC+1, Christian Hammond wrote: > > Hey Dan, > > Would you be able to provide us with the raw contents of that svn diff > command? We need some te

Re: rbt diff object of type 'NoneType' has no len(). RBTools 0.7.2 with svn

2015-04-08 Thread Richard Parks
Thank you very much Griffin, I've pulled in your modified process.py and my diff works fine now. Thanks for taking the time to reproduce my issue and fixing it so fast! -- Supercharge your Review Board with Power Pack: https://www.reviewboard.org/powerpack/ Want us to host Review Board for you

Re: rbt diff object of type 'NoneType' has no len(). RBTools 0.7.2 with svn

2015-04-08 Thread Richard Parks
Oh, so my diff works but a post still fails! rbt post --debug 12192 12218 >>> RBTools 0.7.2 >>> Python 2.7.6 (default, Nov 10 2013, 19:24:18) [MSC v.1500 32 bit (Intel )] >>> Running on Windows-7-6.1.7601-SP1 >>> Home = C:\Users\myuser\AppData\Roaming >>> Current directory = C:\subversion\myproje

Re: rbt diff object of type 'NoneType' has no len(). RBTools 0.7.2 with svn

2015-04-08 Thread Richard Parks
Oh, so my diff works but a post still fails! rbt post --debug 12192 12218 >>> RBTools 0.7.2 >>> Python 2.7.6 (default, Nov 10 2013, 19:24:18) [MSC v.1500 32 bit (Intel )] >>> Running on Windows-7-6.1.7601-SP1 >>> Home = C:\Users\myuser\AppData\Roaming >>> Current directory = C:\subversion\myprojec

Re: limiting users to PowerPack (trial)

2015-04-08 Thread David Carson
Thanks Christian. The powers that be decided it was OK to leave it wide open. But I still do think it should be an option in a future release. Thanks for your timely response. I'll use the trial as-is for now. David On Tuesday, April 7, 2015 at 6:14:50 PM UTC-4, Christian Hammond wrote: > >

Re: CRITICAL: not well-formed (invalid token): line 3, column 28

2015-04-08 Thread Griffin Myers
Can you share the output of: svn log -r 12192 -l 1 --xml The less sanitized the better, although I guess line 3 will be the most critical. I suspect some sort of encoding issue, but I don't know why this is system dependent. Perhaps something to do with locales? On Tuesday, April 7, 2015 at

Re: rbt diff object of type 'NoneType' has no len(). RBTools 0.7.2 with svn

2015-04-08 Thread Griffin Myers
Hi Richard, Your initial issue was a legitimate bug. I think this is something separate, but still likely due to moved/deleted files. I don't do very many post-commit reviews, but coincidentally I needed to do one today and I think I ran into the latest issue you are seeing. I fought with it