Re: post-review: Ignore whitespace?

2013-07-12 Thread Robert Dailey
atch and do some processing > and caching on the result. > > Christian > > > > Robert Dailey wrote: >> >> Is there a way to tell post-review to ignore whitespace in diffs that >> it uploads? I use this tool through P4V (perforce visual client). >> Some

post-review: Ignore whitespace?

2013-07-12 Thread Robert Dailey
Is there a way to tell post-review to ignore whitespace in diffs that it uploads? I use this tool through P4V (perforce visual client). Sometimes a file is full of tabs to spaces conversions and it mixes in with real, useful code changes. I'd like to see a way to upload a diff without whitespace O

Re: Python 3.3.2 x86_64 support?

2013-06-27 Thread Robert Dailey
Thanks guys. Looking forward to the future! On Thu, Jun 27, 2013 at 1:16 PM, Stephen Gallagher wrote: > On 06/27/2013 02:13 PM, Christian Hammond wrote: >> Plus not all dependencies support it fully yet, last I looked. >> >> We're not set up to easily maintain and test our stack for both 2.x and

Re: Python 3.3.2 x86_64 support?

2013-06-27 Thread Robert Dailey
I could but it's rather inconvenient, since everything else I'm using works on Python 3. Is support for Python 3 coming soon? It's already been out for a few years, not sure why things still use 2.x :-( On Wed, Jun 26, 2013 at 12:54 PM, Matthew Woehlke wrote: > On 2013-06-26 13

Python 3.3.2 x86_64 support?

2013-06-26 Thread Robert Dailey
I'm attempting to install RBTools via easy_install -U but getting failure. I'm using Python 3.3.2 x64 on Windows 7: C:\Work>easy_install -U RBTools Searching for RBTools Reading https://pypi.python.org/simple/RBTools/ Reading http://www.reviewboard.org/ Reading http://downloads.reviewboard.org/rel

Re: Problem with RBTools 0.5

2013-04-16 Thread Robert Dailey
ng post-review for now, as it hasn't > changed, but going forward as it's deprecated and replaced by rbt you'll > need to upgrade your version of Review Board. 1.6.3 is actually pretty old, > and should probably be upgraded for many other reasons as well. > > - Steve &g

Re: Problem with RBTools 0.5

2013-04-15 Thread Robert Dailey
/api/review-requests/44580/diffs/";, "method": "POST"}}} On Mon, Apr 8, 2013 at 5:12 PM, Steven MacLeod wrote: > It appears your Apache server won't allow a HEAD request. > > Can I see the output of: > > $ curl http://reviewboard.corp.good.com/api/rev

Re: Problem with RBTools 0.5

2013-04-08 Thread Robert Dailey
lipped by in my inbox. > > Could you please send me the output of: > > $ curl http://reviewboard.corp.good.com/api/review-requests/44580/diffs/ > -I > > Basically I need to take a look at the headers being returned in the HTTP > request to RB. > > > > On Fri,

Re: Problem with RBTools 0.5

2013-04-08 Thread Robert Dailey
iewboard.corp.good.com/api/review-requests/44580/diffs/ > -I > > Basically I need to take a look at the headers being returned in the HTTP > request to RB. > > > > On Fri, Mar 29, 2013 at 1:40 PM, Robert Dailey > wrote: >> >> Using RB version 1.6.3. Output of th

Re: Problem with RBTools 0.5

2013-03-29 Thread Robert Dailey
at 12:20 PM, Steven MacLeod wrote: > Hi Robert, > > What version of Review Board is there server running? Could you also provide > the output of rbt post -d --server=http://my.review.board.url --p4-client=$c > --p4-port=$p --username=myuser --password=mypassword %C > > The

Re: Problem with RBTools 0.5

2013-03-29 Thread Robert Dailey
No one will help? On Thu, Mar 28, 2013 at 3:54 PM, Robert Dailey wrote: > I am getting the following error uploading my review from a Perforce > changelist. This impedes my day to day work, so I hope that someone > can help me quickly. The error I get while trying to run '

Problem with RBTools 0.5

2013-03-28 Thread Robert Dailey
I am getting the following error uploading my review from a Perforce changelist. This impedes my day to day work, so I hope that someone can help me quickly. The error I get while trying to run 'rbt post' is: CRITICAL:root:'ResourceList' object has no attribute 'upload_diff' I have only transitio

Re: Screenshots for version 1.7?

2012-09-28 Thread Robert Dailey
2 11:26:36 AM UTC-7, Robert Dailey wrote: >> >> Are there any screenshots I can look at for various UI improvements >> for version 1.7? What do code review pages look like? > > > Have you seen the live site http://reviews.reviewboard.org/r/3347/ ? > > -- > Want

Screenshots for version 1.7?

2012-09-26 Thread Robert Dailey
Are there any screenshots I can look at for various UI improvements for version 1.7? What do code review pages look like? -- 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/ -~--~~--

Question about diffs for moved code

2012-09-26 Thread Robert Dailey
Sometimes I like to take 100-200 lines of code from one file, make modifications to that code, and then move it to a separate file as a refactoring step. A couple of people on my team have complained about this since now it is impossible to diff the individual changes to the code since it was move

Re: Problem with uploading consecutive diffs with updates

2012-05-15 Thread Robert Dailey
nd annoying without > some patch management system inbetween (like quilt or Git). > > > Christian > > -- > Christian Hammond - chip...@chipx86.com > Review Board - http://www.reviewboard.org > VMware, Inc. - http://www.vmware.com > > > On Mon, May 14, 2012 at 10:28

Re: Problem with uploading consecutive diffs with updates

2012-05-14 Thread Robert Dailey
on for the files to be incorrect. > > Christian > > -- > Christian Hammond - chip...@chipx86.com > Review Board - http://www.reviewboard.org > VMware, Inc. - http://www.vmware.com > > > On Mon, May 7, 2012 at 8:39 AM, Robert Dailey wrote: > >> Can someone hel

Re: Problem with uploading consecutive diffs with updates

2012-05-07 Thread Robert Dailey
Can someone help me out here? On Wed, May 2, 2012 at 1:37 PM, Robert Dailey wrote: > So I'm using Perforce and I upload my initial reviews (as well as follow > up changes and updates) using this command. Note that this is a "Custom > Tool" in P4V: > > %C --server=

Problem with uploading consecutive diffs with updates

2012-05-02 Thread Robert Dailey
So I'm using Perforce and I upload my initial reviews (as well as follow up changes and updates) using this command. Note that this is a "Custom Tool" in P4V: %C --server=http://reviewboard.company.com --p4-client=$c --p4-port=$p --username=user --password=password %C is the changelist number in