Re: Review Board Ticket #4932: Error creating a review request with a SVN repository

[Daniel Morgan]

--
To reply, visit https://hellosplat.com/s/beanbag/tickets/4932/
--

New update by dkmorgan
For Beanbag, Inc. > Review Board > Ticket #4932


Reply:

As far as I can tell no.
I got the same error with the addition of the attached file to the 
repository. (created for testing purposes)
It looks like 'modified_file_details' is undefined in this case.
If I comment out the following two lines, I no longer get the error, but 
obviously this isn't a very good workaround...

590if parsed_file.modified_file_details.endswith(b'(nonexistent)'):
591parsed_file.deleted = True


Files:
- test.c
  

-- 
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 view this discussion on the web visit 
https://groups.google.com/d/msgid/reviewboard-issues/20210711223740.27628.26649%40ip-10-1-54-209.ec2.internal.

Review Board Ticket #4932: Error creating a review request with a SVN repository

[Daniel Morgan]

--
To reply, visit https://hellosplat.com/s/beanbag/tickets/4932/
--

New ticket #4932 by dkmorgan
For Beanbag, Inc. > Review Board

Status: New
Tags: Priority:Medium, Type:Defect


--
Error creating a review request with a SVN repository
==

# What version are you running?
4.02

# What's the URL of the page containing the problem?
/r/new/

# What steps will reproduce the problem?
1. Create a new review request
2. Select a SVN repository
3. Select a commit


# What is the expected output? What do you see instead?
Expect to be able to create a new review request but an error is shown instead.

# What operating system are you using? What browser?
Server side: Ubuntu 18.04 LTS
Client side: Windows 10, Edge browser

# Please provide any additional information below.
The error in the log is as follows:

 File 
"/usr/local/lib/python2.7/dist-packages/reviewboard/scmtools/svn/__init__.py", 
line 590, in parse_diff_header
if parsed_file.modified_file_details.endswith(b'(nonexistent)'):
AttributeError: 'NoneType' object has no attribute 'endswith'

The original code on lines 590-591 did not produce an error with Review Board 
3.x.
My guess is that the merge into the 4.x branch had some adverse effects.


--

-- 
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 view this discussion on the web visit 
https://groups.google.com/d/msgid/reviewboard-issues/20210627233826.19716.88348%40ip-10-1-54-209.ec2.internal.

Re: Review Board Ticket #4847: Pressing enter on upload file dialog cancels it instead of uploading - maddening

[Daniel]

--
To reply, visit https://hellosplat.com/s/beanbag/tickets/4847/
--

New update by brokenalarms
For Beanbag, Inc. > Review Board > Ticket #4847


Reply:

Thanks! You'll make a lot of people at LinkedIn happy.

-- 
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 view this discussion on the web visit 
https://groups.google.com/d/msgid/reviewboard-issues/20200110230912.31991.35481%40ip-10-1-54-209.ec2.internal.

Review Board Ticket #4847: Pressing enter on upload file dialog cancels it instead of uploading - maddening

[Daniel]

--
To reply, visit https://hellosplat.com/s/beanbag/tickets/4847/
--

New ticket #4847 by brokenalarms
For Beanbag, Inc. > Review Board

Status: New
Tags: Priority:Medium, Type:Defect


--
Pressing enter on upload file dialog cancels it instead of uploading - maddening
==

# What version are you running?
3.0.5

# What's the URL of the page containing the problem?
Internal corp RB

# What steps will reproduce the problem?
1. click update -> add file - upload is disabled
2. add a file - upload now enabled
3. press enter - window disappears, changes lost


# What is the expected output? What do you see instead?
The same as any other web form, for it to save and not lose all your input. 
Happens to me by virtue of standard keyboard trigger reflex every second time I 
(or multiple others I work with) do this.


# What operating system are you using? What browser?
MacOS Mojave, latest Chrome


# Please provide any additional information below.
Issue has been present for at least the past two years so finally submitting 
myself out of frustration. Thank you :)

--

-- 
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 view this discussion on the web visit 
https://groups.google.com/d/msgid/reviewboard-issues/20191024002754.30923.52109%40ip-10-1-54-209.ec2.internal.

Re: Review Board 3.0.14 is out

[Daniel]

Stephen, thank you for helping me to a better understanding.

Since yesterday I read up about EPEL usage with RHEL and now i think i have 
a better grasp of this. 

If i got it right, to do a "yum install reviewboard" with more up to date 
RB-versions it would be feasible to wait a bit until i can get my hands on 
an RHEL8 machine, get the EPEL configured (epel-release-latest-8 
<https://dl.fedoraproject.org/pub/epel/epel-release-latest-8.noarch.rpm>), 
regularly check for good news concerning RB via EPEL and upon arrival be 
happily installing via yum (not that i am not happy already, having 
realized once more what an amount of time you pour into this!)

So thank you once more and now i am back in lurkmode :)

Am Dienstag, 20. August 2019 17:03:42 UTC+2 schrieb Stephen Gallagher:
>
> On Tue, Aug 20, 2019 at 11:00 AM Daniel  > wrote: 
> > 
> > Hi Christian and Stephen, 
> > 
> > I just scanned the forums for possible updates concerning the 
> version-compatilities/ version-availabilites and found this post from 
> March. 
> > 
> > Some time ago when i was even more unexperienced in the ways of how the 
> system needs to be to be able to run reviewboard i asked about EPELs and 
> RHELs for revieboard 3.x (
> https://groups.google.com/d/msg/reviewboard/mSQPKhkKmMw/zFkNNf3gBgAJ) 
> > 
> > Since that time we are using RB in the Version 2.5.17 (the Version 
> available via yum like Paul mentioned). 
> > 
> > If i understand it correctly, than an EPEL can be configured for the 
> RHEL7 and afterwards i can use yum to install something out of that EPEL7 
> repo (e.g. reviewboard) 
> > 
> > Stephen was hoping that RB4 might be able to be delivered that way and 
> Christian was reducing that hope as there are still other hindering factors 
> so that the earliest maybe possible version of reviewboard available via an 
> EPEL again might be RB5. But this still is just a possibility and not 
> certain. 
> > 
> > We are working with RHEL7 Machines and i personally do not have much 
> legroom to customize that install ... so i was curious what your last 
> messages in this thread might mean for me and my usecase. 
> > 
> > I would be very happy if you would maybe find the time to describe the 
> way (if there is any) in that i might be able to install a "more modern 
> than 2.5.17"-version of RB into a RHEL7 system without manually installing 
> too many dependencies. 
> > 
>
> Short version: I probably can't provide RB 3.0+ on EPEL 7 any time 
> soon. However, I do expect to be able to deliver 3.0 and later on EPEL 
> 8 within the next six months. We have support for 
> Modularity/AppStreams in the works there and I will be able to package 
> the newer versions of Review Board for RHEL/CentOS 8.1+ 
>

-- 
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.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/reviewboard/17f1309a-8576-4bbd-b3fe-5d081e392771%40googlegroups.com.

Re: Review Board 3.0.14 is out

[Daniel]

Hi Christian and Stephen,

I just scanned the forums for possible updates concerning the 
version-compatilities/ version-availabilites and found this post from March.

Some time ago when i was even more unexperienced in the ways of how the 
system needs to be to be able to run reviewboard i asked about EPELs and 
RHELs for revieboard 3.x 
(https://groups.google.com/d/msg/reviewboard/mSQPKhkKmMw/zFkNNf3gBgAJ)

Since that time we are using RB in the Version 2.5.17 (the Version 
available via yum like Paul mentioned).

If i understand it correctly, than an EPEL can be configured for the RHEL7 
and afterwards i can use yum to install something out of that EPEL7 repo 
(e.g. reviewboard)

Stephen was hoping that RB4 might be able to be delivered that way and 
Christian was reducing that hope as there are still other hindering factors 
so that the earliest maybe possible version of reviewboard available via an 
EPEL again might be RB5. But this still is just a possibility and not 
certain.

We are working with RHEL7 Machines and i personally do not have much 
legroom to customize that install ... so i was curious what your last 
messages in this thread might mean for me and my usecase. 

I would be very happy if you would maybe find the time to describe the way 
(if there is any) in that i might be able to install a "more modern than 
2.5.17"-version of RB into a RHEL7 system without manually installing too 
many dependencies.

TIA and cheers
Daniel

>

-- 
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.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/reviewboard/51e99052-d01c-4658-ae84-a1431e193b9d%40googlegroups.com.

Review Board Ticket #4815: Diff is not displayed after uploading a diff file

[Daniel Viorreta]

--
To reply, visit https://hellosplat.com/s/beanbag/tickets/4815/
--

New ticket #4815 by dviorreta
For Beanbag, Inc. > Review Board

Status: New
Tags: Priority:Medium, Type:Defect


--
Diff is not displayed after uploading a diff file
==

# What version are you running?
3.0.14

# What's the URL of the page containing the problem?
/r/{review_request_id}/

# What steps will reproduce the problem?
1. Create a new review without a diff, for example using the api
2. Go the new review request and upload a file: Update --> Upload diff 
3. The diff is uploaded but it is not displayed
4. In Safari (only) refreshing the page the diff is displayed. 


# What is the expected output? What do you see instead?
In previous reviewboard version (3.0.8 for example), after uploading a diff 
file, the diff was automatically displayed for verification before publishing 
the review. 
In 3.0.14 after uploading the diff, the diff header doesn't appear. 


# What operating system are you using? What browser?
MacOS, Safari. For chrome and firefox the diff is not displayed after 
publishing the review request.

# Please provide any additional information below.


--

-- 
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: Reviewboard 3.0 RPM availability

[daniel . forumsmail]

Thank you for all your efforts and also that you try to keep on! You have 
my sympathy and understanding for not mentioning anything about something 
concerning the concept of a timeframe!


Am Mittwoch, 18. Juli 2018 19:23:09 UTC+2 schrieb Stephen Gallagher:
>
>
>
> On Wed, Jul 18, 2018 at 12:18 PM > 
> wrote:
>
>> Hi Stephen,
>>
>> would you maybe have had the possibility to look into a rpm for EPEL7? 
>> (not sure about either grammar or EPEL naming ... i think i am asking for 
>> something i can install via yum :)[...]
>>
> Getting this working on RHEL/CentOS 7 is something I very much want to do, 
> but please understand that I do this in my spare time. I'm not paid to work 
> on ReviewBoard at all. I have ReviewBoard 3.0 now available in Fedora 28 
> and I am looking into getting it working on EPEL 7, but I won't commit to 
> any particular timeframe, sorry.
>

-- 
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.

Re: Reviewboard 3.0 RPM availability

[daniel . forumsmail]

Hi Stephen,

would you maybe have had the possibility to look into a rpm for EPEL7? (not 
sure about either grammar or EPEL naming ... i think i am asking for 
something i can install via yum :)

I was just asking something in a different thread which got a nice and 
comprehensive reply by Christian Hammond ... also concerning a RB 3.x for 
yum ... quoting from 
(https://groups.google.com/d/msg/reviewboard-dev/2MuigbF_-X8/wU-aPDPKBgAJ)
 

> For c), I'm not sure of the current status there. This is not a Review 
> Board issue exactly, but rather a packaging issue, which I think is being 
> solved now? Basically, we need some older versions of some modules for very 
> good reasons I don't want to go into at the moment (various 
> compatibility... things...) and this makes packaging a bit harder. I think 
> that's being addressed through a new modular packaging system in Fedora. 
> Stephen can go into details more.
>

This quote is about helping Reviewboard 3.x being made available for Redhat 
rpm yum stuff ... as he mentioned that you might comment on that, i'd like 
to take the opportunity to ask you about it :)

Sadly, i am just a consumer and probably not versed enough to be helpful 
concerning your plea for help in supporting this kind of backporting stuff 
that is needed to make RB 3.x available via EPEL.

I tried to install RB in a newer Version  by following the instructions 
using pip but that lead to just a bunch of errors that i could not 
comprehend ... after that unsuccessful attempt i had the machine get 
reverted to previous "untouched" state and afterwards installed the rpm for 
2.5.17 ... and now, like stated, i am thumbing my nose at all those nice 
stuff that is coming in with the 3.x releases :o)

cheers 
Daniel


Am Freitag, 20. April 2018 18:57:52 UTC+2 schrieb Stephen Gallagher:
>
>
>
> On Fri, Apr 20, 2018 at 5:10 AM > wrote:
>
>> For Reviewboard 2.x there's EPEL RPM packages available, I'm just 
>> wondering if Reviewboard is planning to make RPM packages available for 
>> version 3.0 as well.
>>
>>
> I've been working on it, but it's difficult because EPEL 7 has a lot of 
> packages that are very old and hard to update. (Also, EPEL has a policy 
> against replacing any package shipped by RHEL/CentOS).
>
> However, I *do* now have packages working against Fedora 28:  
> https://bodhi.fedoraproject.org/updates/FEDORA-MODULAR-2018-d31893afa3
>
> It can be installed on F28 Beta by running the command:
> `dnf install @reviewboard:3.0`
>
> I'm hoping to get an update out to EPEL 7 at some point, but it's slow 
> going and my dayjob is coming down hard on me at the moment. For the time 
> being, I'm only committing to maintaining updates on the 2.5.x line.
>
> As always, anyone who would like to help me maintain these in Fedora/EPEL 
> should speak up.
>

-- 
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.

jQuery 1.8 vulnerability

[Daniel]

Hello,

My corp security department prevents me using the ReviewBoard because there 
are publicly known vulnerabilities in the one of RB's components 
(particularly jQuery 1.8).

Would it possible for the community to mitigate those issues ?


CVE-2015-9251 

CVE-2012-6708 

 

https://www.cvedetails.com/vulnerability-list/vendor_id-6538/product_id-11031/version_id-235567/opxss-1/Jquery-Jquery-1.8.1.html
 


I saw that master branch in GitHub already has jQuery 3.3, hopefully RB 4.0 
will met security requirements. Would it possible to backport jQuery3.3 to 
2.0 and/or 3.0 ?


Thanks in advance.

-- 
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.

Re: Issue installing reviewbot-worker on latest RB

[Daniel CABRERA]

I have done all that already, with same results:-(

On Fri, 16 Mar. 2018, 11:37 am David Trowbridge,  wrote:

> As I suggested, disable and enable the extension while keeping an eye on
> the log files. And double check the permissions on the directory to verify
> that the web server can write to it.
> On Thu, Mar 15, 2018 at 5:14 PM MoonWalker 
> wrote:
>
>> David,
>>
>> Not sure what else to try 
>>
>> --
>> 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.
>>
> --
> 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/VIwOcAmKC-k/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.

Re: rb-extension-pack Compability with Review Board 3.0

[Daniel Morgan]

Christian,

Thank you for the reply.
I will wait for the patch to be applied and check again.

-- 
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.

Re: rb-extension-pack Compability with Review Board 3.0

[Daniel Morgan]

David,

Thank you for the reply.
I will check again once the Extensions get updated.

-- 
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.

rb-extension-pack Compability with Review Board 3.0

[Daniel Morgan]

Are there any plans to check the compatibility for the Extensions contained 
in rb-extension-pack with Review Board 3.0?
I am using a few of the Extensions there and have encountered errors using 
Review Board 3.0 RC1.

For example, when configuring the message for the "Message of the Day" 
Extension, I get the following error.
The log output is as follows (sensitive details edited):

None - User1 - /admin/extensions/rbmotd.extension.MotdExtension/config/ - 
djblets.log.middleware - Exception thrown for user User1 at 
http://x.x.x.x/admin/extensions/rbmotd.extension.MotdExtension/config/

'MotdSettingsForm' object has no attribute 'siteconfig'
Traceback (most recent call last):
  File 
"/usr/local/lib/python2.7/dist-packages/django/core/handlers/base.py", line 
112, in get_response
response = wrapped_callback(request, *callback_args, **callback_kwargs)
  File "/usr/local/lib/python2.7/dist-packages/django/utils/decorators.py", 
line 99, in _wrapped_view
response = view_func(request, *args, **kwargs)
  File 
"/usr/local/lib/python2.7/dist-packages/reviewboard/extensions/views.py", 
line 15, in configure_extension
template_name)
  File "/usr/local/lib/python2.7/dist-packages/django/utils/decorators.py", 
line 99, in _wrapped_view
response = view_func(request, *args, **kwargs)
  File 
"/usr/local/lib/python2.7/dist-packages/django/contrib/admin/views/decorators.py",
 
line 17, in _checklogin
return view_func(request, *args, **kwargs)
  File 
"/usr/local/lib/python2.7/dist-packages/djblets/extensions/views.py", line 
64, in configure_extension
form.save()
  File 
"/usr/local/lib/python2.7/dist-packages/rbmotd-1.0-py2.7.egg/rbmotd/forms.py", 
line 22, in save
self.siteconfig.set(
AttributeError: 'MotdSettingsForm' object has no attribute 'siteconfig'

-- 
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.

Re: Error opening existing review after upgrading to Review Board 3.0 RC1

[Daniel Morgan]

Thank you for the prompt reply.

Yes, you were spot on.
I didn't copy the SECRET_KEY from the production server over to the test 
server.
Once I did, the errors stopped occurring.

-- 
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.

Error opening existing review after upgrading to Review Board 3.0 RC1

[Daniel Morgan]

I just completed upgrading Review Board 2.5.16 to 3.0 RC1 on a test server.
Logged in without any issues and tried opening an existing review which 
resulted in a "Something broke! (Error 500)".

The log output is as follows (sensitive details edited):

None - user1 - /r/605/ - djblets.log.middleware - Exception thrown for user 
user1 at http://x.x.x.x/r/605/

'utf8' codec can't decode byte 0x87 in position 1: invalid start byte
Traceback (most recent call last):
  File 
"/usr/local/lib/python2.7/dist-packages/django/core/handlers/base.py", line 
112, in get_response
response = wrapped_callback(request, *callback_args, **callback_kwargs)
  File 
"/usr/local/lib/python2.7/dist-packages/django/views/generic/base.py", line 
69, in view
return self.dispatch(request, *args, **kwargs)
  File 
"/usr/local/lib/python2.7/dist-packages/djblets/views/generic/base.py", 
line 46, in dispatch
request, *args, **kwargs)
  File "/usr/local/lib/python2.7/dist-packages/django/utils/decorators.py", 
line 29, in _wrapper
return bound_func(*args, **kwargs)
  File 
"/usr/local/lib/python2.7/dist-packages/reviewboard/accounts/decorators.py", 
line 23, in _check
return login_required(view_func)(*args, **kwargs)
  File 
"/usr/local/lib/python2.7/dist-packages/django/contrib/auth/decorators.py", 
line 22, in _wrapped_view
return view_func(request, *args, **kwargs)
  File "/usr/local/lib/python2.7/dist-packages/django/utils/decorators.py", 
line 25, in bound_func
return func(self, *args2, **kwargs2)
  File 
"/usr/local/lib/python2.7/dist-packages/reviewboard/accounts/mixins.py", 
line 38, in dispatch
*args, **kwargs)
  File "/usr/local/lib/python2.7/dist-packages/django/utils/decorators.py", 
line 29, in _wrapper
return bound_func(*args, **kwargs)
  File 
"/usr/local/lib/python2.7/dist-packages/reviewboard/site/decorators.py", 
line 36, in _check
return view_func(request, local_site=local_site, *args, **kwargs)
  File "/usr/local/lib/python2.7/dist-packages/django/utils/decorators.py", 
line 25, in bound_func
return func(self, *args2, **kwargs2)
  File "/usr/local/lib/python2.7/dist-packages/reviewboard/site/mixins.py", 
line 53, in dispatch
request, *args, **kwargs)
  File 
"/usr/local/lib/python2.7/dist-packages/djblets/views/generic/base.py", 
line 91, in dispatch
super(PrePostDispatchViewMixin, self).dispatch(*args, **kwargs)
  File 
"/usr/local/lib/python2.7/dist-packages/djblets/views/generic/etag.py", 
line 50, in dispatch
**kwargs)
  File 
"/usr/local/lib/python2.7/dist-packages/django/views/generic/base.py", line 
87, in dispatch
return handler(request, *args, **kwargs)
  File 
"/usr/local/lib/python2.7/dist-packages/django/views/generic/base.py", line 
154, in get
context = self.get_context_data(**kwargs)
  File 
"/usr/local/lib/python2.7/dist-packages/reviewboard/reviews/views.py", line 
663, in get_context_data
context.update(make_review_request_context(request, review_request))
  File 
"/usr/local/lib/python2.7/dist-packages/reviewboard/reviews/context.py", 
line 105, in make_review_request_context
upload_diff_form = UploadDiffForm(review_request, request=request)
  File 
"/usr/local/lib/python2.7/dist-packages/reviewboard/reviews/forms.py", line 
129, in __init__
data, *args, **kwargs)
  File 
"/usr/local/lib/python2.7/dist-packages/reviewboard/diffviewer/forms.py", 
line 41, in __init__
if self.repository.get_scmtool().diffs_use_absolute_paths:
  File 
"/usr/local/lib/python2.7/dist-packages/reviewboard/scmtools/models.py", 
line 244, in get_scmtool
return self.scmtool_class(self)
  File 
"/usr/local/lib/python2.7/dist-packages/reviewboard/scmtools/svn/__init__.py", 
line 69, in __init__
credentials = repository.get_credentials()
  File 
"/usr/local/lib/python2.7/dist-packages/reviewboard/scmtools/models.py", 
line 326, in get_credentials
password = self.password
  File 
"/usr/local/lib/python2.7/dist-packages/reviewboard/scmtools/models.py", 
line 220, in _get_password
password = decrypt_password(password).decode('utf-8')
  File "/usr/lib/python2.7/encodings/utf_8.py", line 16, in decode
return codecs.utf_8_decode(input, errors, True)
UnicodeDecodeError: 'utf8' codec can't decode byte 0x87 in position 1: 
invalid start byte


The same review can be opened without any errors using Review Board 2.5.16 
so I am assuming there is some kind of Unicode Decoding issue with 3.0 RC1.
I'm not sure if this information will help, but all existing reviews result 
in the same error when opened.
All reviews contain a mixture of English and Japanese characters.

-- 
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 

Re: cipher AES in CFB8 mode is not supported by this backend

[Daniel Kuecker]

Fixed it ... cryptography==2.0 needed to be upgraded to cryptography==2.0.3.

WHEW that was stressful!

On Monday, August 28, 2017 at 12:55:39 PM UTC-5, Daniel Kuecker wrote:
>
> I'm having an issue updating the user and passowrd for one of our 
> repositories via the admin page.  When I enter the info and hit save, I get 
> the following error:
>
> 2017-08-28 17:49:57,696 - ERROR - None - 888 - 
> /admin/db/scmtools/repository/1/ - djblets.log.middleware - Exception 
> thrown for user 88 at http:///admin/db/scmtools/repository/1/
>
> cipher AES in CFB8 mode is not supported by this backend.
> Traceback (most recent call last):
>   File "/usr/lib/python2.7/site-packages/django/core/handlers/base.py", 
> line 112, in get_response
> response = wrapped_callback(request, *callback_args, **callback_kwargs)
>   File "/usr/lib/python2.7/site-packages/django/contrib/admin/options.py", 
> line 465, in wrapper
> return self.admin_site.admin_view(view)(*args, **kwargs)
>   File "/usr/lib/python2.7/site-packages/django/utils/decorators.py", line 
> 99, in _wrapped_view
> response = view_func(request, *args, **kwargs)
>   File 
> "/usr/lib/python2.7/site-packages/django/views/decorators/cache.py", line 
> 52, in _wrapped_view_func
> response = view_func(request, *args, **kwargs)
>   File "/usr/lib/python2.7/site-packages/django/contrib/admin/sites.py", 
> line 198, in inner
> return view(request, *args, **kwargs)
>   File "/usr/lib/python2.7/site-packages/django/utils/decorators.py", line 
> 29, in _wrapper
> return bound_func(*args, **kwargs)
>   File "/usr/lib/python2.7/site-packages/django/utils/decorators.py", line 
> 99, in _wrapped_view
> response = view_func(request, *args, **kwargs)
>   File "/usr/lib/python2.7/site-packages/django/utils/decorators.py", line 
> 25, in bound_func
> return func(self, *args2, **kwargs2)
>   File "/usr/lib/python2.7/site-packages/django/db/transaction.py", line 
> 371, in inner
> return func(*args, **kwargs)
>   File "/usr/lib/python2.7/site-packages/django/contrib/admin/options.py", 
> line 1246, in change_view
> new_object = self.save_form(request, form, change=True)
>   File "/usr/lib/python2.7/site-packages/django/contrib/admin/options.py", 
> line 887, in save_form
> return form.save(commit=False)
>   File "/usr/lib/python2.7/site-packages/reviewboard/scmtools/forms.py", 
> line 1181, in save
> repository.password = self.cleaned_data['password'] or ''
>   File "/usr/lib/python2.7/site-packages/reviewboard/scmtools/models.py", 
> line 192, in _set_password
> encrypt_password(value.encode('utf-8')))
>   File 
> "/usr/lib/python2.7/site-packages/reviewboard/scmtools/crypto_utils.py", 
> line 150, in encrypt_password
> return base64.b64encode(aes_encrypt(password, key=key))
>   File 
> "/usr/lib/python2.7/site-packages/reviewboard/scmtools/crypto_utils.py", 
> line 88, in aes_encrypt
> encryptor = cipher.encryptor()
>   File 
> "/usr/lib64/python2.7/site-packages/cryptography/hazmat/primitives/ciphers/base.py",
>  
> line 104, in encryptor
> self.algorithm, self.mode
>   File 
> "/usr/lib64/python2.7/site-packages/cryptography/hazmat/backends/multibackend.py",
>  
> line 53, in create_symmetric_encryption_ctx
> _Reasons.UNSUPPORTED_CIPHER
> UnsupportedAlgorithm: cipher AES in CFB8 mode is not supported by this 
> backend.
>
>
> I'm running on RHEL 7.
>

-- 
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.

Re: cipher AES in CFB8 mode is not supported by this backend

[Daniel Kuecker]

My Pip freeze contents:

appdirs==1.4.2
asn1crypto==0.22.0
backports.ssl-match-hostname==3.4.0.2
bcrypt==3.1.3
cffi==1.10.0
chardet==2.2.1
configobj==4.7.2
configshell-fb==1.1.18
cryptography==2.0
decorator==3.4.0
Django==1.6.11.6
django-evolution==0.7.7
django-haystack==2.4.1
django-multiselectfield==0.1.7
django-pipeline==1.3.27
Djblets==0.9.8
dnspython==1.15.0
docutils==0.13.1
ecdsa==0.13
enum34==1.1.6
ethtool==0.8
feedparser==5.2.1
flup==1.0.2
futures==3.1.1
idna==2.5
iniparse==0.4
iotop==0.6
ipaddress==1.0.18
jabber.py==0.3.post1
javapackages==1.0.0
jsonpointer==1.9
kerberos==1.1
kitchen==1.1.1
kmod==0.1
langtable==0.0.31
lxml==3.2.1
M2Crypto==0.21.1
Markdown==2.4.1
mechanize==0.2.5
mercurial==2.6.2
mimeparse==0.1.3
MySQL-python==1.2.3
netaddr==0.7.5
nose==1.3.0
olefile==0.44
p4python==2017.1.1526044
packaging==16.8
paramiko==2.2.1
pciutils==1.7.3
pcp==1.0
perf==0.1
Pillow==4.2.1
pillowfight==0.3
ply==3.4
psutil==2.2.1
psycopg2==2.5.1
publicsuffix==1.1.0
pyasn1==0.2.3
pycparser==2.18
pycrypto==2.6.1
pycurl==7.19.0
Pygments==2.2.0
pygobject==3.14.0
pygpgme==0.3
pyliblzma==0.5.3
PyNaCl==1.1.2
pyOpenSSL==0.13.1
pyparsing==2.1.10
python-augeas==0.5.0
python-dateutil==2.6.1
python-dmidecode==3.10.13
python-ldap==2.4.15
python-memcached==1.58
python-mimeparse==1.6.0
python-nss==0.16.0
python-ntlm==1.1.0
python-yubico==1.2.3
pytz==2017.2
pyudev==0.15
pyusb==1.0.0b1
pyxattr==0.5.1
qrcode==5.0.1
rbmotd==1.0
rbnotefield==1.0.1
rbseverity==0.5.1
rbstopwatch==1.0.1
recaptcha-client==1.0.6
ReviewBoard==2.5.13.1
ReviewBoardPowerPack==1.4.2
rhnlib==2.5.77
rtslib-fb==2.1.57
simplejson==3.10.0
six==1.10.0
slip==0.4.0
slip.dbus==0.4.0
SSSDConfig==1.13.0
targetcli-fb===2.1.fb41
templateconfig==3.7
urlgrabber==3.10
urwid==1.1.1
Whoosh==2.7.4
yum-langpacks==0.4.2
yum-metadata-parser==1.1.4


On Monday, August 28, 2017 at 12:55:39 PM UTC-5, Daniel Kuecker wrote:
>
> I'm having an issue updating the user and passowrd for one of our 
> repositories via the admin page.  When I enter the info and hit save, I get 
> the following error:
>
> 2017-08-28 17:49:57,696 - ERROR - None - 888 - 
> /admin/db/scmtools/repository/1/ - djblets.log.middleware - Exception 
> thrown for user 88 at http:///admin/db/scmtools/repository/1/
>
> cipher AES in CFB8 mode is not supported by this backend.
> Traceback (most recent call last):
>   File "/usr/lib/python2.7/site-packages/django/core/handlers/base.py", 
> line 112, in get_response
> response = wrapped_callback(request, *callback_args, **callback_kwargs)
>   File "/usr/lib/python2.7/site-packages/django/contrib/admin/options.py", 
> line 465, in wrapper
> return self.admin_site.admin_view(view)(*args, **kwargs)
>   File "/usr/lib/python2.7/site-packages/django/utils/decorators.py", line 
> 99, in _wrapped_view
> response = view_func(request, *args, **kwargs)
>   File 
> "/usr/lib/python2.7/site-packages/django/views/decorators/cache.py", line 
> 52, in _wrapped_view_func
> response = view_func(request, *args, **kwargs)
>   File "/usr/lib/python2.7/site-packages/django/contrib/admin/sites.py", 
> line 198, in inner
> return view(request, *args, **kwargs)
>   File "/usr/lib/python2.7/site-packages/django/utils/decorators.py", line 
> 29, in _wrapper
> return bound_func(*args, **kwargs)
>   File "/usr/lib/python2.7/site-packages/django/utils/decorators.py", line 
> 99, in _wrapped_view
> response = view_func(request, *args, **kwargs)
>   File "/usr/lib/python2.7/site-packages/django/utils/decorators.py", line 
> 25, in bound_func
> return func(self, *args2, **kwargs2)
>   File "/usr/lib/python2.7/site-packages/django/db/transaction.py", line 
> 371, in inner
> return func(*args, **kwargs)
>   File "/usr/lib/python2.7/site-packages/django/contrib/admin/options.py", 
> line 1246, in change_view
> new_object = self.save_form(request, form, change=True)
>   File "/usr/lib/python2.7/site-packages/django/contrib/admin/options.py", 
> line 887, in save_form
> return form.save(commit=False)
>   File "/usr/lib/python2.7/site-packages/reviewboard/scmtools/forms.py", 
> line 1181, in save
> repository.password = self.cleaned_data['password'] or ''
>   File "/usr/lib/python2.7/site-packages/reviewboard/scmtools/models.py", 
> line 192, in _set_password
> encrypt_password(value.encode('utf-8')))
>   File 
> "/usr/lib/python2.7/site-packages/reviewboard/scmtools/crypto_utils.py", 
> line 150, in encrypt_password
> return base64.b64encode(aes_encrypt(password, key=key))
>   File 
> "/usr/lib/python2.7/site-packages/reviewboard/scmtools/crypto_utils.py", 
> line 88, in aes_encrypt
> encryp

cipher AES in CFB8 mode is not supported by this backend

[Daniel Kuecker]

I'm having an issue updating the user and passowrd for one of our 
repositories via the admin page.  When I enter the info and hit save, I get 
the following error:

2017-08-28 17:49:57,696 - ERROR - None - 888 - 
/admin/db/scmtools/repository/1/ - djblets.log.middleware - Exception 
thrown for user 88 at http:///admin/db/scmtools/repository/1/

cipher AES in CFB8 mode is not supported by this backend.
Traceback (most recent call last):
  File "/usr/lib/python2.7/site-packages/django/core/handlers/base.py", 
line 112, in get_response
response = wrapped_callback(request, *callback_args, **callback_kwargs)
  File "/usr/lib/python2.7/site-packages/django/contrib/admin/options.py", 
line 465, in wrapper
return self.admin_site.admin_view(view)(*args, **kwargs)
  File "/usr/lib/python2.7/site-packages/django/utils/decorators.py", line 
99, in _wrapped_view
response = view_func(request, *args, **kwargs)
  File "/usr/lib/python2.7/site-packages/django/views/decorators/cache.py", 
line 52, in _wrapped_view_func
response = view_func(request, *args, **kwargs)
  File "/usr/lib/python2.7/site-packages/django/contrib/admin/sites.py", 
line 198, in inner
return view(request, *args, **kwargs)
  File "/usr/lib/python2.7/site-packages/django/utils/decorators.py", line 
29, in _wrapper
return bound_func(*args, **kwargs)
  File "/usr/lib/python2.7/site-packages/django/utils/decorators.py", line 
99, in _wrapped_view
response = view_func(request, *args, **kwargs)
  File "/usr/lib/python2.7/site-packages/django/utils/decorators.py", line 
25, in bound_func
return func(self, *args2, **kwargs2)
  File "/usr/lib/python2.7/site-packages/django/db/transaction.py", line 
371, in inner
return func(*args, **kwargs)
  File "/usr/lib/python2.7/site-packages/django/contrib/admin/options.py", 
line 1246, in change_view
new_object = self.save_form(request, form, change=True)
  File "/usr/lib/python2.7/site-packages/django/contrib/admin/options.py", 
line 887, in save_form
return form.save(commit=False)
  File "/usr/lib/python2.7/site-packages/reviewboard/scmtools/forms.py", 
line 1181, in save
repository.password = self.cleaned_data['password'] or ''
  File "/usr/lib/python2.7/site-packages/reviewboard/scmtools/models.py", 
line 192, in _set_password
encrypt_password(value.encode('utf-8')))
  File 
"/usr/lib/python2.7/site-packages/reviewboard/scmtools/crypto_utils.py", 
line 150, in encrypt_password
return base64.b64encode(aes_encrypt(password, key=key))
  File 
"/usr/lib/python2.7/site-packages/reviewboard/scmtools/crypto_utils.py", 
line 88, in aes_encrypt
encryptor = cipher.encryptor()
  File 
"/usr/lib64/python2.7/site-packages/cryptography/hazmat/primitives/ciphers/base.py",
 
line 104, in encryptor
self.algorithm, self.mode
  File 
"/usr/lib64/python2.7/site-packages/cryptography/hazmat/backends/multibackend.py",
 
line 53, in create_symmetric_encryption_ctx
_Reasons.UNSUPPORTED_CIPHER
UnsupportedAlgorithm: cipher AES in CFB8 mode is not supported by this 
backend.


I'm running on RHEL 7.

-- 
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.

Re: Review Board Ticket #4533: Interdiffs are not working on some files

[Daniel Kuecker]

--
To reply, visit https://hellosplat.com/s/beanbag/tickets/4533/
--

New update by besperon
For Beanbag, Inc. > Review Board > Ticket #4533


Reply:

Any updates on this?  My users are very frustrated...

-- 
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: Error: too many trys to get lock

[Daniel Kuecker]

Thanks for the quick reply Christian!  I see apache did not have write
access to the data folder.  I enabled it and will watch for future errors.

Thanks!

--DK

On Tue, May 23, 2017 at 3:17 PM, Christian Hammond <christ...@beanbaginc.com
> wrote:

> Hi Daniel,
>
> I haven't, but the first thing I'd check is whether that directory and its
> contents are writable by the web server.
>
> There's going to be a new Review Board release out by the end of the week
> that, amongst other things, makes some large improvements (and has better
> error handling) for Perforce authentication. I don't know if it would help
> much in this situation, but possibly.
>
> Christian
>
> --
> Christian Hammond
> President/CEO of Beanbag <https://www.beanbaginc.com/>
> Makers of Review Board <https://www.reviewboard.org/>
>
> On Tue, May 23, 2017 at 1:16 PM, Daniel Kuecker <dan...@kuecker.net>
> wrote:
>
>> I'm getting this error on our production RB server.  We are using P4 with
>> LDAP authentication.
>>
>> SCMError: [P4#run] Errors during command execution( "p4 login" )
>>
>>
>>  [Error]: "Fatal client error; disconnecting!\nOperation
>> 'client-SetPassword' failed.\nToo many trys to get lock
>> /opt/apps/reviewboard/rbinstall/review-board..com/
>> data/.p4tickets.lck."
>>
>> Has anyone seen a similar error?
>>
>> --DK
>>
>> --
>> 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.
>>
>
> --
> 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 "reviewboard" group.
> To unsubscribe from this topic, visit https://groups.google.com/d/
> topic/reviewboard/UjzdTSfwh34/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 
"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.

Error: too many trys to get lock

[Daniel Kuecker]

I'm getting this error on our production RB server.  We are using P4 with 
LDAP authentication.

SCMError: [P4#run] Errors during command execution( "p4 login" )


 [Error]: "Fatal client error; disconnecting!\nOperation 
'client-SetPassword' failed.\nToo many trys to get lock 
/opt/apps/reviewboard/rbinstall/review-board..com/data/.p4tickets.lck."

Has anyone seen a similar error?

--DK

-- 
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.

Repository type is empty for GitHub Enterprise

[Pablo Daniel Dionisi]

I did the default installation (on windows with bitnami distribution) and 
I'm trying to add a GitHub enterprise repository buy the Repository 
Type comboBox is empty.

Could be an installation error?

Regards,
Pablo.

-- 
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.

Re: update_index always indexes the users class

[Daniel Kuecker]

Sure thing. 

Python: 2.7.5.  
I used easy_install to install RB.

On Monday, May 1, 2017 at 4:28:14 PM UTC-5, Christian Hammond wrote:
>
> Actually, rather than the Elasticsearch patch, I'd like you to try a 
> couple patches to the search indexing code. Turns out it was more 
> inefficient than I had thought. Alternatively, I could send you a build. 
> Just need to know the Python version and whether you used easy_install or 
> pip to install Review Board (or some other method).
>
> Christian
>
> -- 
> Christian Hammond
> President/CEO of Beanbag <https://www.beanbaginc.com/>
> Makers of Review Board <https://www.reviewboard.org/>
>
> On Mon, May 1, 2017 at 1:28 PM, Christian Hammond <chri...@beanbaginc.com 
> > wrote:
>
>> Sure thing. I realized though that out of the box, this won't work 
>> without a patch to Review Board (the proper Elasticsearch support has only 
>> been thoroughly tested in 3.0). How comfortable do you feel applying 
>> patches?
>>
>> Christian
>>
>>
>> -- 
>> Christian Hammond
>> President/CEO of Beanbag <https://www.beanbaginc.com/>
>> Makers of Review Board <https://www.reviewboard.org/>
>>
>> On Mon, May 1, 2017 at 1:17 PM, Daniel Kuecker <dan...@kuecker.net 
>> > wrote:
>>
>>> Thanks for the quick reply Christian!
>>>
>>> I would like the info on configuring Elasticsearch if you don't mind.  
>>>
>>> --DK
>>>
>>> On Monday, May 1, 2017 at 3:12:06 PM UTC-5, Christian Hammond wrote:
>>>>
>>>> Hi Daniel,
>>>>
>>>> I believe this happens because unlike review requests, we have no way 
>>>> of determining with the search indexing stuff we use if a user's 
>>>> information is "new" to the index. Basically, we have no "last updated" 
>>>> for 
>>>> the users.
>>>>
>>>> I do see an area in the search index for users that isn't as optimal, 
>>>> so we can probably shave down the number of queries a fair amount, which 
>>>> will help.
>>>>
>>>> I assume you're using the default search backend of Whoosh? If so, at 
>>>> your scale, it might be time to install Elasticsearch. We can give you 
>>>> some 
>>>> info on how to configure that in settings_local.py (the upcoming Review 
>>>> Board 3.0 makes this very easy -- just select it as your search backend 
>>>> and 
>>>> fill in fields). Indexing with Elasticsearch is more efficient for large 
>>>> amounts of data.
>>>>
>>>> Christian
>>>>
>>>> -- 
>>>> Christian Hammond
>>>> President/CEO of Beanbag <https://www.beanbaginc.com/>
>>>> Makers of Review Board <https://www.reviewboard.org/>
>>>>
>>>> On Mon, May 1, 2017 at 12:59 PM, Daniel Kuecker <dan...@kuecker.net> 
>>>> wrote:
>>>>
>>>>> I have a fresh 2.5.9 install with a db taken from a backup of another 
>>>>> 2.5.9 install (basically a backup of my production instance).
>>>>>
>>>>> On the production instance, I am experiencing 100% cpu usage every 
>>>>> time the update_index cron job runs causing my users to complain about 
>>>>> responsiveness of the site.
>>>>>
>>>>> On my production system, the cron job shows it is indexing all the 
>>>>> records in the user table while only the new records in the review 
>>>>> requests 
>>>>> table.
>>>>>
>>>>> For debugging, I restored the production db backup to my dev machine 
>>>>> and set up a 2.5.9 instance pointing to it.  When update_index runs on 
>>>>> the 
>>>>> dev box (given the "-v 3" arg), I get the following warning: 
>>>>>
>>>>>>
>>>>>> WARNING:py.warnings:/usr/lib/python2.7/site-packages/haystack/indexes.py:167:
>>>>>>  
>>>>>> UserWarning: No updated date field found for 'User' - not restricting by 
>>>>>> age.
>>>>>>   warnings.warn(update_field_msg)
>>>>>> Indexing 1971 users
>>>>>>   indexed 1 - 1000 of 1971 (by 16819).
>>>>>>   indexed 1001 - 1971 of 1971 (by 16819).
>>>>>
>>>>>
>>>>> It then proceeds to index the entire user table and only the changed 
>>>>> review requests just as my pro

Re: update_index always indexes the users class

[Daniel Kuecker]

Thanks for the quick reply Christian!

I would like the info on configuring Elasticsearch if you don't mind.  

--DK

On Monday, May 1, 2017 at 3:12:06 PM UTC-5, Christian Hammond wrote:
>
> Hi Daniel,
>
> I believe this happens because unlike review requests, we have no way of 
> determining with the search indexing stuff we use if a user's information 
> is "new" to the index. Basically, we have no "last updated" for the users.
>
> I do see an area in the search index for users that isn't as optimal, so 
> we can probably shave down the number of queries a fair amount, which will 
> help.
>
> I assume you're using the default search backend of Whoosh? If so, at your 
> scale, it might be time to install Elasticsearch. We can give you some info 
> on how to configure that in settings_local.py (the upcoming Review Board 
> 3.0 makes this very easy -- just select it as your search backend and fill 
> in fields). Indexing with Elasticsearch is more efficient for large amounts 
> of data.
>
> Christian
>
> -- 
> Christian Hammond
> President/CEO of Beanbag <https://www.beanbaginc.com/>
> Makers of Review Board <https://www.reviewboard.org/>
>
> On Mon, May 1, 2017 at 12:59 PM, Daniel Kuecker <dan...@kuecker.net 
> > wrote:
>
>> I have a fresh 2.5.9 install with a db taken from a backup of another 
>> 2.5.9 install (basically a backup of my production instance).
>>
>> On the production instance, I am experiencing 100% cpu usage every time 
>> the update_index cron job runs causing my users to complain about 
>> responsiveness of the site.
>>
>> On my production system, the cron job shows it is indexing all the 
>> records in the user table while only the new records in the review requests 
>> table.
>>
>> For debugging, I restored the production db backup to my dev machine and 
>> set up a 2.5.9 instance pointing to it.  When update_index runs on the dev 
>> box (given the "-v 3" arg), I get the following warning: 
>>
>>>
>>> WARNING:py.warnings:/usr/lib/python2.7/site-packages/haystack/indexes.py:167:
>>>  
>>> UserWarning: No updated date field found for 'User' - not restricting by 
>>> age.
>>>   warnings.warn(update_field_msg)
>>> Indexing 1971 users
>>>   indexed 1 - 1000 of 1971 (by 16819).
>>>   indexed 1001 - 1971 of 1971 (by 16819).
>>
>>
>> It then proceeds to index the entire user table and only the changed 
>> review requests just as my production instance does.
>>
>> I have run the rebuild_index command on both systems.
>>
>> Am I missing something?  I have the cron job set to run ever 1/2 hour and 
>> am about to have to turn search indexing off to ensure the site is 
>> responsive.
>>
>> --DK
>>
>>
>>
>> -- 
>> 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...@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 
"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.

update_index always indexes the users class

[Daniel Kuecker]

I have a fresh 2.5.9 install with a db taken from a backup of another 2.5.9 
install (basically a backup of my production instance).

On the production instance, I am experiencing 100% cpu usage every time the 
update_index cron job runs causing my users to complain about 
responsiveness of the site.

On my production system, the cron job shows it is indexing all the records 
in the user table while only the new records in the review requests table.

For debugging, I restored the production db backup to my dev machine and 
set up a 2.5.9 instance pointing to it.  When update_index runs on the dev 
box (given the "-v 3" arg), I get the following warning: 

>
> WARNING:py.warnings:/usr/lib/python2.7/site-packages/haystack/indexes.py:167: 
> UserWarning: No updated date field found for 'User' - not restricting by 
> age.
>   warnings.warn(update_field_msg)
> Indexing 1971 users
>   indexed 1 - 1000 of 1971 (by 16819).
>   indexed 1001 - 1971 of 1971 (by 16819).


It then proceeds to index the entire user table and only the changed review 
requests just as my production instance does.

I have run the rebuild_index command on both systems.

Am I missing something?  I have the cron job set to run ever 1/2 hour and 
am about to have to turn search indexing off to ensure the site is 
responsive.

--DK



-- 
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.

Review Board Ticket #4541: Going from < 300 character summary field content to 300 character content produces clipping in the "Review request changed" diff.

[Daniel Kuecker]

--
To reply, visit https://hellosplat.com/s/beanbag/tickets/4541/
--

New ticket #4541 by dkuecker
For Beanbag, Inc. > Review Board

Status: New
Tags: Priority:Medium, Type:Defect

File attachments:

 * rbdiffclipping.png
   

 * rbdiffclipping2.png
   



--
Going from < 300 character summary field content to 300 character content 
produces clipping in the "Review request changed" diff.
==

# What version are you running?
2.5.9


# What's the URL of the page containing the problem?
(internal page)  http://review-board..com/r/179573/

# What steps will reproduce the problem?
1. Create and submit a review request and enter a summary with less than 300 
characters.
2. Change the summary to include 300 characters and submit changes


# What is the expected output? What do you see instead?
Expected the full change to be displayed in the "Review request changed" diff.
Instead, I see the diff clipped at 273 characters


# What operating system are you using? What browser?
Chrome

# Please provide any additional information below.
Desktop resolution = 1920x1080

--

-- 
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 #4398: Make date of review / change description / status an anchor link to that item

[Daniel Kuecker]

--
To reply, visit https://hellosplat.com/s/beanbag/tickets/4398/
--

New update by mconley
For Beanbag, Inc. > Review Board > Ticket #4398


Reply:

Can we get this change added to 2.5.10?  My users are asking for it.

-- 
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.

diffviewer_filediffdata table after 1.7.27 to 2.5.9 upgrade

[Daniel Kuecker]

I recently moved our RB 1.7.27 instance to a 2.5.9 instance.  I restored a 
backup of the 1.7.27 db to the 2.5.9 instance and ran rb-site install and 
waited and waited.  Everything converted successfully and we are now 
running 2.5.9 (our users love it! yay!)

NOTE:  Before the restore, to avoid the InnoDB conversion evolution, I 
manually edited the sql dump, changed the table engines to InnoDB and the 
charset to UTF8 for all tables. 

I noticed I still have a diffviewer_filediffdata table that is the same 
size as the original but has zero records.  Is that table needed anymore? 
 I'd like to get rid of it as it is 28GB in size.

Thanks!

--DK

-- 
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.

Re: Review Board Ticket #4479: I sometimes want to view only one side of the diff. I find larger fonts more readable, but run out of screen width when viewing both sides of the diff.

[Daniel Lipovetsky]

--
To reply, visit https://hellosplat.com/s/beanbag/tickets/4479/
--

New update by daniel
For Beanbag, Inc. > Review Board > Ticket #4479


Reply:

Ticket 4159 is asking for a different diff representation (that happens to 
use a single-column view), whereas I am asking to temporarily hide one column 
of the existing two-column diff.

-- 
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.

Review Board Ticket #4479: I sometimes want to view only one side of the diff. I find larger fonts more readable, but run out of screen width when viewing both sides of the diff.

[Daniel Lipovetsky]

--
To reply, visit https://hellosplat.com/s/beanbag/tickets/4479/
--

New ticket #4479 by daniel
For Beanbag, Inc. > Review Board

Status: New
Tags: Priority:Medium, Type:Enhancement


--
I sometimes want to view only one side of the diff. I find larger fonts more 
readable, but run out of screen width when viewing both sides of the diff.
==

# What version are you running?

rbcommons.com

# What's the URL of the page this enhancement relates to, if any?

https://rbcommons.com/s/platform9/r//diff/

# Describe the enhancement and the motivation for it.

I sometimes want to view only one side of the diff. I find larger fonts more 
readable, but run out of screen width when viewing both sides of the diff. As a 
result, the text in both sides of the diff is wrapped somewhere between 40-60 
columns and becomes difficult to read.

# What operating system are you using? What browser?

OS X 10.11.6, Chrome

# Please provide any additional information below.

I have this issue when using two different displays: Macbook Air 13" (16:10 
ratio) display, and 19" 1280x1024 (4:3 ratio).

--

-- 
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.

How big is your Review Board site?

[Daniel Kuecker]

With Review Board's recent 10th anniversary, I thought it would be 
interesting to compare the size of everyone's Review Board site.

We are sitting at 125,000 reviews, 2032 users and a 26GB database size on 
disk.

What's yours at?

-- 
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.

Re: Building RBTools 0.7.6 into a single-file exe for Windows

[Daniel Kuecker]

YES!  Thats a wonderful idea (the single shared install)!  Thanks for the
idea!  I think it will work for us!

--Daniel

On Fri, Sep 23, 2016 at 8:26 AM, Christian Hammond <christ...@beanbaginc.com
> wrote:

> Sorry, I don't have any pointers here. Tools like py2exe drastically
> changes the way Python projects execute, which can end up causing a large
> number of tricky to diagnose issues. Having done a lot of
> py2exe/pyinstaller work in the past, it's not a route I'd want to go again.
>
> While I'd accept reasonable patches for things RBTools is doing, it's
> important to note that some things simply will not work, and those are
> things we must ship with. Examples would be any commands that reference a
> shipped data file, or otherwise need to deal with something relative to a
> module path.
>
> I understand the desire to have a single .exe you can drop on development
> machines, but am not sold on this being the way to go. What a lot of
> companies do is have a single shared install of RBTools in some common
> directory on a file server, and have that directory added to the path. Some
> use a wrapper script to set the PYTHON_PATH and such for that. This gives
> you central management of RBTools, usage of the RBTools API, and wouldn't
> require the kinds of work that would be needed to get things in shape for
> tools like py2exe to work.
>
> Christian
>
> --
> Christian Hammond
> President/CEO of Beanbag <https://www.beanbaginc.com/>
> Makers of Review Board <https://www.reviewboard.org/>
>
> On Fri, Sep 23, 2016 at 6:22 AM, Daniel Kuecker <dan...@kuecker.net>
> wrote:
>
>> Any pointers?
>>
>> On Thu, Sep 22, 2016 at 10:54 PM, john levin <smart.le...@gmail.com>
>> wrote:
>>
>>> I did.. and i was successful..
>>>
>>> On Sep 23, 2016 12:30 AM, "Daniel Kuecker" <dan...@kuecker.net> wrote:
>>>
>>>> Has anyone been successful building RBTools 0.7.6 into a single-file
>>>> exe for distribution on windows?
>>>>
>>>> I'm trying to wrap rbt post functionality into an exe file and both
>>>> py2exe and pyinstaller built exes are having issues when pkg_resources
>>>> tries to resolve the rbtools package.
>>>>
>>>> The exe builds fine, but when I run it, I get a
>>>> "pkg_resources.distributionnotfound: the 'rbtools' distribution was
>>>> not found and is required by the application" error.
>>>>
>>>> I've tried listing rbtools as hidden imports and importing an egg
>>>> (although the egg import was never successful), to no avail.
>>>>
>>>> Thanks in advance!
>>>>
>>>> --Daniel
>>>>
>>>>
>>>>
>>>>
>>>> --
>>>> 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.
>>>>
>>> --
>>> 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.
>>>
>>
>> --
>> 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+unsub

Re: Building RBTools 0.7.6 into a single-file exe for Windows

[Daniel Kuecker]

Hey Christian,

Our clients would rather not install an app on their development machines.
We have been using an older version of RBTools, that we made a custom
post-review for, and built an exe.  The exe is dropped on the client
machine (among many other things) as a tool via a scripted setup script we
run and is called from other build-related scripts.

I only really need post and patch functionality and we hard-code the RB url.

Thanks for the response!!!

--Daniel


On Fri, Sep 23, 2016 at 12:05 AM, Christian Hammond <
christ...@beanbaginc.com> wrote:

> Out of curiosity, why not use the RBTools installer we already provide?
>
> The single .exe route will have limitations. I've done a lot of
> py2exe/pyinstaller work in the past for other projects, and it's not a
> route I'd recommend if you can avoid it, at least for more complex projects.
>
> Christian
>
> --
> Christian Hammond
> President/CEO of Beanbag <https://www.beanbaginc.com/>
> Makers of Review Board <https://www.reviewboard.org/>
>
> On Thu, Sep 22, 2016 at 11:59 AM, Daniel Kuecker <dan...@kuecker.net>
> wrote:
>
>> Has anyone been successful building RBTools 0.7.6 into a single-file exe
>> for distribution on windows?
>>
>> I'm trying to wrap rbt post functionality into an exe file and both
>> py2exe and pyinstaller built exes are having issues when pkg_resources
>> tries to resolve the rbtools package.
>>
>> The exe builds fine, but when I run it, I get a
>> "pkg_resources.distributionnotfound: the 'rbtools' distribution was not
>> found and is required by the application" error.
>>
>> I've tried listing rbtools as hidden imports and importing an egg
>> (although the egg import was never successful), to no avail.
>>
>> Thanks in advance!
>>
>> --Daniel
>>
>>
>>
>>
>> --
>> 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.
>>
>
> --
> 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.
>

-- 
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.

Re: Building RBTools 0.7.6 into a single-file exe for Windows

[Daniel Kuecker]

Any pointers?

On Thu, Sep 22, 2016 at 10:54 PM, john levin <smart.le...@gmail.com> wrote:

> I did.. and i was successful..
>
> On Sep 23, 2016 12:30 AM, "Daniel Kuecker" <dan...@kuecker.net> wrote:
>
>> Has anyone been successful building RBTools 0.7.6 into a single-file exe
>> for distribution on windows?
>>
>> I'm trying to wrap rbt post functionality into an exe file and both
>> py2exe and pyinstaller built exes are having issues when pkg_resources
>> tries to resolve the rbtools package.
>>
>> The exe builds fine, but when I run it, I get a
>> "pkg_resources.distributionnotfound: the 'rbtools' distribution was not
>> found and is required by the application" error.
>>
>> I've tried listing rbtools as hidden imports and importing an egg
>> (although the egg import was never successful), to no avail.
>>
>> Thanks in advance!
>>
>> --Daniel
>>
>>
>>
>>
>> --
>> 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.
>>
> --
> 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.
>

-- 
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.

Re: rbt patch "file not found"

[Daniel Kuecker]

Perforce repo and the diff was generated with and older version of RBTools
(post-review era.).

On Fri, Sep 23, 2016 at 12:06 AM Christian Hammond <christ...@beanbaginc.com>
wrote:

> What type of repository is this, and how was the diff generated/uploaded?
>
> Christian
>
> --
> Christian Hammond
> President/CEO of Beanbag <https://www.beanbaginc.com/>
> Makers of Review Board <https://www.reviewboard.org/>
>
> On Thu, Sep 22, 2016 at 6:55 AM, Daniel Kuecker <dan...@kuecker.net>
> wrote:
>
>> Oops, forgot to mention I am running this on Windows 10.
>>
>> RBTools:  0.7.6
>> RB: tried on versions 1.7.27 and 2.5.6.1
>>
>>
>> On Thursday, September 22, 2016 at 8:50:04 AM UTC-5, Daniel Kuecker wrote:
>>>
>>> Running "rbt patch " command results in a file not found
>>> error for every file in the request.  I verified the files are synced to my
>>> local drive.  Any ideas?
>>>
>>> Command line info (path info removed for anonymity):
>>> C:\P4>rbt patch 127325
>>> Patch is being applied from request 127325 with diff revision 1.
>>> Failed to execute command: [u'patch', u'-i',
>>> u'c:\\users\\***\\appdata\\local\\temp\\tmpjq4zbu']
>>> can't find file to patch at input line 3
>>> Perhaps you should have used the -p or --strip option?
>>> The text leading up to this was:
>>> --
>>> |---
>>> ///***/setupscripts/trunk/16.0/source//_common_impl_details.rb
>>>
>>> ///***/setupscripts/trunk/16.0/source//_common_impl_details.rb#6
>>> |+++
>>> ///***/setupscripts/trunk/16.0/source//_common_impl_details.rb
>>>   2016-02-18 09:21:42
>>> --
>>> File to patch:
>>> Skip this patch? [y]
>>> Skipping patch.
>>> 1 out of 1 hunk ignored
>>> missing header for unified diff at line 17 of patch
>>> can't find file to patch at input line 17
>>> Perhaps you should have used the -p or --strip option?
>>> The text leading up to this was:
>>> --
>>> |/setupscripts/trunk/16.0/source//_setup_common_impl_details.rb
>>>
>>> ///***/setupscripts/trunk/16.0/source//_setup_common_impl_details.rb#2
>>> |+++
>>> ///***/setupscripts/trunk/16.0/source//_setup_common_impl_details.rb
>>> 2016-02-18 09:21:43
>>> --
>>> File to patch:
>>> Skip this patch? [y]
>>> Skipping patch.
>>> 1 out of 1 hunk ignored
>>> missing header for unified diff at line 27 of patch
>>> can't find file to patch at input line 27
>>> Perhaps you should have used the -p or --strip option?
>>> The text leading up to this was:
>>> --
>>> |/***/setupscripts/trunk/16.0/source//_setup_impl_details.rb
>>>
>>>  
>>> ///***/setupscripts/trunk/16.0/source//_setup_impl_details.rb#2
>>> |+++
>>> ///***/setupscripts/trunk/16.0/source//_setup_impl_details.rb
>>>2016-02-18 09:21:43
>>> --
>>> File to patch:
>>> Skip this patch? [y]
>>> Skipping patch.
>>> 1 out of 1 hunk ignored
>>>
>>>
>>> --
>> 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.
>>
>
> --
> 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.
>

-- 
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.

Re: How to access the review board site from other system via browser?

[Daniel Kuecker]

Are you getting an error?

Did you copy your apache-wsgi.conf to /etc/httpd/conf.d/?

If you did that, you might need to enable httpd port passthru in the 
firewall.

On Thursday, September 22, 2016 at 9:19:22 AM UTC-5, P P Gobind Chandra 
Praharaj wrote:
>
> I have set-up all prerequisites and all the installation required for 
> local reviewboard along with rb-site, server is apache httpd with mod_wsi. 
>
>
> Can any one help me on how to access the review bpoard which I have 
> installed locally from different systems via web browser.
>
>
> Below are the steps detailed:
>
>
>
>1. yum -y install zlib zlib-devel 
>2. yum -y install openssl-devel 
>3. yum -y install memcached 
>4. yum -y install python-devel
>5.  yum -y install python-setuptools 
>6. yum -y install patch easy_install
>7.  mysql-python yum -y install git-core
>8. easy_install ReviewBoard 
>9. *rb-site install /var/www/ReviewBoard*
>
>
> *Steps followed during Installing rb-site*
>
>
>-  Domain name localhost:80
>-  Root Path : reviews
>- And other configurations are default.
>
>
> All steps are executed successfully.
>
>
> But I do not know how can I access the review board over browser.
>
> Please help me?
>

-- 
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.

Building RBTools 0.7.6 into a single-file exe for Windows

[Daniel Kuecker]

Has anyone been successful building RBTools 0.7.6 into a single-file exe 
for distribution on windows? 

I'm trying to wrap rbt post functionality into an exe file and both py2exe 
and pyinstaller built exes are having issues when pkg_resources tries to 
resolve the rbtools package.

The exe builds fine, but when I run it, I get a 
"pkg_resources.distributionnotfound: the 'rbtools' distribution was not 
found and is required by the application" error.

I've tried listing rbtools as hidden imports and importing an egg (although 
the egg import was never successful), to no avail.

Thanks in advance!

--Daniel




-- 
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.

Re: rbt patch "file not found"

[Daniel Kuecker]

Oops, forgot to mention I am running this on Windows 10.

RBTools:  0.7.6
RB: tried on versions 1.7.27 and 2.5.6.1

On Thursday, September 22, 2016 at 8:50:04 AM UTC-5, Daniel Kuecker wrote:
>
> Running "rbt patch " command results in a file not found error 
> for every file in the request.  I verified the files are synced to my local 
> drive.  Any ideas?
>
> Command line info (path info removed for anonymity): 
> C:\P4>rbt patch 127325
> Patch is being applied from request 127325 with diff revision 1.
> Failed to execute command: [u'patch', u'-i', 
> u'c:\\users\\***\\appdata\\local\\temp\\tmpjq4zbu']
> can't find file to patch at input line 3
> Perhaps you should have used the -p or --strip option?
> The text leading up to this was:
> --
> |--- 
> ///***/setupscripts/trunk/16.0/source//_common_impl_details.rb
>  
>   
> ///***/setupscripts/trunk/16.0/source//_common_impl_details.rb#6
> |+++ 
> ///***/setupscripts/trunk/16.0/source//_common_impl_details.rb
>  
>   2016-02-18 09:21:42
> --
> File to patch:
> Skip this patch? [y]
> Skipping patch.
> 1 out of 1 hunk ignored
> missing header for unified diff at line 17 of patch
> can't find file to patch at input line 17
> Perhaps you should have used the -p or --strip option?
> The text leading up to this was:
> --
> |/setupscripts/trunk/16.0/source//_setup_common_impl_details.rb 
>   
> ///***/setupscripts/trunk/16.0/source//_setup_common_impl_details.rb#2
> |+++ 
> ///***/setupscripts/trunk/16.0/source//_setup_common_impl_details.rb
>  
> 2016-02-18 09:21:43
> --
> File to patch:
> Skip this patch? [y]
> Skipping patch.
> 1 out of 1 hunk ignored
> missing header for unified diff at line 27 of patch
> can't find file to patch at input line 27
> Perhaps you should have used the -p or --strip option?
> The text leading up to this was:
> --
> |/***/setupscripts/trunk/16.0/source//_setup_impl_details.rb   
>   
>  
> ///***/setupscripts/trunk/16.0/source//_setup_impl_details.rb#2
> |+++ 
> ///***/setupscripts/trunk/16.0/source//_setup_impl_details.rb 
>2016-02-18 09:21:43
> --
> File to patch:
> Skip this patch? [y]
> Skipping patch.
> 1 out of 1 hunk ignored
>
>
>

-- 
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.

rbt patch "file not found"

[Daniel Kuecker]

Running "rbt patch " command results in a file not found error 
for every file in the request.  I verified the files are synced to my local 
drive.  Any ideas?

Command line info (path info removed for anonymity): 
C:\P4>rbt patch 127325
Patch is being applied from request 127325 with diff revision 1.
Failed to execute command: [u'patch', u'-i', 
u'c:\\users\\***\\appdata\\local\\temp\\tmpjq4zbu']
can't find file to patch at input line 3
Perhaps you should have used the -p or --strip option?
The text leading up to this was:
--
|--- 
///***/setupscripts/trunk/16.0/source//_common_impl_details.rb 
  
///***/setupscripts/trunk/16.0/source//_common_impl_details.rb#6
|+++ 
///***/setupscripts/trunk/16.0/source//_common_impl_details.rb 
  2016-02-18 09:21:42
--
File to patch:
Skip this patch? [y]
Skipping patch.
1 out of 1 hunk ignored
missing header for unified diff at line 17 of patch
can't find file to patch at input line 17
Perhaps you should have used the -p or --strip option?
The text leading up to this was:
--
|/setupscripts/trunk/16.0/source//_setup_common_impl_details.rb 
  
///***/setupscripts/trunk/16.0/source//_setup_common_impl_details.rb#2
|+++ 
///***/setupscripts/trunk/16.0/source//_setup_common_impl_details.rb
 
2016-02-18 09:21:43
--
File to patch:
Skip this patch? [y]
Skipping patch.
1 out of 1 hunk ignored
missing header for unified diff at line 27 of patch
can't find file to patch at input line 27
Perhaps you should have used the -p or --strip option?
The text leading up to this was:
--
|/***/setupscripts/trunk/16.0/source//_setup_impl_details.rb   
  
 ///***/setupscripts/trunk/16.0/source//_setup_impl_details.rb#2
|+++ 
///***/setupscripts/trunk/16.0/source//_setup_impl_details.rb 
   2016-02-18 09:21:43
--
File to patch:
Skip this patch? [y]
Skipping patch.
1 out of 1 hunk ignored


-- 
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.

Re: rbt patch on Windows troubles

[Daniel Kuecker]

I'm also having issues, but on Windows 10.

I was able to get patch.exe to run via the link Igor posted above 
(http://math.nist.gov/oommf/software-patchsets/patch_on_Windows7.html).

When I manually run patch with the temporary file "rbt patch" creates, it 
crashes.  An inspection of the temporary patch file shows windows line 
endings (LF) instead of unix line endings (CR/LF).  Adding the "--binary" 
option to the patch.exe call 
(per http://gnuwin32.sourceforge.net/packages/patch.htm) works when 
manually run.

Is there any way to get rbt patch to create a temporary patch file with 
proper line endings?

FWIW:  I ran this from a linux machine and it works fine.

Thanks!

--Daniel

On Tuesday, April 8, 2014 at 3:07:23 PM UTC-5, Igor Berger wrote:
>
> Hello,
>
> This is not a Review Board problem. I'm just wondering whether anyone is 
> using "rbt patch" on Windows 7.
>
> I figured out that in order for "rbt patch" to work, patch itself has to 
> be installed separately.
> I've found these precompiled binaries for Windows: 
> http://gnuwin32.sourceforge.net/packages/patch.htm
>
> However, patch crashes even if I use workarounds listed here:
> http://math.nist.gov/oommf/software-patchsets/patch_on_Windows7.html
>
> Any ideas?
>
> Thank you,
> Igor.
>
>

-- 
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.

Re: Review Board Ticket #3987: Allow collapsing/hiding files in diff viewer

[Daniel D]

--
To reply, visit https://hellosplat.com/s/beanbag/tickets/3987/
--

New update by tyralek
For Beanbag, Inc. > Review Board > Ticket #3987


Reply:

I can't seem to subscribe, so I'll just say, this would be a great addition!

-- 
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 #4061: rbt.exe failing to find diff on Windows 10

[Daniel D]

--
To reply, visit https://hellosplat.com/s/beanbag/tickets/4061/
--

New update by bittebrown
For Beanbag, Inc. > RBTools > Ticket #4061


Reply:

I'm on Windows 10, Python 2.7.11 also, and it works if I have cygwin's 
diff.exe in my PATH, but fails otherwise with:
"GNU diff is required in order to generate diffs. Make sure it is installed 
and in the path."

For others at my company, it finds the wrong diff.exe first.
This is a significant impediment for them.

I didn't notice this requirement anywhere in the documentation for running 
rbt post on Windows.
Could that be updated?

-- 
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: Reviewboard upgrade failure

['Daniel Laird' via reviewboard]

All,

I am trying to upgrade from 2.0.20 -> 2.5.3.
I do this by taking a backup of the 2.0.20 database then importing into 
MySQL and then running the rb-site upgrade.

However I get the following output:

Updating database. This may take a while.

The log output below, including warnings and errors,
can be ignored unless upgrade fails.

--  --
Creating tables ...
There are unapplied evolutions for accounts.
There are unapplied evolutions for attachments.
There are unapplied evolutions for diffviewer.
There are unapplied evolutions for reviews.
There are unapplied evolutions for webapi.
Project signature has changed - an evolution is required
Installing custom SQL ...
Installing indexes ...
Installed 0 object(s) from 0 fixture(s)
/usr/local/lib/python2.7/dist-packages/ReviewBoardPowerPack-1.4-py2.7.egg/rbpowerpack/scmtools/tfs.py:9:
 
DeprecationWarning: django.utils.simplejson is deprecated; use json instead.

CommandError: Error applying evolution: (1060, "Duplicate column name 
'visibility'")
...

Any ideas on why I am getting this error or how I can get more debug to 
help?
Much appreciated
Dan

On Friday, 15 January 2016 19:44:43 UTC, Ben Cooksley wrote:
>
> On Sat, Jan 16, 2016 at 6:09 AM, Christian Hammond  > wrote: 
> > Hi Ben, 
> > 
> > You'll have a much easier time restoring from a backup. It's hard to say 
> how 
> > far it went through the evolution process, and unfortunately today it 
> > doesn't keep track of how far it got and what it'd have to do to 
> recover. 
> > You'd have a lot of trial and error to fix it manually. You can try it, 
> > though. 
> > 
> > Basically, you'll need to dump the SQL that the evolutions want to 
> apply, 
> > and go through and hand-undo each thing it did until you get back to the 
> > point of where it was. You'd definitely want to do a backup first, 
> though. 
>
> Unfortunately people had started using it already so this was the 
> easiest approach :( 
>
> Would it be possible to get a copy of a normally, safely upgraded 
> schema so I can double check I haven't clobbered anything? 
>
> The queries I ended up having to run to revert things to a state where 
> the upgrade process would work was: 
>
> 160115 19:34:12 2197763 Query   ALTER TABLE 
> accounts_reviewrequestvisit DROP COLUMN visibility 
> 160115 19:34:24 2197763 Query   DROP INDEX 
> `accounts_reviewrequestvisit_05ee5d21` ON 
> `accounts_reviewrequestvisit` 
> 160115 19:34:45 2197763 Query   ALTER TABLE attachments_fileattachment 
> DROP COLUMN attachment_revision, DROP COLUMN attachment_history_id 
> 160115 19:34:58 2197763 Query   ALTER TABLE diffviewer_filediff DROP 
> COLUMN raw_diff_hash_id, DROP COLUMN raw_parent_diff_hash_id 
> 160115 19:36:18 2197763 Query   ALTER TABLE `reviews_group` DROP 
> COLUMN `email_list_only`, DROP COLUMN is_default_group 
> 160115 19:36:25 2197763 Query   DROP TABLE 
> reviews_reviewrequest_file_attachment_histories 
>
> Note that I observed that the Reviewboard process tries to reverse 
> it's failed upgrade by doing a rollback. It is noted in the case of 
> InnoDB that schema changes cannot be rolled back (see 
> http://www.sitepoint.com/mysql-transaction-gotchas-good-parts/) 
>
> > 
> > Christian 
>
> Cheers, 
> Ben 
>
> > 
> > -- 
> > Christian Hammond - chi...@chipx86.com  
> > Review Board - https://www.reviewboard.org 
> > Beanbag, Inc. - https://www.beanbaginc.com 
> > 
> > On Fri, Jan 15, 2016 at 2:26 AM, Ben Cooksley  > wrote: 
> >> 
> >> On Fri, Jan 15, 2016 at 11:09 PM, Ben Cooksley  > wrote: 
> >> > On Fri, Jan 15, 2016 at 11:04 PM, Christian Hammond 
> >> >  wrote: 
> >> >> Hi Ben, 
> >> >> 
> >> >> This is due to a mismatch between MySQL table types. The existing 
> >> >> tables are 
> >> >> likely MyISAM, with MySQL now defaulting to InnoDB for new ones. 
> You'll 
> >> >> need 
> >> >> to either migrate all the existing tables, or tell MySQL to use the 
> >> >> existing 
> >> >> type for new tables. 
> >> >> 
> >> >> (It's a pretty terrible error, but unfortunately, beyond our 
> control. I 
> >> >> just 
> >> >> recognize this sort of problem.) 
> >> > 
> >> > Argh. Our systems usually have InnoDB as default, guess that isn't 
> the 
> >> > case when we originally had Reviewboard provisioned. 
> >> > I shouldn't see any issues migrating all tables into InnoDB correct? 
> >> 
> >> Seems it is safe. 
> >> Unfortunately it looks like one of the evolutions got part way through 
> >> the process. 
> >> 
> >> CommandError: Error applying evolution: (1060, "Duplicate column name 
> >> 'visibility'") 
> >> 
> >> Any suggestions (I could restore from backups, but if I can avoid 
> it...)? 
> >> 
> >> > 
> >> >> 
> >> >> Christian 
> >> > 
> >> > Cheers, 
> >> > Ben 
> >> 
> >> Thanks, 
> >> Ben 
> >> 
> >> > 
> >> >> 
> >> >> -- 
> >> >> Christian Hammond - chri...@beanbaginc.com  
> >> >> Review Board - https://www.reviewboard.org 

Re: Reviewboard upgrade failure

['Daniel Laird' via reviewboard]

I too had this issue (MySQL default changing from MyISAM to InnoDB).
Although I understand that this is caused by MySQL version being updated at 
the same time as a Reviewboard Update (and migrate).
But would there be any way for the 'upgrade' logic of Reviewboard to post a 
more useful error message (DB Table formats dont match) or something like 
that.
As I am sure it will save a lot of time for some people (but I dont have 
the skills to make these changes sadly)

Dan

On Friday, 15 January 2016 07:27:08 UTC, Ben Cooksley wrote:
>
> Hi everyone, 
>
> While upgrading from Reviewboard 2.0.17 to 2.5.2 i've encountered a 
> few SQL errors. 
> Reviewboard itself appears to boot and run fine though, based on the 
> nature of the errors I suspect it's a case of index name collisions. 
>
> Output from the upgrade process is below: 
>
> (virtualenv)reviewboard@mimi:~$ rb-site upgrade 
> /srv/www/reviewboard/git.reviewboard.kde.org/ 
> Rebuilding directory structure 
> Updating database. This may take a while. 
>
> The log output below, including warnings and errors, 
> can be ignored unless upgrade fails. 
>
> --  -- 
> Creating tables ... 
> Creating table accounts_trophy 
>
> [!] There was an error synchronizing the database. Make sure the 
> database is created and has the appropriate permissions, and then 
> continue. 
> [!] Details: (1005, "Can't create table 'reviewboard_git.#sql- 
> 331d_214215' (errno: 150)") 
>
> Press Enter to continue 
> Creating tables ... 
> Creating table attachments_fileattachmenthistory 
> Creating table diffviewer_rawfilediffdata 
> Creating table notifications_webhooktarget_repositories 
>
> [!] There was an error synchronizing the database. Make sure the 
> database is created and has the appropriate permissions, and then 
> continue. 
> [!] Details: (1005, "Can't create table 'reviewboard_git.#sql- 
> 331d_214215' (errno: 150)") 
>
> Press Enter to continue 
> Creating tables ... 
> Creating table notifications_webhooktarget 
>
> [!] There was an error synchronizing the database. Make sure the 
> database is created and has the appropriate permissions, and then 
> continue. 
> [!] Details: (1005, "Can't create table 'reviewboard_git.#sql- 
> 331d_214215' (errno: 150)") 
>
> Press Enter to continue 
> Creating tables ... 
> Creating table webapi_webapitoken 
>
> [!] There was an error synchronizing the database. Make sure the 
> database is created and has the appropriate permissions, and then 
> continue. 
> [!] Details: (1005, "Can't create table 'reviewboard_git.#sql- 
> 331d_214215' (errno: 150)") 
>
> Press Enter to continue 
> Creating tables ... 
> Upgrading Review Board from 2.0.17 to 2.5.2 
> There are unapplied evolutions for accounts. 
> There are unapplied evolutions for attachments. 
> There are unapplied evolutions for diffviewer. 
> There are unapplied evolutions for notifications. 
> There are unapplied evolutions for reviews. 
> There are unapplied evolutions for webapi. 
> Adding baseline version for new models 
> Evolutions in notifications baseline: webhooktarget_extra_state, 
> webhooktarget_extra_data_null 
> Project signature has changed - an evolution is required 
> Installing custom SQL ... 
> Installing indexes ... 
> Installed 0 object(s) from 0 fixture(s) 
> CommandError: Error applying evolution: (1005, "Can't create table 
> 'reviewboard_git.#sql-331d_214238' (errno: 150)") 
>
> Any pointers? 
>
> Cheers, 
> Ben Cooksley 
> KDE Sysadmin 
>

-- 
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.

Depends-On, Blocks and Circular Reference

[Daniel Laird]

WE are running 2.0.17 and have managed to create a situation and dont know 
how to resolve it - or whether its a defect

Someone created 2 reviews with code changes that were dependent.
They marked Review 2 as Depends On Review 1 -  this then updated the 
'Blocks' field.
They then marked Review 1 as Depends On Review 2 - as they felt that both 
reviews had to be committed together.

However the server then starts to give us lots of errors:
Traceback (most recent call last):

  File 
/usr/local/lib/python2.7/dist-packages/Django-1.6.11-py2.7.egg/django/core/handlers/base.py,
 
line 199, in get_response
response = middleware_method(request, response)

  File 
/usr/local/lib/python2.7/dist-packages/Django-1.6.11-py2.7.egg/django/middleware/http.py,
 
line 14, in process_response
response['Content-Length'] = str(len(response.content))

  File 
/usr/local/lib/python2.7/dist-packages/Djblets-0.8.20-py2.7.egg/djblets/webapi/responses.py,
 
line 109, in _get_content
**self.encoder_kwargs)

  File 
/usr/local/lib/python2.7/dist-packages/Djblets-0.8.20-py2.7.egg/djblets/webapi/encoders.py,
 
line 110, in encode
return super(JSONEncoderAdapter, self).encode(o)

  File /usr/lib/python2.7/json/encoder.py, line 201, in encode
chunks = self.iterencode(o, _one_shot=True)

  File /usr/lib/python2.7/json/encoder.py, line 264, in iterencode
return _iterencode(o, 0)

ValueError: Circular reference detected

It maybe that the double linking is incorrect usage - however having been 
able to get into that situation.  If we try to remove one of the 'Blocks' 
fields we just get a spinning 'Loading' icon and are unable to move past 
this to remove the Circular Reference.

Not sure if you think this is user error, a defect or a combination of both 
- but thought I would share it.

Cheers
Dan

-- 
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.

Re: Problems upgrading to 2.0.13 and 2.0.15

[Daniel Dominguez]

That fixed it.  Many Thanks!

On Friday, April 17, 2015 at 2:05:03 PM UTC-7, Christian Hammond wrote:

 Hi Daniel,

 Sorry, I've been pretty swamped.

 First, make a backup of your database. Then, try:

 DELETE FROM django_project_version WHERE id=4;

 Then perform the upgrade again.

 Christian

 -- 
 Christian Hammond - chi...@chipx86.com javascript:
 Review Board - http://www.reviewboard.org
 Beanbag, Inc. - http://www.beanbaginc.com

 On Fri, Apr 17, 2015 at 10:29 AM, Daniel Dominguez daniel.j@gmail.com 
 javascript: wrote:

 Any update here?


 On Friday, March 20, 2015 at 1:29:07 PM UTC-7, Daniel Dominguez wrote:

 Running --hint --evolve had not been done before.  It was just my 
 experimenting after seeing this issue (with a current backup of the db 
 first of course).

 Anyhow, I've attached the sql dump you requested.

 Thanks!

 On Friday, March 20, 2015 at 8:28:36 AM UTC-7, Daniel Dominguez wrote:

 When I performed the upgrade form 2.0.12 to 2.0.13, the site-upgrade 
 failed.  It didn't seem to break our reviewboard instance, so I just 
 ignored it at the time.  I recently upgraded to 2.0.15, and the result is 
 the same.  Again, our reviewboard server seems to be fine, but this is 
 starting to make me nervous.


 -- begin log output --
 Creating tables ...
 Project signature has changed - an evolution is required
 Installing custom SQL ...
 Installing indexes ...
 Installed 0 object(s) from 0 fixture(s)
 The stored evolutions do not completely resolve all model changes.

 Run `./manage.py evolve --hint` to see a suggestion for the changes 
 required.

 The following are the changes that could not be resolved:
 In model reviews.ReviewRequest:
 Field 'description_rich_text' has been added
 Field 'testing_done_rich_text' has been added
 In model reviews.Review:
 Field 'body_top_rich_text' has been added
 Field 'body_bottom_rich_text' has been added
 In model reviews.ReviewRequestDraft:
 Field 'description_rich_text' has been added
 Field 'testing_done_rich_text' has been added
 In model accounts.Profile:
 Field 'default_use_rich_text' has been added
 CommandError: Your models contain changes that Django Evolution cannot 
 resolve automatically.

 
 I've trying running evolve -- --hint --execute, but that fails with the 
 following

 CommandError: Error applying evolution: (1060, Duplicate column name 
 'default_use_rich_text')



 I've checked that I have the latest django_evolution installed for my 
 platform (0.7.4).  This is running on a MySql database.

 It seems like the evolution is trying to add columns that already 
 exist.  Maybe a version table didn't get incremented in the prior 
 evolution?  Any hints on how I can troubleshoot this?



  -- 
 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...@googlegroups.com javascript:.
 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 
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.

Re: Problems upgrading to 2.0.13 and 2.0.15

[Daniel Dominguez]

Any update here?

On Friday, March 20, 2015 at 1:29:07 PM UTC-7, Daniel Dominguez wrote:

 Running --hint --evolve had not been done before.  It was just my 
 experimenting after seeing this issue (with a current backup of the db 
 first of course).

 Anyhow, I've attached the sql dump you requested.

 Thanks!

 On Friday, March 20, 2015 at 8:28:36 AM UTC-7, Daniel Dominguez wrote:

 When I performed the upgrade form 2.0.12 to 2.0.13, the site-upgrade 
 failed.  It didn't seem to break our reviewboard instance, so I just 
 ignored it at the time.  I recently upgraded to 2.0.15, and the result is 
 the same.  Again, our reviewboard server seems to be fine, but this is 
 starting to make me nervous.


 -- begin log output --
 Creating tables ...
 Project signature has changed - an evolution is required
 Installing custom SQL ...
 Installing indexes ...
 Installed 0 object(s) from 0 fixture(s)
 The stored evolutions do not completely resolve all model changes.

 Run `./manage.py evolve --hint` to see a suggestion for the changes 
 required.

 The following are the changes that could not be resolved:
 In model reviews.ReviewRequest:
 Field 'description_rich_text' has been added
 Field 'testing_done_rich_text' has been added
 In model reviews.Review:
 Field 'body_top_rich_text' has been added
 Field 'body_bottom_rich_text' has been added
 In model reviews.ReviewRequestDraft:
 Field 'description_rich_text' has been added
 Field 'testing_done_rich_text' has been added
 In model accounts.Profile:
 Field 'default_use_rich_text' has been added
 CommandError: Your models contain changes that Django Evolution cannot 
 resolve automatically.

 
 I've trying running evolve -- --hint --execute, but that fails with the 
 following

 CommandError: Error applying evolution: (1060, Duplicate column name 
 'default_use_rich_text')



 I've checked that I have the latest django_evolution installed for my 
 platform (0.7.4).  This is running on a MySql database.

 It seems like the evolution is trying to add columns that already exist. 
  Maybe a version table didn't get incremented in the prior evolution?  Any 
 hints on how I can troubleshoot this?





-- 
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.

Re: RBtools 0.7 - issue posting a review

[Daniel Laird]

On a new Ubuntu 14.04 installation I am seeing something similar:
(We are testing 14.04 before wider rollout.)
 Running: svn info --non-interactive
 Running: diff --version
 Running: svn --version -q
 repository info: Path: http://REVIEWBOARD_URL/NAME, Base path: PATH TO 
CHANGES, Supports changesets: False
 Running: svn status -q --ignore-externals
 Running: svn diff --diff-cmd=diff --notice-ancestry -r BASE
Traceback (most recent call last):
  File /usr/local/bin/rbt, line 9, in module
load_entry_point('RBTools==0.7.2', 'console_scripts', 'rbt')()
  File 
/usr/local/lib/python2.7/dist-packages/RBTools-0.7.2-py2.7.egg/rbtools/commands/main.py,
 
line 133, in main
command.run_from_argv([RB_MAIN, command_name] + args)
  File 
/usr/local/lib/python2.7/dist-packages/RBTools-0.7.2-py2.7.egg/rbtools/commands/__init__.py,
 
line 555, in run_from_argv
exit_code = self.main(*args) or 0
  File 
/usr/local/lib/python2.7/dist-packages/RBTools-0.7.2-py2.7.egg/rbtools/commands/post.py,
 
line 639, in main
extra_args=extra_args)
  File 
/usr/local/lib/python2.7/dist-packages/RBTools-0.7.2-py2.7.egg/rbtools/clients/svn.py,
 
line 319, in diff
diff = self._run_svn(diff_cmd, split_lines=True)
  File 
/usr/local/lib/python2.7/dist-packages/RBTools-0.7.2-py2.7.egg/rbtools/clients/svn.py,
 
line 707, in _run_svn
return execute(cmdline, *args, **kwargs)
  File 
/usr/local/lib/python2.7/dist-packages/RBTools-0.7.2-py2.7.egg/rbtools/utils/process.py,
 
line 104, in execute
data = [line.decode('utf-8') for line in data]
  File /usr/lib/python2.7/encodings/utf_8.py, line 16, in decode
return codecs.utf_8_decode(input, errors, True)
UnicodeDecodeError: 'utf8' codec can't decode byte 0xe4 in position 17: 
invalid continuation byte

I appear to have all the correct locates installed - but no luck.
It is running SVN 1.8.8 and Ubuntu 14.04 and the latest RBT- any ideas what 
to debug - it does not appear to be log related this time
This command:
svn diff --diff-cmd=diff --notice-ancestry -r BASE
Runs fine on the commandline standalone

Dan

On Sunday, 15 February 2015 21:20:00 UTC, Daniel Laird wrote:

 After some investigation we found the following:

 # TODO: This can break on systems that don't have the en_US local

 # installed (which isn't very many). Ideally in this case, we could

 # put something in the config file, but that's not plumbed through to here.

 env['LC_ALL'] = 'en_US.UTF-8'

 env['LANGUAGE'] = 'en_US.UTF-8'

 We can confirm that it does break things!

 With the 'en_US.UTF-8' locale installed the problem goes away - is there a 
 way to check this locale is installed and error nicely rather than breaking?
 Cheers
 Dan

 On Saturday, 31 January 2015 18:49:02 UTC, David Trowbridge wrote:

 Hi Daniel,

 Do you think you could add some code to print out the contents of 'log' 
 inside _convert_symbolic_revision before we try to construct the etree?

 Thanks,
 -David


 On Fri Jan 30 2015 at 4:48:37 AM Daniel Laird daniel@googlemail.com 
 wrote:

 We have a shared Debian server and we have updated RBtools to 0.7.

 When we try to post a review we get the following:

 SERVER_NAME rbt post -d NUM
  RBTools 0.7
  Python 2.7.3 (default, Mar 13 2014, 11:03:55) 
 [GCC 4.7.2]
  Running on Linux-3.2.0-4-amd64-x86_64-with-debian-7.8
  Home = /v/home/rad/USERNAME
  Current directory = /v/space/SERVER_NAME/USERNAME/
 wc/memory_corruption
  Checking for a Subversion repository...
  Running: svn info --non-interactive
  Running: diff --version
  repository info: Path: https://SVN_SERVER/svn-restricted/PROJNAME, 
 Base path: /branches/users/bob/memory_corruption, Supports changesets: 
 False
  Making HTTP GET request to http://REVIEWBOARD_SERVER/api/
  HTTP GET request to http://REVIEWBOARD_SERVER/api/ cannot be cached
  Making HTTP GET request to http://REVIEWBOARD_SERVER/api/
 repositories/
  HTTP GET request to http://REVIEWBOARD_SERVER/api/repositories/ 
 cannot be cached
  Making HTTP GET request to http://REVIEWBOARD_SERVER/api/
 repositories/?max-results=25start=25
  HTTP GET request to http://REVIEWBOARD_SERVER/api/
 repositories/?max-results=25start=25 cannot be cached
  Making HTTP GET request to http://REVIEWBOARD_SERVER/api/
 repositories/1/info/
  HTTP GET request to http://REVIEWBOARD_SERVER/api/
 repositories/1/info/ cannot be cached
  Making HTTP GET request to http://REVIEWBOARD_SERVER/api/
 repositories/2/info/
  HTTP GET request to http://REVIEWBOARD_SERVER/api/
 repositories/2/info/ cannot be cached
  Making HTTP GET request to http://REVIEWBOARD_SERVER/api/
 repositories/3/info/
  HTTP GET request to http://REVIEWBOARD_SERVER/api/
 repositories/3/info/ cannot be cached
  Making HTTP GET request to http://REVIEWBOARD_SERVER/api/
 repositories/4/info/
  HTTP GET request to http://REVIEWBOARD_SERVER/api/
 repositories/4/info/ cannot be cached
  Making HTTP GET request to http://REVIEWBOARD_SERVER/api/
 repositories/5/info/
  HTTP GET request to http://REVIEWBOARD_SERVER/api

Re: Review Board version mismatch

[Daniel Bansch]

I used easy_install.

On Friday, February 20, 2015 at 10:45:53 AM UTC-6, Daniel Bansch wrote:

I'm trying to install review board on redhat with apache, mod_wsgi and 
 mysql.  I get this error when I go to my review board site:


 ---

 Manual server updates required

 A recent upgrade requires manual updates to be made on this server. After 
 these changes are made, you should restart your web server. 

 If you have any questions or problems, please contact us on our mailing 
 list http://groups.google.com/group/reviewboard. 
 Review Board version mismatch 
  
 The version of Review Board running does not match the version the site 
 was last upgraded to. You are running *2.0.13* and the site was last 
 upgraded to *2.0.12*. 

 Please upgrade your site to fix this by running: 

 $ rb-site upgrade /var/www/mysite
 ---

 I've run the rb-site upgrade /var/www/mysite and restarted everything, but 
 the same error pops up.

 I have ReviewBoard-2.0.12-py2.7.egg and ReviewBoard-2.0.13-py2.7.egg in my 
 python site-packages directory.

 Does anyone have any suggestions?

 Thanks,
 Daniel



-- 
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.

Re: Review Board version mismatch

[Daniel Bansch]

I was able to get it to run by changing the directory name of 
ReviewBoard-2.0.13-py2.7.egg 
to ReviewBoard-2.0.12-py2.7.egg.

On Friday, February 20, 2015 at 10:45:53 AM UTC-6, Daniel Bansch wrote:

I'm trying to install review board on redhat with apache, mod_wsgi and 
 mysql.  I get this error when I go to my review board site:


 ---

 Manual server updates required

 A recent upgrade requires manual updates to be made on this server. After 
 these changes are made, you should restart your web server. 

 If you have any questions or problems, please contact us on our mailing 
 list http://groups.google.com/group/reviewboard. 
 Review Board version mismatch 
  
 The version of Review Board running does not match the version the site 
 was last upgraded to. You are running *2.0.13* and the site was last 
 upgraded to *2.0.12*. 

 Please upgrade your site to fix this by running: 

 $ rb-site upgrade /var/www/mysite
 ---

 I've run the rb-site upgrade /var/www/mysite and restarted everything, but 
 the same error pops up.

 I have ReviewBoard-2.0.12-py2.7.egg and ReviewBoard-2.0.13-py2.7.egg in my 
 python site-packages directory.

 Does anyone have any suggestions?

 Thanks,
 Daniel



-- 
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.

Review Board version mismatch

[Daniel Bansch]

   I'm trying to install review board on redhat with apache, mod_wsgi and 
mysql.  I get this error when I go to my review board site:

---

Manual server updates required

A recent upgrade requires manual updates to be made on this server. After 
these changes are made, you should restart your web server. 

If you have any questions or problems, please contact us on our mailing list 
http://groups.google.com/group/reviewboard. 
Review Board version mismatch 
 
The version of Review Board running does not match the version the site was 
last upgraded to. You are running *2.0.13* and the site was last upgraded 
to *2.0.12*. 

Please upgrade your site to fix this by running: 

$ rb-site upgrade /var/www/mysite
---

I've run the rb-site upgrade /var/www/mysite and restarted everything, but 
the same error pops up.

I have ReviewBoard-2.0.12-py2.7.egg and ReviewBoard-2.0.13-py2.7.egg in my 
python site-packages directory.

Does anyone have any suggestions?

Thanks,
Daniel

   

-- 
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.

Re: RBtools 0.7 - issue posting a review

[Daniel Laird]

After some investigation we found the following:

# TODO: This can break on systems that don't have the en_US local

# installed (which isn't very many). Ideally in this case, we could

# put something in the config file, but that's not plumbed through to here.

env['LC_ALL'] = 'en_US.UTF-8'

env['LANGUAGE'] = 'en_US.UTF-8'

We can confirm that it does break things!

With the 'en_US.UTF-8' locale installed the problem goes away - is there a 
way to check this locale is installed and error nicely rather than breaking?
Cheers
Dan

On Saturday, 31 January 2015 18:49:02 UTC, David Trowbridge wrote:

 Hi Daniel,

 Do you think you could add some code to print out the contents of 'log' 
 inside _convert_symbolic_revision before we try to construct the etree?

 Thanks,
 -David


 On Fri Jan 30 2015 at 4:48:37 AM Daniel Laird daniel@googlemail.com 
 javascript: wrote:

 We have a shared Debian server and we have updated RBtools to 0.7.

 When we try to post a review we get the following:

 SERVER_NAME rbt post -d NUM
  RBTools 0.7
  Python 2.7.3 (default, Mar 13 2014, 11:03:55) 
 [GCC 4.7.2]
  Running on Linux-3.2.0-4-amd64-x86_64-with-debian-7.8
  Home = /v/home/rad/USERNAME
  Current directory = /v/space/SERVER_NAME/USERNAME/
 wc/memory_corruption
  Checking for a Subversion repository...
  Running: svn info --non-interactive
  Running: diff --version
  repository info: Path: https://SVN_SERVER/svn-restricted/PROJNAME, 
 Base path: /branches/users/bob/memory_corruption, Supports changesets: 
 False
  Making HTTP GET request to http://REVIEWBOARD_SERVER/api/
  HTTP GET request to http://REVIEWBOARD_SERVER/api/ cannot be cached
  Making HTTP GET request to http://REVIEWBOARD_SERVER/api/
 repositories/
  HTTP GET request to http://REVIEWBOARD_SERVER/api/repositories/ 
 cannot be cached
  Making HTTP GET request to http://REVIEWBOARD_SERVER/api/
 repositories/?max-results=25start=25
  HTTP GET request to http://REVIEWBOARD_SERVER/api/
 repositories/?max-results=25start=25 cannot be cached
  Making HTTP GET request to http://REVIEWBOARD_SERVER/api/
 repositories/1/info/
  HTTP GET request to http://REVIEWBOARD_SERVER/api/
 repositories/1/info/ cannot be cached
  Making HTTP GET request to http://REVIEWBOARD_SERVER/api/
 repositories/2/info/
  HTTP GET request to http://REVIEWBOARD_SERVER/api/
 repositories/2/info/ cannot be cached
  Making HTTP GET request to http://REVIEWBOARD_SERVER/api/
 repositories/3/info/
  HTTP GET request to http://REVIEWBOARD_SERVER/api/
 repositories/3/info/ cannot be cached
  Making HTTP GET request to http://REVIEWBOARD_SERVER/api/
 repositories/4/info/
  HTTP GET request to http://REVIEWBOARD_SERVER/api/
 repositories/4/info/ cannot be cached
  Making HTTP GET request to http://REVIEWBOARD_SERVER/api/
 repositories/5/info/
  HTTP GET request to http://REVIEWBOARD_SERVER/api/
 repositories/5/info/ cannot be cached
  Making HTTP GET request to http://REVIEWBOARD_SERVER/api/
 repositories/6/info/
  HTTP GET request to http://REVIEWBOARD_SERVER/api/
 repositories/6/info/ cannot be cached
  Making HTTP GET request to http://REVIEWBOARD_SERVER/api/
 repositories/7/info/
  HTTP GET request to http://REVIEWBOARD_SERVER/api/
 repositories/7/info/ cannot be cached
  Making HTTP GET request to http://REVIEWBOARD_SERVER/api/
 repositories/8/info/
  HTTP GET request to http://REVIEWBOARD_SERVER/api/
 repositories/8/info/ cannot be cached
  Making HTTP GET request to http://REVIEWBOARD_SERVER/api/
 repositories/9/info/
  HTTP GET request to http://REVIEWBOARD_SERVER/api/
 repositories/9/info/ cannot be cached
  Making HTTP GET request to http://REVIEWBOARD_SERVER/api/
 repositories/10/info/
  HTTP GET request to http://REVIEWBOARD_SERVER/api/
 repositories/10/info/ cannot be cached
  Making HTTP GET request to http://REVIEWBOARD_SERVER/api/
 repositories/11/info/
  HTTP GET request to http://REVIEWBOARD_SERVER/api/
 repositories/11/info/ cannot be cached
  Making HTTP GET request to http://REVIEWBOARD_SERVER/api/
 repositories/12/info/
  HTTP GET request to http://REVIEWBOARD_SERVER/api/
 repositories/12/info/ cannot be cached
  Making HTTP GET request to http://REVIEWBOARD_SERVER/api/
 repositories/13/info/
  HTTP GET request to http://REVIEWBOARD_SERVER/api/
 repositories/13/info/ cannot be cached
  Making HTTP GET request to http://REVIEWBOARD_SERVER/api/
 repositories/14/info/
  HTTP GET request to http://REVIEWBOARD_SERVER/api/
 repositories/14/info/ cannot be cached
  repository info: Path: https://SVN_SERVER/svn-restricted/PROJNAME, 
 Base path: /branches/users/bob/memory_corruption, Supports changesets: 
 False
  Running: svn log -r REVNUM -l 1 --xml
 Traceback (most recent call last):
   File /v/home/rad/USERNAME/.local/bin/rbt, line 9, in module
 load_entry_point('RBTools==0.7', 'console_scripts', 'rbt')()
   File /v/home/rad/USERNAME/.local/lib/python2.7/site-packages/
 RBTools-0.7-py2.7.egg/rbtools/commands/main.py, line 133, in main
 command.run_from_argv([RB_MAIN

RBtools 0.7 - issue posting a review

[Daniel Laird]

We have a shared Debian server and we have updated RBtools to 0.7.

When we try to post a review we get the following:

SERVER_NAME rbt post -d NUM
 RBTools 0.7
 Python 2.7.3 (default, Mar 13 2014, 11:03:55) 
[GCC 4.7.2]
 Running on Linux-3.2.0-4-amd64-x86_64-with-debian-7.8
 Home = /v/home/rad/USERNAME
 Current directory = /v/space/SERVER_NAME/USERNAME/wc/memory_corruption
 Checking for a Subversion repository...
 Running: svn info --non-interactive
 Running: diff --version
 repository info: Path: https://SVN_SERVER/svn-restricted/PROJNAME, Base 
path: /branches/users/bob/memory_corruption, Supports changesets: False
 Making HTTP GET request to http://REVIEWBOARD_SERVER/api/
 HTTP GET request to http://REVIEWBOARD_SERVER/api/ cannot be cached
 Making HTTP GET request to http://REVIEWBOARD_SERVER/api/repositories/
 HTTP GET request to http://REVIEWBOARD_SERVER/api/repositories/ cannot 
be cached
 Making HTTP GET request to 
http://REVIEWBOARD_SERVER/api/repositories/?max-results=25start=25
 HTTP GET request to 
http://REVIEWBOARD_SERVER/api/repositories/?max-results=25start=25 cannot 
be cached
 Making HTTP GET request to 
http://REVIEWBOARD_SERVER/api/repositories/1/info/
 HTTP GET request to http://REVIEWBOARD_SERVER/api/repositories/1/info/ 
cannot be cached
 Making HTTP GET request to 
http://REVIEWBOARD_SERVER/api/repositories/2/info/
 HTTP GET request to http://REVIEWBOARD_SERVER/api/repositories/2/info/ 
cannot be cached
 Making HTTP GET request to 
http://REVIEWBOARD_SERVER/api/repositories/3/info/
 HTTP GET request to http://REVIEWBOARD_SERVER/api/repositories/3/info/ 
cannot be cached
 Making HTTP GET request to 
http://REVIEWBOARD_SERVER/api/repositories/4/info/
 HTTP GET request to http://REVIEWBOARD_SERVER/api/repositories/4/info/ 
cannot be cached
 Making HTTP GET request to 
http://REVIEWBOARD_SERVER/api/repositories/5/info/
 HTTP GET request to http://REVIEWBOARD_SERVER/api/repositories/5/info/ 
cannot be cached
 Making HTTP GET request to 
http://REVIEWBOARD_SERVER/api/repositories/6/info/
 HTTP GET request to http://REVIEWBOARD_SERVER/api/repositories/6/info/ 
cannot be cached
 Making HTTP GET request to 
http://REVIEWBOARD_SERVER/api/repositories/7/info/
 HTTP GET request to http://REVIEWBOARD_SERVER/api/repositories/7/info/ 
cannot be cached
 Making HTTP GET request to 
http://REVIEWBOARD_SERVER/api/repositories/8/info/
 HTTP GET request to http://REVIEWBOARD_SERVER/api/repositories/8/info/ 
cannot be cached
 Making HTTP GET request to 
http://REVIEWBOARD_SERVER/api/repositories/9/info/
 HTTP GET request to http://REVIEWBOARD_SERVER/api/repositories/9/info/ 
cannot be cached
 Making HTTP GET request to 
http://REVIEWBOARD_SERVER/api/repositories/10/info/
 HTTP GET request to http://REVIEWBOARD_SERVER/api/repositories/10/info/ 
cannot be cached
 Making HTTP GET request to 
http://REVIEWBOARD_SERVER/api/repositories/11/info/
 HTTP GET request to http://REVIEWBOARD_SERVER/api/repositories/11/info/ 
cannot be cached
 Making HTTP GET request to 
http://REVIEWBOARD_SERVER/api/repositories/12/info/
 HTTP GET request to http://REVIEWBOARD_SERVER/api/repositories/12/info/ 
cannot be cached
 Making HTTP GET request to 
http://REVIEWBOARD_SERVER/api/repositories/13/info/
 HTTP GET request to http://REVIEWBOARD_SERVER/api/repositories/13/info/ 
cannot be cached
 Making HTTP GET request to 
http://REVIEWBOARD_SERVER/api/repositories/14/info/
 HTTP GET request to http://REVIEWBOARD_SERVER/api/repositories/14/info/ 
cannot be cached
 repository info: Path: https://SVN_SERVER/svn-restricted/PROJNAME, Base 
path: /branches/users/bob/memory_corruption, Supports changesets: False
 Running: svn log -r REVNUM -l 1 --xml
Traceback (most recent call last):
  File /v/home/rad/USERNAME/.local/bin/rbt, line 9, in module
load_entry_point('RBTools==0.7', 'console_scripts', 'rbt')()
  File 
/v/home/rad/USERNAME/.local/lib/python2.7/site-packages/RBTools-0.7-py2.7.egg/rbtools/commands/main.py,
 
line 133, in main
command.run_from_argv([RB_MAIN, command_name] + args)
  File 
/v/home/rad/USERNAME/.local/lib/python2.7/site-packages/RBTools-0.7-py2.7.egg/rbtools/commands/__init__.py,
 
line 538, in run_from_argv
exit_code = self.main(*args) or 0
  File 
/v/home/rad/USERNAME/.local/lib/python2.7/site-packages/RBTools-0.7-py2.7.egg/rbtools/commands/post.py,
 
line 618, in main
self.revisions = get_revisions(self.tool, self.cmd_args)
  File 
/v/home/rad/USERNAME/.local/lib/python2.7/site-packages/RBTools-0.7-py2.7.egg/rbtools/utils/review_request.py,
 
line 73, in get_revisions
revisions = tool.parse_revision_spec(cmd_args)
  File 
/v/home/rad/USERNAME/.local/lib/python2.7/site-packages/RBTools-0.7-py2.7.egg/rbtools/clients/svn.py,
 
line 128, in parse_revision_spec
revision = self._convert_symbolic_revision(revision)
  File 
/v/home/rad/USERNAME/.local/lib/python2.7/site-packages/RBTools-0.7-py2.7.egg/rbtools/clients/svn.py,
 
line 173, in _convert_symbolic_revision
root = 

Adding a hyperlink to the navbar

[Daniel Cappuccio]

Hopefully this is a simple question with a simple answer as I am very new 
and unexperienced with the code for this web page.

My company uses review board for code reviews and also has a company wiki 
page with the procedures used for code reviews.  I would like to have a 
link to that wiki page appear in the navbar of review board.  (The light 
blue region at the top of the page with other internal links (MyDashboard, 
New Review Request, etc)).  

I found the source html file on the git hub page, but, i cannot find that 
file on the installed and running review board server.  

Is this change even possible?  Is there a process to do this posted 
somewhere else?

Thanks for your patience, understanding, and responses,
Dan

-- 
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.

Email all users......

[Daniel Laird]

All,

Before I try and write some python that queries all users (ideally active) 
and gets their email addresses.  Has anyone else done this and would you be 
willing to share?
I want to be able to announce things like RBtools 0.7 to all active users 
and cant think of another way (other than the MOTD banner).

If you have something I could use/tweak let me know

Dan

-- 
Get the Review Board Power Pack at http://www.reviewboard.org/powerpack/
---
Sign up for Review Board hosting at RBCommons: https://rbcommons.com/
---
Happy user? Let us know at http://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, SVN and a strange issue

[Daniel Laird]

I am using RBtools 0.6.3 and am trying to post a review to ReviewBoard.
We use SVN as our version control system.

We are not using the .reviewboardrc files but the reviewboard:url property 
as this means less files in the workspace.
For most of our repos this works fine and rbt post creates a review request.

However for 1 of our repos (may be more - not tested all of them) I got the 
following error:
 Error data: {u'stat': u'fail', u'repository': u'SVN_SERVER/REPONAME/', 
u'err': {u'msg': u'The repository path specified is not in the list of 
known repositories.', u'code': 206}}
Traceback (most recent call last):
  File /usr/local/bin/rbt, line 9, in module
load_entry_point('RBTools==0.6.3', 'console_scripts', 'rbt')()
  File 
/usr/local/lib/python2.7/dist-packages/RBTools-0.6.3-py2.7.egg/rbtools/commands/main.py,
 
line 134, in main
command.run_from_argv([RB_MAIN, command_name] + args)
  File 
/usr/local/lib/python2.7/dist-packages/RBTools-0.6.3-py2.7.egg/rbtools/commands/__init__.py,
 
line 416, in run_from_argv
exit_code = self.main(*args) or 0
  File 
/usr/local/lib/python2.7/dist-packages/RBTools-0.6.3-py2.7.egg/rbtools/commands/post.py,
 
line 791, in main
submit_as=self.options.submit_as)
  File 
/usr/local/lib/python2.7/dist-packages/RBTools-0.6.3-py2.7.egg/rbtools/commands/post.py,
 
line 507, in post_request
raise CommandError(Error creating review request: %s % e)
rbtools.commands.CommandError: Error creating review request: The 
repository path specified is not in the list of known repositories. (HTTP 
400, API Error 206)

After some investigation I checked the Repo setup in Reviewboard.
I found that for most of my repositories I had:
http://SVN_SERVER/svn/REPONAME

However for the failing one in question I had:
http://SVN_SERVER/svn/REPONAME/

It seems the addition of the trailing '/' stopped rbtools from posting to 
the server.

With the -d option it shows that it finds the Repo in Reviewboard, only 
when it attempts to POST the diff did it fail.
This might be intentional, however if not.  Perhaps it can be made a little 
more forgiving - maybe don't allow trailing '/' when a repo is configured.

Cheers
Dan

-- 
Get the Review Board Power Pack at http://www.reviewboard.org/powerpack/
---
Sign up for Review Board hosting at RBCommons: https://rbcommons.com/
---
Happy user? Let us know at http://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.

Re: Reviewboard 1.7 - Reviewboard 2.0.x update

[Daniel Laird]

HI Christian,

Many thanks for the response, it was as you describe, our custom script 
(which is for following externals) - is meant to strip the URL.  But the 
wildcard match failed sometimes, leaving the full URL in the diff file. 
 Which RB then processed.
On moving the data from old server - new server (with 2.0.11) - I ran some 
sed scripts to ensure the SVN paths were consistent. 
Now because the BaseUrl matches the file path RB displays it all fine.

I have another issue, but will mail the list about this.

Cheers
Dan

On Friday, 19 December 2014 22:38:42 UTC, Christian Hammond wrote:

 Hi Daniel, 

 We never intended the base directory to contain a URL. It sounds like the 
 custom script is probably responsible for this. 

 Can you confirm whether the base directory's URL exactly matches the 
 configured URL for the repository? 

 In the case that it does match, we take the URL as-is, but if it does not 
 match, we normalize and start building a path (by combining the repository 
 path, base directory, and file path in the diff). This was not intended to 
 handle URLs in the path, but it may be the cause of your problems. 

 Christian 

 --   
 Christian Hammond - chi...@chipx86.com javascript: 
 Review Board - https://www.reviewboard.org 
 Beanbag, Inc. - https://www.beanbaginc.com 

 -Original Message- 
 From: Daniel Laird daniel@googlemail.com javascript: 
 Reply: revie...@googlegroups.com javascript: revie...@googlegroups.com 
 javascript: 
 Date: December 19, 2014 at 3:29:19 AM 
 To: revie...@googlegroups.com javascript: revie...@googlegroups.com 
 javascript: 
 Cc: chi...@chipx86.com javascript: chi...@chipx86.com javascript: 
 Subject:  Re: Reviewboard 1.7 - Reviewboard 2.0.x update 

  Christian 

  Many thanks for getting back to me. 
  I think I know why it is breaking but don't know how to fix it. 

  The diff looks as I would expect (with all files being something like 
 this): 
  Index: libs/A/Source/File.cpp 
  === 
  --- libs/A/Source/File.cpp (revision 143446) 
  +++ libs/A/Source/File.cpp (working copy) 

  I used the webapi to query the Review and spotted the following on the 
 diff 
  resource associated with the review request: 
  http://SVN_SERVER/svn/REPO_NAME/branches/features/software   

  I suspect that maybe in the old GUI (Reviewboard 1.7) it used to combine 
  this basedir and the file path in the diff to draw the differences. 
  I think that maybe we have a problem where it appends the Repo Path + 
  BaseDir + FilePath 
  Maybe there is a difference in RB2.0.x? 

  Our databases on legacy SW used a lot of SVN externals, so someone wrote 
 a 
  script to generate a single diff file relative to the SVN_SERVER root. 
  It allowed us a single review which was better, but it probably set the 
  Basedir to be the complete http path. 
  Maybe it was not supposed to work that way, but we just got away with 
 it. 

  Interested to see what you think and whether there is a regression or a 
  misuse by us that has been cleared up in RB2.0, the only problem now is 
 we 
  have 100s of requests that are like this 

  Hope you can help, and happy to try patches. 
  Cheers 
  Dan 


  On Thursday, 18 December 2014 20:40:16 UTC, Christian Hammond wrote: 
   
   Hi Dan, 
   
   Would you be able to provide some of the data from that diff 
   (specifically, the 'Index:' line, '---', and '+++' lines in the diff)? 
   
   A lot of work was done on Subversion since 1.7.x, so something may 
 have 
   regressed. I don't recall being able to handle file paths containing 
   'http:' in it, though. 
   
   Christian 
   
   -- 
   Christian Hammond - chi...@chipx86.com   
   Review Board - http://www.reviewboard.org 
   Beanbag, Inc. - http://www.beanbaginc.com 
   
   On Thu, Dec 18, 2014 at 4:32 AM, Daniel Lairdwrote: 
   
   To follow up the error message we get from the diff viewer is: 
   The file '/svn/Common_PythonTests/ 
   http://SVN_SERVER_NAME/svn/Common_PythonTests/path/to/file/base.py' 
   (r1838) could not be found in the repository: 
   
 '/svn/Common_PythonTests/!svn/rvr/1838/http:/SVN_SERVER/svn/Common_PythonTests/path/to/file/base.py'
  
   
   path not found 
   
   The review was created before the migration (1.7.25) - we dumped the 
   database, re-imported and off we went. 
   New requests seem to be ok and many of the old requests are fine. 
 This 
   was probably created via the GUI in 1.7.25 and uploaded a diff file 
   produced by SVN diff. 
   
   Cheers 
   Dan 
   
   On Thursday, 18 December 2014 11:29:18 UTC, Daniel Laird wrote: 
   
   All, 
   
   We were running Reviewboard 1.7.25 and people used a number of 
 methods 
   to create review requests: 
   svn diff - manually upload 
   post-review 
   rbt 
   
   I have just migrated us to Reviewboard 2.0.11 and seem to have hit 
 an 
   issue. 
   
   Some of our reviews no longer display

Backup and Restore of Reviewboard 2.0 failing (mysqldump issues)

[Daniel Laird]

All,

Since RB 1.5 I have backed up the RB installation everynight be by using
mysqldump user=x password=y reviewboard  backup.sql

I have also backed up all attachments etc.
I could then restore the data easily with another script.

I used this method to upgrade from 1.7.25 - 2.0.11 and with the additional 
rb-site upgrade etc, it all worked great.
However, after 2 days I noticed an error on my setup and re-created the 
site and re-imported the data.

At this point it was a RB 2.0.11 backup being restored to a RB2.0.11 
database.
It all appeared to go ok, but when I clicked on any review i got
Error 500, something broke, I also got the following email:

Traceback (most recent call last):

  File 
/usr/local/lib/python2.7/dist-packages/Django-1.6.8-py2.7.egg/django/core/handlers/base.py,
 
line 112, in get_response
response = wrapped_callback(request, *callback_args, **callback_kwargs)

  File 
/usr/local/lib/python2.7/dist-packages/ReviewBoard-2.0.11-py2.7.egg/reviewboard/accounts/decorators.py,
 
line 23, in _check
return view_func(*args, **kwargs)

  File 
/usr/local/lib/python2.7/dist-packages/ReviewBoard-2.0.11-py2.7.egg/reviewboard/site/decorators.py,
 
line 35, in _check
return view_func(request, local_site=local_site, *args, **kwargs)

  File 
/usr/local/lib/python2.7/dist-packages/ReviewBoard-2.0.11-py2.7.egg/reviewboard/reviews/views.py,
 
line 710, in review_detail
'screenshots': screenshots,

  File 
/usr/local/lib/python2.7/dist-packages/ReviewBoard-2.0.11-py2.7.egg/reviewboard/reviews/context.py,
 
line 92, in make_review_request_context
upload_diff_form = UploadDiffForm(review_request, request=request)

  File 
/usr/local/lib/python2.7/dist-packages/ReviewBoard-2.0.11-py2.7.egg/reviewboard/reviews/forms.py,
 
line 95, in __init__
data, *args, **kwargs)

  File 
/usr/local/lib/python2.7/dist-packages/ReviewBoard-2.0.11-py2.7.egg/reviewboard/diffviewer/forms.py,
 
line 41, in __init__
if self.repository.get_scmtool().get_diffs_use_absolute_paths():

  File 
/usr/local/lib/python2.7/dist-packages/ReviewBoard-2.0.11-py2.7.egg/reviewboard/scmtools/models.py,
 
line 230, in get_scmtool
return cls(self)

  File 
/usr/local/lib/python2.7/dist-packages/ReviewBoard-2.0.11-py2.7.egg/reviewboard/scmtools/svn/__init__.py,
 
line 73, in __init__
local_site_name)

  File 
/usr/local/lib/python2.7/dist-packages/ReviewBoard-2.0.11-py2.7.egg/reviewboard/scmtools/svn/__init__.py,
 
line 415, in build_client
client = Client(config_dir, repopath, username, password)

  File 
/usr/local/lib/python2.7/dist-packages/ReviewBoard-2.0.11-py2.7.egg/reviewboard/scmtools/svn/pysvn.py,
 
line 41, in __init__
self.client.set_default_password(six.text_type(password))

UnicodeDecodeError: 'ascii' codec can't decode byte 0xd5 in position 0: 
ordinal not in range(128)

Has something changed in the internal data structures?
Should I be using some options like --default-character-set=utf8 when 
dumping/importing the database?

Hope you can help
Dan

-- 
Get the Review Board Power Pack at http://www.reviewboard.org/powerpack/
---
Sign up for Review Board hosting at RBCommons: https://rbcommons.com/
---
Happy user? Let us know at http://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.

Re: Reviewboard 1.7 - Reviewboard 2.0.x update

[Daniel Laird]

Christian

Many thanks for getting back to me.
I think I know why it is breaking but don't know how to fix it.

The diff looks as I would expect (with all files being something like this):
Index: libs/A/Source/File.cpp
===
--- libs/A/Source/File.cpp (revision 143446)
+++ libs/A/Source/File.cpp (working copy)

I used the webapi to query the Review and spotted the following on the diff 
resource associated with the review request:
basedirhttp://SVN_SERVER/svn/REPO_NAME/branches/features/software/basedir

I suspect that maybe in the old GUI (Reviewboard 1.7) it used to combine 
this basedir and the file path in the diff to draw the differences.
I think that maybe we have a problem where it appends the Repo Path + 
BaseDir + FilePath
Maybe there is a difference in RB2.0.x?

Our databases on legacy SW used a lot of SVN externals, so someone wrote a 
script to generate a single diff file relative to the SVN_SERVER root.
It allowed us a single review which was better, but it probably set the 
Basedir to be the complete http path.
Maybe it was not supposed to work that way, but we just got away with it.

Interested to see what you think and whether there is a regression or a 
misuse by us that has been cleared up in RB2.0, the only problem now is we 
have 100s of requests that are like this

Hope you can help, and happy to try patches.
Cheers
Dan


On Thursday, 18 December 2014 20:40:16 UTC, Christian Hammond wrote:

 Hi Dan,

 Would you be able to provide some of the data from that diff 
 (specifically, the 'Index:' line, '---', and '+++' lines in the diff)?

 A lot of work was done on Subversion since 1.7.x, so something may have 
 regressed. I don't recall being able to handle file paths containing 
 'http:' in it, though.

 Christian

 -- 
 Christian Hammond - chi...@chipx86.com javascript:
 Review Board - http://www.reviewboard.org
 Beanbag, Inc. - http://www.beanbaginc.com

 On Thu, Dec 18, 2014 at 4:32 AM, Daniel Laird daniel@googlemail.com 
 javascript: wrote:

 To follow up the error message we get from the diff viewer is:
 The file '/svn/Common_PythonTests/
 http://SVN_SERVER_NAME/svn/Common_PythonTests/path/to/file/base.py' 
 (r1838) could not be found in the repository: 
 '/svn/Common_PythonTests/!svn/rvr/1838/http:/SVN_SERVER/svn/Common_PythonTests/path/to/file/base.py'
  
 path not found

 The review was created before the migration (1.7.25) - we dumped the 
 database, re-imported and off we went.
 New requests seem to be ok and many of the old requests are fine.  This 
 was probably created via the GUI in 1.7.25 and uploaded a diff file 
 produced by SVN diff.

 Cheers
 Dan 

 On Thursday, 18 December 2014 11:29:18 UTC, Daniel Laird wrote:

 All,

 We were running Reviewboard 1.7.25 and people used a number of methods 
 to create review requests:
 svn diff - manually upload
 post-review
 rbt

 I have just migrated us to Reviewboard 2.0.11 and seem to have hit an 
 issue.

 Some of our reviews no longer display it suggests that a file cant be 
 found.
 And seems to have prepended some sub part of the path before the full 
 file path.
 So I get something like

 /svn/REPONAME/http://SVN_SERVER/svn/REPONAME/path/to/file.cpp cant be 
 found.

 Have we got to change something to make these work?

 I can provide logs offline if that helps?

 Dan

  -- 
 Get the Review Board Power Pack at http://www.reviewboard.org/powerpack/
 ---
 Sign up for Review Board hosting at RBCommons: https://rbcommons.com/
 ---
 Happy user? Let us know at http://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...@googlegroups.com javascript:.
 For more options, visit https://groups.google.com/d/optout.



-- 
Get the Review Board Power Pack at http://www.reviewboard.org/powerpack/
---
Sign up for Review Board hosting at RBCommons: https://rbcommons.com/
---
Happy user? Let us know at http://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.

Reviewboard 1.7 - Reviewboard 2.0.x update

[Daniel Laird]

All,

We were running Reviewboard 1.7.25 and people used a number of methods to 
create review requests:
svn diff - manually upload
post-review
rbt

I have just migrated us to Reviewboard 2.0.11 and seem to have hit an issue.

Some of our reviews no longer display it suggests that a file cant be found.
And seems to have prepended some sub part of the path before the full file 
path.
So I get something like

/svn/REPONAME/http://SVN_SERVER/svn/REPONAME/path/to/file.cpp cant be found.

Have we got to change something to make these work?

I can provide logs offline if that helps?

Dan

-- 
Get the Review Board Power Pack at http://www.reviewboard.org/powerpack/
---
Sign up for Review Board hosting at RBCommons: https://rbcommons.com/
---
Happy user? Let us know at http://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.

Re: Reviewboard 1.7 - Reviewboard 2.0.x update

[Daniel Laird]

To follow up the error message we get from the diff viewer is:
The file 
'/svn/Common_PythonTests/http://SVN_SERVER_NAME/svn/Common_PythonTests/path/to/file/base.py'
 
(r1838) could not be found in the repository: 
'/svn/Common_PythonTests/!svn/rvr/1838/http:/SVN_SERVER/svn/Common_PythonTests/path/to/file/base.py'
 
path not found

The review was created before the migration (1.7.25) - we dumped the 
database, re-imported and off we went.
New requests seem to be ok and many of the old requests are fine.  This was 
probably created via the GUI in 1.7.25 and uploaded a diff file produced by 
SVN diff.

Cheers
Dan 

On Thursday, 18 December 2014 11:29:18 UTC, Daniel Laird wrote:

 All,

 We were running Reviewboard 1.7.25 and people used a number of methods to 
 create review requests:
 svn diff - manually upload
 post-review
 rbt

 I have just migrated us to Reviewboard 2.0.11 and seem to have hit an 
 issue.

 Some of our reviews no longer display it suggests that a file cant be 
 found.
 And seems to have prepended some sub part of the path before the full file 
 path.
 So I get something like

 /svn/REPONAME/http://SVN_SERVER/svn/REPONAME/path/to/file.cpp cant be 
 found.

 Have we got to change something to make these work?

 I can provide logs offline if that helps?

 Dan



-- 
Get the Review Board Power Pack at http://www.reviewboard.org/powerpack/
---
Sign up for Review Board hosting at RBCommons: https://rbcommons.com/
---
Happy user? Let us know at http://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.

Subversion not updating revisions

[Daniel Dominguez]

Hi,

I have a new reviewboard installation (2.0.5), and we've setup a subversion 
repository.  However, the repository is never synchronized with the server 
after it is first added.  I can re-add the repo and am able to see the most 
recent revisions, but after that, the view into the repository becomes 
stale.  By the way, this is when we use the New Review Request link on 
the web UI.  Not sure yet if this is an issue or not with RBTools.  We are 
able to successfully use RBTools for pre-commit reviews.

I've enabled logging and inspected both the reviewboard and the apache log 
files, but nothing interesting shows up.

What is the mechanism to trigger a repository refresh?  Where else can I 
look for useful troubleshooting information?  I'd love to get my 
distributed team to adopt reviewboard, but this is a bit of a blocker at 
the moment.

Any help is appreciated.


OS: Debian 7 64-bit
Python 2.7.3
Using python-svn (since subvertpy was unable to connect to our subversion 
repo)

-- 
Get the Review Board Power Pack at http://www.reviewboard.org/powerpack/
---
Sign up for Review Board hosting at RBCommons: https://rbcommons.com/
---
Happy user? Let us know at http://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.

TeamForge and Reviewboard

[Daniel Laird]

All,

My company have a TeamForge service running and are using GIT as the 
default SCM.
I have a local Reviewboard server and would like to point it at the GIT 
repo - instead of using the default code review tool of gerrit

I am using Reviewboard 1.7.25 currently
The TeamForge project says that the clone URL is:
git clone ssh://MY_USERNAME@gerritforge.DOMAIN:PORT/dashboard

I have tried to setup a repo using my MY_USERNAME/PASSWORD, I have alos 
tried without.
But I am not having much luck.

Has anyone done this, or got it working or any ideas how I could get this 
working?

Many thanks
Dan

-- 
Get the Review Board Power Pack at http://www.reviewboard.org/powerpack/
---
Sign up for Review Board hosting at RBCommons: https://rbcommons.com/
---
Happy user? Let us know at http://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.

Error using post-review and Reviewboard 1.7.25

[Daniel Laird]

All,

We are currently using post-review and Reviewboard 1.7.25 (RBtools 0.5.1).
We added a new Repo to Reviewboard and its listed as repository 26.
However when we try to post a new review to Reviewboard we get:

DEBUG:root:repository info: Path: http://server/svn/repo_name, Base path: 
/trunk, Supports changesets: False
  Attempting to create review request on http://server/svn/repo_name for 
None   HTTP POSTing to
http://reviewboard-server/api/review-requests/: 
{'repository': u'http://server/svn/repo_name'}   Got API Error 206 (HTTP 
code 400): The repository path specified is not in the list of known 
repositories   Error data: {u'stat': u'fail', u'repository': 
u'http://server/svn/repo_name', u'err': {u'msg': u'The repository path 
specified is not in the list of known repositories', u'code': 206}}

Any ideas why it sees the repo in the list, generates the diff and then 
when posting it falls over?

Cheers
Dan

-- 
Get the Review Board Power Pack at http://www.reviewboard.org/powerpack/
---
Sign up for Review Board hosting at RBCommons: https://rbcommons.com/
---
Happy user? Let us know at http://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.

Re: Renaming Repositories

[Daniel Laird]

That sorted it - thank you.
Cheers
Dan

On Tuesday, 13 May 2014 21:49:26 UTC+1, David Trowbridge wrote:

 The fix itself is in djblets 1.7.26. If you upgrade to Review Board 1.7.25 
 you'll get the latest djblets along with it.

 -David


 On Tue, May 13, 2014 at 1:01 PM, Daniel Laird 
 daniel@googlemail.comjavascript:
  wrote:

 I am running 1.7.18 and having this problem.
 I could upgrade to 1.7.25 if you think that will fix it?

 Dan

 --
 Get the Review Board Power Pack at http://www.reviewboard.org/powerpack/
 ---
 Sign up for Review Board hosting at RBCommons: https://rbcommons.com/
 ---
 Happy user? Let us know at http://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...@googlegroups.com javascript:.
 For more options, visit https://groups.google.com/d/optout.




-- 
Get the Review Board Power Pack at http://www.reviewboard.org/powerpack/
---
Sign up for Review Board hosting at RBCommons: https://rbcommons.com/
---
Happy user? Let us know at http://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.

Renaming Repositories

[Daniel Laird]

All,

I wish to change the name of a repository in Reviewboard (am using it in a 
RB extension).
However when i change the field and enter the authentication details I just 
get a box that says 
'Please correct the error below'

However nothing in shown.
I have check the logs and nothing is showing here either.

Are you able to change a repository name (the name RB gives it)?
I am running RB 1.7.18 on Ubuntu Linux

Cheers
Dan 

-- 
Get the Review Board Power Pack at http://www.reviewboard.org/powerpack/
---
Sign up for Review Board hosting at RBCommons: https://rbcommons.com/
---
Happy user? Let us know at http://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.

Re: Renaming Repositories

[Daniel Laird]

We are running 0.7.9
/usr/local/lib/python2.6/dist-packages/Djblets-0.7.9-py2.6.egg

Cheers
Dan

On Tuesday, 13 May 2014 10:07:55 UTC+1, David Trowbridge wrote:

 Dan,

 What version of djblets is installed?

 -David


 On Tue, May 13, 2014 at 1:28 AM, Daniel Laird 
 daniel@googlemail.comjavascript:
  wrote:

 All,

 I wish to change the name of a repository in Reviewboard (am using it in 
 a RB extension).
 However when i change the field and enter the authentication details I 
 just get a box that says 
 'Please correct the error below'

 However nothing in shown.
 I have check the logs and nothing is showing here either.

 Are you able to change a repository name (the name RB gives it)?
 I am running RB 1.7.18 on Ubuntu Linux

 Cheers
 Dan 
  
 -- 
 Get the Review Board Power Pack at http://www.reviewboard.org/powerpack/
 ---
 Sign up for Review Board hosting at RBCommons: https://rbcommons.com/
 ---
 Happy user? Let us know at http://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...@googlegroups.com javascript:.
 For more options, visit https://groups.google.com/d/optout.




-- 
Get the Review Board Power Pack at http://www.reviewboard.org/powerpack/
---
Sign up for Review Board hosting at RBCommons: https://rbcommons.com/
---
Happy user? Let us know at http://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.

Re: Renaming Repositories

[Daniel Laird]

I am running 1.7.18 and having this problem.
I could upgrade to 1.7.25 if you think that will fix it?

Dan

-- 
Get the Review Board Power Pack at http://www.reviewboard.org/powerpack/
---
Sign up for Review Board hosting at RBCommons: https://rbcommons.com/
---
Happy user? Let us know at http://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.

Re: Downgrading Reviewboard 2.0RC2 - 1.7.25

[Daniel Laird]

My original email was not very clear.
On my backup server I want to remove the 2.0RC2 installation and get a 
1.7.x installation running.
However I removed all the RB2.0RC2 files and then dropped the reviewboard 
table from mysql database.
I then did a clean installation of 1.7.25.
I then imported the data from my old server (which was running 1.7.18) - at 
this point I would have thought I would have a 'clean' installation.
However the rb-site upgrade fails and when I visit the homepage I get the 
errors below.

It is almost as if there are some settings files that are stored someone 
not in my installation directory and not in the mysql database.
Might there be some files related to django evolutions that I have to 
delete?

Dan



On Friday, 9 May 2014 20:27:23 UTC+1, Christian Hammond wrote:

 Hi Dan,

 There are many reasons why a 2.0 database will not cleanly turn into a 
 1.7.x database. We stick certain data in new tables/columns that don't 
 exist in 1.7. We've migrated data from those older columns. 
 Django-Evolution has no concept of a downgrade and will likely just break 
 every future attempt to ever upgrade your database again.

 You're looking to keep the 2.0 data in 1.7.x? Any chance you can just 
 upgrade to 2.0 RC 3 and stick with that until 2.0 is out (very very soon)?

 Christian

 -- 
 Christian Hammond - chi...@chipx86.com javascript:
 Review Board - http://www.reviewboard.org
 Beanbag, Inc. - http://www.beanbaginc.com


 On Fri, May 9, 2014 at 8:04 AM, Daniel Laird 
 daniel@googlemail.comjavascript:
  wrote:

 All,

 Due to various issue my development server has to be re-purposed as the 
 main server.
 This has meant I have to revert from 2.0RC2 - 1.7.25 (the latest stable 
 version).

 At this point I usually import the database and media files and all works 
 ok.
 However I get the following:
 Traceback (most recent call last):

   File 
 /usr/local/lib/python2.6/dist-packages/Django-1.4.12-py2.6.egg/django/core/handlers/base.py,
  
 line 89, in get_response
 response = middleware_method(request)

   File 
 /usr/local/lib/python2.6/dist-packages/ReviewBoard-1.7.25-py2.6.egg/reviewboard/accounts/middleware.py,
  
 line 12, in process_request
 user = Profile.objects.get(user=request.user)

   File 
 /usr/local/lib/python2.6/dist-packages/Django-1.4.12-py2.6.egg/django/db/models/manager.py,
  
 line 131, in get
 return self.get_query_set().get(*args, **kwargs)

   File 
 /usr/local/lib/python2.6/dist-packages/Django-1.4.12-py2.6.egg/django/db/models/query.py,
  
 line 361, in get
 num = len(clone)

   File 
 /usr/local/lib/python2.6/dist-packages/Django-1.4.12-py2.6.egg/django/db/models/query.py,
  
 line 85, in __len__
 self._result_cache = list(self.iterator())

   File 
 /usr/local/lib/python2.6/dist-packages/Django-1.4.12-py2.6.egg/django/db/models/query.py,
  
 line 291, in iterator
 for row in compiler.results_iter():

   File 
 /usr/local/lib/python2.6/dist-packages/Django-1.4.12-py2.6.egg/django/db/models/sql/compiler.py,
  
 line 763, in results_iter
 for rows in self.execute_sql(MULTI):

   File 
 /usr/local/lib/python2.6/dist-packages/Django-1.4.12-py2.6.egg/django/db/models/sql/compiler.py,
  
 line 818, in execute_sql
 cursor.execute(sql, params)

   File 
 /usr/local/lib/python2.6/dist-packages/Django-1.4.12-py2.6.egg/django/db/backends/mysql/base.py,
  
 line 114, in execute
 return self.cursor.execute(query, args)

   File 
 /usr/local/lib/python2.6/dist-packages/MySQL_python-1.2.3-py2.6-linux-x86_64.egg/MySQLdb/cursors.py,
  
 line 174, in execute
 self.errorhandler(self, exc, value)

   File 
 /usr/local/lib/python2.6/dist-packages/MySQL_python-1.2.3-py2.6-linux-x86_64.egg/MySQLdb/connections.py,
  
 line 36, in defaulterrorhandler
 raise errorclass, errorvalue

 DatabaseError: (1054, Unknown column 'accounts_profile.show_submitted' 
 in 'field list')

 I thought this could be a left over column (from the 2.0 installation) - 
 so I dropped the tables and re-installed from scratch.
 I then checked at my 'live' server is running 1.7.18 whereas this is 
 running 1.7.25.
 So I then tried an rb-site upgrade and get:
 Rebuilding directory structure
 Updating database. This may take a while.

 The log output below, including warnings and errors,
 can be ignored unless upgrade fails.

 -- begin log output --
 Creating tables ...

 [!] Unable to execute the manager command syncdb: No module named
 db.fields

 [!] Unable to execute the manager command evolve: No module named
 db.fields
 --- end log output ---

 Resetting in-database caches.

 Upgrade complete!

 But still no luck.
 I know I am doing something stupid - but can anyone help me spot it
 Any help will be much appreciated

 Dan

 -- 
 Get the Review Board Power Pack at http://www.reviewboard.org/powerpack/
 ---
 Sign up for Review Board hosting at RBCommons: https://rbcommons.com

Downgrading Reviewboard 2.0RC2 - 1.7.25

[Daniel Laird]

All,

Due to various issue my development server has to be re-purposed as the 
main server.
This has meant I have to revert from 2.0RC2 - 1.7.25 (the latest stable 
version).

At this point I usually import the database and media files and all works 
ok.
However I get the following:
Traceback (most recent call last):

  File 
/usr/local/lib/python2.6/dist-packages/Django-1.4.12-py2.6.egg/django/core/handlers/base.py,
 
line 89, in get_response
response = middleware_method(request)

  File 
/usr/local/lib/python2.6/dist-packages/ReviewBoard-1.7.25-py2.6.egg/reviewboard/accounts/middleware.py,
 
line 12, in process_request
user = Profile.objects.get(user=request.user)

  File 
/usr/local/lib/python2.6/dist-packages/Django-1.4.12-py2.6.egg/django/db/models/manager.py,
 
line 131, in get
return self.get_query_set().get(*args, **kwargs)

  File 
/usr/local/lib/python2.6/dist-packages/Django-1.4.12-py2.6.egg/django/db/models/query.py,
 
line 361, in get
num = len(clone)

  File 
/usr/local/lib/python2.6/dist-packages/Django-1.4.12-py2.6.egg/django/db/models/query.py,
 
line 85, in __len__
self._result_cache = list(self.iterator())

  File 
/usr/local/lib/python2.6/dist-packages/Django-1.4.12-py2.6.egg/django/db/models/query.py,
 
line 291, in iterator
for row in compiler.results_iter():

  File 
/usr/local/lib/python2.6/dist-packages/Django-1.4.12-py2.6.egg/django/db/models/sql/compiler.py,
 
line 763, in results_iter
for rows in self.execute_sql(MULTI):

  File 
/usr/local/lib/python2.6/dist-packages/Django-1.4.12-py2.6.egg/django/db/models/sql/compiler.py,
 
line 818, in execute_sql
cursor.execute(sql, params)

  File 
/usr/local/lib/python2.6/dist-packages/Django-1.4.12-py2.6.egg/django/db/backends/mysql/base.py,
 
line 114, in execute
return self.cursor.execute(query, args)

  File 
/usr/local/lib/python2.6/dist-packages/MySQL_python-1.2.3-py2.6-linux-x86_64.egg/MySQLdb/cursors.py,
 
line 174, in execute
self.errorhandler(self, exc, value)

  File 
/usr/local/lib/python2.6/dist-packages/MySQL_python-1.2.3-py2.6-linux-x86_64.egg/MySQLdb/connections.py,
 
line 36, in defaulterrorhandler
raise errorclass, errorvalue

DatabaseError: (1054, Unknown column 'accounts_profile.show_submitted' in 
'field list')

I thought this could be a left over column (from the 2.0 installation) - so 
I dropped the tables and re-installed from scratch.
I then checked at my 'live' server is running 1.7.18 whereas this is 
running 1.7.25.
So I then tried an rb-site upgrade and get:
Rebuilding directory structure
Updating database. This may take a while.

The log output below, including warnings and errors,
can be ignored unless upgrade fails.

-- begin log output --
Creating tables ...

[!] Unable to execute the manager command syncdb: No module named
db.fields

[!] Unable to execute the manager command evolve: No module named
db.fields
--- end log output ---

Resetting in-database caches.

Upgrade complete!

But still no luck.
I know I am doing something stupid - but can anyone help me spot it
Any help will be much appreciated

Dan

-- 
Get the Review Board Power Pack at http://www.reviewboard.org/powerpack/
---
Sign up for Review Board hosting at RBCommons: https://rbcommons.com/
---
Happy user? Let us know at http://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.

Re: Using --revision-range with single revision in subversion

[Daniel F]

Perfect - thanks!
I'll try to test when I have some free time.

On Thursday, January 30, 2014 12:05:29 PM UTC-5, Daniel F wrote:

 I haven't been able to successfully use *rbt post 
 --revision-range=SINGLE_REVISION*.
 It works fine if I use the syntax *rbt post --revision-range=*
 *OLD_REV:NEW_REV*
 Am I doing something wrong?


 My steps:

1. Commit some files to svn revision 123.  Note: there are still 
modified files in my working copy which I do NOT want to review.
2. I want to review just the files committed in revision 123, so I 
type:
*rbt post --revision-range=123*
This creates a review request with a diff of the *changes in my 
working copy* instead of showing me a diff of what was committed.  It 
was as if I posted a pre-commit review by typing: *rbt post* .
3. Suddenly it works if I type: *rbt post --revision-range=122:123*


 Using:
 RBTools 0.5.5, ReviewBoard 1.7.21, subversion 1.7.8, python 2.7.5


-- 
Get the Review Board Power Pack at http://www.reviewboard.org/powerpack/
---
Sign up for Review Board hosting at RBCommons: https://rbcommons.com/
---
Happy user? Let us know at http://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/groups/opt_out.

Word-break making it difficult to read numbers of reviews in dashboard navbar

[Daniel Kan]

In the dashboard navbar, one of my users noticed that if a review group is 
sufficiently long and the group has =100 reviews, the UI wraps both the 
group name and the number of reviews. Wrapping the group name isn't too big 
a deal. However, wrapping the number of reviews makes it a bit hard to 
read, especially when a user is a member of many review groups. I changed 
word-break from 'break-all' to 'normal' in common.min.css which seemed to 
keep the number from wrapping. By doing this though, am I ignoring any 
design decision that led to 'word-break break-all' from being used 
explicitly?

I'm using 1.7.18 btw... Thanks!!!

-- 
Get the Review Board Power Pack at http://www.reviewboard.org/powerpack/
---
Sign up for Review Board hosting at RBCommons: https://rbcommons.com/
---
Happy user? Let us know at http://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/groups/opt_out.

Re: Change in LDAP authentication behavior in 1.7.14 breaks environments that don't allow anonymous searches

[Daniel Kan]

David,
  Sorry for the massive delay! I've finally posted it at 
https://reviews.reviewboard.org/r/4953/.

On Friday, October 18, 2013 1:51:59 AM UTC-7, David Trowbridge wrote:

 Daniel,

 I'd suggest making a quick clone of the reviewboard repository, making 
 your change, and then posting your diff against the reviewboard repository 
 by either creating a diff manually (git diff --full-index) or using rbt 
 post.

 -David


 On Tue, Oct 15, 2013 at 4:57 PM, Daniel Kan danie...@gmail.comjavascript:
  wrote:

 David,
   Sure, although I'm not sure how to post a bare diff there without 
 specifying a repository. Is that possible? Thanks!


 On Thursday, September 26, 2013 2:43:43 PM UTC-7, David Trowbridge wrote:

 Would you mind submitting your change to http://reviews.reviewboard.org/?

 Thanks!
 -David


 On Tue, Sep 24, 2013 at 7:32 PM, Daniel Kan danie...@gmail.com wrote:

 I am using RB 1.7.x with an LDAP authentication backend and it's been 
 working fine up through 1.7.13. With 1.7.14, the authentication mechanism 
 no longer works in my environment. After some digging, it appears that 
 there was a change in reviewboard/accounts/backends.py to search 
 anonymously for the user before attempting to bind. The LDAP server in my 
 environment is set up to not return any results to anonymous queries and 
 so 
 the authentication attempt fails. Is there a reason this was changed? 
 Thanks!!!

 In an attempt to preserve the existing behavior of doing an anonymous 
 search but then attempting a bind as a specific user if the anonymous 
 search returns nothing, I made the following change which works for me:
 --- reviewboard/accounts/backends.py.orig   2013-09-24 
 16:18:47.0 -0700
 +++ reviewboard/accounts/backends.py2013-09-24 16:20:01.0 
 -0700
 @@ -210,7 +210,10 @@
   search = ldapo.search_s(settings.LDAP_BASE_DN,
  ldap.SCOPE_SUBTREE,
  uid)
 -userbinding = search[0][0]
 +   if (len(search)  0):
 +   userbinding = search[0][0]
 +   else:
 +   userbinding=','.join([uid,
 settings.LDAP_BASE_DN])
  ldapo.bind_s(userbinding, password)
  
  return self.get_or_create_user(username, None, ldapo)

 -- 
 Get the Review Board Power Pack at http://www.reviewboard.org/
 powerpack/
 ---
 Sign up for Review Board hosting at RBCommons: https://rbcommons.com/
 ---
 Happy user? Let us know at http://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...@googlegroups.com.

 For more options, visit https://groups.google.com/groups/opt_out.


  -- 
 Get the Review Board Power Pack at http://www.reviewboard.org/powerpack/
 ---
 Sign up for Review Board hosting at RBCommons: https://rbcommons.com/
 ---
 Happy user? Let us know at http://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...@googlegroups.com javascript:.
 For more options, visit https://groups.google.com/groups/opt_out.




-- 
Get the Review Board Power Pack at http://www.reviewboard.org/powerpack/
---
Sign up for Review Board hosting at RBCommons: https://rbcommons.com/
---
Happy user? Let us know at http://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/groups/opt_out.

Re: Change in LDAP authentication behavior in 1.7.14 breaks environments that don't allow anonymous searches

[Daniel Kan]

David,
  Sure, although I'm not sure how to post a bare diff there without 
specifying a repository. Is that possible? Thanks!

On Thursday, September 26, 2013 2:43:43 PM UTC-7, David Trowbridge wrote:

 Would you mind submitting your change to http://reviews.reviewboard.org/ ?

 Thanks!
 -David


 On Tue, Sep 24, 2013 at 7:32 PM, Daniel Kan danie...@gmail.comjavascript:
  wrote:

 I am using RB 1.7.x with an LDAP authentication backend and it's been 
 working fine up through 1.7.13. With 1.7.14, the authentication mechanism 
 no longer works in my environment. After some digging, it appears that 
 there was a change in reviewboard/accounts/backends.py to search 
 anonymously for the user before attempting to bind. The LDAP server in my 
 environment is set up to not return any results to anonymous queries and so 
 the authentication attempt fails. Is there a reason this was changed? 
 Thanks!!!

 In an attempt to preserve the existing behavior of doing an anonymous 
 search but then attempting a bind as a specific user if the anonymous 
 search returns nothing, I made the following change which works for me:
 --- reviewboard/accounts/backends.py.orig   2013-09-24 
 16:18:47.0 -0700
 +++ reviewboard/accounts/backends.py2013-09-24 16:20:01.0 
 -0700
 @@ -210,7 +210,10 @@
  search = ldapo.search_s(settings.LDAP_BASE_DN,
  ldap.SCOPE_SUBTREE,
  uid)
 -userbinding = search[0][0]
 +   if (len(search)  0):
 +   userbinding = search[0][0]
 +   else:
 +   userbinding=','.join([uid,settings.LDAP_BASE_DN])
  ldapo.bind_s(userbinding, password)
  
  return self.get_or_create_user(username, None, ldapo)

 -- 
 Get the Review Board Power Pack at http://www.reviewboard.org/powerpack/
 ---
 Sign up for Review Board hosting at RBCommons: https://rbcommons.com/
 ---
 Happy user? Let us know at http://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...@googlegroups.com javascript:.
 For more options, visit https://groups.google.com/groups/opt_out.




-- 
Get the Review Board Power Pack at http://www.reviewboard.org/powerpack/
---
Sign up for Review Board hosting at RBCommons: https://rbcommons.com/
---
Happy user? Let us know at http://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/groups/opt_out.

Change in LDAP authentication behavior in 1.7.14 breaks environments that don't allow anonymous searches

[Daniel Kan]

I am using RB 1.7.x with an LDAP authentication backend and it's been 
working fine up through 1.7.13. With 1.7.14, the authentication mechanism 
no longer works in my environment. After some digging, it appears that 
there was a change in reviewboard/accounts/backends.py to search 
anonymously for the user before attempting to bind. The LDAP server in my 
environment is set up to not return any results to anonymous queries and so 
the authentication attempt fails. Is there a reason this was changed? 
Thanks!!!

In an attempt to preserve the existing behavior of doing an anonymous 
search but then attempting a bind as a specific user if the anonymous 
search returns nothing, I made the following change which works for me:
--- reviewboard/accounts/backends.py.orig   2013-09-24 
16:18:47.0 -0700
+++ reviewboard/accounts/backends.py2013-09-24 16:20:01.0 -0700
@@ -210,7 +210,10 @@
 search = ldapo.search_s(settings.LDAP_BASE_DN,
 ldap.SCOPE_SUBTREE,
 uid)
-userbinding = search[0][0]
+   if (len(search)  0):
+   userbinding = search[0][0]
+   else:
+   userbinding=','.join([uid,settings.LDAP_BASE_DN])
 ldapo.bind_s(userbinding, password)
 
 return self.get_or_create_user(username, None, ldapo)

-- 
Get the Review Board Power Pack at http://www.reviewboard.org/powerpack/
---
Sign up for Review Board hosting at RBCommons: https://rbcommons.com/
---
Happy user? Let us know at http://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/groups/opt_out.

Error with home directory path expansion when using rbt?

[Daniel Kan]

New user to ReviewBoard here. Great tool with an intuitive interface, 
thanks for your efforts! Recently, one of my users was using 'rbt' from 
RBTools 0.5.1 and noticed that it would not accept '~' in the value for 
--diff-filename. My guess was that rbt does not run os.path.expanduser on 
the value, so I modified rbtools/commands/post.py with the following:
--- orig/rbtools/commands/post.py   2013-05-30 00:36:44.0 -0700
+++ RBTools-0.5.1.1-py2.7.egg/rbtools/commands/post.py  2013-06-11 
15:33:05.449908000 -0700
@@ -455,8 +455,11 @@
 diff = sys.stdin.read()
 else:
 try:
-diff_path = os.path.join(origcwd,
- self.options.diff_filename)
+if (self.options.diff_filename.startswith('~')):
+   diff_path = 
os.path.expanduser(self.options.diff_filename)
+else:
+diff_path = os.path.join(origcwd,
+ 
self.options.diff_filename)
 fp = open(diff_path, 'r')
 diff = fp.read()
 fp.close()

My question is, is this sufficient to fix the problem or are there more 
places where I need to run expanduser? Also, although debug output 
indicates that it is unable to open the file, running rbt in normal mode is 
silent about not being able to find the difffile. Thanks!!!

-- 
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
--- 
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/groups/opt_out.

How can I remove an extension?

[Daniel Kelly]

I tried and failed to install the ascii art shipit extension.

2013-06-10 18:41:31,649 - ERROR -  - Error loading extension
shipit_ascii_art_extension: No module named
shipit_ascii_art_extension.extension
2013-06-10 18:41:36,053 - ERROR -  - Error loading extension
shipit_ascii_art_extension: No module named
shipit_ascii_art_extension.extension
2013-06-10 18:42:51,401 - ERROR -  - Error loading extension
shipit_ascii_art_extension: No module named
shipit_ascii_art_extension.extension
2013-06-10 18:43:53,995 - ERROR -  - Error loading extension
shipit_ascii_art_extension: No module named
shipit_ascii_art_extension.extension
2013-06-10 18:44:15,601 - ERROR -  - Error loading extension
shipit_ascii_art_extension: No module named
shipit_ascii_art_extension.extension
2013-06-10 18:46:30,115 - ERROR -  - Error loading extension
shipit_ascii_art_extension: No module named
shipit_ascii_art_extension.extension
2013-06-10 18:46:31,594 - ERROR -  - Error loading extension
shipit_ascii_art_extension: No module named
shipit_ascii_art_extension.extension
2013-06-10 18:46:36,524 - ERROR -  - Error loading extension
shipit_ascii_art_extension: No module named
shipit_ascii_art_extension.extension
2013-06-10 18:47:52,215 - ERROR -  - Error loading extension
shipit_ascii_art_extension: No module named
shipit_ascii_art_extension.extension
2013-06-10 18:48:54,037 - ERROR -  - Error loading extension
shipit_ascii_art_extension: No module named
shipit_ascii_art_extension.extension

any idea how to remove it?

-- 
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
--- 
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/groups/opt_out.

Re: How can I remove an extension?

[Daniel Kelly]

 Extensions should be disabled from within the API before uninstalling from
 the filesystem. We should make this more bullet-proof though, maybe disable
 if it's not found.

 Are you still able to use RB?


RB works fine


 Is it just spewing logs?


Yes


 If so, go into the admin UI - Database - Extensions, find the ascii art
 extension, and disable it from there.


It's not listed.
According to that, there are no extensions registered


On Jun 10, 2013, at 11:50 AM, Daniel Kelly arrays.tostr...@gmail.com
 wrote:

 I tried and failed to install the ascii art shipit extension.

 2013-06-10 18:41:31,649 - ERROR -  - Error loading extension
 shipit_ascii_art_extension: No module named
 shipit_ascii_art_extension.extension
 2013-06-10 18:41:36,053 - ERROR -  - Error loading extension
 shipit_ascii_art_extension: No module named
 shipit_ascii_art_extension.extension
 2013-06-10 18:42:51,401 - ERROR -  - Error loading extension
 shipit_ascii_art_extension: No module named
 shipit_ascii_art_extension.extension
 2013-06-10 18:43:53,995 - ERROR -  - Error loading extension
 shipit_ascii_art_extension: No module named
 shipit_ascii_art_extension.extension
 2013-06-10 18:44:15,601 - ERROR -  - Error loading extension
 shipit_ascii_art_extension: No module named
 shipit_ascii_art_extension.extension
 2013-06-10 18:46:30,115 - ERROR -  - Error loading extension
 shipit_ascii_art_extension: No module named
 shipit_ascii_art_extension.extension
 2013-06-10 18:46:31,594 - ERROR -  - Error loading extension
 shipit_ascii_art_extension: No module named
 shipit_ascii_art_extension.extension
 2013-06-10 18:46:36,524 - ERROR -  - Error loading extension
 shipit_ascii_art_extension: No module named
 shipit_ascii_art_extension.extension
 2013-06-10 18:47:52,215 - ERROR -  - Error loading extension
 shipit_ascii_art_extension: No module named
 shipit_ascii_art_extension.extension
 2013-06-10 18:48:54,037 - ERROR -  - Error loading extension
 shipit_ascii_art_extension: No module named
 shipit_ascii_art_extension.extension

 any idea how to remove it?


 --
 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
 ---
 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/groups/opt_out.




  --
 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
 ---
 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/groups/opt_out.




-- 
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
--- 
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/groups/opt_out.

Re: How can I remove an extension?

[Daniel Kelly]

No that's not it.  I've rebooted this box (  a VM ) multiple times since
the problem started happening.


On Mon, Jun 10, 2013 at 8:12 PM, Christian Hammond chip...@chipx86.comwrote:

 On Jun 10, 2013, at 1:07 PM, Daniel Kelly arrays.tostr...@gmail.com
 wrote:


 Extensions should be disabled from within the API before uninstalling
 from the filesystem. We should make this more bullet-proof though, maybe
 disable if it's not found.

 Are you still able to use RB?


 RB works fine


  Is it just spewing logs?


 Yes


 If so, go into the admin UI - Database - Extensions, find the ascii art
 extension, and disable it from there.


 It's not listed.
 According to that, there are no extensions registered


 Try reloading Apache. There may just be other threads that still think it
 should be there.

 --
 Christian Hammond - chip...@chipx86.com
 Review Board - http://www.reviewboard.org
 Beanbag, Inc. - http://www.beanbaginc.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+unsubscr...@googlegroups.com
 For more options, visit this group at
 http://groups.google.com/group/reviewboard?hl=en
 ---
 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/groups/opt_out.




-- 
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
--- 
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/groups/opt_out.

myisam or innodb ?

[Daniel Kelly]

This is probably a Django issue, but if I'm using mysql, should my tables 
be in innodb format or myisam?

-- 
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
--- 
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/groups/opt_out.

Re: Chrome Extension: Review Board Notifier

[Daniel Laird]

I am logged into Reviewboard via Chome but still getting the spinning icon 
- is there a way to find out which cookie file its using?

Dan

On Monday, April 22, 2013 9:17:00 PM UTC+1, Kirollos Risk wrote:

 Ah, once installed and opened, it uses the the appropriate Review Board 
 cookie.

 If it spins indefinitely, it may mean that you haven't logged in to Review 
 Board, thus the authentication cookie isn't set.  

 Currently, you *cannot* log in via the extension itself (I am working on 
 that).

 - Kiro


 On Monday, 22 April 2013 08:56:14 UTC-7, Daniel Laird wrote:

 I have downloaded and installed it - I have then pointed it at my company 
 server.
 But it just spins - with the busy icon.

 Does it require anonymous access or anything as it does not seem to be 
 asking for username/password?

 Dan

 On Saturday, April 20, 2013 12:15:31 AM UTC+1, Kirollos Risk wrote:

 Hey everyone!

 Here's a Chrome extension, *Review Board Notifie*r:

 *Features:*

- Quick glance on an review board requests assigned to you, whether 
they are waiting for *ship it*, or have been given *ship it*
- Click on an item to quickly get to the details
- Notifies you when there are new RB requests assigned to you
- Configurable via the settings page


 *Screenshots:*



 https://lh5.googleusercontent.com/-JIBwdna5Pso/UXHMoITr7eI/G0o/3QNhzGqN1po/s1600/Screen+Shot+2013-04-19+at+3.54.03+PM.png














































 https://lh4.googleusercontent.com/-Lz-NmhJCzPA/UXHNSGlCA7I/G04/fB8AWAAchRU/s1600/Screen+Shot+2013-04-19+at+4.03.15+PM.png










































 You can install it for free, via https://github.com/krisk/rbn

 - Kiro



-- 
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
--- 
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/groups/opt_out.

Re: Chrome Extension: Review Board Notifier

[Daniel Laird]

I have downloaded and installed it - I have then pointed it at my company 
server.
But it just spins - with the busy icon.

Does it require anonymous access or anything as it does not seem to be 
asking for username/password?

Dan

On Saturday, April 20, 2013 12:15:31 AM UTC+1, Kirollos Risk wrote:

 Hey everyone!

 Here's a Chrome extension, *Review Board Notifie*r:

 *Features:*

- Quick glance on an review board requests assigned to you, whether 
they are waiting for *ship it*, or have been given *ship it*
- Click on an item to quickly get to the details
- Notifies you when there are new RB requests assigned to you
- Configurable via the settings page


 *Screenshots:*



 https://lh5.googleusercontent.com/-JIBwdna5Pso/UXHMoITr7eI/G0o/3QNhzGqN1po/s1600/Screen+Shot+2013-04-19+at+3.54.03+PM.png














































 https://lh4.googleusercontent.com/-Lz-NmhJCzPA/UXHNSGlCA7I/G04/fB8AWAAchRU/s1600/Screen+Shot+2013-04-19+at+4.03.15+PM.png










































 You can install it for free, via https://github.com/krisk/rbn

 - Kiro



-- 
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
--- 
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/groups/opt_out.

rb-extension-pack - installation and usage?

[Daniel Laird]

I just found this pack of extensions and saw the XML Review Extension.
However there is little explaining what this does or how it enhances the 
process of reviewing XML.
Can anyone tell me what it adds? I thought I had installed it properly and 
then looked at a review with XML files and noted nothing different.

Cheers
Dan


-- 
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 RC 1 is released!

[Daniel Laird]

Very exciting - I look forward to getting this installed.
One question I have is about RBTools?

I am using ReviewBot and that depends on the API branch of RBtools.
Do you know if the API branch is going to be merged / released at the same 
time?

Also I understand there was a plan for a Python API for 1.7 (as well as the 
RESTful API) - is that still planned or has that slipped to 1.8 (or later)

Cheers
Dan

On Thursday, November 29, 2012 10:08:07 PM UTC, Christian Hammond wrote:

 Review Board Better Late Than Never 1.7 RC 1 is out. We had hoped to get 
 this out earlier, but hit some problems and annoyances during some 
 production use that we felt should be fixed first. So trust me, it was 
 worth the wait :)

 There's a lot of fixes, enhancements, and some small extension interface 
 changes in this release.

 See the news and release notes for more information.

 http://www.reviewboard.org/news/2012/11/29/review-board-1-7-rc-1-released/

 We'd like to ask those who plan to upgrade to 1.7 to please give it a try. 
 Aside from major fixes, we are hoping there won't be many changes made 
 until the final 1.7, so now's the time to report any regressions you hit.

 Christian

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

SVN 1.7 support

[Daniel Laird]

Has anyone managed to patch or modify reviewboard so that diffs generated 
under SVN 1.7 actually work with post-review and ReviewBoard?

The upgrade to SVN is about to hit my department and it will be a shame to 
stop reviewing code as a result of this change

Dan

-- 
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

[Daniel Laird]

I checked permissions - it turned out that ext had permissions of 755 and 
infact needs 775 as the user is in group www-data but the www-data needs w 
access to ext.
Perhaps that could be added to the debug message to help other people who 
make the same mistake as me.

Cheers
Dan

On Friday, August 3, 2012 7:49:01 PM UTC+1, Christian Hammond wrote:

 Hi Daniel,

 The CACHE_* warnings you get are harmless. Just new APIs in Django we have 
 to migrate to.

 We can't detect the config file location. The user may be using our 
 auto-generated ones, or they may have copied/pasted into their Apache 
 configuration. There's no way for us to know.

 So I wasn't completely sure, but did you create the ext directory and set 
 the ownership on it? Can you show me a ls -ld of that directory? That 
 should be the last thing you'd need to configure.

 Christian

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


 On Fri, Aug 3, 2012 at 6:39 AM, Daniel Laird 
 daniel.j.la...@googlemail.com wrote:

 I have tried to install this release.
 I had to manually installed Djblets 0.7 and then I could install by 
 specifying -f to easy_install.
 Onc done I then did 
 rb-site upgrade /path/my/site

 This took a while and generated the following:
 Upgrading Review Board from 1.6.11 to 1.7 beta 1
 There are unapplied evolutions for auth.
 There are unapplied evolutions for accounts.
 There are unapplied evolutions for diffviewer.
 Adding baseline version for new models
 Project signature has changed - an evolution is required
 Installing custom SQL ...
 Installing indexes ...
 Installed 0 object(s) from 0 fixture(s)

 Evolution successful.
 Resetting in-database caches.
 Upgrade complete.

 The location of static media files (CSS, JavaScript, images)
 has changed. You will need to make manual changes to 
 your web server configuration.

 For Apache, you will need to add:

 Alias /static /var/www/reviewboard-satv.tandbergtv.lan/htdocs/static

 For lighttpd, add the following to alias.url:

 /static = /var/www/reviewboard-satv.tandbergtv.lan/htdocs/static

 Once you have made these changes, type the following 
 to resolve this:

 $ rb-site manage /var/www/reviewboard-satv.tandbergtv.lan 
 resolve-check static-media

 I think the help could tell you which conf file to update with the Alias. 
  
 However did that and then ran the command specified.

 I then restarted memcached and apache.
 However I now get a splash screen that says:

 *Manual server updates required
 **A recent upgrade requires manual updates to be made on this server. 
 After these changes are made, you should restart your server.
 **If you have any questions or problems, please contact us on our 
 mailing list.*


 But I have done this:
 It also says:

 *Your site's extensions media directory isn't properly set up. This 
 directory is where Review Board will store various extension media files.
 **Your extensions media directory is currently at: 
 /var/www/reviewboard-satv.tandbergtv.lan/htdocs/media/ext
 **Permission problems
 **The directory must be writable by the web server. On Linux/Unix/Mac, 
 you can fix this by typing:
 **$ sudo chown -R www-data 
 /var/www/reviewboard-satv.tandbergtv.lan/htdocs/media/ext
 **On Windows, right-click the data directory and change the ownership 
 to www-data.*

  
 But I have also checked this.
 Have you got any logging I can try and turn on to debug further?
 Finally I am also getting a lot of these when running any rb-site 
 commands:

 */usr/local/lib/python2.6/dist-packages/Django-1.4.1-py2.6.egg/django/core/cache/__init__.py:82:
  
 DeprecationWarning: settings.CACHE_* is deprecated; use settings.CACHES 
 instead.
 **  DeprecationWarning*


 I am running on Ubuntu 10.04 64 bit Server.

 Cheers
 Dan


 On Friday, August 3, 2012 9:49:58 AM UTC+1, 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/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 - chip...@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+unsubscr...@googlegroups.com

Re: Review Board 1.7 beta 1 released

[Daniel Laird]

I have tried to install this release.
I had to manually installed Djblets 0.7 and then I could install by 
specifying -f to easy_install.
Onc done I then did 
rb-site upgrade /path/my/site

This took a while and generated the following:
Upgrading Review Board from 1.6.11 to 1.7 beta 1
There are unapplied evolutions for auth.
There are unapplied evolutions for accounts.
There are unapplied evolutions for diffviewer.
Adding baseline version for new models
Project signature has changed - an evolution is required
Installing custom SQL ...
Installing indexes ...
Installed 0 object(s) from 0 fixture(s)

Evolution successful.
Resetting in-database caches.
Upgrade complete.

The location of static media files (CSS, JavaScript, images)
has changed. You will need to make manual changes to 
your web server configuration.

For Apache, you will need to add:

Alias /static /var/www/reviewboard-satv.tandbergtv.lan/htdocs/static

For lighttpd, add the following to alias.url:

/static = /var/www/reviewboard-satv.tandbergtv.lan/htdocs/static

Once you have made these changes, type the following 
to resolve this:

$ rb-site manage /var/www/reviewboard-satv.tandbergtv.lan resolve-check 
static-media

I think the help could tell you which conf file to update with the Alias.  
However did that and then ran the command specified.

I then restarted memcached and apache.
However I now get a splash screen that says:

*Manual server updates required
 **A recent upgrade requires manual updates to be made on this server. 
 After these changes are made, you should restart your server.
 **If you have any questions or problems, please contact us on our mailing 
 list.*


But I have done this:
It also says:

*Your site's extensions media directory isn't properly set up. This 
 directory is where Review Board will store various extension media files.
 **Your extensions media directory is currently at: 
 /var/www/reviewboard-satv.tandbergtv.lan/htdocs/media/ext
 **Permission problems
 **The directory must be writable by the web server. On Linux/Unix/Mac, 
 you can fix this by typing:
 **$ sudo chown -R www-data 
 /var/www/reviewboard-satv.tandbergtv.lan/htdocs/media/ext
 **On Windows, right-click the data directory and change the ownership to 
 www-data.*

 
But I have also checked this.
Have you got any logging I can try and turn on to debug further?
Finally I am also getting a lot of these when running any rb-site commands:

 */usr/local/lib/python2.6/dist-packages/Django-1.4.1-py2.6.egg/django/core/cache/__init__.py:82:
  
 DeprecationWarning: settings.CACHE_* is deprecated; use settings.CACHES 
 instead.
 **  DeprecationWarning*


I am running on Ubuntu 10.04 64 bit Server.

Cheers
Dan

On Friday, August 3, 2012 9:49:58 AM UTC+1, 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 - chip...@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+unsubscr...@googlegroups.com
For more options, visit this group at 
http://groups.google.com/group/reviewboard?hl=en

Post-Review and Non publically accessible repositories

[Daniel Laird]

I am using post-review to generate my diff requests and post them to my 
reviewboard instance.
One of my SVN servers has restricted access, as such I mark it as non 
publically accessible and add all the Users /Groups that can access it.
The problem is when I run post-review it fails to find the repo and I get 
an error that the repo is not accessible.

However if I make it publically accessible it all works fine.
It seems to be a problem where post-review does not do any authentication 
until after it has tried to locate the repo - but this wont work.

Has anyone else seen the same issue and if so did you solve it OR is this a 
bigger problem in post-review that will take some hacking or should I not 
be doing this.

Cheers
Dan

-- 
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

rb-site help

[Daniel Laird]

Hi,

I am going to use rb-site to dump the data and then import onto a new 
webserver.
Whilst trying to check out the options I noticed that when I do rb-site -h 
there is no help regarding the manage options.
I am going to check the source code - but is this intentional or just an 
oversight?

Cheers
Dan

-- 
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

Phantom Review Closer

[Daniel]

We've been experiencing something weird within ReviewBoard. Reviews are 
being closed and either A) users aren't remembering that they're doing it 
(happening too often, so I don't believe this to be the case) B) Something 
else is closing the reviews.

The main problem is we can't figure out what is closing them as there is no 
record of who closed them, and I can't find any evidence in the logs to 
show when something is closed and an indication to who did it.

Any help will be much appreciated.

Thanks,
Daniel

-- 
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

Can Connect SVN Repository thru proxy

[Daniel Etchen]

Hi all,

We're using Review Board 1.6.4.1.
We access a Subversion server thru a proxy which is configured in /
root/.subversion/servers.
The 'svn info http://svn.foo.com:8080/repos/3845' command return a
correct response.

But when we try to create a new repository in reviewboard, the server
kicks back an error:
2012-06-20 09:34:08,240 - ERROR - SVN: Failed to get repository
information for http://svn.foo.com:8080/repos/3845: OPTIONS of 'http://
svn.foo.com:8080/repos/3845': Could not resolve hostname
`svn.foo.com': Host not found (http://svn.foo.com:8080)

Where is the problem ? It seems that the proxy configuration
configured in /root/.subversion/servers is not read by reviewboard

Thank you for your help.

-- 
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: Problem uploading screenshots

[Daniel]

Attached is the screenshot of the console you asked for.


Thanks,

Daniel

https://lh6.googleusercontent.com/-zI4uTyoe4ys/T6QParQvlfI/AFQ/jEq2pTO8Q1E/s1600/ScreenShot003.jpg


On Thursday, April 26, 2012 2:48:54 PM UTC-6, Christian Hammond wrote:

 What version of Review Board?

 Can you use Chrome or Firefox's developer tools (javascript console and 
 such) to see what the network transfer looks like for that request?

 Christian

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


 On Thu, Apr 26, 2012 at 1:04 PM, Daniel doub...@mcdewey.com wrote:

 We are having a problem where we cannot upload screenshots. Whenever we 
 try we have an error that looks like the following:

 *Error Code:* 0
 *Error Text:* n/a
 *Request URL:* 
 http://reviewboard.ks.quantum.com/api/review-requests/7424/screenshots/
 *Request Data:* (none)
 *Response Data:*
 There may be useful error details below. The following error page may be 
 useful to your system administrator or when reporting a 
 bughttps://mail.quantum.com/owa/redir.aspx?C=01f5dced1e83444a9406c2539e8b4474URL=http%3a%2f%2fwww.reviewboard.org%2fbugs%2fnew%2f.
  
 To save the page, right-click the error below and choose Save Page As, if 
 available, or View Source and save the result as a .html file.
 *Warning:* Be sure to remove any sensitive material that may exist in 
 the error page before reporting a bug!

 We've already installed PIL, and have already double checked the 
 permissions of the media folder. I can't find information in the apache or 
 rb logs. Any help appreciated.

 Thanks,
 Daniel

 -- 
 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: Password reset email said to come from example.com despite correct server name set in General Settings

[Daniel]

Yep, that did it. Thanks.

Daniel

On Monday, March 26, 2012 11:56:45 PM UTC-6, brajesh wrote:

 Hi Daniel,

 You need to make changes under Database - Sites

 Thanks
 Brajesh

 On Tue, Mar 27, 2012 at 6:06 AM, Daniel  wrote:

 One of my Review Board users attempted to reset his password. He got it 
 correctly, however it didn't look right. He got the following:

 Subject: Password reset on *example.com*

  You're receiving this e-mail because you requested a password reset for 
 your account at *example.com*.

  Please go to the following page and choose a new password:
 Correct URL was here

  In case you've forgotten, your username is John.Doe

  Thanks for using Review Board!

 Does anyone know why example.com is showing and where I can fix it. It's 
 set correctly in the admin site under General Settings.

 Thanks,
 Daniel

 --




-- 
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

Password reset email said to come from example.com despite correct server name set in General Settings

[Daniel]

One of my Review Board users attempted to reset his password. He got it 
correctly, however it didn't look right. He got the following:

Subject: Password reset on *example.com*

 You're receiving this e-mail because you requested a password reset for 
your account at *example.com*.

 Please go to the following page and choose a new password:
Correct URL was here

 In case you've forgotten, your username is John.Doe

 Thanks for using Review Board!

Does anyone know why example.com is showing and where I can fix it. It's 
set correctly in the admin site under General Settings.

Thanks,
Daniel

-- 
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

Paid Support

[Daniel]

Is there anybody out there who does or is willing to give us support
for a fee?

We've come across several problems with SQLite, and we are trying to
get moved off of it, however Django and SQLite stop us every step of
the way, and I'm at my wits end. We need to migrate while preserving
data. I've tried asking for help on this discussion and haven't gotten
anything, and in the meantime our Reviewboard is continually plagued
with database locked errors that we can't get away from. Especially
when django comes back and gives me a process killed with no
explanation. And as far as I can tell, that was with verbose turned
on.

Any and all help and offers are appreciated.

Thanks,
Daniel

-- 
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

Trying to move from sqlite to mysql running into disk image is malformed problems

[Daniel]

I'm in the midst of trying to move from SQLite to MySQL. I know that
we shouldn't be running SQLite on a production server, that's the
reason for the migration. However whenever I try to dump the data from
the database using:
rb-site manage . dumpdata  sqlite.json
I get the following error:

Traceback (most recent call last):
  File /usr/local/bin/rb-site, line 8, in module
load_entry_point('ReviewBoard==1.5.4', 'console_scripts', 'rb-
site')()
  File /usr/local/lib/python2.6/dist-packages/ReviewBoard-1.5.4-
py2.6.egg/reviewboard/cmdline/rbsite.py, line 1611, in main
command.run()
  File /usr/local/lib/python2.6/dist-packages/ReviewBoard-1.5.4-
py2.6.egg/reviewboard/cmdline/rbsite.py, line 1545, in run
site.run_manage_command(args[0], args[1:])
  File /usr/local/lib/python2.6/dist-packages/ReviewBoard-1.5.4-
py2.6.egg/reviewboard/cmdline/rbsite.py, line 397, in
run_manage_command
execute_manager(reviewboard.settings, [__file__, cmd] + params)
  File /usr/local/lib/python2.6/dist-packages/Django-1.2.5-py2.6.egg/
django/core/management/__init__.py, line 438, in execute_manager
utility.execute()
  File /usr/local/lib/python2.6/dist-packages/Django-1.2.5-py2.6.egg/
django/core/management/__init__.py, line 379, in execute
self.fetch_command(subcommand).run_from_argv(self.argv)
  File /usr/local/lib/python2.6/dist-packages/Django-1.2.5-py2.6.egg/
django/core/management/base.py, line 191, in run_from_argv
self.execute(*args, **options.__dict__)
  File /usr/local/lib/python2.6/dist-packages/Django-1.2.5-py2.6.egg/
django/core/management/base.py, line 220, in execute
output = self.handle(*args, **options)
  File /usr/local/lib/python2.6/dist-packages/Django-1.2.5-py2.6.egg/
django/core/management/commands/dumpdata.py, line 84, in handle
objects.extend(model._default_manager.using(using).all())
  File /usr/local/lib/python2.6/dist-packages/Django-1.2.5-py2.6.egg/
django/db/models/query.py, line 82, in __len__
self._result_cache.extend(list(self._iter))
  File /usr/local/lib/python2.6/dist-packages/Django-1.2.5-py2.6.egg/
django/db/models/query.py, line 271, in iterator
for row in compiler.results_iter():
  File /usr/local/lib/python2.6/dist-packages/Django-1.2.5-py2.6.egg/
django/db/models/sql/compiler.py, line 677, in results_iter
for rows in self.execute_sql(MULTI):
  File /usr/local/lib/python2.6/dist-packages/Django-1.2.5-py2.6.egg/
django/db/models/sql/compiler.py, line 752, in execute_sql
return list(result)
  File /usr/local/lib/python2.6/dist-packages/Django-1.2.5-py2.6.egg/
django/db/models/sql/compiler.py, line 746, in lambda
result = iter((lambda: cursor.fetchmany(GET_ITERATOR_CHUNK_SIZE)),
sqlite3.DatabaseError: database disk image is malformed

I need some help getting this resolved. This data cannot be lost. Our
actual RB site appears to do just fine, except that we get database
locked errors because of high traffic. Another reason we are trying to
migrate to MySQL. Any help would be greatly appreciated.

Thanks,
Daniel

-- 
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-receive hook for Reviewboard?

[Daniel Dieckmann]

Hey,

I installed ReviewBoard a couple of days ago and now I am wondering
if there is any possibility to make a post-receive hook in a git repo,
that creates a new review request.

So I want to do something like this:
1. Someone writes some code an does a commit. Now he pushes this
one to the server where ReviewBoard is running and which also has
a clone of the original git repo.
2. After the push there is some post-receive hook creating a new
review
request using post-review.

But none of my solutions worked, have zou any idea how to solve this
problem?

Thanks in advance,
Daniel

-- 
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: Bzr support in post-review

[Daniel Benamy]

Now that 0.3.3 is out, would this work be merged if it was fixed up?

On Jun 26, 3:37 am, Christian Hammond chip...@chipx86.com wrote:
 Hi Dan,

 We need to do a full review and make sure it's suitable. I don't know if the
 original author is still around (there weren't any updates made after the
 current reviews), but we'd need someone familiar with bzr to help finish up
 any changes and test them.

 Right now, we're freezing RBTools while some restructuring work finishes up
 in a branch. It's too tedious to keep things in sync right now. Once that
 work is in, we can start accepting patches again.

 Christian

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







 On Sat, Jun 25, 2011 at 3:29 PM, Daniel Benamy dben...@gmail.com wrote:
  Hi,

  Adding bzr support to post-review seems to be in limbo. There's a
  patch for it at
 http://reviews.reviewboard.org/r/1739/
  and a thread about it at

 http://groups.google.com/group/reviewboard/browse_thread/thread/888a4...

  There's a comment in the review that says that changing the regex to
  BRANCH_REGEX = r'\w*(branch|branch root): (?Pbranch_path.+)$'
  makes it work. That regex matches the output of my bzr info too and
  I think I saw one or two comments on other lists of it working for
  people but it was a while ago.

  Is there anything holding up the merging of that patch + new regex?

  Thanks!
  Dan

  --
  Want to help the Review Board project? Donate today at
 http://www.reviewboard.org/donate/
  Happy user? Let us know athttp://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: Bzr support in post-review

[Daniel Benamy]

Hi Christian,

I can spend a little time working on the patch and testing. I'm not
bzr expert, but I use it daily. Would that be ok?

Who needs to do the full review?

Do you know when the freeze should be done? Ie, when should I poke at
this issue again?

Thanks!
Dan

On Jun 26, 3:37 am, Christian Hammond chip...@chipx86.com wrote:
 Hi Dan,

 We need to do a full review and make sure it's suitable. I don't know if the
 original author is still around (there weren't any updates made after the
 current reviews), but we'd need someone familiar with bzr to help finish up
 any changes and test them.

 Right now, we're freezing RBTools while some restructuring work finishes up
 in a branch. It's too tedious to keep things in sync right now. Once that
 work is in, we can start accepting patches again.

 Christian

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







 On Sat, Jun 25, 2011 at 3:29 PM, Daniel Benamy dben...@gmail.com wrote:
  Hi,

  Adding bzr support to post-review seems to be in limbo. There's a
  patch for it at
 http://reviews.reviewboard.org/r/1739/
  and a thread about it at

 http://groups.google.com/group/reviewboard/browse_thread/thread/888a4...

  There's a comment in the review that says that changing the regex to
  BRANCH_REGEX = r'\w*(branch|branch root): (?Pbranch_path.+)$'
  makes it work. That regex matches the output of my bzr info too and
  I think I saw one or two comments on other lists of it working for
  people but it was a while ago.

  Is there anything holding up the merging of that patch + new regex?

  Thanks!
  Dan

  --
  Want to help the Review Board project? Donate today at
 http://www.reviewboard.org/donate/
  Happy user? Let us know athttp://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