Re: Customization the Review Board

2012-12-26 Thread Matthew Woehlke
before publishing and how to publish it also. What is the work flow. I'm having a similar issue. If you manage to create a working example, PLEASE share it, as one is desperately needed for those of us that aren't intimately familiar with RB's source. -- Matthew -- Want to

Re: trouble editing review with code comments

2012-12-21 Thread Matthew Woehlke
.. Thanks for the help (also thanks to Stephen, again, for the package... sheesh, 0.7.7 didn't last long, did it? :-) ). -- Matthew -- Want to help the Review Board project? Donate today at http://www.reviewboard.org/donate/ Happy user? Let us know at http://www.review

Re: trouble editing review with code comments

2012-12-21 Thread Matthew Woehlke
httpd?) -- Matthew -- 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

trouble editing review with code comments

2012-12-21 Thread Matthew Woehlke
s this a known issue? I feel like I remember hearing about something similar on the list... I'm on RB 1.7.1 upgraded from 1.7.0 rc1 from 1.7.0 beta 4; maybe I missed a step upgrading? -- Matthew -- Want to help the Review Board project? Donate today at http://www.reviewboard.org/donate

Re: Review Board 1.7.1 released

2012-12-21 Thread Matthew Woehlke
the '+', and as I recall, I had to put it there. I'm also on Fedora 18.) p.s. Big thanks to Stephen for the packages! -- Matthew -- Want to help the Review Board project? Donate today at http://www.reviewboard.org/donate/ Happy user? Let us know at http://www.review

Re: Review Board, Git and patch-series

2012-12-16 Thread Matthew Woehlke
On 2012-12-15 14:13, Matthew Woehlke wrote: On Monday, October 31, 2011 10:45:15 AM UTC-4, Stephen Gallagher wrote: I've been trying to work out for some time now how to accomplish patch-series in Git with Review Board. [...] Modify post-review so that it will create an individual revi

Re: Extension generator throwing exception

2012-12-15 Thread Matthew Woehlke
On 2012-11-08 19:53, Matthew Woehlke wrote: On 2012-10-11 00:43, kvr wrote: I'm trying to generate an extension using extension_generator.py but it is throwing the following exception> *jinja2.exceptions.TemplateNotFound: extension/templates/extension/dashboard.html* But actua

Re: cannot add remote repository in 1.7 rc1?

2012-12-14 Thread Matthew Woehlke
...) -- Matthew -- 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

cannot add remote repository in 1.7 rc1?

2012-12-14 Thread Matthew Woehlke
but is not visible... -- Matthew -- 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 reviewboar

Re: record git branch name in request?

2012-12-13 Thread Matthew Woehlke
this in existing 'change request') We're also going to want/need a few local fields, but not things that would make sense for inclusion in RB proper. Ability to view individual commit diffs in a multi-commit request would be awesome (you probably know that already) but I expect

FYI 1.7 rc1 results (was: Will there be a 1.6.14 release)

2012-12-13 Thread Matthew Woehlke
olling the Fedora package that djblets 0.7.2 is no longer sufficient for rc1. (I just rebuilt the RPM bumping the source to 0.7.6 with no issue, so am using that; not sure what the actual requirement is.) -- Matthew -- Want to help the Review Board project? Donate today at http://www.revie

record git branch name in request?

2012-12-13 Thread Matthew Woehlke
short term solution (longer term I would rather have the SHA there). Another would be an RB extension to add a custom field. In the latter case, does anyone have an example of doing this, both in terms of the extension, and also how to retrieve the value from the server? -- Matthew -- Want to

Re: Will there be a 1.6.14 release

2012-12-12 Thread Matthew Woehlke
not going to try to test it today, though; I don't have a VM set up for a test install, so I don't plan on dropping the new build in without at least a database backup in case something goes wrong, and I don't expect to get to that immediately. -- Matthew -- Want to help the Rev

Re: Will there be a 1.6.14 release

2012-12-12 Thread Matthew Woehlke
ine 63, in run self.run_command('build_media') File "/usr/lib64/python2.7/distutils/cmd.py", line 326, in run_command self.distribution.run_command(command) File "/usr/lib64/python2.7/distutils/dist.py", line 972, in run_command cmd_obj.run() File &

Re: open issue without code comment?

2012-12-07 Thread Matthew Woehlke
On 2012-12-07 14:55, Christian Hammond wrote: On Dec 7, 2012, at 11:52 AM, Matthew Woehlke wrote: Is it possible to create an issue for a review request without it being associated with a code comment? (I don't see a way, but want to check if I'm missing something before filing an e

open issue without code comment?

2012-12-07 Thread Matthew Woehlke
it has e.g. introduced a behavioral regression, but don't know what specific code is the culprit. I think it makes sense for tracking purposes to be able to open an issue describing the problem, rather than just a review stating that a problem exists. Thanks, -- Matthew -- Want to help t

Re: Configuring Review Board for GIT

2012-12-07 Thread Matthew Woehlke
still get an error. ...looks reasonable. I would guess the Raw File URL is the issue. (Have you checked the server's logs to see what it is trying to do?) What about the id2 in the raw file URL mask - should I leave it as id2 or should I use just id ? This I don't know... whatever w

Re: post-review fails if no tracking branch, when not needed?

2012-12-06 Thread Matthew Woehlke
k that needs to be done to refactor this code and clean it up. Would you mind filing a bug to help us track this? Sure. -- Matthew -- Want to help the Review Board project? Donate today at http://www.reviewboard.org/donate/ Happy user? Let us know at http://www.review

post-review doesn't work with extraneous git-svn remote?

2012-12-04 Thread Matthew Woehlke
? -- Matthew -- 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

post-review fails if no tracking branch, when not needed?

2012-12-04 Thread Matthew Woehlke
I'm seeing this out of post-review: post-review --revision-range=: --output-diff --debug RBTools 0.4.1 Home = /home/matthew HTTP GETting api/ HTTP GETting https://manticore/api/info/ Using the new web API Failed to execute command: ['git', 'merge-base', 'origin/m

Re: I have implemented --file-list in post-review

2012-12-04 Thread Matthew Woehlke
the same branch. We would also like to record the SHA that was posted (mostly to be able to verify when doing a merge that the branch hasn't changed) as well as the name of the branch that was posted, but this needs server-side changes of course. -- Matthew -- Want to help the Revie

Re: Error using post-review with git

2012-12-04 Thread Matthew Woehlke
ply returns. That could mean the value is an empty string (it shouldn't be), or the command failed. What does 'git config core.bare; echo $?' say? Note: what *should* happen is it echoes 'false' (unless run in a bare repository, in which case it would echo 't

Re: Configuring Review Board for GIT

2012-12-04 Thread Matthew Woehlke
27;s for experimenting, read up on git's 'ls-tree' command. (Short form: 'git ls-tree ' will give you the list of blobs (files) with their respective ID's in as of .) -- Matthew -- Want to help the Review Board project? Donate today at http://www.reviewbo

Re: Integration of RB with Gitlab

2012-11-20 Thread Matthew Woehlke
clone the gitorious repo for such a service (and at that point, you might as well keep the clone where RB can talk to it directly). -- Matthew -- Want to help the Review Board project? Donate today at http://www.reviewboard.org/donate/ Happy user? Let us know at http://www.r

Re: Integration of RB with Gitlab

2012-11-20 Thread Matthew Woehlke
p.s. If you end up patching Gitlab yourself, what you need is basically a URL that gives you the output of 'git show '. The path isn't actually needed for git. -- Matthew -- Want to help the Review Board project? Donate today at http://www.reviewboard.org/donate/

owner cannot drop issues?

2012-11-16 Thread Matthew Woehlke
If I create an issue on a request, and later realize it is not an issue, is there no way for me (as the issue creator) to drop the issue? If not, would it be possible to add this ability? -- Matthew -- Want to help the Review Board project? Donate today at http://www.reviewboard.org/donate

Re: Extension generator throwing exception

2012-11-08 Thread Matthew Woehlke
have the same problem, except it is complaining about extension/__init__.py. Help would be appreciated. -- Matthew -- Want to help the Review Board project? Donate today at http://www.reviewboard.org/donate/ Happy user? Let us know at http://www.reviewboard.o

Re: cannot join review group?

2012-11-08 Thread Matthew Woehlke
eta. Okay; good to know it is a known issue. Thanks. -- Matthew -- 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

cannot join review group?

2012-11-08 Thread Matthew Woehlke
checked and 'invite only' not checked. Am I doing something wrong? Is there a permission somewhere that is not set properly? -- Matthew -- Want to help the Review Board project? Donate today at http://www.reviewboard.org/donate/ Happ

custom fields redux

2012-11-07 Thread Matthew Woehlke
(especially with working completion) would be ideal, but what we're doing now with gerrit is requiring a machine-readable tag in the request description, so this would be okay too. -- Matthew -- Want to help the Review Board project? Donate today at http://www.reviewboard.org/donate/ Happy user

Re: Error 207 when using post-review with github

2012-06-19 Thread Matthew Gertner
On Wednesday, June 20, 2012 8:34:41 AM UTC+2, Christian Hammond wrote: > > Which version of RB though? This is important as GitHub made some major > changes recently that are only compatible with the latest 1.6.x releases. > > Christian > > Aha, we're still on 1.6.4.1. I'll ask our sysadmin to upd

Re: Error 207 when using post-review with github

2012-06-19 Thread Matthew Gertner
BTW, I just tried uploading the diff manually (no parent diff) and I get exactly the same error message about the file not being in the repo. The only thing I can think of is perhaps their are some Windows line endings in the file. Would that bother RB? -- Want to help the Review Board project

Re: Error 207 when using post-review with github

2012-06-19 Thread Matthew Gertner
chip...@chipx86.com > Review Board - http://www.reviewboard.org > VMware, Inc. - http://www.vmware.com > > > On Tue, Jun 19, 2012 at 9:37 AM, Matthew Gertner > wrote: > >> I'm trying to post a review request for a changeset I created on a new >> branch of our public git

Error 207 when using post-review with github

2012-06-19 Thread Matthew Gertner
I'm trying to post a review request for a changeset I created on a new branch of our public github repo. Since I already committed the change, I enter: $ post-review --parent 5f7fc000274abab9cbfb7a00a5b9602314ae6268 I'm getting the dreaded 207 (the file was not found in the repository) error.

Re: Unable to get post-review to recognise a P4 repository

2012-03-27 Thread Matthew Waddilove
I think that this is similar to a problem that I've come across, or might be the same tbh. :) I had all sorts of issues as we're both using a perforce proxy and reviewboard at a remote location so had to do some fiddling to get it all to work, both configuring reviewboard, and poking ssh tunnels

Post-review thinks CVS diff file is empty

2011-09-26 Thread Matthew
The cvs command issued by post-review executes, but post-review doesn't seem to think there are any diffs to upload. I'll include the result of post-review --output-diff --debug at the end of this message Although I have a software background it has been a long time since I've done any sort of sys

Re: Post-review thinks CVS diff file is empty

2011-09-26 Thread Matthew
raise SCMError(errmsg) SCMError: cvs [checkout aborted]: Absolute module reference invalid: `/ usr/local/cvsroot/WorldWind/src/gov/nasa/worldwind/Version.ja va' On Sep 26, 4:05 pm, Christian Hammond wrote: > Hi Matthew, > > It's entirely possible that that error is misleading.

Method names in condensed diff view - I love you all

2011-09-14 Thread Matthew Flint
Hey, We just upgraded ReviewBoard, and I see there are now method names shown even when that part of the file is hidden. It is brilliant. I love you all. Matthew -- Want to help the Review Board project? Donate today at http://www.reviewboard.org/donate/ Happy user? Let us know at http

Re: Search not enabled on Ubuntu Server 11.04

2011-07-28 Thread Matthew Blair
David - yes, running in apache, and have executed several restarts of httpd after the install of pylucene. The restarts were related to other configuration items though, such as enabling SSL. Thanks, Matthew On Jul 28, 12:33 pm, David Trowbridge wrote: > Are you running in apache? Have

Search not enabled on Ubuntu Server 11.04

2011-07-28 Thread Matthew Blair
guide for Linux seems to pretty indicate that PyLucene must be installed, then search must be enabled via the admin page, followed by setting up a cron job. What is my next step for debugging this problem? Thanks for any help you can provide, Matthew -- Want to help the Review Board project

Re: Review Board 1.0.8 released

2010-05-17 Thread Matthew Woehlke
ly. Thanks for this improvement! -- Matthew -- 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, sen

Re: How do I install RBTools under windows?

2010-05-17 Thread Matthew Woehlke
sy_install RBTools". See http://pypi.python.org/pypi/setuptools#installation-instructions . Hope that helps. -- Matthew -- 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/ -~--~~---

Re: Review Board 1.0.8 released

2010-05-17 Thread Matthew Woehlke
re how one achieves that? Or does that mean it is safe to just move the log file and RB will notice and adjust? -- Matthew -- Want to help the Review Board project? Donate today at http://www.reviewboard.org/donate/ Happy user? Let us know at http://www.reviewboard

Re: how to rotate reviewboard log?

2010-04-19 Thread Matthew Woehlke
ble. (Maybe I will even take a look at what would be required...) For now I may just deal with the lighttpd restart, maybe I'll use weekly instead of daily rotation to minimize disruption. Thanks for the useful info. -- Matthew -- Want to help the Review Board project? Donate today at

Re: how to rotate reviewboard log?

2010-04-14 Thread Matthew Woehlke
On 2010-04-14 12:42, Jan Koprowski wrote: On Apr 14, 7:05 pm, Matthew Woehlke wrote: So, while wondering why a request was taking a while to post, I was poking around for the logs, and noticed that the lighttpd log had grown to over 240 MiB. "Okay," I said to myself, "I need t

how to rotate reviewboard log?

2010-04-14 Thread Matthew Woehlke
uot; Well, various sources say I can do that for lighttpd with logrotate and 'service lighttpd reload'. But will that work for Review Board's log? What is the proper way to rotate Review Board's log file? -- Matthew -- Want to help the Review Board project? Donate today at

Re: granting staff permissions without superuser permission?

2010-03-10 Thread Matthew Woehlke
ynonymous. I guess I should bug Django about it? -- Matthew -- 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

granting staff permissions without superuser permission?

2010-03-10 Thread Matthew Woehlke
peruser. Is there a permission to prevent that? I'm using RB 1.0.5.1. -- Matthew -- 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 un

Re: Multiple diffs per review?

2010-02-05 Thread Matthew Woehlke
On 2010-02-04 18:15, Stodge wrote: On Feb 4, 6:21 pm, Matthew Woehlke wrote: On 2010-02-04 10:13, Stodge wrote: [...] we need to be able to group multiple revisions on one review. Is this possible with RB? You could upload the diff of the first revision, publish it, upload the cumulative

Re: Multiple diffs per review?

2010-02-04 Thread Matthew Woehlke
. I use this frequently when I have vcs:r1 == a bunch of whitespace changes and vcs:r2 == interesting changes. Note that you can assign the review initially to a dummy user to avoid spamming everyone when you incrementally upload+publish diffs. -- Matthew -- Want to help the Review Board

Re: setting up memcache (for RB) on Fedora 12?

2009-12-07 Thread Matthew Woehlke
On 2009-12-03 15:59, Matthew Woehlke wrote: > Is libmemcache/cmemcache still needed, or is RB using some other way to > talk to memcached? I wonder because the latest "rc2" of libmemcache is > almost three years old. So... apparently python-memcached suffices. At least, I got

setting up memcache (for RB) on Fedora 12?

2009-12-03 Thread Matthew Woehlke
Is libmemcache/cmemcache still needed, or is RB using some other way to talk to memcached? I wonder because the latest "rc2" of libmemcache is almost three years old. -- Matthew -- Want to help the Review Board project? Donate today at http://www.reviewboard.org/donate/ Happy us

Re: ReviewBoard + GIT Integration

2009-09-16 Thread Matthew Woehlke
dows machine, I think it should work. Obviously, you'll also need Python :-). -- Matthew --~--~-~--~~~---~--~~ You received this message because you are subscribed to the Google Groups "reviewboard" group. To post to this group, send email to reviewb

Re: ReviewBoard + GIT Integration

2009-09-15 Thread Matthew Woehlke
just snag postreview.py and make sure everyone has a copy.) -- Matthew --~--~-~--~~~---~--~~ You received this message because you are subscribed to the Google Groups "reviewboard" group. To post to this group, send email to reviewboard@googlegroups.com T

Re: Review Board has moved to Git - Changes required

2009-09-10 Thread Matthew Woehlke
no > trouble rebasing your patch until it gets accepted! True :-). (Actually, the patch still applied cleanly, sans needing to type manually the name of the file to patch, since "post-review" is now named "postreview.py". Alas, Review Board didn't see

Re: Review Board has moved to Git - Changes required

2009-09-09 Thread Matthew Woehlke
That's not NNTP :-). (IMO HTTP interfaces are even worse than mail. /me laments underappreciated NNTP which is still by far the best interface for discussion lists.) -- Matthew --~--~-~--~~~---~--~~ You received this message because you are subscribed to

Re: Review Board has moved to Git - Changes required

2009-09-09 Thread Matthew Woehlke
ow to get post-review to work with rbtools git? (Also, I hate using actual mail for mailing lists ;-), anyone mind if I ask gmane to carry reviewbo...@?) -- Matthew --~--~-~--~~~---~--~~ You received this message because you are subscribed to the Google Groups &q

Re: View Diff is broken: Diff currently unavailable.

2009-02-03 Thread Matthew Von Bargen
- > Christian Hammond - chip...@chipx86.com > VMware, Inc. > > > On Tue, Feb 3, 2009 at 2:43 PM, Matthew Von Bargen > wrote: >> >> I don't know... >> >> If running "sudo easy_install --upgrade ReviewBoard" upgrades Djblets >> then I

Re: View Diff is broken: Diff currently unavailable.

2009-02-03 Thread Matthew Von Bargen
jblets as well? > > Christian > > -- > Christian Hammond - chip...@chipx86.com > VMware, Inc. > > > On Tue, Feb 3, 2009 at 2:35 PM, Matthew Von Bargen > wrote: >> >> Okay. Thanks. >> >> Did you mean: >> >> key += "-%s" % settings.AJ

Re: View Diff is broken: Diff currently unavailable.

2009-02-03 Thread Matthew Von Bargen
Okay. Thanks. Did you mean: key += "-%s" % settings.AJAX_SERIAL instead of : str += "-%s" % settings.AJAX_SERIAL ? If I do what you say, but assign to +key+ instead of +str+, then things mostly work. But I am still seeing one "Diff currently unavailable." error: Diff currently unavailable.

<    1   2