Sounds like we're doing a poor job of normalizing the URL. That should be fixed.
Christian -- Christian Hammond - [email protected] Review Board - http://www.reviewboard.org VMware, Inc. - http://www.vmware.com On Sun, Feb 28, 2010 at 8:02 PM, danielc <[email protected]> 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 <[email protected]> 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 - [email protected] > > Review Board -http://www.reviewboard.org > > VMware, Inc. -http://www.vmware.com > > > > On Thu, Feb 25, 2010 at 6:38 PM, danielc <[email protected]> wrote: > > > update: > > > Found this post by Christian: > > >http://www.mail-archive.com/[email protected]/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 <[email protected]> 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 <[email protected]> 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 - [email protected] > > > > > Review Board -http://www.reviewboard.org > > > > > VMware, Inc. -http://www.vmware.com > > > > > > > On Thu, Feb 25, 2010 at 4:39 PM, danielc <[email protected]> > 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 <[email protected]> wrote: > > > > > > > Looks like this guy here describes the same sort of problem: > > > > >http://www.mail-archive.com/[email protected]/msg02695.html > > > > > > > but I didn install pysvn into the python packages folder. (used > > > their > > > > > > > win32 installer) > > > > > > > > > On Feb 26, 9:16 am, Christian Hammond <[email protected]> > 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 - [email protected] > > > > > > > > Review Board -http://www.reviewboard.org > > > > > > > > VMware, Inc. -http://www.vmware.com > > > > > > > > > > On Thu, Feb 25, 2010 at 2:15 PM, danielc < > [email protected]> > > > 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 <[email protected] > > > > > wrote: > > > > > > > > > > Glad it works :) > > > > > > > > > > > > Christian > > > > > > > > > > > > -- > > > > > > > > > > Christian Hammond - [email protected] > > > > > > > > > > Review Board -http://www.reviewboard.org > > > > > > > > > > VMware, Inc. -http://www.vmware.com > > > > > > > > > > > > On Wed, Feb 24, 2010 at 8:49 PM, danielc < > > > [email protected]> > > > > > > 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 < > [email protected] > > > > > > > > wrote: > > > > > > > > > > > > Try removing the Alias for /reviews/ and adding back > the > > > media > > > > > > and > > > > > > > > > > > errordocs > > > > > > > > > > > > ones. > > > > > > > > > > > > > > Christian > > > > > > > > > > > > > > -- > > > > > > > > > > > > Christian Hammond - [email protected] > > > > > > > > > > > > Review Board -http://www.reviewboard.org > > > > > > > > > > > > VMware, Inc. -http://www.vmware.com > > > > > > > > > > > > > > On Wed, Feb 24, 2010 at 8:30 PM, danielc < > > > [email protected]> > > > > > > > > > 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 > [email protected]<reviewboard%[email protected]> > 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 [email protected] For more options, visit this group at http://groups.google.com/group/reviewboard?hl=en
