Re: Screenshots for version 1.7?

[Christian Hammond]

It's not a radical change. It's a refinement on what we had.

Christian

-- 
Christian Hammond - chip...@chipx86.com
Review Board - http://www.reviewboard.org
VMware, Inc. - http://www.vmware.com


On Fri, Sep 28, 2012 at 12:33 PM, Robert Dailey wrote:

> Thanks, I wasn't aware of the live site :)
>
> Doesn't look too much different to me. The reviews/diff pages which
> are what I look at the most, seem mostly different other than a few
> border changes.
>
> On Thu, Sep 27, 2012 at 4:04 PM, Chris Clark 
> wrote:
> > On Wednesday, September 26, 2012 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 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
> > reviewboard+unsubscr...@googlegroups.com
> > For more options, visit this group at
> > http://groups.google.com/group/reviewboard?hl=en
>
> --
> 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
> reviewboard+unsubscr...@googlegroups.com
> For more options, visit this group at
> http://groups.google.com/group/reviewboard?hl=en
>

-- 
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 
reviewboard+unsubscr...@googlegroups.com
For more options, visit this group at 
http://groups.google.com/group/reviewboard?hl=en

Re: Screenshots for version 1.7?

[Robert Dailey]

Thanks, I wasn't aware of the live site :)

Doesn't look too much different to me. The reviews/diff pages which
are what I look at the most, seem mostly different other than a few
border changes.

On Thu, Sep 27, 2012 at 4:04 PM, Chris Clark  wrote:
> On Wednesday, September 26, 2012 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 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
> reviewboard+unsubscr...@googlegroups.com
> For more options, visit this group at
> http://groups.google.com/group/reviewboard?hl=en

-- 
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 
reviewboard+unsubscr...@googlegroups.com
For more options, visit this group at 
http://groups.google.com/group/reviewboard?hl=en

Re: easy_install doesn't find 1.6.12

[Igor Berger]

Thanks anyway.


On Thursday, September 27, 2012 5:17:16 PM UTC-4, Christian Hammond wrote:
>
> I wish I had some idea, but that's just plain weird.
>
> Christian
>
> -- 
> Christian Hammond - chi...@chipx86.com 
> Review Board - http://www.reviewboard.org
> VMware, Inc. - http://www.vmware.com
>
>
> On Thu, Sep 27, 2012 at 12:29 PM, Igor Berger 
> > wrote:
>
>> Different error:
>>
>> 
>> # easy_install -U ReviewBoard==1.6.12
>> Searching for ReviewBoard==1.6.12
>> Reading http://pypi.python.org/simple/ReviewBoard/
>> Reading http://www.reviewboard.org/
>> Reading http://downloads.reviewboard.org/releases/ReviewBoard/1.5/
>> Reading http://downloads.reviewboard.org/releases/ReviewBoard/1.6/
>> Reading http://www.review-board.org/
>> Reading http://downloads.review-board.org/releases/ReviewBoard/1.0/
>> Reading http://downloads.review-board.org/releases/
>> Reading http://downloads.reviewboard.org/releases/ReviewBoard/1.0/
>> No local packages or download links found for ReviewBoard==1.6.12
>> error: Could not find suitable distribution for 
>> Requirement.parse('ReviewBoard==1.6.12')
>> 
>>
>> I've seen this in the past. Eventually it works.
>> Perhaps, something is cached somewhere and I need to wait for it to 
>> expire.
>> But I can't think of what this could be. :(
>>
>>
>> On Thursday, September 27, 2012 3:23:04 PM UTC-4, Christian Hammond wrote:
>>>
>>> Very weird. How about ReviewBoard==1.6.12 ?
>>>
>>> Christian
>>>
>>  -- 
>> 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 
>> reviewboard...@googlegroups.com 
>> For more options, visit this group at 
>> http://groups.google.com/group/reviewboard?hl=en
>>
>
>

-- 
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 
reviewboard+unsubscr...@googlegroups.com
For more options, visit this group at 
http://groups.google.com/group/reviewboard?hl=en

Re: post-review issue with .reviewboardrc file on windows (clearcase as SCM)

[Christian Hammond]

Believe it or not, this has never come up before, despite all the Windows
users.

You can name the file something in your text editor and then rename it on
the command line to ".reviewboardrc"

Christian

-- 
Christian Hammond - chip...@chipx86.com
Review Board - http://www.reviewboard.org
VMware, Inc. - http://www.vmware.com


On Fri, Sep 28, 2012 at 6:26 AM, Amit Priyadarshi <
amit.priyadarshi@gmail.com> wrote:

> Hi All,
>
> I wanted to use reviewboard with clearcase and we have dynamic views on
> windows machine.
> I installed reviewboard and then tried to use post-review.
> However I run into an error:
> "Unable to find a Review Board server for this source code tree."
>
> When i had a quick look at code
>
> http://code.google.com/p/reviewboard/source/browse/trunk/reviewboard/contrib/tools/post-review?spec=svn1812&r=1812
>
> it seems it looks for some .reviewboardrc file.
> I have a windows box and we cannot have such a file name?
> do we have a patch already for this?
>
> --
> 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
> reviewboard+unsubscr...@googlegroups.com
> For more options, visit this group at
> http://groups.google.com/group/reviewboard?hl=en

-- 
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 
reviewboard+unsubscr...@googlegroups.com
For more options, visit this group at 
http://groups.google.com/group/reviewboard?hl=en

Re: Review Board 1.7 beta 1 released

[Christian Hammond]

You'll get an entirely different error on Linux. I'd need to see it to give you 
any information on a fix.

Christian



On Sep 28, 2012, at 3:49, kvr  wrote:

> No even I tried in Ubuntu, but was not successful.It failed while building 
> the media files in Ubuntu also.
> 
> On Friday, September 28, 2012 12:55:08 AM UTC+5:30, Christian Hammond wrote:
> We don't build packages on Windows, so likely new parts of our build 
> infrastructure are assuming unixisms.
> 
> Christian
> 
> 
> On Sep 27, 2012, at 4:21, kvr  wrote:
> 
>> 
>> 
>> On Thursday, September 27, 2012 11:22:07 AM UTC+5:30, Christian Hammond 
>> wrote:
>> Can you show me what happened when you tried? There are certain things that 
>> must be installed in order to build the package.
>> 
>> Christian
>>  
>> I issued the following command to build the beta 1.7 code.
>>  
>>  setup.py bdist_egg
>> 
>> And the result is given below.
>> 
>> running bdist_egg
>> running egg_info
>> running build_media
>> error: %1 is not a valid Win32 application 
>> 
>> But 1.6.x code got successfully built.
>>  
>> On Sep 26, 2012, at 20:26, kvr  wrote:
>> 
>>> May I know how did you make the egg file?
>>> 
>>> Because I tried with easy_install bdist_egg, but did not work out.
>>> 
>>> On Saturday, September 15, 2012 4:41:12 AM UTC+5:30, Christian Hammond 
>>> wrote:
>>> In that case, grab the upstream Git version. Some of our RB 1.7 
>>> dependencies have updated and are incompatible with a source build of 1.7.
>>> 
>>> Christian
>>> 
>>> -- 
>>> Christian Hammond - chi...@chipx86.com
>>> Review Board - http://www.reviewboard.org
>>> VMware, Inc. - http://www.vmware.com
>>> 
>>> 
>>> On Thu, Sep 13, 2012 at 4:34 AM, kvr  wrote:
>>> Thanks for your reply.
>>> 
>>> But I've added some widgets in admin module that's why I want to install 
>>> from source.
>>> 
>>> 
>>> On Friday, August 3, 2012 2:19:58 PM UTC+5:30, Christian Hammond wrote:
>>> Hey everyone,
>>> 
>>> I'm happy to announce that we've pushed out the release of Review Board 1.7 
>>> beta 1. There's many new and exciting things in this release, but rather 
>>> than repeat myself, I'll just link you to the news post.
>>> 
>>> http://www.reviewboard.org/news/2012/08/03/review-board-17-beta-1-released/
>>> 
>>> Please let me know if you hit any problems. And remember, it's a beta, so 
>>> use your best judgment when deciding where to deploy it. Any testing would 
>>> definitely be helpful, though, as that will help ensure a higher-quality 
>>> release.
>>> 
>>> Christian
>>> 
>>> -- 
>>> Christian Hammond - chi...@chipx86.com
>>> 
>>> Review Board - http://www.reviewboard.org
>>> VMware, Inc. - http://www.vmware.com
>>> -- 
>>> 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 
>>> reviewboard...@googlegroups.com
>>> For more options, visit this group at 
>>> http://groups.google.com/group/reviewboard?hl=en
>>> 
>>> -- 
>>> 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 
>>> reviewboard...@googlegroups.com
>>> For more options, visit this group at 
>>> http://groups.google.com/group/reviewboard?hl=en
>> 
>> -- 
>> 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 reviewboard...@googlegroups.com
>> For more options, visit this group at 
>> http://groups.google.com/group/reviewboard?hl=en
> 
> -- 
> 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 
> reviewboard+unsubscr...@googlegroups.com
> For more options, visit this group at 
> http://groups.google.com/group/reviewboard?hl=en

-- 
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 
reviewboard+unsubscr...@googlegroups.com
For more options, visit this group at 
http://groups.google.com/group/reviewboard?hl=en

Re: Parent Dif - Options

[siva kumar]

Thanks You Christian ,

I got this error using post-review 


DEBUG:root:Using candidate path 'default': 'https://sivakumar/hgtools'
DEBUG:root:repository info: Path: https://sivakumar/hgtools, Base path: , 
Supports changesets: False
>>> Finished checking the repository type.
>>> HTTP GETting api/
>>> HTTP GETting http://sivakumar/reviews/api/info/
>>> Using the new web API
>>> Attempting to create review request on HGTOOLS_NEW for None
>>> HTTP POSTing to http://sivakumar/reviews/api/review-requests/: 
{'repository': 'HGTOOLS_NEW'}
>>> Review request created
>>> Uploading diff, size: 994
>>> HTTP POSTing to http://sivakumar/reviews/api/review-requests/27/diffs/: 
{}
>>> Got API Error 207 (HTTP code 400): The file was not found in the 
repository
>>> Error data: {u'stat': u'fail', u'file': u'settings.py', u'err': 
{u'msg': u'The file was not found in the repository', u'code': 207}, 
u'revision': u'd5ade09376df'}

*Error uploading diff

Your review request still exists, but the diff is not attached.*


Thanks,
Siva

On Friday, September 28, 2012 2:35:04 PM UTC+5:30, Christian Hammond wrote:
>
> Hi Siva,
>
> I recommend you use our post-review tool or the third-party 'hg 
> post-review' to post review requests instead of filling in this form.
>
> To answer your question, though, Parent Diff is optional and only needed 
> if you want to show a diff that's on top of commits that don't exist in the 
> central repository.
>
> Christian
>
> -- 
> Christian Hammond - chi...@chipx86.com 
> Review Board - http://www.reviewboard.org
> VMware, Inc. - http://www.vmware.com
>
>
> On Thu, Sep 27, 2012 at 10:53 PM, siva kumar <85s...@gmail.com
> > wrote:
>
>>
>> I have configured hg repository module in review board .
>>
>> Parent Diff :  What i have to give here ( I have given base diff 
>> contents but its not shown reviewboard )
>>
>> ( An optional diff that the main diff is based on. This is usually used 
>> for distributed revision control systems (Git, Mercurial, etc.)  ) 
>>
>> If i missed anything pl let us know
>>
>> Thanks,
>> Siva
>>
>>  -- 
>> 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 
>> reviewboard...@googlegroups.com 
>> For more options, visit this group at 
>> http://groups.google.com/group/reviewboard?hl=en
>
>
>

-- 
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 
reviewboard+unsubscr...@googlegroups.com
For more options, visit this group at 
http://groups.google.com/group/reviewboard?hl=en

post-review issue with .reviewboardrc file on windows (clearcase as SCM)

[Amit Priyadarshi]

Hi All,

I wanted to use reviewboard with clearcase and we have dynamic views on 
windows machine.
I installed reviewboard and then tried to use post-review.
However I run into an error:
"Unable to find a Review Board server for this source code tree."

When i had a quick look at code
http://code.google.com/p/reviewboard/source/browse/trunk/reviewboard/contrib/tools/post-review?spec=svn1812&r=1812

it seems it looks for some .reviewboardrc file.
I have a windows box and we cannot have such a file name?
do we have a patch already for this?

-- 
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 
reviewboard+unsubscr...@googlegroups.com
For more options, visit this group at 
http://groups.google.com/group/reviewboard?hl=en

Re: Review Board 1.7 beta 1 released

[kvr]

No even I tried in Ubuntu, but was not successful.It failed while building 
the media files in Ubuntu also.

On Friday, September 28, 2012 12:55:08 AM UTC+5:30, Christian Hammond wrote:
>
> We don't build packages on Windows, so likely new parts of our build 
> infrastructure are assuming unixisms.
>
> Christian
>
>
> On Sep 27, 2012, at 4:21, kvr > wrote:
>
>
>
> On Thursday, September 27, 2012 11:22:07 AM UTC+5:30, Christian Hammond 
> wrote:
>>
>> Can you show me what happened when you tried? There are certain things 
>> that must be installed in order to build the package.
>>
>> Christian
>>
>  
>
>> I issued the following command to build the beta 1.7 code.
>>
>  
>  setup.py bdist_egg
>
> And the result is given below.
>
> running bdist_egg
> running egg_info
> running build_media
> error: %1 is not a valid Win32 application 
>
>>
>> But 1.6.x code got successfully built.
>>
>  
>
>> On Sep 26, 2012, at 20:26, kvr  wrote:
>>
>> May I know how did you make the egg file?
>>
>> Because I tried with easy_install bdist_egg, but did not work out.
>>
>> On Saturday, September 15, 2012 4:41:12 AM UTC+5:30, Christian Hammond 
>> wrote:
>>>
>>> In that case, grab the upstream Git version. Some of our RB 1.7 
>>> dependencies have updated and are incompatible with a source build of 1.7.
>>>
>>> Christian
>>>
>>> -- 
>>> Christian Hammond - chi...@chipx86.com
>>> Review Board - http://www.reviewboard.org
>>> VMware, Inc. - http://www.vmware.com
>>>
>>>
>>> On Thu, Sep 13, 2012 at 4:34 AM, kvr  wrote:
>>>
 Thanks for your reply.

 But I've added some widgets in admin module that's why I want to 
 install from source.


 On Friday, August 3, 2012 2:19:58 PM UTC+5:30, Christian Hammond wrote:

> Hey everyone,
>
> I'm happy to announce that we've pushed out the release of Review 
> Board 1.7 beta 1. There's many new and exciting things in this release, 
> but 
> rather than repeat myself, I'll just link you to the news post.
>
> http://www.reviewboard.org/**news/2012/08/03/review-board-**
> 17-beta-1-released/
>
> Please let me know if you hit any problems. And remember, it's a beta, 
> so use your best judgment when deciding where to deploy it. Any testing 
> would definitely be helpful, though, as that will help ensure a 
> higher-quality release.
>
> Christian
>
> -- 
> Christian Hammond - chi...@chipx86.com
>
> Review Board - http://www.reviewboard.org
> VMware, Inc. - http://www.vmware.com
>
  -- 
 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 
 reviewboard...@googlegroups.com
 For more options, visit this group at 
 http://groups.google.com/group/reviewboard?hl=en

>>>
>>>  -- 
>> 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 
>> reviewboard...@googlegroups.com
>> For more options, visit this group at 
>> http://groups.google.com/group/reviewboard?hl=en
>>
>>  -- 
> 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 
> reviewboard...@googlegroups.com 
> For more options, visit this group at 
> http://groups.google.com/group/reviewboard?hl=en
>
>

-- 
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 
reviewboard+unsubscr...@googlegroups.com
For more options, visit this group at 
http://groups.google.com/group/reviewboard?hl=en

Re: Parent Dif - Options

[Christian Hammond]

Hi Siva,

I recommend you use our post-review tool or the third-party 'hg
post-review' to post review requests instead of filling in this form.

To answer your question, though, Parent Diff is optional and only needed if
you want to show a diff that's on top of commits that don't exist in the
central repository.

Christian

-- 
Christian Hammond - chip...@chipx86.com
Review Board - http://www.reviewboard.org
VMware, Inc. - http://www.vmware.com


On Thu, Sep 27, 2012 at 10:53 PM, siva kumar <85s...@gmail.com> wrote:

>
> I have configured hg repository module in review board .
>
> Parent Diff :  What i have to give here ( I have given base diff
> contents but its not shown reviewboard )
>
> ( An optional diff that the main diff is based on. This is usually used
> for distributed revision control systems (Git, Mercurial, etc.)  )
>
> If i missed anything pl let us know
>
> Thanks,
> Siva
>
>  --
> 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
> reviewboard+unsubscr...@googlegroups.com
> For more options, visit this group at
> http://groups.google.com/group/reviewboard?hl=en

-- 
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 
reviewboard+unsubscr...@googlegroups.com
For more options, visit this group at 
http://groups.google.com/group/reviewboard?hl=en