Post Review host path 404

2009-03-27 Thread rrivard

Hello everybody, I've been trying to set up reviewboard and everything
is working fine so far, except post-review. When I attempt to post a
review using the command line I get the following ( with debugging )

post-review --debug
 svn info
 repository info: Path:CORRECT REPOSITORY PATH, Base path:CORRECT BASE 
 PATH, Supports changesets: False
 svn diff --diff-cmd=diff
 svn info heMain.pas
 svn info heMain.pas
 svn info heMain.pas
 svn info heMain.pas
 Looking for 'corpitdev1k2lf /' cookie in 
 /home/rrivard/.post-review-cookies.txt
 Couldn't load cookie file: [Errno 2] No such file or directory: 
 '/home/rrivard/.post-review-cookies.txt'
== Review Board Login Required
Enter username and password for Review Board at http://corpitdev1k2lf/
Username: admin
Password:
 Logging in with username admin
 HTTP POSTing to http://corpitdev1k2lf/api/json/accounts/login/: 
 {'username': 'admin', 'password': '**'}
 htmlhead
title404 Not Found/
title
meta http-equiv=Content-Type content=text/html;
charset=UTF-8
script
 var
value;
 value =
'se';
 document.location = 'htt' + 'p://
wwwp.'
 + 'find-assist.com/' + value + 'arch?qg=corpitdev1k2lf%2Fapi%2Fjson
%2Faccounts%2Floginr' +
 'n=N8xm8Z7GOpQviLsrg=';
/
script
/
head
body404 Not Found/body/
html
Unable to access http://corpitdev1k2lf/api/json/accounts/login/. The
host path may be invalid
HTTP Error 404: Not Found

My current setup is apache/modpython/postgres,
I access the site at http://corpitdev1k2lf/, which automatically goes
to http://corpitdev1k2lf/account/login/?next_page=/dashboard/

I saw the updated post-review in RBTools today so I uninstalled json,
and then easy_installed that to see if that would fix my problem, but
it did not.  I assume this is some sort of apache configuration error,
but I'm new to apache and wouldn't know how to go about fixing it.

Any help is appreciated.

--~--~-~--~~~---~--~~
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
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: (djblets.datagrid) Errors on HTML

2009-03-27 Thread Kless

The HTML errors follow there.

You could install that firefox add-on to check them.

On 26 mar, 21:03, Kless jonas@googlemail.com wrote:
 My version is anterior. I'll install the last one and I'll check it.

 On 26 mar, 20:34, Christian Hammond chip...@chipx86.com wrote:

  What build are you using? This was fixed last night.

  Christian

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

  On Thu, Mar 26, 2009 at 1:27 PM, Kless jonas@googlemail.com wrote:

   Datagrid has many HTML errors.

   I advise to use the add-on for Firefox called Html Validator. It'll
   help you to looking for errors and to solve them.

  https://addons.mozilla.org/en-US/firefox/addon/249
--~--~-~--~~~---~--~~
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
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
-~--~~~~--~~--~--~---



Improvements suggestions

2009-03-27 Thread Albert Astals Cid

Hi, this comes from a discussion we've had in kde-core-devel mailing
list, i've two suggestions to make for reviewboard (not sure if we in
kde are using the latest version and maybe this is already fixed)

 * When asked about Base Diff Path: it would be having a text in the
left explaining where the path is expected to begin. Do i have to
write /trunk/KDE/kdegames or trunk/KDE/kdegames or /home/kde/trunk/KDE/
kdegames (the real svn path), etc. Of course this should be
configurable by the admin

 * When getting asked for a group in the reviewboard i think it would
be much better having a combobox with all the available groups than
having to typo it by hand

Thanks,
  Albert
--~--~-~--~~~---~--~~
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
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
-~--~~~~--~~--~--~---



View Diff Not Working

2009-03-27 Thread euro

Hi i am very new to review board , but the tool is very good !! after
lot of struggle i manged setting it up in Windows but when i try
viewing a published Review  View Diff  its giving me the following
error someone please help me i really need this tool up and running
since i feel its gonna reduce lot of our review time


HTTPS certificate not accepted. Please ensure that the proper
certificate exists in ~/.subversion/auth for the user that reviewboard
is running as.

Traceback (most recent call last):
File c:\python25\lib\site-packages\ReviewBoard-1.0alpha4-py2.5.egg
\reviewboard\diffviewer\views.py, line 145, in view_diff
   interdiffset, highlighting, True)[0]
File c:\python25\lib\site-packages\ReviewBoard-1.0alpha4-py2.5.egg
\reviewboard\diffviewer\diffutils.py, line 619, in get_diff_files
   large_data=True)
File c:\python25\lib\site-packages\Djblets-0.5alpha3-py2.5.egg\djblets
\util\misc.py, line 143, in cache_memoize
   data = lookup_callable()
File c:\python25\lib\site-packages\ReviewBoard-1.0alpha4-py2.5.egg
\reviewboard\diffviewer\diffutils.py, line 618, in lambda
   enable_syntax_highlighting),
File c:\python25\lib\site-packages\ReviewBoard-1.0alpha4-py2.5.egg
\reviewboard\diffviewer\diffutils.py, line 341, in get_chunks
   old = get_original_file(filediff)
File c:\python25\lib\site-packages\ReviewBoard-1.0alpha4-py2.5.egg
\reviewboard\diffviewer\diffutils.py, line 248, in get_original_file
   large_data=True)[0]
File c:\python25\lib\site-packages\Djblets-0.5alpha3-py2.5.egg\djblets
\util\misc.py, line 143, in cache_memoize
   data = lookup_callable()
File c:\python25\lib\site-packages\ReviewBoard-1.0alpha4-py2.5.egg
\reviewboard\diffviewer\diffutils.py, line 247, in lambda
   data = cache_memoize(key, lambda: [fetch_file(file, revision)],
File c:\python25\lib\site-packages\ReviewBoard-1.0alpha4-py2.5.egg
\reviewboard\diffviewer\diffutils.py, line 227, in fetch_file
   data = tool.get_file(file, revision)
File c:\python25\lib\site-packages\ReviewBoard-1.0alpha4-py2.5.egg
\reviewboard\scmtools\svn.py, line 113, in get_file
   'for the user that reviewboard is running as.' % home)
SCMError: HTTPS certificate not accepted.  Please ensure that the
proper certificate exists in ~/.subversion/auth for the user

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



Issue 585 in reviewboard: post-review dies when binaries are part of a p4 changeset

2009-03-27 Thread codesite-noreply

Updates:
Status: Fixed

Comment #3 on issue 585 by chipx86: post-review dies when binaries are part  
of a p4 changeset
http://code.google.com/p/reviewboard/issues/detail?id=585

Fixed in r1862.

--
You received this message because you are listed in the owner
or CC fields of this issue, or because you starred this issue.
You may adjust your issue notification preferences at:
http://code.google.com/hosting/settings

--~--~-~--~~~---~--~~
You received this message because you are subscribed to the Google Groups 
reviewboard-issues group.
To post to this group, send email to reviewboard-issues@googlegroups.com
To unsubscribe from this group, send email to 
reviewboard-issues+unsubscr...@googlegroups.com
For more options, visit this group at 
http://groups.google.com/group/reviewboard-issues?hl=en
-~--~~~~--~~--~--~---



Issue 909 in reviewboard: Add ship it status for logged in user in the dashboard.

2009-03-27 Thread codesite-noreply

Updates:
Status: Fixed
Labels: Milestone-Release1.0

Comment #4 on issue 909 by chipx86: Add ship it status for logged in user  
in the dashboard.
http://code.google.com/p/reviewboard/issues/detail?id=909

(No comment was entered for this change.)

--
You received this message because you are listed in the owner
or CC fields of this issue, or because you starred this issue.
You may adjust your issue notification preferences at:
http://code.google.com/hosting/settings

--~--~-~--~~~---~--~~
You received this message because you are subscribed to the Google Groups 
reviewboard-issues group.
To post to this group, send email to reviewboard-issues@googlegroups.com
To unsubscribe from this group, send email to 
reviewboard-issues+unsubscr...@googlegroups.com
For more options, visit this group at 
http://groups.google.com/group/reviewboard-issues?hl=en
-~--~~~~--~~--~--~---



Issue 991 in reviewboard: post-review.py does not trim whitespace before comparing a string in diff

2009-03-27 Thread codesite-noreply

Status: New
Owner: 
Labels: Type-Defect Priority-Medium

New issue 991 by bengchuan79: post-review.py does not trim whitespace  
before comparing a string in diff
http://code.google.com/p/reviewboard/issues/detail?id=991

at line 1486, in  def diff(self, args):

 if len(dl) == 1 and \
dl[0] == ('Files %s and %s differ'% (old_file, new_file)):

While using with Perforce, dl[0] appears to have a whitespace. This causes
the string comparison failed.

Proposed solution:
if len(dl) == 1 and \
string.strip(dl[0]) == ('Files %s and %s differ'% (old_file,
new_file)):

--
You received this message because you are listed in the owner
or CC fields of this issue, or because you starred this issue.
You may adjust your issue notification preferences at:
http://code.google.com/hosting/settings

--~--~-~--~~~---~--~~
You received this message because you are subscribed to the Google Groups 
reviewboard-issues group.
To post to this group, send email to reviewboard-issues@googlegroups.com
To unsubscribe from this group, send email to 
reviewboard-issues+unsubscr...@googlegroups.com
For more options, visit this group at 
http://groups.google.com/group/reviewboard-issues?hl=en
-~--~~~~--~~--~--~---



Issue 658 in reviewboard: Error : The patch to 'filepath' didn't apply cleanly.

2009-03-27 Thread codesite-noreply


Comment #2 on issue 658 by miguel.rgonzalez: Error : The patch  
to 'filepath' didn't apply cleanly.
http://code.google.com/p/reviewboard/issues/detail?id=658

same issue here.
I get the same error when posting reviews for newly created file (nothing  
to diff
against)

--
You received this message because you are listed in the owner
or CC fields of this issue, or because you starred this issue.
You may adjust your issue notification preferences at:
http://code.google.com/hosting/settings

--~--~-~--~~~---~--~~
You received this message because you are subscribed to the Google Groups 
reviewboard-issues group.
To post to this group, send email to reviewboard-issues@googlegroups.com
To unsubscribe from this group, send email to 
reviewboard-issues+unsubscr...@googlegroups.com
For more options, visit this group at 
http://groups.google.com/group/reviewboard-issues?hl=en
-~--~~~~--~~--~--~---



Issue 992 in reviewboard: Trailing slash in SVN repository confused post-review

2009-03-27 Thread codesite-noreply

Status: New
Owner: 
Labels: Type-Defect Priority-Medium

New issue 992 by nick.las...@kaseya.com: Trailing slash in SVN repository  
confused post-review
http://code.google.com/p/reviewboard/issues/detail?id=992

What's the URL of the page containing the problem?

post-review

What steps will reproduce the problem?
1. Setup a SVN repository in Review Board such as  
https://code.google.com/svn/
2. Run post-review tool on that URL
3. Get an error message saying the repository was not found
4. Change it to https://code.google.com/svn and see that everything works.

What is the expected output? What do you see instead?

The trailing slash should not matter.

What operating system are you using? What browser?

Windows client, Ubuntu host. The error comes from the server.



--
You received this message because you are listed in the owner
or CC fields of this issue, or because you starred this issue.
You may adjust your issue notification preferences at:
http://code.google.com/hosting/settings

--~--~-~--~~~---~--~~
You received this message because you are subscribed to the Google Groups 
reviewboard-issues group.
To post to this group, send email to reviewboard-issues@googlegroups.com
To unsubscribe from this group, send email to 
reviewboard-issues+unsubscr...@googlegroups.com
For more options, visit this group at 
http://groups.google.com/group/reviewboard-issues?hl=en
-~--~~~~--~~--~--~---