Re: [reviewboard for git] Error uploading diff

2016-06-02 Thread David Trowbridge
The "Raw File URL Mask" is wrong. Review Board requires interfaces which
are not available in the git remote APIs, so when using plain git
repositories (those not hosted by a service), you either need to have a
local checkout or use a frontend that allows individual file access such as
GitWeb or cgit.

See
https://www.reviewboard.org/docs/manual/2.5/admin/configuration/repositories/#raw-file-urls
for
details.

-David

On Thu, Jun 2, 2016 at 7:38 AM 尹慧贤  wrote:

>
>
>
>1.  The review board is installed on windows 7.
>2.  the git is deployed on ubuntu.
>3. the review board configuration as
>
>
>
> 
>4. the review board rc file:
>
>   REVIEWBOARD_URL = "http://135.242.156.181/reviewboard;
>   REPOSITORY = "http://192.168.112.128/git/itfm.git;
>  REPOSITORY = 'internal'
>  USERNAME = "andy"
>  PASSWORD = "123456"
>
>
> and the error output is :
>   root@ubuntu:/tmp/itfm# post-review --debug --parent=HEAD~8
> >>> RBTools 0.3.4
> >>> Home = /root
> >>> svn info
> >>> git rev-parse --git-dir
> >>> git config core.bare
> >>> git symbolic-ref -q HEAD
> >>> git config --get branch.master.merge
> >>> git config --get branch.master.remote
> >>> git config --get remote.origin.url
> >>> repository info: Path: http://192.168.112.128/git/itfm.git, Base
> path: , Supports changesets: False
> >>> HTTP GETting api/
> >>> HTTP GETting http://135.242.156.181/reviewboard/api/info/
> >>> Using the new web API
> >>> git merge-base origin/master refs/heads/master
> >>> git diff --no-color --full-index --no-ext-diff HEAD~8
> >>> git diff --no-color --full-index --no-ext-diff
> 793a79c2afe0beeabf03c39b7d718a1b12f45711..HEAD~8
> >>> Attempting to create review request on internal for None
> >>> HTTP POSTing to
> http://135.242.156.181/reviewboard/api/review-requests/: {'repository':
> 'internal'}
> >>> Review request created
> >>> Uploading diff, size: 82038
> >>> Uploading parent diff, size: 82013
> >>> HTTP POSTing to
> http://135.242.156.181/reviewboard/api/review-requests/35/diffs/: {}
> >>> Got API Error 207 (HTTP code 400): The file was not found in the
> repository.
> >>> Error data: {u'stat': u'fail', u'file': u'confd-wrapper/IDatabase.h',
> u'err': {u'msg': u'The file was not found in the repository.', u'code':
> 207}, u'revision': u'd1f51fe8636a7918edbe3c20d9528b40b64347e6'}
>
>
> Appreciate for any help...
>
>
>
>
>
> --
> Supercharge your Review Board with Power Pack:
> https://www.reviewboard.org/powerpack/
> Want us to host Review Board for you? Check out RBCommons:
> https://rbcommons.com/
> Happy user? Let us know! https://www.reviewboard.org/users/
> ---
> You received this message because you are subscribed to the Google Groups
> "reviewboard" group.
> To unsubscribe from this group and stop receiving emails from it, send an
> email to reviewboard+unsubscr...@googlegroups.com.
> For more options, visit https://groups.google.com/d/optout.
>

-- 
Supercharge your Review Board with Power Pack: 
https://www.reviewboard.org/powerpack/
Want us to host Review Board for you? Check out RBCommons: 
https://rbcommons.com/
Happy user? Let us know! https://www.reviewboard.org/users/
--- 
You received this message because you are subscribed to the Google Groups 
"reviewboard" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to reviewboard+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.


[reviewboard for git] Error uploading diff

2016-06-02 Thread 尹慧贤




   1.  The review board is installed on windows 7.
   2.  the git is deployed on ubuntu.
   3. the review board configuration as
   
   
   

   4. the review board rc file:
   
  REVIEWBOARD_URL = "http://135.242.156.181/reviewboard;
  REPOSITORY = "http://192.168.112.128/git/itfm.git;
 REPOSITORY = 'internal'
 USERNAME = "andy"
 PASSWORD = "123456"


and the error output is :
  root@ubuntu:/tmp/itfm# post-review --debug --parent=HEAD~8
>>> RBTools 0.3.4
>>> Home = /root
>>> svn info
>>> git rev-parse --git-dir
>>> git config core.bare
>>> git symbolic-ref -q HEAD
>>> git config --get branch.master.merge
>>> git config --get branch.master.remote
>>> git config --get remote.origin.url
>>> repository info: Path: http://192.168.112.128/git/itfm.git, Base path: 
, Supports changesets: False
>>> HTTP GETting api/
>>> HTTP GETting http://135.242.156.181/reviewboard/api/info/
>>> Using the new web API
>>> git merge-base origin/master refs/heads/master
>>> git diff --no-color --full-index --no-ext-diff HEAD~8
>>> git diff --no-color --full-index --no-ext-diff 
793a79c2afe0beeabf03c39b7d718a1b12f45711..HEAD~8
>>> Attempting to create review request on internal for None
>>> HTTP POSTing to 
http://135.242.156.181/reviewboard/api/review-requests/: {'repository': 
'internal'}
>>> Review request created
>>> Uploading diff, size: 82038
>>> Uploading parent diff, size: 82013
>>> HTTP POSTing to 
http://135.242.156.181/reviewboard/api/review-requests/35/diffs/: {}
>>> Got API Error 207 (HTTP code 400): The file was not found in the 
repository.
>>> Error data: {u'stat': u'fail', u'file': u'confd-wrapper/IDatabase.h', 
u'err': {u'msg': u'The file was not found in the repository.', u'code': 
207}, u'revision': u'd1f51fe8636a7918edbe3c20d9528b40b64347e6'}


Appreciate for any help...

   
   
 

-- 
Supercharge your Review Board with Power Pack: 
https://www.reviewboard.org/powerpack/
Want us to host Review Board for you? Check out RBCommons: 
https://rbcommons.com/
Happy user? Let us know! https://www.reviewboard.org/users/
--- 
You received this message because you are subscribed to the Google Groups 
"reviewboard" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to reviewboard+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.


How to install reviewboard from GIT

2015-05-18 Thread Sri Ram Kannan
Hi,

I have to make a particular dev - change in reviewboard .

For this, I cloned the new reviewboard version from GIT repository.

When I tried to install using setup.py as :

python setup.py install

It throws an error:

Traceback (most recent call last):
  File contrib/internal/build-media.py, line 29, in module
ret = call_command('collectstatic', interactive=False, verbosity=2)
  File 
/usr/local/lib/python2.7/dist-packages/django/core/management/__init__.py, 
line 159, in call_command
return klass.execute(*args, **defaults)
  File 
/usr/local/lib/python2.7/dist-packages/django/core/management/base.py, 
line 285, in execute
output = self.handle(*args, **options)
  File 
/usr/local/lib/python2.7/dist-packages/django/core/management/base.py, 
line 415, in handle
return self.handle_noargs(**options)
  File 
/usr/local/lib/python2.7/dist-packages/django/contrib/staticfiles/management/commands/collectstatic.py,
 
line 173, in handle_noargs
collected = self.collect()
  File 
/usr/local/lib/python2.7/dist-packages/django/contrib/staticfiles/management/commands/collectstatic.py,
 
line 119, in collect
for original_path, processed_path, processed in processor:
  File 
/usr/local/lib/python2.7/dist-packages/django_pipeline-1.3.27-py2.7.egg/pipeline/storage.py,
 
line 32, in post_process
packager.pack_stylesheets(package)
  File 
/usr/local/lib/python2.7/dist-packages/django_pipeline-1.3.27-py2.7.egg/pipeline/packager.py,
 
line 94, in pack_stylesheets
variant=package.variant, **kwargs)
  File 
/usr/local/lib/python2.7/dist-packages/django_pipeline-1.3.27-py2.7.egg/pipeline/packager.py,
 
line 103, in pack
paths = self.compile(package.paths, force=True)
  File 
/usr/local/lib/python2.7/dist-packages/django_pipeline-1.3.27-py2.7.egg/pipeline/packager.py,
 
line 97, in compile
return self.compiler.compile(paths, force=force)
  File 
/usr/local/lib/python2.7/dist-packages/django_pipeline-1.3.27-py2.7.egg/pipeline/compilers/__init__.py,
 
line 55, in compile
return list(executor.map(_compile, paths))
  File 
/usr/local/lib/python2.7/dist-packages/futures-3.0.2-py2.7.egg/concurrent/futures/_base.py,
 
line 579, in result_iterator
yield future.result()
  File 
/usr/local/lib/python2.7/dist-packages/futures-3.0.2-py2.7.egg/concurrent/futures/_base.py,
 
line 403, in result
return self.__get_result()
  File 
/usr/local/lib/python2.7/dist-packages/futures-3.0.2-py2.7.egg/concurrent/futures/thread.py,
 
line 55, in run
result = self.fn(*self.args, **self.kwargs)
  File 
/usr/local/lib/python2.7/dist-packages/django_pipeline-1.3.27-py2.7.egg/pipeline/compilers/__init__.py,
 
line 40, in _compile
outdated=outdated, force=force)
  File 
/usr/local/lib/python2.7/dist-packages/django_pipeline-1.3.27-py2.7.egg/pipeline/compilers/less.py,
 
line 22, in compile_file
return self.execute_command(command, cwd=dirname(infile))
  File 
/usr/local/lib/python2.7/dist-packages/django_pipeline-1.3.27-py2.7.egg/pipeline/compilers/__init__.py,
 
line 99, in execute_command
raise CompilerError(stderr)
*pipeline.exceptions.CompilerError: /usr/bin/env: lessc: No such file or 
directory*

Traceback (most recent call last):
  File setup.py, line 231, in module
Topic :: Software Development :: Quality Assurance,
  File /usr/lib/python2.7/distutils/core.py, line 151, in setup
dist.run_commands()
  File /usr/lib/python2.7/distutils/dist.py, line 953, in run_commands
self.run_command(cmd)
  File /usr/lib/python2.7/distutils/dist.py, line 972, in run_command
cmd_obj.run()
  File /usr/lib/python2.7/dist-packages/setuptools/command/install.py, 
line 73, in run
self.do_egg_install()
  File /usr/lib/python2.7/dist-packages/setuptools/command/install.py, 
line 88, in do_egg_install
self.run_command('bdist_egg')
  File /usr/lib/python2.7/distutils/cmd.py, line 326, in run_command
self.distribution.run_command(command)
  File /usr/lib/python2.7/distutils/dist.py, line 972, in run_command
cmd_obj.run()
  File /usr/lib/python2.7/dist-packages/setuptools/command/bdist_egg.py, 
line 177, in run
self.run_command(egg_info)
  File /usr/lib/python2.7/distutils/cmd.py, line 326, in run_command
self.distribution.run_command(command)
  File /usr/lib/python2.7/distutils/dist.py, line 972, in run_command
cmd_obj.run()
  File setup.py, line 80, in run
self.run_command('build_media')
  File /usr/lib/python2.7/distutils/cmd.py, line 326, in run_command
self.distribution.run_command(command)
  File /usr/lib/python2.7/distutils/dist.py, line 972, in run_command
cmd_obj.run()
  File setup.py, line 100, in run
raise RuntimeError('Failed to build media files')
*RuntimeError: Failed to build media files*

I have seen many posts with same error, but none of them helped me to solve 
this issue.

There was post mentioning that settings_local.py file must be present on 
the top directory of reviewboard to avoid this annoying error, but I didnt 
see any

Re: How to install reviewboard from GIT

2015-05-18 Thread Christian Hammond
Hi,

You'll need to install the latest versions of nodejs, lesscss, and uglifyjs
to package Review Board. We generally don't recommend that people modify
Review Board itself, but rather write extensions for this, so we don't
currently have any documentation covering this aspect of the packaging.

Christian

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

On Mon, May 18, 2015 at 3:10 PM, Sri Ram Kannan sriram.ss...@gmail.com
wrote:

 Hi,

 I have to make a particular dev - change in reviewboard .

 For this, I cloned the new reviewboard version from GIT repository.

 When I tried to install using setup.py as :

 python setup.py install

 It throws an error:

 Traceback (most recent call last):
   File contrib/internal/build-media.py, line 29, in module
 ret = call_command('collectstatic', interactive=False, verbosity=2)
   File
 /usr/local/lib/python2.7/dist-packages/django/core/management/__init__.py,
 line 159, in call_command
 return klass.execute(*args, **defaults)
   File
 /usr/local/lib/python2.7/dist-packages/django/core/management/base.py,
 line 285, in execute
 output = self.handle(*args, **options)
   File
 /usr/local/lib/python2.7/dist-packages/django/core/management/base.py,
 line 415, in handle
 return self.handle_noargs(**options)
   File
 /usr/local/lib/python2.7/dist-packages/django/contrib/staticfiles/management/commands/collectstatic.py,
 line 173, in handle_noargs
 collected = self.collect()
   File
 /usr/local/lib/python2.7/dist-packages/django/contrib/staticfiles/management/commands/collectstatic.py,
 line 119, in collect
 for original_path, processed_path, processed in processor:
   File
 /usr/local/lib/python2.7/dist-packages/django_pipeline-1.3.27-py2.7.egg/pipeline/storage.py,
 line 32, in post_process
 packager.pack_stylesheets(package)
   File
 /usr/local/lib/python2.7/dist-packages/django_pipeline-1.3.27-py2.7.egg/pipeline/packager.py,
 line 94, in pack_stylesheets
 variant=package.variant, **kwargs)
   File
 /usr/local/lib/python2.7/dist-packages/django_pipeline-1.3.27-py2.7.egg/pipeline/packager.py,
 line 103, in pack
 paths = self.compile(package.paths, force=True)
   File
 /usr/local/lib/python2.7/dist-packages/django_pipeline-1.3.27-py2.7.egg/pipeline/packager.py,
 line 97, in compile
 return self.compiler.compile(paths, force=force)
   File
 /usr/local/lib/python2.7/dist-packages/django_pipeline-1.3.27-py2.7.egg/pipeline/compilers/__init__.py,
 line 55, in compile
 return list(executor.map(_compile, paths))
   File
 /usr/local/lib/python2.7/dist-packages/futures-3.0.2-py2.7.egg/concurrent/futures/_base.py,
 line 579, in result_iterator
 yield future.result()
   File
 /usr/local/lib/python2.7/dist-packages/futures-3.0.2-py2.7.egg/concurrent/futures/_base.py,
 line 403, in result
 return self.__get_result()
   File
 /usr/local/lib/python2.7/dist-packages/futures-3.0.2-py2.7.egg/concurrent/futures/thread.py,
 line 55, in run
 result = self.fn(*self.args, **self.kwargs)
   File
 /usr/local/lib/python2.7/dist-packages/django_pipeline-1.3.27-py2.7.egg/pipeline/compilers/__init__.py,
 line 40, in _compile
 outdated=outdated, force=force)
   File
 /usr/local/lib/python2.7/dist-packages/django_pipeline-1.3.27-py2.7.egg/pipeline/compilers/less.py,
 line 22, in compile_file
 return self.execute_command(command, cwd=dirname(infile))
   File
 /usr/local/lib/python2.7/dist-packages/django_pipeline-1.3.27-py2.7.egg/pipeline/compilers/__init__.py,
 line 99, in execute_command
 raise CompilerError(stderr)
 *pipeline.exceptions.CompilerError: /usr/bin/env: lessc: No such file or
 directory*

 Traceback (most recent call last):
   File setup.py, line 231, in module
 Topic :: Software Development :: Quality Assurance,
   File /usr/lib/python2.7/distutils/core.py, line 151, in setup
 dist.run_commands()
   File /usr/lib/python2.7/distutils/dist.py, line 953, in run_commands
 self.run_command(cmd)
   File /usr/lib/python2.7/distutils/dist.py, line 972, in run_command
 cmd_obj.run()
   File /usr/lib/python2.7/dist-packages/setuptools/command/install.py,
 line 73, in run
 self.do_egg_install()
   File /usr/lib/python2.7/dist-packages/setuptools/command/install.py,
 line 88, in do_egg_install
 self.run_command('bdist_egg')
   File /usr/lib/python2.7/distutils/cmd.py, line 326, in run_command
 self.distribution.run_command(command)
   File /usr/lib/python2.7/distutils/dist.py, line 972, in run_command
 cmd_obj.run()
   File /usr/lib/python2.7/dist-packages/setuptools/command/bdist_egg.py,
 line 177, in run
 self.run_command(egg_info)
   File /usr/lib/python2.7/distutils/cmd.py, line 326, in run_command
 self.distribution.run_command(command)
   File /usr/lib/python2.7/distutils/dist.py, line 972, in run_command
 cmd_obj.run()
   File setup.py, line 80, in run
 self.run_command('build_media')
   File /usr

Re: Issue 3213 in reviewboard: rbtools git client fails if core.bare is unset

2014-10-21 Thread reviewboard


Comment #6 on issue 3213 by nano...@gmail.com: rbtools git client fails if  
core.bare is unset

https://code.google.com/p/reviewboard/issues/detail?id=3213

damned, you're right :-(

sorry for this noise

--
You received this message because this project is configured to send all  
issue notifications to this address.

You may adjust your notification preferences at:
https://code.google.com/hosting/settings

--
You received this message because you are subscribed to the Google Groups 
reviewboard-issues group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to reviewboard-issues+unsubscr...@googlegroups.com.
To post to this group, send email to reviewboard-issues@googlegroups.com.
Visit this group at http://groups.google.com/group/reviewboard-issues.
For more options, visit https://groups.google.com/d/optout.


Re: Issue 3213 in reviewboard: rbtools git client fails if core.bare is unset

2014-10-17 Thread reviewboard


Comment #4 on issue 3213 by nano...@gmail.com: rbtools git client fails if  
core.bare is unset

https://code.google.com/p/reviewboard/issues/detail?id=3213

Hi David,

could you please cherry-pick 6a44356 to release-0.6.x so that it could be  
included in next 0.6.3 release?


Regards

Nicolas

--
You received this message because this project is configured to send all  
issue notifications to this address.

You may adjust your notification preferences at:
https://code.google.com/hosting/settings

--
You received this message because you are subscribed to the Google Groups 
reviewboard-issues group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to reviewboard-issues+unsubscr...@googlegroups.com.
To post to this group, send email to reviewboard-issues@googlegroups.com.
Visit this group at http://groups.google.com/group/reviewboard-issues.
For more options, visit https://groups.google.com/d/optout.


Re: Issue 3213 in reviewboard: rbtools git client fails if core.bare is unset

2014-10-17 Thread reviewboard


Comment #5 on issue 3213 by trowb...@gmail.com: rbtools git client fails if  
core.bare is unset

https://code.google.com/p/reviewboard/issues/detail?id=3213

This commit was made before we created the release-0.6.x branch (from  
master), so it's included in all 0.6.x releases.


--
You received this message because this project is configured to send all  
issue notifications to this address.

You may adjust your notification preferences at:
https://code.google.com/hosting/settings

--
You received this message because you are subscribed to the Google Groups 
reviewboard-issues group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to reviewboard-issues+unsubscr...@googlegroups.com.
To post to this group, send email to reviewboard-issues@googlegroups.com.
Visit this group at http://groups.google.com/group/reviewboard-issues.
For more options, visit https://groups.google.com/d/optout.


Issue 3213 in reviewboard: rbtools git client fails if core.bare is unset

2014-01-28 Thread reviewboard

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

New issue 3213 by nano...@gmail.com: rbtools git client fails if core.bare  
is unset

http://code.google.com/p/reviewboard/issues/detail?id=3213

*** READ THIS BEFORE POSTING!
***
*** You must complete this form in its entirety, or your bug report will be
rejected.
***
*** For customer support, please post to reviewbo...@googlegroups.com
***
*** If you have a patch, please submit it to
http://reviews.reviewboard.org/
***
*** Do not post confidential information in this bug report!


What version are you running?
rbtools 0.5.1

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

What steps will reproduce the problem?
1. repo sync or git clone a git repo
2. core.bare is not set in .git/config
3. post-review

What is the expected output? What do you see instead?
post-review fails because git config core.bare exit 1 and exception is not  
caught

if git config core.bare exit 1 it means core.bare != true

What operating system are you using? What browser?
NA

Please provide any additional information below.

--
You received this message because this project is configured to send all  
issue notifications to this address.

You may adjust your notification preferences at:
https://code.google.com/hosting/settings

--
You received this message because you are subscribed to the Google Groups 
reviewboard-issues group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to reviewboard-issues+unsubscr...@googlegroups.com.
To post to this group, send email to reviewboard-issues@googlegroups.com.
Visit this group at http://groups.google.com/group/reviewboard-issues.
For more options, visit https://groups.google.com/groups/opt_out.


Re: Issue 3213 in reviewboard: rbtools git client fails if core.bare is unset

2014-01-28 Thread reviewboard


Comment #1 on issue 3213 by nano...@gmail.com: rbtools git client fails if  
core.bare is unset

http://code.google.com/p/reviewboard/issues/detail?id=3213

when using git-repo tool without mirroring support (to be confirmed), git  
clones are setup without core.bare set.


--
You received this message because this project is configured to send all  
issue notifications to this address.

You may adjust your notification preferences at:
https://code.google.com/hosting/settings

--
You received this message because you are subscribed to the Google Groups 
reviewboard-issues group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to reviewboard-issues+unsubscr...@googlegroups.com.
To post to this group, send email to reviewboard-issues@googlegroups.com.
Visit this group at http://groups.google.com/group/reviewboard-issues.
For more options, visit https://groups.google.com/groups/opt_out.


Re: Issue 3213 in reviewboard: rbtools git client fails if core.bare is unset

2014-01-28 Thread reviewboard


Comment #2 on issue 3213 by nano...@gmail.com: rbtools git client fails if  
core.bare is unset

http://code.google.com/p/reviewboard/issues/detail?id=3213

A proposal to fix this issue at
https://reviews.reviewboard.org/r/5342

--
You received this message because this project is configured to send all  
issue notifications to this address.

You may adjust your notification preferences at:
https://code.google.com/hosting/settings

--
You received this message because you are subscribed to the Google Groups 
reviewboard-issues group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to reviewboard-issues+unsubscr...@googlegroups.com.
To post to this group, send email to reviewboard-issues@googlegroups.com.
Visit this group at http://groups.google.com/group/reviewboard-issues.
For more options, visit https://groups.google.com/groups/opt_out.


Re: Issue 3213 in reviewboard: rbtools git client fails if core.bare is unset

2014-01-28 Thread reviewboard

Updates:
Status: Fixed

Comment #3 on issue 3213 by trowb...@gmail.com: rbtools git client fails if  
core.bare is unset

http://code.google.com/p/reviewboard/issues/detail?id=3213

Fixed in rbtools master (6a44356). Thanks!

--
You received this message because this project is configured to send all  
issue notifications to this address.

You may adjust your notification preferences at:
https://code.google.com/hosting/settings

--
You received this message because you are subscribed to the Google Groups 
reviewboard-issues group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to reviewboard-issues+unsubscr...@googlegroups.com.
To post to this group, send email to reviewboard-issues@googlegroups.com.
Visit this group at http://groups.google.com/group/reviewboard-issues.
For more options, visit https://groups.google.com/groups/opt_out.


Re: Issue 3169 in reviewboard: (RBTools) Git-p4 repo throws 'UnboundLocalError' on post-review

2013-12-22 Thread reviewboard

Updates:
Status: Fixed

Comment #4 on issue 3169 by trowb...@gmail.com: (RBTools) Git-p4 repo  
throws 'UnboundLocalError' on post-review

http://code.google.com/p/reviewboard/issues/detail?id=3169

Fixed in release-0.5.x (0f88c6a)

--
You received this message because this project is configured to send all  
issue notifications to this address.

You may adjust your notification preferences at:
https://code.google.com/hosting/settings

--
You received this message because you are subscribed to the Google Groups 
reviewboard-issues group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to reviewboard-issues+unsubscr...@googlegroups.com.
To post to this group, send email to reviewboard-issues@googlegroups.com.
Visit this group at http://groups.google.com/group/reviewboard-issues.
For more options, visit https://groups.google.com/groups/opt_out.


Issue 3169 in reviewboard: (RBTools) Git-p4 repo throws 'UnboundLocalError' on post-review

2013-12-16 Thread reviewboard

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

New issue 3169 by dpr...@gmail.com: (RBTools) Git-p4 repo  
throws 'UnboundLocalError' on post-review

http://code.google.com/p/reviewboard/issues/detail?id=3169

What version are you running?
0.5.2 (also tested with git 74ebdf)

What's the URL of the page containing the problem?
N/A (RBTools issue)


What steps will reproduce the problem?
1. Sync a perforce repository using git-p4
2. Configure reviewboard settings for Perforce depot
2. Create new branch off master and create a commit you want to post to  
reviewboard

3. Attempt to post with 'post-review'

What is the expected output? What do you see instead?
Unknown expected output, python throws a UnboundLocalError (log attached)

What operating system are you using? What browser?
Fedora 19

Please provide any additional information below.


Attachments:
git-p4-post-review-log.txt  3.1 KB

--
You received this message because this project is configured to send all  
issue notifications to this address.

You may adjust your notification preferences at:
https://code.google.com/hosting/settings

--
You received this message because you are subscribed to the Google Groups 
reviewboard-issues group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to reviewboard-issues+unsubscr...@googlegroups.com.
To post to this group, send email to reviewboard-issues@googlegroups.com.
Visit this group at http://groups.google.com/group/reviewboard-issues.
For more options, visit https://groups.google.com/groups/opt_out.


Re: Issue 3169 in reviewboard: (RBTools) Git-p4 repo throws 'UnboundLocalError' on post-review

2013-12-16 Thread reviewboard


Comment #1 on issue 3169 by dpr...@gmail.com: (RBTools) Git-p4 repo  
throws 'UnboundLocalError' on post-review

http://code.google.com/p/reviewboard/issues/detail?id=3169

It would appear 'base_path' is not getting assigned as the regular  
expression is incorrect.


From git.py:445:
 m = re.search(r'repo-paths = (.+): change = (\d+)\]', log, re.M)

repo-paths should be depot-paths

git log attached

--
You received this message because this project is configured to send all  
issue notifications to this address.

You may adjust your notification preferences at:
https://code.google.com/hosting/settings

--
You received this message because you are subscribed to the Google Groups 
reviewboard-issues group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to reviewboard-issues+unsubscr...@googlegroups.com.
To post to this group, send email to reviewboard-issues@googlegroups.com.
Visit this group at http://groups.google.com/group/reviewboard-issues.
For more options, visit https://groups.google.com/groups/opt_out.


Re: Issue 3169 in reviewboard: (RBTools) Git-p4 repo throws 'UnboundLocalError' on post-review

2013-12-16 Thread reviewboard


Comment #3 on issue 3169 by dpr...@gmail.com: (RBTools) Git-p4 repo  
throws 'UnboundLocalError' on post-review

http://code.google.com/p/reviewboard/issues/detail?id=3169

A fix for this issue was submitted to ReviewBoard.

https://reviews.reviewboard.org/r/5120/

--
You received this message because this project is configured to send all  
issue notifications to this address.

You may adjust your notification preferences at:
https://code.google.com/hosting/settings

--
You received this message because you are subscribed to the Google Groups 
reviewboard-issues group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to reviewboard-issues+unsubscr...@googlegroups.com.
To post to this group, send email to reviewboard-issues@googlegroups.com.
Visit this group at http://groups.google.com/group/reviewboard-issues.
For more options, visit https://groups.google.com/groups/opt_out.


Issue 3125 in reviewboard: rbtools git parent diff support broken for revision ranges

2013-10-31 Thread reviewboard

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

New issue 3125 by ade...@gmail.com: rbtools git parent diff support broken  
for revision ranges

http://code.google.com/p/reviewboard/issues/detail?id=3125

post-review in RBTools 0.5.2 with --revision-range isn't uploading the  
parent diff to the RB server. It's generating both diffs but due to a bug  
in git.py it's not properly registering the parent_diff, causing the caller  
to ignore it.


I've found and fixed the problem; just filing this bug so that others will  
know about the breakage.


--
You received this message because this project is configured to send all  
issue notifications to this address.

You may adjust your notification preferences at:
https://code.google.com/hosting/settings

--
You received this message because you are subscribed to the Google Groups 
reviewboard-issues group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to reviewboard-issues+unsubscr...@googlegroups.com.
To post to this group, send email to reviewboard-issues@googlegroups.com.
Visit this group at http://groups.google.com/group/reviewboard-issues.
For more options, visit https://groups.google.com/groups/opt_out.


Re: Issue 3125 in reviewboard: rbtools git parent diff support broken for revision ranges

2013-10-31 Thread reviewboard


Comment #1 on issue 3125 by ade...@gmail.com: rbtools git parent diff  
support broken for revision ranges

http://code.google.com/p/reviewboard/issues/detail?id=3125

Pull request: https://github.com/reviewboard/rbtools/pull/29


--
You received this message because this project is configured to send all  
issue notifications to this address.

You may adjust your notification preferences at:
https://code.google.com/hosting/settings

--
You received this message because you are subscribed to the Google Groups 
reviewboard-issues group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to reviewboard-issues+unsubscr...@googlegroups.com.
To post to this group, send email to reviewboard-issues@googlegroups.com.
Visit this group at http://groups.google.com/group/reviewboard-issues.
For more options, visit https://groups.google.com/groups/opt_out.


Re: Issue 3125 in reviewboard: rbtools git parent diff support broken for revision ranges

2013-10-31 Thread reviewboard


Comment #2 on issue 3125 by ade...@gmail.com: rbtools git parent diff  
support broken for revision ranges

http://code.google.com/p/reviewboard/issues/detail?id=3125

Forgot you guys don't use pull requests. Here's the RB link:  
https://reviews.reviewboard.org/r/4870/


--
You received this message because this project is configured to send all  
issue notifications to this address.

You may adjust your notification preferences at:
https://code.google.com/hosting/settings

--
You received this message because you are subscribed to the Google Groups 
reviewboard-issues group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to reviewboard-issues+unsubscr...@googlegroups.com.
To post to this group, send email to reviewboard-issues@googlegroups.com.
Visit this group at http://groups.google.com/group/reviewboard-issues.
For more options, visit https://groups.google.com/groups/opt_out.


Re: Issue 3125 in reviewboard: rbtools git parent diff support broken for revision ranges

2013-10-31 Thread reviewboard

Updates:
Status: Fixed

Comment #3 on issue 3125 by trowb...@gmail.com: rbtools git parent diff  
support broken for revision ranges

http://code.google.com/p/reviewboard/issues/detail?id=3125

Fixed in rbtools release-0.5.x (50cf71e). Thanks!

--
You received this message because this project is configured to send all  
issue notifications to this address.

You may adjust your notification preferences at:
https://code.google.com/hosting/settings

--
You received this message because you are subscribed to the Google Groups 
reviewboard-issues group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to reviewboard-issues+unsubscr...@googlegroups.com.
To post to this group, send email to reviewboard-issues@googlegroups.com.
Visit this group at http://groups.google.com/group/reviewboard-issues.
For more options, visit https://groups.google.com/groups/opt_out.


ReviewBoard with git. “There don't seem to be any diffs!” but I see them.

2013-07-08 Thread Emanuele Ianni


I'm trying this codereview tool. I'm having problems with git. This is the 
output of git status:

# On branch master
# Changes not staged for commit:
#   (use git add file... to update what will be committed)
#   (use git checkout -- file... to discard changes in working directory)
#
#   modified:   path/to/something/modified.js
#
# Untracked files:
#   (use git add file... to include in what will be committed)
#
#   .gitignore
#   .reviewboardrc

If i try to run post-review --guess-summary --guess-description -p under 
this status I get this message:

There don't seem to be any diffs!

Now, if I run git add -u, my git status is:

# On branch master
# Changes to be committed:
#   (use git reset HEAD file... to unstage)
#
#   modified:   path/to/something/modified.js
#
# Untracked files:
#   (use git add file... to include in what will be committed)
#
#   .gitignore
#   .reviewboardrc

With this situation, if I run again post-review --guess-summary 
--guess-description -p I still get the same message:

There don't seem to be any diffs!

At this point if I commit it with the command git commit -m my commit I 
have the following message (obviously):

[master 027e044] my commit 1 file changed, 1 insertion(+)

and I run again the post-review command I have the output:

Review request #22 posted.

http://192.168.133.218/r/22/

This puzzles me because If you go to read the ReviewBoard FAQ here's the 
message:

A basic pre-commit workflow with git would look like this:

Clone the central repository.
Make a change you want reviewed, but do not commit it yet.
Run post-review (or otherwise submit a diff).
Get reviews, update your change as needed.
When the change is marked to ship, commit it to master and push it to the 
origin.

Since my intention was doing a pre-commit, can you explain to me why there 
are no diff after I stage changes for commit?

-- 
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
--- 
You received this message because you are subscribed to the Google Groups 
reviewboard group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to reviewboard+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/groups/opt_out.




Re: ReviewBoard with git. “There don't seem to be any diffs!” but I see them.

2013-07-08 Thread David Trowbridge
With git, pre-commit really means pre-merge or pre-push. Most of us work in 
topic branches on our local clone, and only merge those changes to master after 
they've been reviewed.

-David

On Jul 8, 2013, at 8:27 AM, Emanuele Ianni die...@gmail.com wrote:

 I'm trying this codereview tool. I'm having problems with git. This is the 
 output of git status:
 
 # On branch master
 # Changes not staged for commit:
 #   (use git add file... to update what will be committed)
 #   (use git checkout -- file... to discard changes in working directory)
 #
 #   modified:   path/to/something/modified.js
 #
 # Untracked files:
 #   (use git add file... to include in what will be committed)
 #
 #   .gitignore
 #   .reviewboardrc
 If i try to run post-review --guess-summary --guess-description -p under this 
 status I get this message:
 
 There don't seem to be any diffs!
 
 Now, if I run git add -u, my git status is:
 
 # On branch master
 # Changes to be committed:
 #   (use git reset HEAD file... to unstage)
 #
 #   modified:   path/to/something/modified.js
 #
 # Untracked files:
 #   (use git add file... to include in what will be committed)
 #
 #   .gitignore
 #   .reviewboardrc
 With this situation, if I run again post-review --guess-summary 
 --guess-description -p I still get the same message:
 
 There don't seem to be any diffs!
 
 At this point if I commit it with the command git commit -m my commit I 
 have the following message (obviously):
 
 [master 027e044] my commit 1 file changed, 1 insertion(+)
 
 and I run again the post-review command I have the output:
 
 Review request #22 posted.
 
 http://192.168.133.218/r/22/
 This puzzles me because If you go to read the ReviewBoard FAQ here's the 
 message:
 
 A basic pre-commit workflow with git would look like this:
 
 Clone the central repository.
 Make a change you want reviewed, but do not commit it yet.
 Run post-review (or otherwise submit a diff).
 Get reviews, update your change as needed.
 When the change is marked to ship, commit it to master and push it to the 
 origin.
 Since my intention was doing a pre-commit, can you explain to me why there 
 are no diff after I stage changes for commit?
 
 -- 
 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
 --- 
 You received this message because you are subscribed to the Google Groups 
 reviewboard group.
 To unsubscribe from this group and stop receiving emails from it, send an 
 email to reviewboard+unsubscr...@googlegroups.com.
 For more options, visit https://groups.google.com/groups/opt_out.
  
  

-- 
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
--- 
You received this message because you are subscribed to the Google Groups 
reviewboard group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to reviewboard+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/groups/opt_out.




Re: Setting up Reviewboard with Git gives Unable to authenticate against this repository using one of the supported authentication types error

2013-04-30 Thread David Trowbridge
Connecting to git repositories requires using the Raw File URL and some
kind of frontend like cgit or gitweb, due to the fact that the git protocol
doesn't support fetching individual blobs. You can't just use the clone url.

Please see
http://www.reviewboard.org/docs/manual/1.7/admin/configuration/repositories/#raw-file-urlsfor
more information.

-David


On Tue, Apr 30, 2013 at 12:38 PM, mamta.nanav...@gmail.com wrote:

 Hi,
 My Git repo is of the form :

 git@ipaddress:www. This is the exact string I use to run git clone.

 I have the ssh keys generated for reviewboard and installed on the Git
 server.

 When i try to setup the repository with Reviewboard with following settings

 *Name*: GitDemo
 *HostingService*: Custom
 *RepositoryType*: Git
 *Path:*git@ipaddress:www

 It gives me the following error:

- Unable to authenticate against this repository using one of the
supported authentication types

 Any pointers will be appreciated.


 Thanks,

 M

 --
 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
 ---
 You received this message because you are subscribed to the Google Groups
 reviewboard group.
 To unsubscribe from this group and stop receiving emails from it, send an
 email to reviewboard+unsubscr...@googlegroups.com.
 For more options, visit https://groups.google.com/groups/opt_out.




-- 
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
--- 
You received this message because you are subscribed to the Google Groups 
reviewboard group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to reviewboard+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/groups/opt_out.




Re: Issue 1483 in reviewboard: Allow git repository to be configured with SourceForge hosting service

2013-02-05 Thread reviewboard


Comment #2 on issue 1483 by trowb...@gmail.com: Allow git repository to be  
configured with SourceForge hosting service

http://code.google.com/p/reviewboard/issues/detail?id=1483

As far as I can tell, the allura platform (which is what SourceForge uses  
these days) doesn't have any way to fetch file blobs directly. We'll need  
that capability before we can do this.


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




Re: Issue 2077 in reviewboard: Incorrect git diff parsing

2012-09-23 Thread reviewboard

Updates:
Status: Fixed
Labels: Milestone-Release1.6.x Component-DiffViewer Component-SCMTools

Comment #5 on issue 2077 by chip...@gmail.com: Incorrect git diff parsing
http://code.google.com/p/reviewboard/issues/detail?id=2077

A lot of work has gone into diff parsing and into GitHub support since  
these bugs. I believe the issues have been fixed, unless you're posting a  
change that has a parent not part of the upstream repository (in which  
case, you want to use --parent=).


Going to close this, but if you're still hitting it, please reopen.

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



reviewboard and git

2012-03-28 Thread Vivek Thakkar
Hi,

I am trying to setup reviewboard server to work with GIT repository. We
have a central GIT repository managed by gitosis (ssh based tool).
My reviewboard server setup seems ok. But I am a little confused with
uploading the code review on it as the documentation on reviewboard site
seems out of date to me (as per the mail archive on this group). The model
I would like to use is the following.

a) All the developers clone from the central repo.
b) They make changes locally and commit them in their local branch.
c) Upload the review using post-review.

For c), I am trying to follow the following steps:

1) git config reviewboard.url http://
IP_reviewboard_server/web_server_name
2) post-review --parent=master --tracking-branch=origin/master

Here, origin would be the central git server (gitoisis@...)

Does reviewboard work with above configuration?

Regards,
-Vivek

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

Re: Issue 2448 in reviewboard: Clarify git repo instructions

2012-01-22 Thread reviewboard

Updates:
Status: Fixed

Comment #1 on issue 2448 by trowb...@gmail.com: Clarify git repo  
instructions

http://code.google.com/p/reviewboard/issues/detail?id=2448

Fixed in release-1.6.x (512b868). Thanks!

--
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 2448 in reviewboard: Clarify git repo instructions

2012-01-11 Thread reviewboard

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

New issue 2448 by joshua.o...@gmail.com: Clarify git repo instructions
http://code.google.com/p/reviewboard/issues/detail?id=2448

*NOTE: If you have a patch, please submit it to
http://reviews.reviewboard.org/


What version are you running?
1.6.3

What's the URL of the page this enhancement relates to, if any?


Describe the enhancement and the motivation for it.
Change text in the git description from
For git, this should be the path to the .git repository on the local disk.
to
For git, this should be the path to the .git directory on the local disk.

Since the former may lead users to giving
/path/to/local
rather than
/path/to/local/.git/

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



Re: Issue 2077 in reviewboard: Incorrect git diff parsing

2011-12-02 Thread reviewboard


Comment #3 on issue 2077 by chris.a@gmail.com: Incorrect git diff  
parsing

http://code.google.com/p/reviewboard/issues/detail?id=2077

I'm running into what appears to be the same issue here. I have a project  
attached to github, and it's failing to parse the diffs. It complains about  
revisions not existing:
Got API Error 207 (HTTP code 400): The file was not found in the  
repository
Error data: {u'stat': u'fail', u'file':  
u'plugins/com.aptana.editor.dtd/parsing/DTD.grammar', u'err': {u'msg':  
u'The file was not found in the repository', u'code': 207},  
u'revision': u'14f130fca4dfd182e42e5b99ad543c39ba8517e3'}


The diff itself is using the SHAs of the blobs representing the file  
contents before and after:


diff --git a/plugins/com.aptana.editor.dtd/parsing/DTD.grammar  
b/plugins/com.aptana.editor.dtd/parsing/DTD.grammar
index  
d5f60c4b1b447b0fb6191c1a34a4a40f248a7905..14f130fca4dfd182e42e5b99ad543c39ba8517e3  
100644


Github expects the commit SHA containing that version of the file in it's  
URLs, like so:

https://github.com/aptana/studio3/blob/5a8245f18a6a844c99eca27b497a134cd67ce431/plugins/com.aptana.editor.dtd/parsing/DTD.grammar

This also seems to be the same issue?
http://www.mail-archive.com/reviewboard@googlegroups.com/msg06300.html

Is there a way I can fix this manually? if I change the repository  
from Github to custom, it shows  
http://github.com/api/v2/yaml/blob/show/aptana/studio3/revision as the  
raw file URL mask, and substituting the blob from the diff there does  
return what I'd expect (though note that it becomes https):

https://github.com/api/v2/yaml/blob/show/aptana/studio3/14f130fca4dfd182e42e5b99ad543c39ba8517e3

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



Re: Issue 2077 in reviewboard: Incorrect git diff parsing

2011-12-02 Thread reviewboard


Comment #4 on issue 2077 by chris.a@gmail.com: Incorrect git diff  
parsing

http://code.google.com/p/reviewboard/issues/detail?id=2077

I changed that file mask to use https and it failed on a different file. I  
then tried setting up a clone of the repo that was on disk and has a cron  
job to update it regularly. Against that repo the same diff file works.


Looking at the github API v2 docs it says there's a rate limit of 60 calls  
a minute. Could that be what is causing the issues here? There's a new  
github v3 APi that rate limits to 5000/hour, maybe that would work better?  
But in any case, I'd assume that since it's likely that 60+ files could be  
changed in a large diff, that handling this for Github should be included  
in the logic...


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



Found blog post: Using Reviewboard with Git

2011-06-21 Thread elf Pavlik
Hi,

I have just come across this blog post:
http://ericholscher.com/blog/2011/jan/24/using-reviewboard-git/

Thought that you may like to add it to the list of posts on you homepage...

Cheers,
elf Pavlik

http://wwelves.org/perpetual-tripper

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


Issue 2077 in reviewboard: Incorrect git diff parsing

2011-04-20 Thread reviewboard

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

New issue 2077 by a.chuk...@gmail.com: Incorrect git diff parsing
http://code.google.com/p/reviewboard/issues/detail?id=2077

* NOTE: Do not post confidential information in this bug report. *
*   If you need immediate support, please contact*
*   reviewbo...@googlegroups.com *

What version are you running?
Review board 1.5.5


What steps will reproduce the problem?
1. Make changes to an existing file.
2. Use post-review to upload it.
3. It fails complaining on non-existent revision.

What is the expected output? What do you see instead?
Seems that it treats git file's SHA1 to be commit id.

Please provide any additional information below.

git diff --full-index --cc -r HEAD^..HEAD
diff --git a/tmp b/tmp
index  
44957fa4b300b0fbee878cb7aa95c495371f4a5c..5a3f4060c35281b591f0d09f0b10815aac6e5b20  
100644

--- a/tmp
+++ b/tmp
@@ -1,2 +1 @@
-artasrt
 Now add small changes
diff --git a/tmp2 b/tmp2
new file mode 100644
index  
..1c975dce2df0cbcd48b1fb67f53abef77919945a

--- /dev/null
+++ b/tmp2
@@ -0,0 +1 @@
+tarstasrt

post-review -d -r 31 --revision-range=HEAD^:HEAD

Got API Error 207 (HTTP code 200): The file was not found in the  
repository
Error data: {u'deprecated': {u'in_version': u'1.5'}, u'stat': u'fail',  
u'file': u'tmp', u'err': {u'msg': u'The file was not found in the  
repository', u'code': 207}, u'revision':  
u'44957fa4b300b0fbee878cb7aa95c495371f4a5c'}



(It thinks that SHA1 of old tmp is a revision ID)

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



Re: Issue 2077 in reviewboard: Incorrect git diff parsing

2011-04-20 Thread reviewboard


Comment #1 on issue 2077 by a.chuk...@gmail.com: Incorrect git diff parsing
http://code.google.com/p/reviewboard/issues/detail?id=2077

I tried different setup for gitweb: either  
http://hostname/?p=repo.git;a=blob_plain;f=filename;h=revision  or  
http://hostname/?p=repo.git;a=blob_plain;f=filename;hb=revision


None of these versions work.

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



Re: Issue 2077 in reviewboard: Incorrect git diff parsing

2011-04-20 Thread reviewboard


Comment #2 on issue 2077 by a.chuk...@gmail.com: Incorrect git diff parsing
http://code.google.com/p/reviewboard/issues/detail?id=2077

The problem was in repository setup:


http://hostname/?p=repo.git;a=blob_plain;f=filename;h=revision
or

http://hostname/?p=repo.git;a=blob_plain;f=filename;hb=revision


Later version looks for commit ID while first version looks file by its  
SHA1 (the thing I need).


P.S. I'm interested why do you still call it revision?

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



Re: ReviewBoard with GIT

2010-07-11 Thread Christian Hammond
Can you run with --output-diff and see if the result looks correct to you?

Christian

On Friday, July 9, 2010, Sapna Vempati sapnavemp...@gmail.com wrote:
 Thanks Christian. After modifying the Apache path, I was able to add the repo 
 !

 From my local workspace (for the added repo), I am trying to post a review. 
 For this, I did the following, but am having issues with the diff.

 postreview.py in my local workspace.
 run postreview as: postreview.py --guess-summary  am getting this error. 
 Review has been posted and I am also able to send the request to the 
 reviewer, but without the diff information.
 $ ../postreview.py --guess-summary
 _make_url: path = api/json/reviewrequests/new/
 _make_url:app = /
 _make_url: joined URL = http://test-url.com/api/json/reviewrequests/new/
 _make_url: Resulting URL = http://test-url.com/api/json/reviewrequests/new/
 _make_url: path = api/json/reviewrequests/80/draft/set/
 _make_url:app = /
 _make_url: joined URL = 
 http://test-url.com/api/json/reviewrequests/80/draft/set/
 _make_url: Resulting URL = 
 http://test-url.com/api/json/reviewrequests/80/draft/set/
 _make_url: path = api/json/reviewrequests/80/draft/save/
 _make_url:app = /
 _make_url: joined URL = 
 http://test-url.com/api/json/reviewrequests/80/draft/save/
 _make_url: Resulting URL = 
 http://test-url.com/api/json/reviewrequests/80/draft/save/
 _make_url: path = api/json/reviewrequests/80/diff/new/
 _make_url:app = /
 _make_url: joined URL = 
 http://test-url.com/api/json/reviewrequests/80/diff/new/
 _make_url: Resulting URL = 
 http://test-url.com/api/json/reviewrequests/80/diff/new/
 Error uploading diff: One or more fields had errors (105)
 Your review request still exists, but the diff is not attached.
 Thanks for the help !
 Sapna

 On Tue, Jul 6, 2010 at 2:43 PM, Christian Hammond chip...@chipx86.com wrote:
 Sounds like git is not in the path. If that's the error you're getting, then 
 it's definitely not finding that in the PATH that Apache sees.


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






 On Tue, Jul 6, 2010 at 2:39 PM, Sapna Vempati sapnavemp...@gmail.com wrote:





 I am still not able to set RB for git.

 This is the local path where I have a workspace. This is the copy of the main 
 repository. To setup initially, will use this local path and not cgit (just 
 by periodic pull of main repository, to this local path):

 /projects/mobcom_andrbuilds/users/reviewboard.

 I specified, the following in RB webpage, under admin -- repositories,

 Hosting service:: custom
 Repository type:: git
 path: /projects/mobcom_andrbuilds/users/reviewboard/brcm_tools/scripts/.git
 mirror path:: empty
 Raw file URL mask:: empty

 When I try to save, I get this error:
 The executable git is not in the path.

 I am not sure what I am missing. BTW, I have RB 1.5 Beta2 installed.

 Thanks for the help again,
 Sapna




 On Thu, Jul 1, 2010 at 6:26 PM, Christian Hammond chip...@chipx86.com wrote:





 On Thursday, July 1, 2010, Sapna Vempati sapnavemp...@gmail.com wrote:
 Hi Christian,
 I am trying to configure a repository on our RB test server, and am having 
 trouble filling these fields below,

 When I am under--- admin --- repositories,

 path:  (It says) For git, this should be the path to the .git repository on 
 the local disk.

 But the repo is not on the local disk, but on a central repository. Can I 
 have the unix path to our central server on ubuntu ? (Since I am going to 
 use cgit, I assume I don't have to clone the central repository to my local 
 ?)


 This only needs to point to a local clone if you're not using the raw
 URL path. Otherwise, it should be the upstream path.



 mirror path: Blank ?

 Raw file URL Mask:
 for example: The upper-level directory url with all paths to the .gits is: 
 http://mobcom-git.sj.broadcom.com/cgi-bin/cgit.cgi/git_repos (say under here 
 I need to set to this repo: git_repos/repo_tools/scripts.git)

 How will I get the filename (can be a lot of file in this repo)  
 revision, as suggested on your website for this field.
  http://mobcom-git.sj.broadcom.com/cgi-bin/cgit.cgi/git_repos/repo_tools/scripts.git/

 It's intended as a mask. The filename and revision are literala,
 which Review Board will automatically replace with the needed
 filename.

 This is being reworked for the final release and will be easier then.

 Christian

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


 --


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


 Want to help the Review Board project? Donate today at 
 http://www.reviewboard.org/donate/
 Happy user? Let us know at http

Re: ReviewBoard with GIT

2010-07-09 Thread Sapna Vempati
Thanks Christian. After modifying the Apache path, I was able to add the
repo !

From my local workspace (for the added repo), I am trying to post a review.
For this, I did the following, but am having issues with the diff.

   - postreview.py in my local workspace.
   - run postreview as: postreview.py --guess-summary  am getting this
   error. Review has been posted and I am also able to send the request to the
   reviewer, but without the diff information.

$ ../postreview.py --guess-summary
_make_url: path = api/json/reviewrequests/new/
_make_url:app = /
_make_url: joined URL =
http://test-url.com/api/json/reviewrequests/new/http://%3ctest-url%3e.com/api/json/reviewrequests/new/
_make_url: Resulting URL =
http://test-url.com/api/json/reviewrequests/new/http://%3ctest-url%3e.com/api/json/reviewrequests/new/
_make_url: path = api/json/reviewrequests/80/draft/set/
_make_url:app = /
_make_url: joined URL =
http://test-url.com/api/json/reviewrequests/80/draft/set/http://%3ctest-url%3e.com/api/json/reviewrequests/80/draft/set/
_make_url: Resulting URL =
http://test-url.com/api/json/reviewrequests/80/draft/set/http://%3ctest-url%3e.com/api/json/reviewrequests/80/draft/set/
_make_url: path = api/json/reviewrequests/80/draft/save/
_make_url:app = /
_make_url: joined URL =
http://test-url.com/api/json/reviewrequests/80/draft/save/http://mob-rb-test.sj.broadcom.com/api/json/reviewrequests/80/draft/save/
_make_url: Resulting URL =
http://test-url.com/api/json/reviewrequests/80/draft/save/http://mob-rb-test.sj.broadcom.com/api/json/reviewrequests/80/draft/save/
_make_url: path = api/json/reviewrequests/80/diff/new/
_make_url:app = /
_make_url: joined URL =
http://test-url.com/api/json/reviewrequests/80/diff/new/http://mob-rb-test.sj.broadcom.com/api/json/reviewrequests/80/diff/new/
_make_url: Resulting URL =
http://test-url.com/api/json/reviewrequests/80/diff/new/http://mob-rb-test.sj.broadcom.com/api/json/reviewrequests/80/diff/new/
Error uploading diff: One or more fields had errors (105)
Your review request still exists, but the diff is not attached.
Thanks for the help !
Sapna

On Tue, Jul 6, 2010 at 2:43 PM, Christian Hammond chip...@chipx86.comwrote:

 Sounds like git is not in the path. If that's the error you're getting,
 then it's definitely not finding that in the PATH that Apache sees.


 Christian

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


   On Tue, Jul 6, 2010 at 2:39 PM, Sapna Vempati sapnavemp...@gmail.comwrote:

   I am still not able to set RB for git.

 This is the local path where I have a workspace. This is the copy of the
 main repository. To setup initially, will use this local path and not cgit
 (just by periodic pull of main repository, to this local path):
 /projects/mobcom_andrbuilds/users/reviewboard.

 I specified, the following in RB webpage, under *admin -- repositories,*
 **
 *Hosting service::* custom
 *Repository type::* git
 *path:*/projects/mobcom_andrbuilds/users/reviewboard/brcm_tools/scripts/.git
 *mirror path::* empty
 *Raw file URL mask::* empty

 When I try to save, I get this error:
 The executable git is not in the path.
 I am not sure what I am missing. BTW, I have RB 1.5 Beta2 installed.

 Thanks for the help again,
 Sapna
   On Thu, Jul 1, 2010 at 6:26 PM, Christian Hammond 
 chip...@chipx86.comwrote:

   On Thursday, July 1, 2010, Sapna Vempati sapnavemp...@gmail.com
 wrote:
  Hi Christian,
  I am trying to configure a repository on our RB test server, and am
 having trouble filling these fields below,
 
  When I am under--- admin --- repositories,
 
  path:  (It says) For git, this should be the path to the .git
 repository on the local disk.
 
  But the repo is not on the local disk, but on a central repository. Can
 I have the unix path to our central server on ubuntu ? (Since I am going to
 use cgit, I assume I don't have to clone the central repository to my local
 ?)


 This only needs to point to a local clone if you're not using the raw
 URL path. Otherwise, it should be the upstream path.

 
 
  mirror path: Blank ?
 
  Raw file URL Mask:
  for example: The upper-level directory url with all paths to the .gits
 is: http://mobcom-git.sj.broadcom.com/cgi-bin/cgit.cgi/git_repos (say
 under here I need to set to this repo: git_repos/repo_tools/scripts.git)
 
  How will I get the filename (can be a lot of file in this repo) 
 revision, as suggested on your website for this field.
   
 http://mobcom-git.sj.broadcom.com/cgi-bin/cgit.cgi/git_repos/repo_tools/scripts.git/
 

 It's intended as a mask. The filename and revision are literala,
 which Review Board will automatically replace with the needed
 filename.

 This is being reworked for the final release and will be easier then.

 Christian

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

 --
  Want to help the Review Board project? Donate today

Re: ReviewBoard with GIT

2010-07-06 Thread Sapna Vempati
I am still not able to set RB for git.

This is the local path where I have a workspace. This is the copy of the
main repository. To setup initially, will use this local path and not cgit
(just by periodic pull of main repository, to this local path):
/projects/mobcom_andrbuilds/users/reviewboard.

I specified, the following in RB webpage, under *admin -- repositories,*
**
*Hosting service::* custom
*Repository type::* git
*path:*/projects/mobcom_andrbuilds/users/reviewboard/brcm_tools/scripts/.git
*mirror path::* empty
*Raw file URL mask::* empty

When I try to save, I get this error:
The executable git is not in the path.
I am not sure what I am missing. BTW, I have RB 1.5 Beta2 installed.

Thanks for the help again,
Sapna
On Thu, Jul 1, 2010 at 6:26 PM, Christian Hammond chip...@chipx86.comwrote:

 On Thursday, July 1, 2010, Sapna Vempati sapnavemp...@gmail.com wrote:
  Hi Christian,
  I am trying to configure a repository on our RB test server, and am
 having trouble filling these fields below,
 
  When I am under--- admin --- repositories,
 
  path:  (It says) For git, this should be the path to the .git repository
 on the local disk.
 
  But the repo is not on the local disk, but on a central repository. Can I
 have the unix path to our central server on ubuntu ? (Since I am going to
 use cgit, I assume I don't have to clone the central repository to my local
 ?)


 This only needs to point to a local clone if you're not using the raw
 URL path. Otherwise, it should be the upstream path.

 
 
  mirror path: Blank ?
 
  Raw file URL Mask:
  for example: The upper-level directory url with all paths to the .gits
 is: http://mobcom-git.sj.broadcom.com/cgi-bin/cgit.cgi/git_repos (say
 under here I need to set to this repo: git_repos/repo_tools/scripts.git)
 
  How will I get the filename (can be a lot of file in this repo) 
 revision, as suggested on your website for this field.
   
 http://mobcom-git.sj.broadcom.com/cgi-bin/cgit.cgi/git_repos/repo_tools/scripts.git/
 

 It's intended as a mask. The filename and revision are literala,
 which Review Board will automatically replace with the needed
 filename.

 This is being reworked for the final release and will be easier then.

 Christian

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

 --
  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.comreviewboard%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

Re: ReviewBoard with GIT

2010-07-06 Thread Christian Hammond
Sounds like git is not in the path. If that's the error you're getting, then
it's definitely not finding that in the PATH that Apache sees.

Christian

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


On Tue, Jul 6, 2010 at 2:39 PM, Sapna Vempati sapnavemp...@gmail.comwrote:

 I am still not able to set RB for git.

 This is the local path where I have a workspace. This is the copy of the
 main repository. To setup initially, will use this local path and not cgit
 (just by periodic pull of main repository, to this local path):
 /projects/mobcom_andrbuilds/users/reviewboard.

 I specified, the following in RB webpage, under *admin -- repositories,*
 **
 *Hosting service::* custom
 *Repository type::* git
 *path:*/projects/mobcom_andrbuilds/users/reviewboard/brcm_tools/scripts/.git
 *mirror path::* empty
 *Raw file URL mask::* empty

 When I try to save, I get this error:
 The executable git is not in the path.
 I am not sure what I am missing. BTW, I have RB 1.5 Beta2 installed.

 Thanks for the help again,
 Sapna
 On Thu, Jul 1, 2010 at 6:26 PM, Christian Hammond chip...@chipx86.comwrote:

 On Thursday, July 1, 2010, Sapna Vempati sapnavemp...@gmail.com wrote:
  Hi Christian,
  I am trying to configure a repository on our RB test server, and am
 having trouble filling these fields below,
 
  When I am under--- admin --- repositories,
 
  path:  (It says) For git, this should be the path to the .git repository
 on the local disk.
 
  But the repo is not on the local disk, but on a central repository. Can
 I have the unix path to our central server on ubuntu ? (Since I am going to
 use cgit, I assume I don't have to clone the central repository to my local
 ?)


 This only needs to point to a local clone if you're not using the raw
 URL path. Otherwise, it should be the upstream path.

 
 
  mirror path: Blank ?
 
  Raw file URL Mask:
  for example: The upper-level directory url with all paths to the .gits
 is: http://mobcom-git.sj.broadcom.com/cgi-bin/cgit.cgi/git_repos (say
 under here I need to set to this repo: git_repos/repo_tools/scripts.git)
 
  How will I get the filename (can be a lot of file in this repo) 
 revision, as suggested on your website for this field.
   
 http://mobcom-git.sj.broadcom.com/cgi-bin/cgit.cgi/git_repos/repo_tools/scripts.git/
 

 It's intended as a mask. The filename and revision are literala,
 which Review Board will automatically replace with the needed
 filename.

 This is being reworked for the final release and will be easier then.

 Christian

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

 --
  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.comreviewboard%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.comreviewboard%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

Re: ReviewBoard with GIT

2010-07-01 Thread Sapna Vempati
Hi Christian,
I am trying to configure a repository on our RB test server, and am having
trouble filling these fields below,

When I am under--- *admin --- repositories,*

*path:*  (*It says)* For git, this should be the path to the .git repository
on the local disk.
But the repo is not on the local disk, but on a central repository. Can I
have the unix path to our central server on ubuntu ? (Since I am going to
use cgit, I assume I don't have to clone the central repository to my local
?)

*mirror path:* Blank ?

*Raw file URL Mask:*
for example: The upper-level directory url with all paths to the .gits is:
http://mobcom-git.sj.broadcom.com/cgi-bin/cgit.cgi/git_repos (say under here
I need to set to this repo:
git_repos/repo_tools/scripts.githttp://mobcom-git.sj.broadcom.com/cgi-bin/cgit.cgi/git_repos/repo_tools/scripts.git/
)
How will I get the filename (can be a lot of file in this repo) 
revision, as suggested on your website for this field.
http://mobcom-git.sj.broadcom.com/cgi-bin/cgit.cgi/git_repos/repo_tools/scripts.git/

Thanks for the help !
Sapna

On Fri, Jun 18, 2010 at 5:53 PM, Christian Hammond chip...@chipx86.comwrote:

 Using cgit is the preferred way. It's faster, easier to set up, requires
 less space, and is less error-prone.

 The URL mask referenced in the docs is an actual mask, not a path to a
 specific file. You can't use this for Path or Mirror Path. It needs to go in
 the Raw File URL Mask field, as described in the documentation. You can try
 browsing your cgit repository and viewing a raw file to see if you have it
 correct.

 Your Path or Mirror Path must point to the result of:

 $ git remote show origin

 On the client machines. post-review is going to use this to match the
 repository on Review Board. Review Board in turn is going to plug the
 filename and revision into the Raw File URL mask and use that to fetch the
 file from cgit.

 Christian

 --
 Christian Hammond - chip...@chipx86.com

 Review Board - http://www.reviewboard.org
 VMware, Inc. - http://www.vmware.com


   On Fri, Jun 18, 2010 at 5:03 PM, Kunjal kunjal.par...@gmail.com wrote:

 So..should I create clone on the server OR should I use CGIT? Which
 one is MOST easy.
 I am hold up on this issue almost 8 months now..Any help is greatly
 appreciated.

 Also, can some one answer how to use CGIT here?

 For example: If I have repository in CGIT as

 http://mobcom-git.sj.broadcom.com/cgi-bin/cgit.cgi/git_repos/repo_kernel/prebuilt.git/
 and I put this path in MIRROR PATH then will it work?

 Thanks
 Kunjal



 On Jun 18, 10:00 am, Dan Savilonis d...@n-cube.org wrote:
  On Jun 16, 6:58 pm, Kunjal kunjal.par...@gmail.com wrote:
 
   Now I get the point..
 
   But If keep clone on the server, I need to keep the clone in-sync with
   every commit happening in developers workspace. Is that correct?
 
  No. This is where the parent branch comes in. There are two
  possiblities: the developer wants to review code based off a revision
  that exists in the central repository, or the developer wants to
  review code based off a local revision (which in turn is based off of
  the central repository at some point). For the first case, post-review
  simply submits the diff and the file blobs in the diff already exist
  on the server. In the second case the review diff references files
  that are *not* in the repository, so a corresponding parent diff is
  required to specify the differences between the base revision in the
  central repository and the base revision upon which the review is
  based.
 
   We already have CGIT configured. How I can re-sue CGIT here?
 
   Should I say PATH ashttp://
 mobcom-git.sj.broadcom.com/cgi-bin/cgit.cgi/git_repos/repo_too...
   and will it work?
 
   Thanks for your great help..
 
   Kunjal
 
  Dan
 
 
 
   On Jun 16, 11:46 am, Christian Hammond chip...@chipx86.com wrote:
 
It's a clone of the repository on the Review Board server itself.
 Path
points to that clone, and Mirror Path would be the remote origin
 that the
other developers are using for the central repository. Think of Path
 as
being what Review Board actually uses for fetching files, and Mirror
 Path as
just being an alias that is matched against for the central server.
 
Using a local clone is not great, though, unless Review Board is
 running on
the same server as the central Git repository (or has that server
 mounted).
You really should use something like cgit/gitweb and then the raw
 file URL
mask field. It's easier to maintain and more scalable.
 
Christian
 
--
Christian Hammond - chip...@chipx86.com
Review Board -http://www.reviewboard.org
VMware, Inc. -http://www.vmware.com
 
On Wed, Jun 16, 2010 at 9:57 AM, Kunjal kunjal.par...@gmail.com
 wrote:
 Chris,
 
 I do not understand how GIT can be supported for 300+ users if the
 Path field is the full path of the .git directory inside the
 checkout.
 How would we know the checkout directory 

Re: ReviewBoard with GIT

2010-07-01 Thread Paul Schulz
Hi Sapna,

With the current ReviewBoard, you need to have a local copy of the git
repository you are attempting to review against.

eg. git clone parent-git-repository

I have a cron job that then does 'git pull parent-git-repository'
every minute to keep this copy up to date, and this works reasonably
well. a 1 min delay (at most) doesn't appear to cause problems as it
seems that it takes most developers at least this amount of time to
make their changes and create the diff for the review.

If you have control over the 'upstream repo' then a 'post-commit' hook
that pushes changes as they are made would be a better solution, but I
haven't done this, or seen any documentation on this. (Anyone here
done this? It would be good to have this in the Reviewboard
documentation, or at least referenced.)

Some more documentation:
http://www.reviewboard.org/docs/manual/1.0/admin/management/repositories/#git

Some references, after some googling.
See: 
http://groups.google.com/group/reviewboard/browse_thread/thread/fce7abb11375cac7?pli=1

Cheers,
Paul

On Fri, Jul 2, 2010 at 7:37 AM, Sapna Vempati sapnavemp...@gmail.com wrote:
 Hi Christian,

 I am trying to configure a repository on our RB test server, and am having
 trouble filling these fields below,

 When I am under--- admin --- repositories,

 path:  (It says) For git, this should be the path to the .git repository on
 the local disk.
 But the repo is not on the local disk, but on a central repository. Can I
 have the unix path to our central server on ubuntu ? (Since I am going to
 use cgit, I assume I don't have to clone the central repository to my local
 ?)

 mirror path: Blank ?

 Raw file URL Mask:
 for example: The upper-level directory url with all paths to the .gits is:
 http://mobcom-git.sj.broadcom.com/cgi-bin/cgit.cgi/git_repos (say under here
 I need to set to this repo: git_repos/repo_tools/scripts.git)
 How will I get the filename (can be a lot of file in this repo) 
 revision, as suggested on your website for this field.

 Thanks for the help !
 Sapna

 On Fri, Jun 18, 2010 at 5:53 PM, Christian Hammond chip...@chipx86.com
 wrote:

 Using cgit is the preferred way. It's faster, easier to set up, requires
 less space, and is less error-prone.

 The URL mask referenced in the docs is an actual mask, not a path to a
 specific file. You can't use this for Path or Mirror Path. It needs to go in
 the Raw File URL Mask field, as described in the documentation. You can try
 browsing your cgit repository and viewing a raw file to see if you have it
 correct.

 Your Path or Mirror Path must point to the result of:

     $ git remote show origin

 On the client machines. post-review is going to use this to match the
 repository on Review Board. Review Board in turn is going to plug the
 filename and revision into the Raw File URL mask and use that to fetch the
 file from cgit.

 Christian

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


 On Fri, Jun 18, 2010 at 5:03 PM, Kunjal kunjal.par...@gmail.com wrote:

 So..should I create clone on the server OR should I use CGIT? Which
 one is MOST easy.
 I am hold up on this issue almost 8 months now..Any help is greatly
 appreciated.

 Also, can some one answer how to use CGIT here?

 For example: If I have repository in CGIT as

 http://mobcom-git.sj.broadcom.com/cgi-bin/cgit.cgi/git_repos/repo_kernel/prebuilt.git/
 and I put this path in MIRROR PATH then will it work?

 Thanks
 Kunjal



 On Jun 18, 10:00 am, Dan Savilonis d...@n-cube.org wrote:
  On Jun 16, 6:58 pm, Kunjal kunjal.par...@gmail.com wrote:
 
   Now I get the point..
 
   But If keep clone on the server, I need to keep the clone in-sync
   with
   every commit happening in developers workspace. Is that correct?
 
  No. This is where the parent branch comes in. There are two
  possiblities: the developer wants to review code based off a revision
  that exists in the central repository, or the developer wants to
  review code based off a local revision (which in turn is based off of
  the central repository at some point). For the first case, post-review
  simply submits the diff and the file blobs in the diff already exist
  on the server. In the second case the review diff references files
  that are *not* in the repository, so a corresponding parent diff is
  required to specify the differences between the base revision in the
  central repository and the base revision upon which the review is
  based.
 
   We already have CGIT configured. How I can re-sue CGIT here?
 
   Should I say PATH
   ashttp://mobcom-git.sj.broadcom.com/cgi-bin/cgit.cgi/git_repos/repo_too...
   and will it work?
 
   Thanks for your great help..
 
   Kunjal
 
  Dan
 
 
 
   On Jun 16, 11:46 am, Christian Hammond chip...@chipx86.com wrote:
 
It's a clone of the repository on the Review Board server itself.
Path
points to that clone, and Mirror Path would be the remote origin
that 

Re: ReviewBoard with GIT

2010-07-01 Thread Christian Hammond
On Thursday, July 1, 2010, Sapna Vempati sapnavemp...@gmail.com wrote:
 Hi Christian,
 I am trying to configure a repository on our RB test server, and am having 
 trouble filling these fields below,

 When I am under--- admin --- repositories,

 path:  (It says) For git, this should be the path to the .git repository on 
 the local disk.

 But the repo is not on the local disk, but on a central repository. Can I 
 have the unix path to our central server on ubuntu ? (Since I am going to use 
 cgit, I assume I don't have to clone the central repository to my local ?)


This only needs to point to a local clone if you're not using the raw
URL path. Otherwise, it should be the upstream path.



 mirror path: Blank ?

 Raw file URL Mask:
 for example: The upper-level directory url with all paths to the .gits is: 
 http://mobcom-git.sj.broadcom.com/cgi-bin/cgit.cgi/git_repos (say under here 
 I need to set to this repo: git_repos/repo_tools/scripts.git)

 How will I get the filename (can be a lot of file in this repo)  
 revision, as suggested on your website for this field.
  http://mobcom-git.sj.broadcom.com/cgi-bin/cgit.cgi/git_repos/repo_tools/scripts.git/

It's intended as a mask. The filename and revision are literala,
which Review Board will automatically replace with the needed
filename.

This is being reworked for the final release and will be easier then.

Christian

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

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


Re: ReviewBoard with GIT

2010-06-18 Thread Kunjal
So..should I create clone on the server OR should I use CGIT? Which
one is MOST easy.
I am hold up on this issue almost 8 months now..Any help is greatly
appreciated.

Also, can some one answer how to use CGIT here?

For example: If I have repository in CGIT as
http://mobcom-git.sj.broadcom.com/cgi-bin/cgit.cgi/git_repos/repo_kernel/prebuilt.git/
and I put this path in MIRROR PATH then will it work?

Thanks
Kunjal



On Jun 18, 10:00 am, Dan Savilonis d...@n-cube.org wrote:
 On Jun 16, 6:58 pm, Kunjal kunjal.par...@gmail.com wrote:

  Now I get the point..

  But If keep clone on the server, I need to keep the clone in-sync with
  every commit happening in developers workspace. Is that correct?

 No. This is where the parent branch comes in. There are two
 possiblities: the developer wants to review code based off a revision
 that exists in the central repository, or the developer wants to
 review code based off a local revision (which in turn is based off of
 the central repository at some point). For the first case, post-review
 simply submits the diff and the file blobs in the diff already exist
 on the server. In the second case the review diff references files
 that are *not* in the repository, so a corresponding parent diff is
 required to specify the differences between the base revision in the
 central repository and the base revision upon which the review is
 based.

  We already have CGIT configured. How I can re-sue CGIT here?

  Should I say PATH 
  ashttp://mobcom-git.sj.broadcom.com/cgi-bin/cgit.cgi/git_repos/repo_too...
  and will it work?

  Thanks for your great help..

  Kunjal

 Dan



  On Jun 16, 11:46 am, Christian Hammond chip...@chipx86.com wrote:

   It's a clone of the repository on the Review Board server itself. Path
   points to that clone, and Mirror Path would be the remote origin that the
   other developers are using for the central repository. Think of Path as
   being what Review Board actually uses for fetching files, and Mirror Path 
   as
   just being an alias that is matched against for the central server.

   Using a local clone is not great, though, unless Review Board is running 
   on
   the same server as the central Git repository (or has that server 
   mounted).
   You really should use something like cgit/gitweb and then the raw file URL
   mask field. It's easier to maintain and more scalable.

   Christian

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

   On Wed, Jun 16, 2010 at 9:57 AM, Kunjal kunjal.par...@gmail.com wrote:
Chris,

I do not understand how GIT can be supported for 300+ users if the
Path field is the full path of the .git directory inside the
checkout.
How would we know the checkout directory for each and every developer?
and how it is possible to use the same PATH variable for N number of
users?
It looks like I am missing somehting here.

Please elobarate...

Thanks for your time

Kunjal

The Path field should be the full path of the .git directory inside
this checkout.

On Jun 15, 10:53 pm, Jan Koprowski jan.koprow...@gmail.com wrote:
 This looks like Christain said - git can't find repo. Read this:
   http://www.reviewboard.org/docs/manual/dev/admin/management/repositor...
 to know how get patches for Git to enter.

 On Jun 16, 1:06 am, Christian Hammond chip...@chipx86.com wrote:

  Git only works with local file paths on the Review Board server, 
  unless
you
  have the Raw URL field set. For that, you need a web-based Git 
  viewer
such
  as cgit or gitweb. See the Raw File URL Mask section athttp://
   www.reviewboard.org/docs/manual/dev/admin/management/repositor...

  Basically, Git has no way to fetch a remote file with a given 
  revision
from
  a server, requiring instead local clones. However, our Raw File URL
mask, if
  set correctly, allows us to fetch the file from a web service that 
  is
on the
  central Git server that has access to all files and revisions.

  Christian

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

  On Tue, Jun 15, 2010 at 3:30 PM, Kunjal kunjal.par...@gmail.com
wrote:
   Thanks !

   My path variable is set to git://
   mobcom-git.sj.broadcom.com/git_repos/repo_tools/scripts.git

   My Mirror Path variable is empty. What should be the value in 
   mirror
   path?

   On Jun 15, 2:09 pm, Christian Hammond chip...@chipx86.com wrote:
Hi,

First thing to check is that git is in your path, accessible by
your web
server.

What are you using for your Path and Mirror Path?

Christian

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

Re: ReviewBoard with GIT

2010-06-18 Thread Christian Hammond
Using cgit is the preferred way. It's faster, easier to set up, requires
less space, and is less error-prone.

The URL mask referenced in the docs is an actual mask, not a path to a
specific file. You can't use this for Path or Mirror Path. It needs to go in
the Raw File URL Mask field, as described in the documentation. You can try
browsing your cgit repository and viewing a raw file to see if you have it
correct.

Your Path or Mirror Path must point to the result of:

$ git remote show origin

On the client machines. post-review is going to use this to match the
repository on Review Board. Review Board in turn is going to plug the
filename and revision into the Raw File URL mask and use that to fetch the
file from cgit.

Christian

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


On Fri, Jun 18, 2010 at 5:03 PM, Kunjal kunjal.par...@gmail.com wrote:

 So..should I create clone on the server OR should I use CGIT? Which
 one is MOST easy.
 I am hold up on this issue almost 8 months now..Any help is greatly
 appreciated.

 Also, can some one answer how to use CGIT here?

 For example: If I have repository in CGIT as

 http://mobcom-git.sj.broadcom.com/cgi-bin/cgit.cgi/git_repos/repo_kernel/prebuilt.git/
 and I put this path in MIRROR PATH then will it work?

 Thanks
 Kunjal



 On Jun 18, 10:00 am, Dan Savilonis d...@n-cube.org wrote:
  On Jun 16, 6:58 pm, Kunjal kunjal.par...@gmail.com wrote:
 
   Now I get the point..
 
   But If keep clone on the server, I need to keep the clone in-sync with
   every commit happening in developers workspace. Is that correct?
 
  No. This is where the parent branch comes in. There are two
  possiblities: the developer wants to review code based off a revision
  that exists in the central repository, or the developer wants to
  review code based off a local revision (which in turn is based off of
  the central repository at some point). For the first case, post-review
  simply submits the diff and the file blobs in the diff already exist
  on the server. In the second case the review diff references files
  that are *not* in the repository, so a corresponding parent diff is
  required to specify the differences between the base revision in the
  central repository and the base revision upon which the review is
  based.
 
   We already have CGIT configured. How I can re-sue CGIT here?
 
   Should I say PATH ashttp://
 mobcom-git.sj.broadcom.com/cgi-bin/cgit.cgi/git_repos/repo_too...
   and will it work?
 
   Thanks for your great help..
 
   Kunjal
 
  Dan
 
 
 
   On Jun 16, 11:46 am, Christian Hammond chip...@chipx86.com wrote:
 
It's a clone of the repository on the Review Board server itself.
 Path
points to that clone, and Mirror Path would be the remote origin that
 the
other developers are using for the central repository. Think of Path
 as
being what Review Board actually uses for fetching files, and Mirror
 Path as
just being an alias that is matched against for the central server.
 
Using a local clone is not great, though, unless Review Board is
 running on
the same server as the central Git repository (or has that server
 mounted).
You really should use something like cgit/gitweb and then the raw
 file URL
mask field. It's easier to maintain and more scalable.
 
Christian
 
--
Christian Hammond - chip...@chipx86.com
Review Board -http://www.reviewboard.org
VMware, Inc. -http://www.vmware.com
 
On Wed, Jun 16, 2010 at 9:57 AM, Kunjal kunjal.par...@gmail.com
 wrote:
 Chris,
 
 I do not understand how GIT can be supported for 300+ users if the
 Path field is the full path of the .git directory inside the
 checkout.
 How would we know the checkout directory for each and every
 developer?
 and how it is possible to use the same PATH variable for N number
 of
 users?
 It looks like I am missing somehting here.
 
 Please elobarate...
 
 Thanks for your time
 
 Kunjal
 
 The Path field should be the full path of the .git directory inside
 this checkout.
 
 On Jun 15, 10:53 pm, Jan Koprowski jan.koprow...@gmail.com
 wrote:
  This looks like Christain said - git can't find repo. Read this:

 http://www.reviewboard.org/docs/manual/dev/admin/management/repositor...
  to know how get patches for Git to enter.
 
  On Jun 16, 1:06 am, Christian Hammond chip...@chipx86.com
 wrote:
 
   Git only works with local file paths on the Review Board
 server, unless
 you
   have the Raw URL field set. For that, you need a web-based Git
 viewer
 such
   as cgit or gitweb. See the Raw File URL Mask section athttp://
www.reviewboard.org/docs/manual/dev/admin/management/repositor...
 
   Basically, Git has no way to fetch a remote file with a given
 revision
 from
   a server, requiring instead local clones. However, our Raw File
 URL
 mask, 

Re: ReviewBoard with GIT

2010-06-16 Thread Christian Hammond
It's a clone of the repository on the Review Board server itself. Path
points to that clone, and Mirror Path would be the remote origin that the
other developers are using for the central repository. Think of Path as
being what Review Board actually uses for fetching files, and Mirror Path as
just being an alias that is matched against for the central server.

Using a local clone is not great, though, unless Review Board is running on
the same server as the central Git repository (or has that server mounted).
You really should use something like cgit/gitweb and then the raw file URL
mask field. It's easier to maintain and more scalable.

Christian

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


On Wed, Jun 16, 2010 at 9:57 AM, Kunjal kunjal.par...@gmail.com wrote:

 Chris,

 I do not understand how GIT can be supported for 300+ users if the
 Path field is the full path of the .git directory inside the
 checkout.
 How would we know the checkout directory for each and every developer?
 and how it is possible to use the same PATH variable for N number of
 users?
 It looks like I am missing somehting here.

 Please elobarate...

 Thanks for your time

 Kunjal


 The Path field should be the full path of the .git directory inside
 this checkout.

 On Jun 15, 10:53 pm, Jan Koprowski jan.koprow...@gmail.com wrote:
  This looks like Christain said - git can't find repo. Read this:
 http://www.reviewboard.org/docs/manual/dev/admin/management/repositor...
  to know how get patches for Git to enter.
 
  On Jun 16, 1:06 am, Christian Hammond chip...@chipx86.com wrote:
 
 
 
   Git only works with local file paths on the Review Board server, unless
 you
   have the Raw URL field set. For that, you need a web-based Git viewer
 such
   as cgit or gitweb. See the Raw File URL Mask section athttp://
 www.reviewboard.org/docs/manual/dev/admin/management/repositor...
 
   Basically, Git has no way to fetch a remote file with a given revision
 from
   a server, requiring instead local clones. However, our Raw File URL
 mask, if
   set correctly, allows us to fetch the file from a web service that is
 on the
   central Git server that has access to all files and revisions.
 
   Christian
 
   --
   Christian Hammond - chip...@chipx86.com
   Review Board -http://www.reviewboard.org
   VMware, Inc. -http://www.vmware.com
 
   On Tue, Jun 15, 2010 at 3:30 PM, Kunjal kunjal.par...@gmail.com
 wrote:
Thanks !
 
My path variable is set to git://
mobcom-git.sj.broadcom.com/git_repos/repo_tools/scripts.git
 
My Mirror Path variable is empty. What should be the value in mirror
path?
 
On Jun 15, 2:09 pm, Christian Hammond chip...@chipx86.com wrote:
 Hi,
 
 First thing to check is that git is in your path, accessible by
 your web
 server.
 
 What are you using for your Path and Mirror Path?
 
 Christian
 
 --
 Christian Hammond - chip...@chipx86.com
 Review Board -http://www.reviewboard.org
 VMware, Inc. -http://www.vmware.com
 
 On Tue, Jun 15, 2010 at 2:05 PM, Kunjal kunjal.par...@gmail.com
 wrote:
  Hi chris,
 
  I have reviewborad 1.5Beta 2 configured for GIT.
  When I create review request, I get below error. any pointers?
 
  ImportError at /r/56/
  Request Method: GET
  Request URL:http://mob-rb-test.sj.broadcom.com/r/56/
  Exception Type: ImportError
  Exception Value:
  Exception Location: /projects/mob_tools/xampp/1.6.4-brcm-v2/lib/
 
 python2.5/site-packages/ReviewBoard-1.5beta2-py2.5.egg/reviewboard/
  scmtools/git.py in __init__, line 202
  Python Executable: /opt/lampp/bin/python
  Python Version: 2.5.2
  Python Path: ['/opt/lampp/htdocs/reviewboard/conf',
 '/opt/lampp/lib/
  python2.5/site-packages/django', '/opt/lampp/lib/python2.5/site-
  packages/reviewboard', '/opt/lampp/lib/python2.5/site-packages/
  MySQL_python-1.2.2-py2.5-linux-i686.egg',
 '/opt/lampp/lib/python2.5/
  site-packages/setuptools-0.6c9-py2.5.egg',
 '/opt/lampp/lib/python2.5/
  site-packages/pytz-2009l-py2.5.egg',
 '/opt/lampp/lib/python2.5/site-
  packages/flup-1.0.3.dev_20090716-py2.5.egg',
 '/opt/lampp/lib/python2.5/
  site-packages/Pygments-1.1.1-py2.5.egg',
 '/opt/lampp/lib/python2.5/
  site-packages/django_evolution-0.0.0-py2.5.egg', '/opt/lampp/lib/
  python2.5/site-packages/PIL-1.1.6-py2.5-linux-i686.egg',
 '/opt/lampp/
 
 lib/python2.5/site-packages/simplejson-2.0.9-py2.5-linux-i686.egg', '/
 
 opt/lampp/lib/python2.5/site-packages/python_ldap-2.3.9-py2.5-linux-
  i686.egg', '/opt/lampp/lib/python2.5/site-packages/Django-1.1.1-
  py2.5.egg',
 '/opt/lampp/lib/python2.5/site-packages/JCC-2.5-py2.5-
  linux-i686.egg',
 '/opt/lampp/lib/python2.5/site-packages/lucene-3.0.0-
  py2.5-linux-i686.egg', '/opt/lampp/lib/python2.5/site-packages/
  RBTools-0.2-py2.5.egg', 

Re: ReviewBoard with GIT

2010-06-16 Thread Kunjal

Now I get the point..

But If keep clone on the server, I need to keep the clone in-sync with
every commit happening in developers workspace. Is that correct?

We already have CGIT configured. How I can re-sue CGIT here?

Should I say PATH as 
http://mobcom-git.sj.broadcom.com/cgi-bin/cgit.cgi/git_repos/repo_tools/scripts.git/
and will it work?

Thanks for your great help..

Kunjal


On Jun 16, 11:46 am, Christian Hammond chip...@chipx86.com wrote:
 It's a clone of the repository on the Review Board server itself. Path
 points to that clone, and Mirror Path would be the remote origin that the
 other developers are using for the central repository. Think of Path as
 being what Review Board actually uses for fetching files, and Mirror Path as
 just being an alias that is matched against for the central server.

 Using a local clone is not great, though, unless Review Board is running on
 the same server as the central Git repository (or has that server mounted).
 You really should use something like cgit/gitweb and then the raw file URL
 mask field. It's easier to maintain and more scalable.

 Christian

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



 On Wed, Jun 16, 2010 at 9:57 AM, Kunjal kunjal.par...@gmail.com wrote:
  Chris,

  I do not understand how GIT can be supported for 300+ users if the
  Path field is the full path of the .git directory inside the
  checkout.
  How would we know the checkout directory for each and every developer?
  and how it is possible to use the same PATH variable for N number of
  users?
  It looks like I am missing somehting here.

  Please elobarate...

  Thanks for your time

  Kunjal

  The Path field should be the full path of the .git directory inside
  this checkout.

  On Jun 15, 10:53 pm, Jan Koprowski jan.koprow...@gmail.com wrote:
   This looks like Christain said - git can't find repo. Read this:
 http://www.reviewboard.org/docs/manual/dev/admin/management/repositor...
   to know how get patches for Git to enter.

   On Jun 16, 1:06 am, Christian Hammond chip...@chipx86.com wrote:

Git only works with local file paths on the Review Board server, unless
  you
have the Raw URL field set. For that, you need a web-based Git viewer
  such
as cgit or gitweb. See the Raw File URL Mask section athttp://
 www.reviewboard.org/docs/manual/dev/admin/management/repositor...

Basically, Git has no way to fetch a remote file with a given revision
  from
a server, requiring instead local clones. However, our Raw File URL
  mask, if
set correctly, allows us to fetch the file from a web service that is
  on the
central Git server that has access to all files and revisions.

Christian

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

On Tue, Jun 15, 2010 at 3:30 PM, Kunjal kunjal.par...@gmail.com
  wrote:
 Thanks !

 My path variable is set to git://
 mobcom-git.sj.broadcom.com/git_repos/repo_tools/scripts.git

 My Mirror Path variable is empty. What should be the value in mirror
 path?

 On Jun 15, 2:09 pm, Christian Hammond chip...@chipx86.com wrote:
  Hi,

  First thing to check is that git is in your path, accessible by
  your web
  server.

  What are you using for your Path and Mirror Path?

  Christian

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

  On Tue, Jun 15, 2010 at 2:05 PM, Kunjal kunjal.par...@gmail.com
  wrote:
   Hi chris,

   I have reviewborad 1.5Beta 2 configured for GIT.
   When I create review request, I get below error. any pointers?

   ImportError at /r/56/
   Request Method: GET
   Request URL:http://mob-rb-test.sj.broadcom.com/r/56/
   Exception Type: ImportError
   Exception Value:
   Exception Location: /projects/mob_tools/xampp/1.6.4-brcm-v2/lib/

  python2.5/site-packages/ReviewBoard-1.5beta2-py2.5.egg/reviewboard/
   scmtools/git.py in __init__, line 202
   Python Executable: /opt/lampp/bin/python
   Python Version: 2.5.2
   Python Path: ['/opt/lampp/htdocs/reviewboard/conf',
  '/opt/lampp/lib/
   python2.5/site-packages/django', '/opt/lampp/lib/python2.5/site-
   packages/reviewboard', '/opt/lampp/lib/python2.5/site-packages/
   MySQL_python-1.2.2-py2.5-linux-i686.egg',
  '/opt/lampp/lib/python2.5/
   site-packages/setuptools-0.6c9-py2.5.egg',
  '/opt/lampp/lib/python2.5/
   site-packages/pytz-2009l-py2.5.egg',
  '/opt/lampp/lib/python2.5/site-
   packages/flup-1.0.3.dev_20090716-py2.5.egg',
  '/opt/lampp/lib/python2.5/
   site-packages/Pygments-1.1.1-py2.5.egg',
  '/opt/lampp/lib/python2.5/
   site-packages/django_evolution-0.0.0-py2.5.egg', '/opt/lampp/lib/
   

Re: ReviewBoard with GIT

2010-06-16 Thread Jan Koprowski
Here is some informations about cgit configuration:
http://www.reviewboard.org/docs/manual/dev/admin/management/repositories/#cgit

On Jun 17, 12:58 am, Kunjal kunjal.par...@gmail.com wrote:
 Now I get the point..

 But If keep clone on the server, I need to keep the clone in-sync with
 every commit happening in developers workspace. Is that correct?

 We already have CGIT configured. How I can re-sue CGIT here?

 Should I say PATH 
 ashttp://mobcom-git.sj.broadcom.com/cgi-bin/cgit.cgi/git_repos/repo_too...
 and will it work?

 Thanks for your great help..

 Kunjal

 On Jun 16, 11:46 am, Christian Hammond chip...@chipx86.com wrote:



  It's a clone of the repository on the Review Board server itself. Path
  points to that clone, and Mirror Path would be the remote origin that the
  other developers are using for the central repository. Think of Path as
  being what Review Board actually uses for fetching files, and Mirror Path as
  just being an alias that is matched against for the central server.

  Using a local clone is not great, though, unless Review Board is running on
  the same server as the central Git repository (or has that server mounted).
  You really should use something like cgit/gitweb and then the raw file URL
  mask field. It's easier to maintain and more scalable.

  Christian

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

  On Wed, Jun 16, 2010 at 9:57 AM, Kunjal kunjal.par...@gmail.com wrote:
   Chris,

   I do not understand how GIT can be supported for 300+ users if the
   Path field is the full path of the .git directory inside the
   checkout.
   How would we know the checkout directory for each and every developer?
   and how it is possible to use the same PATH variable for N number of
   users?
   It looks like I am missing somehting here.

   Please elobarate...

   Thanks for your time

   Kunjal

   The Path field should be the full path of the .git directory inside
   this checkout.

   On Jun 15, 10:53 pm, Jan Koprowski jan.koprow...@gmail.com wrote:
This looks like Christain said - git can't find repo. Read this:
  http://www.reviewboard.org/docs/manual/dev/admin/management/repositor...
to know how get patches for Git to enter.

On Jun 16, 1:06 am, Christian Hammond chip...@chipx86.com wrote:

 Git only works with local file paths on the Review Board server, 
 unless
   you
 have the Raw URL field set. For that, you need a web-based Git viewer
   such
 as cgit or gitweb. See the Raw File URL Mask section athttp://
  www.reviewboard.org/docs/manual/dev/admin/management/repositor...

 Basically, Git has no way to fetch a remote file with a given revision
   from
 a server, requiring instead local clones. However, our Raw File URL
   mask, if
 set correctly, allows us to fetch the file from a web service that is
   on the
 central Git server that has access to all files and revisions.

 Christian

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

 On Tue, Jun 15, 2010 at 3:30 PM, Kunjal kunjal.par...@gmail.com
   wrote:
  Thanks !

  My path variable is set to git://
  mobcom-git.sj.broadcom.com/git_repos/repo_tools/scripts.git

  My Mirror Path variable is empty. What should be the value in mirror
  path?

  On Jun 15, 2:09 pm, Christian Hammond chip...@chipx86.com wrote:
   Hi,

   First thing to check is that git is in your path, accessible by
   your web
   server.

   What are you using for your Path and Mirror Path?

   Christian

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

   On Tue, Jun 15, 2010 at 2:05 PM, Kunjal kunjal.par...@gmail.com
   wrote:
Hi chris,

I have reviewborad 1.5Beta 2 configured for GIT.
When I create review request, I get below error. any pointers?

ImportError at /r/56/
Request Method: GET
Request URL:http://mob-rb-test.sj.broadcom.com/r/56/
Exception Type: ImportError
Exception Value:
Exception Location: /projects/mob_tools/xampp/1.6.4-brcm-v2/lib/

   python2.5/site-packages/ReviewBoard-1.5beta2-py2.5.egg/reviewboard/
scmtools/git.py in __init__, line 202
Python Executable: /opt/lampp/bin/python
Python Version: 2.5.2
Python Path: ['/opt/lampp/htdocs/reviewboard/conf',
   '/opt/lampp/lib/
python2.5/site-packages/django', '/opt/lampp/lib/python2.5/site-
packages/reviewboard', '/opt/lampp/lib/python2.5/site-packages/
MySQL_python-1.2.2-py2.5-linux-i686.egg',
   '/opt/lampp/lib/python2.5/
site-packages/setuptools-0.6c9-py2.5.egg',
   '/opt/lampp/lib/python2.5/
site-packages/pytz-2009l-py2.5.egg',
   '/opt/lampp/lib/python2.5/site-

Re: ReviewBoard with GIT

2010-06-15 Thread Christian Hammond
Hi,

First thing to check is that git is in your path, accessible by your web
server.

What are you using for your Path and Mirror Path?

Christian

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


On Tue, Jun 15, 2010 at 2:05 PM, Kunjal kunjal.par...@gmail.com wrote:

 Hi chris,

 I have reviewborad 1.5Beta 2 configured for GIT.
 When I create review request, I get below error. any pointers?

 ImportError at /r/56/
 Request Method: GET
 Request URL: http://mob-rb-test.sj.broadcom.com/r/56/
 Exception Type: ImportError
 Exception Value:
 Exception Location: /projects/mob_tools/xampp/1.6.4-brcm-v2/lib/
 python2.5/site-packages/ReviewBoard-1.5beta2-py2.5.egg/reviewboard/
 scmtools/git.py in __init__, line 202
 Python Executable: /opt/lampp/bin/python
 Python Version: 2.5.2
 Python Path: ['/opt/lampp/htdocs/reviewboard/conf', '/opt/lampp/lib/
 python2.5/site-packages/django', '/opt/lampp/lib/python2.5/site-
 packages/reviewboard', '/opt/lampp/lib/python2.5/site-packages/
 MySQL_python-1.2.2-py2.5-linux-i686.egg', '/opt/lampp/lib/python2.5/
 site-packages/setuptools-0.6c9-py2.5.egg', '/opt/lampp/lib/python2.5/
 site-packages/pytz-2009l-py2.5.egg', '/opt/lampp/lib/python2.5/site-
 packages/flup-1.0.3.dev_20090716-py2.5.egg', '/opt/lampp/lib/python2.5/
 site-packages/Pygments-1.1.1-py2.5.egg', '/opt/lampp/lib/python2.5/
 site-packages/django_evolution-0.0.0-py2.5.egg', '/opt/lampp/lib/
 python2.5/site-packages/PIL-1.1.6-py2.5-linux-i686.egg', '/opt/lampp/
 lib/python2.5/site-packages/simplejson-2.0.9-py2.5-linux-i686.egg', '/
 opt/lampp/lib/python2.5/site-packages/python_ldap-2.3.9-py2.5-linux-
 i686.egg', '/opt/lampp/lib/python2.5/site-packages/Django-1.1.1-
 py2.5.egg', '/opt/lampp/lib/python2.5/site-packages/JCC-2.5-py2.5-
 linux-i686.egg', '/opt/lampp/lib/python2.5/site-packages/lucene-3.0.0-
 py2.5-linux-i686.egg', '/opt/lampp/lib/python2.5/site-packages/
 RBTools-0.2-py2.5.egg', '/opt/lampp/lib/python2.5/site-packages/
 ReviewBoard-1.5beta2-py2.5.egg', '/opt/lampp/lib/python2.5/site-
 packages/recaptcha_client-1.0.5-py2.5.egg', '/opt/lampp/lib/python2.5/
 site-packages/paramiko-1.7.6-py2.5.egg', '/opt/lampp/lib/python2.5/
 site-packages/pycrypto-2.0.1-py2.5-linux-i686.egg', '/opt/lampp/lib/
 python2.5/site-packages/Djblets-0.6.2-py2.5.egg', '/opt/lampp/lib/
 python25.zip', '/opt/lampp/lib/python2.5', '/opt/lampp/lib/python2.5/
 plat-linux2', '/opt/lampp/lib/python2.5/lib-tk', '/opt/lampp/lib/
 python2.5/lib-dynload', '/opt/lampp/lib/python2.5/site-packages']
 Server time: Tue, 15 Jun 2010 14:04:07 -0700

 --
 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.comreviewboard%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

Re: ReviewBoard with GIT

2010-06-15 Thread Christian Hammond
Git only works with local file paths on the Review Board server, unless you
have the Raw URL field set. For that, you need a web-based Git viewer such
as cgit or gitweb. See the Raw File URL Mask section at
http://www.reviewboard.org/docs/manual/dev/admin/management/repositories/

Basically, Git has no way to fetch a remote file with a given revision from
a server, requiring instead local clones. However, our Raw File URL mask, if
set correctly, allows us to fetch the file from a web service that is on the
central Git server that has access to all files and revisions.

Christian

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


On Tue, Jun 15, 2010 at 3:30 PM, Kunjal kunjal.par...@gmail.com wrote:

 Thanks !

 My path variable is set to git://
 mobcom-git.sj.broadcom.com/git_repos/repo_tools/scripts.git

 My Mirror Path variable is empty. What should be the value in mirror
 path?

 On Jun 15, 2:09 pm, Christian Hammond chip...@chipx86.com wrote:
  Hi,
 
  First thing to check is that git is in your path, accessible by your web
  server.
 
  What are you using for your Path and Mirror Path?
 
  Christian
 
  --
  Christian Hammond - chip...@chipx86.com
  Review Board -http://www.reviewboard.org
  VMware, Inc. -http://www.vmware.com
 
 
 
  On Tue, Jun 15, 2010 at 2:05 PM, Kunjal kunjal.par...@gmail.com wrote:
   Hi chris,
 
   I have reviewborad 1.5Beta 2 configured for GIT.
   When I create review request, I get below error. any pointers?
 
   ImportError at /r/56/
   Request Method: GET
   Request URL:http://mob-rb-test.sj.broadcom.com/r/56/
   Exception Type: ImportError
   Exception Value:
   Exception Location: /projects/mob_tools/xampp/1.6.4-brcm-v2/lib/
   python2.5/site-packages/ReviewBoard-1.5beta2-py2.5.egg/reviewboard/
   scmtools/git.py in __init__, line 202
   Python Executable: /opt/lampp/bin/python
   Python Version: 2.5.2
   Python Path: ['/opt/lampp/htdocs/reviewboard/conf', '/opt/lampp/lib/
   python2.5/site-packages/django', '/opt/lampp/lib/python2.5/site-
   packages/reviewboard', '/opt/lampp/lib/python2.5/site-packages/
   MySQL_python-1.2.2-py2.5-linux-i686.egg', '/opt/lampp/lib/python2.5/
   site-packages/setuptools-0.6c9-py2.5.egg', '/opt/lampp/lib/python2.5/
   site-packages/pytz-2009l-py2.5.egg', '/opt/lampp/lib/python2.5/site-
   packages/flup-1.0.3.dev_20090716-py2.5.egg', '/opt/lampp/lib/python2.5/
   site-packages/Pygments-1.1.1-py2.5.egg', '/opt/lampp/lib/python2.5/
   site-packages/django_evolution-0.0.0-py2.5.egg', '/opt/lampp/lib/
   python2.5/site-packages/PIL-1.1.6-py2.5-linux-i686.egg', '/opt/lampp/
   lib/python2.5/site-packages/simplejson-2.0.9-py2.5-linux-i686.egg', '/
   opt/lampp/lib/python2.5/site-packages/python_ldap-2.3.9-py2.5-linux-
   i686.egg', '/opt/lampp/lib/python2.5/site-packages/Django-1.1.1-
   py2.5.egg', '/opt/lampp/lib/python2.5/site-packages/JCC-2.5-py2.5-
   linux-i686.egg', '/opt/lampp/lib/python2.5/site-packages/lucene-3.0.0-
   py2.5-linux-i686.egg', '/opt/lampp/lib/python2.5/site-packages/
   RBTools-0.2-py2.5.egg', '/opt/lampp/lib/python2.5/site-packages/
   ReviewBoard-1.5beta2-py2.5.egg', '/opt/lampp/lib/python2.5/site-
   packages/recaptcha_client-1.0.5-py2.5.egg', '/opt/lampp/lib/python2.5/
   site-packages/paramiko-1.7.6-py2.5.egg', '/opt/lampp/lib/python2.5/
   site-packages/pycrypto-2.0.1-py2.5-linux-i686.egg', '/opt/lampp/lib/
   python2.5/site-packages/Djblets-0.6.2-py2.5.egg', '/opt/lampp/lib/
   python25.zip', '/opt/lampp/lib/python2.5', '/opt/lampp/lib/python2.5/
   plat-linux2', '/opt/lampp/lib/python2.5/lib-tk', '/opt/lampp/lib/
   python2.5/lib-dynload', '/opt/lampp/lib/python2.5/site-packages']
   Server time: Tue, 15 Jun 2010 14:04:07 -0700
 
   --
   Want to help the Review Board project? Donate today at
  http://www.reviewboard.org/donate/
   Happy user? Let us know athttp://www.reviewboard.org/users/
   -~--~~~~--~~--~--~---
   To unsubscribe from this group, send email to
   reviewboard+unsubscr...@googlegroups.comreviewboard%2bunsubscr...@googlegroups.com
 reviewboard%2bunsubscr...@googlegr­oups.com
   For more options, visit this group at
  http://groups.google.com/group/reviewboard?hl=en- Hide quoted text -
 
  - Show quoted text -

 --
 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.comreviewboard%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 

Re: ReviewBoard with GIT

2010-06-15 Thread Jan Koprowski
This looks like Christain said - git can't find repo. Read this:
http://www.reviewboard.org/docs/manual/dev/admin/management/repositories/#git
to know how get patches for Git to enter.

On Jun 16, 1:06 am, Christian Hammond chip...@chipx86.com wrote:
 Git only works with local file paths on the Review Board server, unless you
 have the Raw URL field set. For that, you need a web-based Git viewer such
 as cgit or gitweb. See the Raw File URL Mask section 
 athttp://www.reviewboard.org/docs/manual/dev/admin/management/repositor...

 Basically, Git has no way to fetch a remote file with a given revision from
 a server, requiring instead local clones. However, our Raw File URL mask, if
 set correctly, allows us to fetch the file from a web service that is on the
 central Git server that has access to all files and revisions.

 Christian

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



 On Tue, Jun 15, 2010 at 3:30 PM, Kunjal kunjal.par...@gmail.com wrote:
  Thanks !

  My path variable is set to git://
  mobcom-git.sj.broadcom.com/git_repos/repo_tools/scripts.git

  My Mirror Path variable is empty. What should be the value in mirror
  path?

  On Jun 15, 2:09 pm, Christian Hammond chip...@chipx86.com wrote:
   Hi,

   First thing to check is that git is in your path, accessible by your web
   server.

   What are you using for your Path and Mirror Path?

   Christian

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

   On Tue, Jun 15, 2010 at 2:05 PM, Kunjal kunjal.par...@gmail.com wrote:
Hi chris,

I have reviewborad 1.5Beta 2 configured for GIT.
When I create review request, I get below error. any pointers?

ImportError at /r/56/
Request Method: GET
Request URL:http://mob-rb-test.sj.broadcom.com/r/56/
Exception Type: ImportError
Exception Value:
Exception Location: /projects/mob_tools/xampp/1.6.4-brcm-v2/lib/
python2.5/site-packages/ReviewBoard-1.5beta2-py2.5.egg/reviewboard/
scmtools/git.py in __init__, line 202
Python Executable: /opt/lampp/bin/python
Python Version: 2.5.2
Python Path: ['/opt/lampp/htdocs/reviewboard/conf', '/opt/lampp/lib/
python2.5/site-packages/django', '/opt/lampp/lib/python2.5/site-
packages/reviewboard', '/opt/lampp/lib/python2.5/site-packages/
MySQL_python-1.2.2-py2.5-linux-i686.egg', '/opt/lampp/lib/python2.5/
site-packages/setuptools-0.6c9-py2.5.egg', '/opt/lampp/lib/python2.5/
site-packages/pytz-2009l-py2.5.egg', '/opt/lampp/lib/python2.5/site-
packages/flup-1.0.3.dev_20090716-py2.5.egg', '/opt/lampp/lib/python2.5/
site-packages/Pygments-1.1.1-py2.5.egg', '/opt/lampp/lib/python2.5/
site-packages/django_evolution-0.0.0-py2.5.egg', '/opt/lampp/lib/
python2.5/site-packages/PIL-1.1.6-py2.5-linux-i686.egg', '/opt/lampp/
lib/python2.5/site-packages/simplejson-2.0.9-py2.5-linux-i686.egg', '/
opt/lampp/lib/python2.5/site-packages/python_ldap-2.3.9-py2.5-linux-
i686.egg', '/opt/lampp/lib/python2.5/site-packages/Django-1.1.1-
py2.5.egg', '/opt/lampp/lib/python2.5/site-packages/JCC-2.5-py2.5-
linux-i686.egg', '/opt/lampp/lib/python2.5/site-packages/lucene-3.0.0-
py2.5-linux-i686.egg', '/opt/lampp/lib/python2.5/site-packages/
RBTools-0.2-py2.5.egg', '/opt/lampp/lib/python2.5/site-packages/
ReviewBoard-1.5beta2-py2.5.egg', '/opt/lampp/lib/python2.5/site-
packages/recaptcha_client-1.0.5-py2.5.egg', '/opt/lampp/lib/python2.5/
site-packages/paramiko-1.7.6-py2.5.egg', '/opt/lampp/lib/python2.5/
site-packages/pycrypto-2.0.1-py2.5-linux-i686.egg', '/opt/lampp/lib/
python2.5/site-packages/Djblets-0.6.2-py2.5.egg', '/opt/lampp/lib/
python25.zip', '/opt/lampp/lib/python2.5', '/opt/lampp/lib/python2.5/
plat-linux2', '/opt/lampp/lib/python2.5/lib-tk', '/opt/lampp/lib/
python2.5/lib-dynload', '/opt/lampp/lib/python2.5/site-packages']
Server time: Tue, 15 Jun 2010 14:04:07 -0700

--
Want to help the Review Board project? Donate today at
   http://www.reviewboard.org/donate/
Happy user? Let us know athttp://www.reviewboard.org/users/
-~--~~~~--~~--~--~---
To unsubscribe from this group, send email to
reviewboard+unsubscr...@googlegroups.comreviewboard%2bunsubscr...@googlegr
 oups.com
  reviewboard%2bunsubscr...@googlegr­oups.com
For more options, visit this group at
   http://groups.google.com/group/reviewboard?hl=en-Hide quoted text -

   - Show quoted text -

  --
  Want to help the Review Board project? Donate today at
 http://www.reviewboard.org/donate/
  Happy user? Let us know athttp://www.reviewboard.org/users/
  -~--~~~~--~~--~--~---
  To unsubscribe from this group, send email to
  reviewboard+unsubscr...@googlegroups.comreviewboard%2bunsubscr...@googlegr 
  oups.com
  For more options, 

Error when creating New Review Request when using ReviewBoard with Git

2010-02-04 Thread shahbhat
I am trying to install ReviewBoard with a git repository, which is
setup remotely. I used 
http://www.reviewboard.org/docs/manual/dev/admin/installation/linux/
for installing on Ubuntu 9. I then upgraded to ReviewBoard 1.1 using
$ easy_install -f http://downloads.reviewboard.org/releases/ReviewBoard/1.1/
-U ReviewBoard

I then created a local directory git and initialized with git as
  git --bare init
I then created link to git repository
  git remote add origin ssh://sbha...@gitserver/git/myproject.git

I was able to then add above repository using admin tool. For testing
purpose, I checked out the same project into a different directory and
modified README file. I then created diff file using git diff.

Next, I tried to create a new review request, but when I submitted ,
it can't find README file. Kindly show me where I misconfigured the
settings.

Here is full message.


FileNotFoundError at /r/new/

The file 'README' (r135b4d8) could not be found in the repository

Request Method: POST
Request URL:http://codereview.weseed.com/r/new/
Exception Type: FileNotFoundError
Exception Value:

The file 'README' (r135b4d8) could not be found in the repository

Exception Location: /usr/local/lib/python2.6/dist-packages/
ReviewBoard-1.1alpha3-py2.6.egg/reviewboard/diffviewer/forms.py in
_process_files, line 143
Python Executable:  /usr/bin/python
Python Version: 2.6.4
Python Path:['/var/www/weseed/conf', '/var/www/weseed/htdocs', '/usr/
local/lib/python2.6/dist-packages/cmemcache-0.95-py2.6-linux-
i686.egg', '/usr/local/lib/python2.6/dist-packages/pytz-2010b-
py2.6.egg', '/usr/local/lib/python2.6/dist-packages/
flup-1.0.3.dev_20091027-py2.6.egg', '/usr/local/lib/python2.6/dist-
packages/Pygments-1.2.2-py2.6.egg', '/usr/local/lib/python2.6/dist-
packages/django_evolution-0.0.0-py2.6.egg', '/usr/local/lib/python2.6/
dist-packages/Django-1.1.1-py2.6.egg', '/usr/local/lib/python2.6/dist-
packages/MySQL_python-1.2.3c1-py2.6-linux-i686.egg', '/usr/local/lib/
python2.6/dist-packages/recaptcha_client-1.0.5-py2.6.egg', '/usr/local/
lib/python2.6/dist-packages/paramiko-1.7.6-py2.6.egg', '/usr/local/lib/
python2.6/dist-packages/PIL-1.1.6-py2.6-linux-i686.egg', '/usr/local/
lib/python2.6/dist-packages/Djblets-0.5.7alpha0-py2.6.egg', '/usr/
local/lib/python2.6/dist-packages/ReviewBoard-1.1alpha3-py2.6.egg', '/
usr/lib/python2.6', '/usr/lib/python2.6/plat-linux2', '/usr/lib/
python2.6/lib-tk', '/usr/lib/python2.6/lib-old', '/usr/lib/python2.6/
lib-dynload', '/usr/lib/python2.6/dist-packages', '/usr/lib/python2.6/
dist-packages/PIL', '/usr/lib/python2.6/dist-packages/gst-0.10', '/usr/
lib/pymodules/python2.6', '/usr/lib/python2.6/dist-packages/gtk-2.0',
'/usr/lib/pymodules/python2.6/gtk-2.0', '/usr/local/lib/python2.6/dist-
packages']
Server time:Thu, 4 Feb 2010 14:09:14 -0800
Traceback Switch to copy-and-paste view

* /usr/local/lib/python2.6/dist-packages/Django-1.1.1-py2.6.egg/
django/core/handlers/base.py in get_response
85. # Apply view middleware
86. for middleware_method in self._view_middleware:
87. response = middleware_method(request, callback,
callback_args, callback_kwargs)
88. if response:
89. return response
90.
91. try:
92. response = callback(request, *callback_args,
**callback_kwargs) ...
93. except Exception, e:
94. # If the view raised an exception, run it through
exception
95. # middleware, and if the exception middleware returns a
96. # response, use that. Otherwise, reraise the exception.
97. for middleware_method in self._exception_middleware:
98. response = middleware_method(request, e)
  ‚ñ∂ Local vars
  Variable  Value
  callback
  function new_review_request at 0x9fa79cc
  callback_args
  ()
  callback_kwargs
  {}
  e
  FileNotFoundError(The file 'README' (r135b4d8) could not be
found in the repository,)
  exc_info
  (class 'reviewboard.scmtools.errors.FileNotFoundError',
FileNotFoundError(The file 'README' (r135b4d8) could not be found in
the repository,), traceback object at 0xa34a4dc)
  exceptions
  module 'django.core.exceptions' from '/usr/local/lib/python2.6/
dist-packages/Django-1.1.1-py2.6.egg/django/core/exceptions.pyc'
  middleware_method
  bound method LoggingMiddleware.process_exception of
djblets.log.middleware.LoggingMiddleware object at 0x9aba4ac
  receivers
  [(function _rollback_on_exception at 0x99848ec, None)]
  request
  WSGIRequest GET:QueryDict: {}, POST:QueryDict:
{u'changenum': [u''], u'basedir': [u''], u'repository': [u'1']},
COOKIES:{'rbsessionid': 'f47f1327124e6ae257aa7be39b649cd3'}, META:
{'CONTENT_LENGTH': '966', 'CONTENT_TYPE': 'multipart/form-data;
boundary=---26011589619811243601686650177',
'DOCUMENT_ROOT': '/var/www/weseed/htdocs', 'GATEWAY_INTERFACE': 'CGI/
1.1', 'HTTP_ACCEPT': 'text/html,application/xhtml+xml