Hi Chris,
Just wanted to let you know that if you set a repo path ending with
"/" in the admin pages, the "post-review" tool returns a 206 error.
I've attached the debug trace for it.
After removed the "/" from the path, it managed to submit the review.

----------------------------------------------------------------------------------------------
C:\SVN\workingCopy\trunk\SomnoLab>post-review --server=http://maskfp01/
reviews/ --revision-range=1537 --username=USER --password=*****  --
repository-url=svn://maskfp01/replay/trunk/SomnoLab -d
>>> svn info svn://maskfp01/replay/trunk/SomnoLab
>>> repository info: Path: svn://maskfp01/replay, Base path: /trunk/SomnoLab, Su
pports changesets: False
>>> svn diff --diff-cmd=diff svn://maskfp01/replay/trunk/somno...@1537 svn://mas
kfp01/replay/trunk/somno...@head
>>> Looking for 'maskfp01 /reviews/' cookie in C:\Documents and Settings\Adminis
trator\Application Data\.post-review-cookies.txt
>>> Cookie file loaded, but no cookie for this server
==> Review Board Login Required
Enter username and password for Review Board at http://maskfp01/reviews/
>>> Logging in with username "daniel"
>>> HTTP POSTing to http://maskfp01/reviews/api/json/accounts/login/: {'username
': 'daniel', 'password': '**************'}
>>> Logged in.
>>> HTTP GETting /api/json/repositories/
>>> HTTP GETting /api/json/repositories/1/info/
>>> repository info: Path: svn://maskfp01/replay, Base path: /trunk/SomnoLab, Su
pports changesets: False
>>> Attempting to create review request on svn://maskfp01/replay for None
>>> HTTP POSTing to http://maskfp01/reviews/api/json/reviewrequests/new/: {'repo
sitory_path': 'svn://maskfp01/replay'}
>>> Got API Error 206 (HTTP code 200): The repository path specified is not in t
he list of known repositories
>>> Error data: {'repository_path': 'svn://maskfp01/replay', 'stat': 'fail', 'er
r': {'msg': 'The repository path specified is not in the list of known
repositor
ies', 'code': 206}}
Error creating review request: The repository path specified is not in
the list
of known repositories (HTTP 200, API Error 206)
----------------------------------------------------------------------------------------------


Cheers,
Daniel


On Feb 27, 5:48 am, Christian Hammond <chip...@chipx86.com> wrote:
> Glad to hear it :)
>
> There seem to be issues with the newer pysvn and Subversion. I suspect pysvn
> isn't properly checking for its own compatibility with the Subversion in the
> path.
>
> Christian
>
> --
> Christian Hammond - chip...@chipx86.com
> Review Board -http://www.reviewboard.org
> VMware, Inc. -http://www.vmware.com
>
> On Thu, Feb 25, 2010 at 6:38 PM, danielc <co.dan...@gmail.com> wrote:
> > update:
> > Found this post by Christian:
> >http://www.mail-archive.com/reviewboard@googlegroups.com/msg02417.html
> > which suggest using pysvn v 1.6.3.
> > SVN repo now shows in the new review request. No errors in log!
>
> > Continuing with testing ...
>
> > On Feb 26, 12:26 pm, danielc <co.dan...@gmail.com> wrote:
> > > Although I haven't really used the product yet, I can see you're doing
> > > a great job. Thanks you.
> > > If I get this to work, I will be happy to share the configuration and
> > > dependencies I've used. Even upload them somewhere.
> > > Even if it's just a bunch of installed, not bundled into one, I am
> > > sure it would help people who are going to install it on windows in
> > > the future.
>
> > > Any words of advice in regards to the pysvn problem?
> > > I've tried running the examples from:
> > > C:\Python25\Lib\site-packages\pysvn\Examples\Client\svn_cmd.py
> > > and it seems to work fine.
>
> > > On Feb 26, 11:59 am, Christian Hammond <chip...@chipx86.com> wrote:
>
> > > > Agreed, we need to do that.
>
> > > > In general, I recommend installing on Linux, which we have more
> > experience
> > > > with and our dependencies are more actively tested on. This can be done
> > in a
> > > > VM or something, if need be.
>
> > > > I'm really hoping we can get a proper Win32 installer in time. We need
> > > > someone to maintain it, but the advantage would be that it would
> > install
> > > > known good versions of the dependencies.
>
> > > > Christian
>
> > > > --
> > > > Christian Hammond - chip...@chipx86.com
> > > > Review Board -http://www.reviewboard.org
> > > > VMware, Inc. -http://www.vmware.com
>
> > > > On Thu, Feb 25, 2010 at 4:39 PM, danielc <co.dan...@gmail.com> wrote:
> > > > > Sorry, I mean I DID install it into the packages folder using the
> > > > > win32 installer.
>
> > > > > BTW, I wanted to recommend you to indicate specific versions of the
> > > > > required prerequisites for reviewboard and maybe links to downloading
> > > > > them, or even include a package of them since they can disappear.
> > > > > Having a list of such 'tested' versions would save a lot of time
> > > > > during installation mix&matching the right versions.
>
> > > > > On Feb 26, 10:29 am, danielc <co.dan...@gmail.com> wrote:
> > > > > > Looks like this guy here describes the same sort of problem:
>
> >http://www.mail-archive.com/reviewboard@googlegroups.com/msg02695.html
> > > > > > but I didn install pysvn into the python packages folder. (used
> > their
> > > > > > win32 installer)
>
> > > > > > On Feb 26, 9:16 am, Christian Hammond <chip...@chipx86.com> wrote:
>
> > > > > > > Sounds like an installation problem with PySVN. The Python module
> > isn't
> > > > > > > finding the DLL, or isn't finding a symbol in it.
>
> > > > > > > Christian
>
> > > > > > > --
> > > > > > > Christian Hammond - chip...@chipx86.com
> > > > > > > Review Board -http://www.reviewboard.org
> > > > > > > VMware, Inc. -http://www.vmware.com
>
> > > > > > > On Thu, Feb 25, 2010 at 2:15 PM, danielc <co.dan...@gmail.com>
> > wrote:
> > > > > > > > Sorry to be a pain, but I am getting the following error in the
> > logs
> > > > > > > > when trying to create a new review request.
> > > > > > > > The reposotory drop down just appears empty in the UI.
>
> > > > > > > > 2010-02-25 17:29:51,671 - ERROR - Error loading SCMTool for
> > > > > repository
> > > > > > > > Replay (ID 1): DLL load failed: The specified procedure could
> > not be
> > > > > > > > found.
> > > > > > > > Traceback (most recent call last):
> > > > > > > >  File
> > "c:\python25\lib\site-packages\ReviewBoard-1.0.5.1-py2.5.egg
> > > > > > > > \reviewboard\reviews\forms.py", line 62, in __init__
> > > > > > > >    self.field_mapping[repo.id] =
> > repo.get_scmtool().get_fields()
> > > > > > > >  File
> > "c:\python25\lib\site-packages\ReviewBoard-1.0.5.1-py2.5.egg
> > > > > > > > \reviewboard\scmtools\models.py", line 43, in get_scmtool
> > > > > > > >    return cls(self)
> > > > > > > >  File
> > "c:\python25\lib\site-packages\ReviewBoard-1.0.5.1-py2.5.egg
> > > > > > > > \reviewboard\scmtools\svn.py", line 49, in __init__
> > > > > > > >    import pysvn
> > > > > > > >  File "C:\Python25\lib\site-packages\pysvn\__init__.py", line
> > 99, in
> > > > > > > > <module>
> > > > > > > >    import _pysvn_2_5
> > > > > > > > ImportError: DLL load failed: The specified procedure could not
> > be
> > > > > > > > found.
>
> > > > > > > > The following is installed:
> > > > > > > > py25-pysvn-svn161-1.7.0-1177
> > > > > > > > with Python 2.5
>
> > > > > > > > any ideas?
>
> > > > > > > > On Feb 25, 3:53 pm, Christian Hammond <chip...@chipx86.com>
> > wrote:
> > > > > > > > > Glad it works :)
>
> > > > > > > > > Christian
>
> > > > > > > > > --
> > > > > > > > > Christian Hammond - chip...@chipx86.com
> > > > > > > > > Review Board -http://www.reviewboard.org
> > > > > > > > > VMware, Inc. -http://www.vmware.com
>
> > > > > > > > > On Wed, Feb 24, 2010 at 8:49 PM, danielc <
> > co.dan...@gmail.com>
> > > > > wrote:
> > > > > > > > > > Christian, Thanks you SO much! You Rock!
> > > > > > > > > > It works. Just in case someone else needs it. Here is the
> > final
> > > > > > > > > > httpd.conf:
>
> > ----------------------------------------------------------------------------------------------------------------------------------
> > > > > > > > > >         Alias /reviews/media "c:/reviewboard/htdocs/media"
> > > > > > > > > >        Alias /reviews/errordocs
> > "c:/reviewboard/htdocs/errordocs"
>
> > > > > > > > > >        <Location "/reviews/">.
> > > > > > > > > >        Options Indexes FollowSymLinks ExecCGI
> > > > > > > > > >        AllowOverride All
>
> > > > > > > > > >    Order allow,deny
> > > > > > > > > >    Allow from all
>
> > > > > > > > > >        PythonPath "['c:/reviewboard/conf'] + sys.path"
> > > > > > > > > >                SetEnv DJANGO_SETTINGS_MODULE
> > reviewboard.settings
> > > > > > > > > >                SetEnv PYTHON_EGG_CACHE
> > > > > "c:/reviewboard/tmp/egg_cache"
> > > > > > > > > >                SetHandler mod_python
> > > > > > > > > >                PythonHandler django.core.handlers.modpython
> > > > > > > > > >                PythonAutoReload Off
> > > > > > > > > >                PythonDebug Off
> > > > > > > > > >                # Used to run multiple mod_python sites in
> > the
> > > > > same
> > > > > > > > apache
> > > > > > > > > >                PythonInterpreter reviewboard_reviewboard
> > > > > > > > > >        </Location>
>
> > > > > > > > > > <Location "/reviews/media">
> > > > > > > > > >        SetHandler None
> > > > > > > > > > </Location>
> > > > > > > > > > <Location "/reviews/errordocs">
> > > > > > > > > >        SetHandler None
> > > > > > > > > > </Location>
>
> > ----------------------------------------------------------------------------------------------------------------------------------
> > > > > > > > > > Going to have a play with it now
>
> > > > > > > > > > On Feb 25, 3:41 pm, Christian Hammond <chip...@chipx86.com
>
> > > > > wrote:
> > > > > > > > > > > Try removing the Alias for /reviews/ and adding back the
> > media
> > > > > and
> > > > > > > > > > errordocs
> > > > > > > > > > > ones.
>
> > > > > > > > > > > Christian
>
> > > > > > > > > > > --
> > > > > > > > > > > Christian Hammond - chip...@chipx86.com
> > > > > > > > > > > Review Board -http://www.reviewboard.org
> > > > > > > > > > > VMware, Inc. -http://www.vmware.com
>
> > > > > > > > > > > On Wed, Feb 24, 2010 at 8:30 PM, danielc <
> > co.dan...@gmail.com>
> > > > > > > > wrote:
> > > > > > > > > > > > THANK YOU!
> > > > > > > > > > > > OK, good and bad news.
> > > > > > > > > > > > Good: it seems to work now. I can login and access
>
> >http://maskfp01/reviews/account/preferences/?next=/reviews/dashboard/
>
> > > > > > > > > > > > BAD. it's doesn't display any images or anything for
> > the
> > > > > media
> > > > > > > > folder.
> > > > > > > > > > > > I Get 404 not found error for them.
>
> > > > > > > > > > > > from my new  httpd.conf:
>
> > ----------------------------------------------------------------------------------------------------------------------------------
> > > > > > > > > > > >         Alias /reviews/ "c:/reviewboard/htdocs"
> > > > > > > > > > > > #       Alias /reviews/media
> > "c:/reviewboard/htdocs/media"
> > > > > > > > > > > > #       Alias /reviews/errordocs
> > > > > "c:/reviewboard/htdocs/errordocs"
>
> > > > > > > > > > > >         <Location "/reviews/">.
> > > > > > > > > > > >         Options Indexes FollowSymLinks ExecCGI
> > > > > > > > > > > >        AllowOverride All
>
> > > > > > > > > > > >    Order allow,deny
> > > > > > > > > > > >    Allow from all
>
> > > > > > > > > > > >        PythonPath "['c:/reviewboard/conf'] + sys.path"
> > > > > > > > > > > >                SetEnv DJANGO_SETTINGS_MODULE
> > > > > reviewboard.settings
> > > > > > > > > > > >                SetEnv PYTHON_EGG_CACHE
> > > > > > > > "c:/reviewboard/tmp/egg_cache"
> > > > > > > > > > > >                SetHandler mod_python
> > > > > > > > > > > >                PythonHandler
> > django.core.handlers.modpython
> > > > > > > > > > > >                PythonAutoReload Off
> > > > > > > > > > > >                PythonDebug Off
> > > > > > > > > > > >                # Used to run multiple mod_python sites
> > in the
> > > > > same
> > > > > > > > > > apache
> > > > > > > > > > > >                PythonInterpreter
> > reviewboard_reviewboard
> > > > > > > > > > > >         </Location>
>
> > > > > > > > > > > > <Location "/reviews/media">
> > > > > > > > > > > >        SetHandler None
> > > > > > > > > > > > </Location>
> > > > > > > > > > > > <Location "/reviews/errordocs">
> > > > > > > > > > > >        SetHandler None
> > > > > > > > > > > > </Location>
>
> ...
>
> read more »

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

Reply via email to