I've noticed a few references to Review Board 2.0.12 such
as https://reviews.reviewboard.org/r/6676/ (2.0.12 was a large
release...) but it doesn't seem to have ever been made public. Should it
have been, or has it just been delayed?
-- Bruce
--
Get the Review Board Power Pack at
I'm currently working on setting up Review Board to work in a corporate
environment where people login with a numeric employee id instead of a
username - for example '12345' instead of 'bruce'.
The problem I'm trying to solve is that the Submitter column displays the
'username' and not the
On Tue, Sep 30, 2014 at 12:23 PM, Felix Wei elixy...@gmail.com wrote:
Hi, I have been trying to install RBTools on windows, but when i run the
easy_install -U RBTools command it gives me this error
U:\easy_install -U RBTools
As an alternative, I built a standalone RBTools 0.6.2 Windows
I spoke too soon! Another user has reported the same problem and asked me
how to get rid of the garbage files. I'll file a bug report and debug
this next week.
--
Bruce
On Mon, Aug 25, 2014 at 10:14 PM, Bruce Cran bruce.c...@gmail.com wrote:
Sorry, I never got around to debugging
Can't you permanently delete it after publishing it with you as the only
reviewer (if you have the necessary permissions)?
--
Bruce
On Wed, Jul 23, 2014 at 12:35 PM, Christian Hammond
christ...@beanbaginc.com wrote:
Hi Joe,
There’s no way to permanently delete it. You should probably
There's a problem with the empty files detection code in at least the
Mercurial client of RBTools 0.6.2: it's adding all the empty files that
exist in the repository. The relevant code is in Mercurial.py,
_handle_empty_files():
tip_empty_files = self._get_files_in_changeset(tip,
There were some non-empty added files, along with modifications; I just ran
'rbt post -o' to create the review - it used the branch, server and
repository from the .reviewboardrc file.
--
Bruce
On Tue, Jul 22, 2014 at 5:25 PM, Anselina Chia anselina.c...@gmail.com
wrote:
Can you give more
From debugging earlier today, added_empty_files and deleted_empty_files
were both the same, and contained the list of empty files that were already
in the repository.
base_files and tip_files looked correct - I'll go back and do some more
debugging tomorrow.
--
Bruce
On Tue, Jul 22, 2014 at
Do you also need to restart memcached?
--
Bruce
On Thu, Jul 17, 2014 at 9:51 AM, Jason Batchkoff firewor...@gmail.com
wrote:
*bump bump*
Anyone? Still haven't figured out how to get a local change to the py
scripts to reflect correctly in the running copy.
--
Get the Review Board
http://reviewboard.org contains a link to RBTools 0.6.1 - it should
probably be updated for 0.6.2?
Also, /docs/releasenotes contain a broken link to Power Pack.
--
Bruce
--
--
Bruce Cran
--
Get the Review Board Power Pack at http://www.reviewboard.org/powerpack/
---
Sign up for Review
I've uploaded a new RBTools Windows installer to
http://bluestop.org/rbtools/RBTools-0.6.2.exe .
--
Bruce
On Sun, Jul 6, 2014 at 8:21 PM, Christian Hammond christ...@beanbaginc.com
wrote:
Hi everyone,
We just put out a release of RBTools 0.6.2. We have some fixes for
Mercurial, Git,
I've used NSIS in the past and I really don't like it: I've found it can
lead to fragile setup packages. You may be able to use the free Visual
Studio 2013 Express with WiX after installing Update 2 to avoid having to
purchase a copy.
To build the installer, you'd just run msbuild /m
I'm running Review Board 2.0.1. Is it normal to get quite a lot of the
following messages in reviewboard.log?
2014-06-26 00:26:59,410 - WARNING - - Attempting to override insert count
on FileDiffData 569c03ed942572b0d790d8db2ea1c4b49dbdcd4e from 196 to None (F
ileDiff None)
2014-06-26
Have you bumped the logging level to DEBUG in the admin settings?
--
Bruce
On Tue, Jun 24, 2014 at 7:02 PM, fireworm firewor...@gmail.com wrote:
I modified the one
in
/usr/local/lib/python2.7/dist-packages/ReviewBoard-2.0.1-py2.7.egg/reviewboard/accounts/backend.py
Then I restarted
Thanks! I've started committing .reviewboardrc files to each repo. I was
slightly worried about the server changing, but it's simple enough to add a
CNAME so things don't break.
Is there anything I can do to help get the Windows installer patch
committed and not forgotten about (presuming it's
? This would avoid having individual developers running 'rbt
setup-repo'.
--
Bruce
On Tue, Jun 17, 2014 at 4:35 PM, Bruce Cran bruce.c...@gmail.com wrote:
I agree, http 500 isn't at all useful: I think on my setup I was always
seeing an error saying there were no matching repositories found
://www.reviewboard.org
Beanbag, Inc. - http://www.beanbaginc.com
On June 18, 2014 at 7:12:19 PM, Bruce Cran (bruce.c...@gmail.com) wrote:
Sorry, it's not actually git but mercurial I'm having trouble with.
After running 'rbt post' I run 'rbt patch reviewid' and get the error:
Patch is being applied from
I've been trying to use the 'rbt patch' command, and it seems that
_get_p_number() is only for subversion repositories.
Since diffs are commonly created with a prefix, and git defaults to adding
a/ and b/, should we automatically try, or retry, using 'patch -p1' without
the user needing to
seeing?
-David
On Wed, Jun 18, 2014 at 7:58 AM, Bruce Cran bruce.c...@gmail.com wrote:
I've been trying to use the 'rbt patch' command, and it seems that
_get_p_number() is only for subversion repositories.
Since diffs are commonly created with a prefix, and git defaults to
adding a/ and b
much ignored
at this point.
Christian
--
Christian Hammond - christ...@beanbaginc.com
Review Board - http://www.reviewboard.org
Beanbag, Inc. - http://www.beanbaginc.com
On June 14, 2014 at 5:59:31 PM, Bruce Cran (bruce.c...@gmail.com)
wrote:
Christian,
Yes, I was hoping I'd
Isn't this covered in
http://www.reviewboard.org/docs/rbtools/0.6/rbt/configuration/ ?
--
Bruce
On Tue, Jun 17, 2014 at 1:49 PM, Sara Rayburn sararayb...@gmail.com wrote:
That makes sense. It would be nice to see a section in the RB Tools setup
documentation about what to do if the
I agree, http 500 isn't at all useful: I think on my setup I was always
seeing an error saying there were no matching repositories found.
Christian's mention of Mirror Path was the hint I needed to look up the
documentation - from there I realised that entering the ssh:// URL that
developers use
I've put together an RBTools installer for Windows for testing that adds
only the rbt command to the environment (i.e. it doesn't pollute %PATH%
with python.exe etc.).
It's available from http://bluestop.org/rbtools/RBTools-0.6.1.exe - I'd
welcome any feedback.
It _should_ work on Windows XP and
Christian,
Yes, I was hoping I'd be able to contribute the code to make future
maintenance simpler.
Which repo and in which directory would it be added? I see there's a old
rb-win32-installer repository on github for Review Board - would it be
there, or under contrib/ in the rbtools repo etc.?
It sounds like there might be a need for a standalone RBTools package for
Windows, created using something like cx_freeze?
--
Bruce
On Fri, Jun 13, 2014 at 7:37 AM, Sara Rayburn sararayb...@gmail.com wrote:
Hi David,
The biggest problem with RB Tools for our windows users is that it
Christian,
No, my patch (also posted at https://reviews.reviewboard.org/r/5965/) only
appears to fix the problem with parent diffs: I've had users complaining
today both that valid patches won't upload or that the diff viewer doesn't
work for existing reviews (FileNotFoundError: The file
I have a review in RB 2.0.1 where someone pasted over 700 lines of console
output into the 'Testing Done' field of a review. Internet Explorer and
Chrome don't have a problem with it, but with Firefox 29 you have to click
through 3 'script timeout' warnings before it will display. It's getting
, Bruce Cran bruce.c...@gmail.com wrote:
I don't know if it used to work with RBTools, but people have said it used
to work with 'hg postreview -m qparent'.
Running 'rbt post tip' or 'rbt post --parent=qparent tip' results in
Review Board saying it can't display the diff where there are changes
Sorry, I won't be able to provide it since it contains internal data
(usernames etc.).
--
Bruce
On Mon, Jun 9, 2014 at 1:47 PM, Christian Hammond christ...@beanbaginc.com
wrote:
If there’s any way you can get me an exact copy of some text that’s
breaking things (as a file attachment), I’ll
Your search index directory appears to be wrong: it should be something
like /var/www/reviews/search-index/ and not the root directory.
--
Bruce
On Thu, Jun 5, 2014 at 2:04 AM, Stefan Bohne
stefan.bo...@sentec-elektronik.de wrote:
Hello all,
since upgrading to RB2.0 and then RB2.0.1 my
Hi Christian,
It reduced rebuild_index to around 15 minutes and update_index to 20:
user@host:~$ sudo time rb-site manage /var/www/reviews rebuild_index
...
All documents removed.
Indexing 23550 review requests
443.30user 15.45system 8:07.01elapsed 94%CPU (0avgtext+0avgdata
1655104maxresident)k
I don't seem to have access to that review.
--
Bruce
On Thu, Jun 5, 2014 at 1:54 PM, Christian Hammond christ...@beanbaginc.com
wrote:
That’s better, but not good enough.
Patch your install with this (on top of the existing patch):
https://reviews.reviewboard.org/r/5937/
--
Get the
That's much better!
With rb-site manage /var/www/reviews update_index -- --age=1 it just
takes a few seconds to run.
--
Bruce
On Thu, Jun 5, 2014 at 2:02 PM, Christian Hammond christ...@beanbaginc.com
wrote:
It would probably be helpful if I published it…
Try now :)
Christian
--
In 1.7, it was possible to generate a review request of the topmost
changeset patch1 - e.g.:
hg qapplied
patch1
patch2
patch3
patch4
... by creating a diff of patch2-patch4 and uploading it via the web
interface as the parent diff. How do I do the same in 2.x using 'rbt
post'? Uploading the
I don't know if it used to work with RBTools, but people have said it used
to work with 'hg postreview -m qparent'.
Running 'rbt post tip' or 'rbt post --parent=qparent tip' results in Review
Board saying it can't display the diff where there are changes in a file
which also has changes in other
On Wed, Jun 4, 2014 at 10:27 AM, Bruce Cran bruce.c...@gmail.com wrote:
Running a full reindex seems to be working, but is this a bug that people
need to be aware of?
Hmm, or maybe not:
user@host:~$ sudo /usr/local/bin/rb-site manage /var/www/reviews index
-- --full
Removing all documents
Thanks. For some reason a rebuild takes 10 minutes while an update takes
25, so they're obviously doing something different - except I'd expect the
update to be much faster.
user@host:~$ sudo time rb-site manage /var/www/reviews rebuild_index
WARNING: This will irreparably remove EVERYTHING from
I'm using Mercurial 3.0, ReviewBoard 2.0.1 and as of today, the latest code
from https://bitbucket.org/ccaughie/hgreviewboard . I did forget to add hg
add file2.txt to the list of steps, but I ran it during testing. Should
the code figure out which revision to use for the parent diff base? It
I'm investigating upgrading our installation to Review Board 2.x and one
issue I've come across is that with the existing code, hg postreview
doesn't work correctly. I've submitted a ticket to
https://bitbucket.org/ccaughie/hgreviewboard, but I was wondering - are
people still planning to use the
Sure.
2014-05-14 04:20:36,301 - ERROR - - Exception thrown for user a_user at
https://example.com/admin/security/
[Errno 13] Permission denied: '/var/www/
example.com/htdocs/media/uploaded/files/exec_check.php'
Traceback (most recent call last):
File
The 'Security checklist' link on the admin page (which links to
/admin/security/) in 2.0 RC3 produces an http 500 error because it tries to
access media/uploaded/files/exec_check.php, which is disallowed via the
section in apache-wsgi.conf:
Prevent the server from processing or allowing the
When I upgraded from 1.7.9 to 2.0 RC2 I had to go via 1.7.24 otherwise I
got an 'Unknown column' error (I can't remember if it was the same one).
--
Bruce
On Wed, Apr 30, 2014 at 1:26 PM, Stephen Gallagher
step...@gallagherhome.com wrote:
It worked cleanly for me.
--
Get the Review
I've not figured out why it only sometimes happens, but I've found that
'rbt post' will sometimes throw an exception on Windows, apparently
complaining about key= lines in hg showconfig where there's no value.
The traceback is:
Running: hg root
Running: hg showconfig
Traceback (most recent call
On Mon, Apr 28, 2014 at 5:59 PM, David Trowbridge trowb...@gmail.comwrote:
Would you mind sharing the output of 'hg showconfig' ?
I've attached the configuration (hopefully attachments are allowed).
--
Bruce
--
Get the Review Board Power Pack at http://www.reviewboard.org/powerpack/
---
I can't actually think of why someone might want to create two reviews from
the same commit.
--
Bruce
--
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
I've been setting up Review Board 2.0 RC2, and found I got an HTTP 500
error with an IntegrityError when trying for a second time to post a review
from a git repository using 'rbt post'.
The first time, I'd forgotten to set the Mirror Path in the repository
configuration, so i got:
ERROR:
I've checked and this appears to be a regression from at least 1.7.9. It
used to be possible to run the same 'rbt post' command in a git repo twice
and end up with two reviews.
--
Bruce
On Thu, Apr 17, 2014 at 8:34 PM, Bruce Cran bruce.c...@gmail.com wrote:
I've been setting up Review Board
Is there a way to restrict access to a specified LDAP group? I see there's
an option when using AD authentication, but I can't get it to work - I
think because I need to use the login details that are only available when
using LDAP.
--
Bruce
--
Get the Review Board Power Pack at
Is there anything in /tmp/reviewboard.log ?
--
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
final mention the 2.6.5 dependency.
-David
On Fri, Feb 21, 2014 at 4:58 PM, Bruce Cran bruce.c...@gmail.com wrote:
After having problems running rb-site to upgrade my installation, I
realised that Ubuntu 9.04 only has Python 2.6.2, while Django needs 2.6.5
or newer (
http://stackoverflow.com
Yes, it's 2.0 beta 3 - which is now running on Ubuntu 13.10.
I have both Djblets 0.7.15 and 0.8beta2 installed - it seems installing
ReviewBoard 2.0 beta 3 installed the newer version.
--
Bruce
On Sat, Feb 22, 2014 at 1:59 PM, Christian Hammond chip...@chipx86.comwrote:
Is this 2.0 beta 3?
log for when you installed the package and it
tried to install PIL?
Christian
--
Christian Hammond - chip...@chipx86.com
Review Board - http://www.reviewboard.org
Beanbag, Inc. - http://www.beanbaginc.com
On Sat, Feb 22, 2014 at 1:12 PM, Bruce Cran bruce.c...@gmail.com wrote:
Yes, it's
For people not familiar with the memory requirements of dedupe, I wonder if
it might be a good idea to add a warning that running rb-site manage
site condensediffs can need _lots_ of memory?
I ran it on our test server (running 2.0 beta 3) which had 8GB RAM and 8GB
swap allocated and with almost
There appears to be a regression between 1.7.9 and 2.0 beta 3. I loaded a
test machine with a snapshot of data from 1.7.9 and upgraded. Most reviews
load fine, but on one that works on 1.7.9, with beta3 I get the exception:
UnicodeDecodeError: 'ascii' codec can't decode byte 0xef in position 176:
Created http://code.google.com/p/reviewboard/issues/detail?id=3261
Thanks.
Bruce
On Sat, Feb 22, 2014 at 8:48 PM, David Trowbridge trowb...@gmail.comwrote:
Bruce,
Can you file a bug about this, including the traceback?
Thanks!
-David
On Feb 22, 2014, at 7:32 PM, Bruce Cran bruce.c
After having problems running rb-site to upgrade my installation, I
realised that Ubuntu 9.04 only has Python 2.6.2, while Django needs 2.6.5
or newer (
http://stackoverflow.com/questions/19365230/django-init-keywords-must-be-strings-error-while-running-runserver
).
As seen in
https://vimeo.com/86448355https://www.google.com/url?q=https%3A%2F%2Fvimeo.com%2F86448355sa=Dsntz=1usg=AFQjCNH1Acw29w1w2l53BMj_KOisQXjHyQ,
it looks like 2.0 beta 3 still has the bug whereby Review Board shows a
negative value for outgoing reviews. Is there a ticket open for this?
I've created http://code.google.com/p/reviewboard/issues/detail?id=3103.
Thanks.
--
Bruce
--
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
from pending to discarded
Review request changed
Updated 1 day, 3 hours ago (Oct. 1, 2013, 5:59 p.m.)
status changed from pending to discarded
Review request changed
Updated 21 hours, 26 minutes ago (Oct. 2, 2013, 12:19 a.m.)
status changed from pending to submitted
--
Bruce Cran
--
Get
and
the Apache log just contains a 403 error.
--
Bruce Cran
On Wednesday, 2 October 2013 21:52:37 UTC+1, David Trowbridge wrote:
It sounds like the review request is posted against either a group or
repository that has access controls. The closed/open state has no impact on
that.
--
Get
/to/file' (abcdef) could not be found in the
repository
--
Bruce Cran
--
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
I think the only thing that matters is that all the tables are in the
same format, but Innodb is now the MySQL default since it's supposed
to be more efficient
(http://dev.mysql.com/doc/refman/5.5/en/innodb-default-se.html).
--
Bruce Cran
On 7 June 2013 16:24, Daniel Kelly arrays.tostr
There seems to be a regression in 1.7.9 with mercurial parent diffs: if a
file is modified in the review diff that wasn't changed in the parent diff,
then Review Board generates the error *Error:* The file 'file2.txt'
(r12345678) could not be found in the repository.
--
Bruce Cran
--
Want
in a Duplicate column name 'extra_data' error).
--
Bruce Cran
--
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
applying evolution: (1005, Can't create table
'db_name.#sql-5b3a_6d' (errno: 150))
--
Bruce Cran
--
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
---
Resetting in-database caches.
Upgrade complete!
--
Bruce Cran
--
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
The homepage points to 1.7.1 (resulting in a 404), not 1.7.0.1.
--
Bruce Cran
--
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
/fad1e1741c19d606/28ccf483f4fd81e1?lnk=gstq=chris+white+space#28ccf483f4fd81e1
Thanks, that was the problem: the Show all whitespace for field
hadn't been set.
--
Bruce Cran
--
Want to help the Review Board project? Donate today at
http://www.reviewboard.org/donate/
Happy user? Let us know at http
68 matches
Mail list logo