Sounds like we're doing a poor job of normalizing the URL. That should be
fixed.

Christian

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


On Sun, Feb 28, 2010 at 8:02 PM, danielc <co.dan...@gmail.com> wrote:

> 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/ <http://maskfp01/%0Areviews/> --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...@1537svn://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<reviewboard%2bunsubscr...@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

Reply via email to