Re: Review Board 2.0.16 is released, with security fixes and more

2015-06-16 Thread Stephen Gallagher
Thanks for the

On Tue, Jun 16, 2015 at 7:03 AM Paul Fee paul.f@gmail.com wrote:

 Hi Stephen,

 EPEL7 upgrade didn't work for me.  Negative karma and details of haystack
 dependency issue here.

 https://admin.fedoraproject.org/updates/FEDORA-EPEL-2015-6685/ReviewBoard-2.0.16-1.el7,python-djblets-0.8.19-1.el7,python-django-evolution-0.7.5-1.el7

 Forgive the formatting, I see bodhi doesn't preserve line endings within
 the comment box.


Thanks for the report; I missed the jump to Haystack 2.3.1. I'll get that
build, update Review Board to require it and get an update out today with
2.0.17 as well.

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


Issue 3887 in reviewboard: Perforce diff submission is broken in 2.0.17

2015-06-16 Thread reviewboard

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

New issue 3887 by erik.l...@gmail.com: Perforce diff submission is broken  
in 2.0.17

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

What version are you running?
ReviewBoard 2.0.17 and RBTools 0.7.2 and 0.7.4

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


What steps will reproduce the problem?
1. Attempt to post any change via perforce


What is the expected output? What do you see instead?
I expect the post to succeed. Instead, I get a 500 error in the client:

Generating diff for pending changeset 179888
ERROR: Error validating diff

HTTP 500


And the error in the server is:
[...]
  
File /usr/local/lib/python2.7/dist-packages/ReviewBoard-2.0.17-py2.7.egg/reviewboard/webapi/resources/validate_diff.py,  
line 144, in create

   save=False)

  
File /usr/local/lib/python2.7/dist-packages/ReviewBoard-2.0.17-py2.7.egg/reviewboard/diffviewer/managers.py,  
line 156, in create_from_upload

   save=save)

  
File /usr/local/lib/python2.7/dist-packages/ReviewBoard-2.0.17-py2.7.egg/reviewboard/diffviewer/managers.py,  
line 180, in create_from_data

   check_existence=(not parent_diff_file_contents)))

  
File /usr/local/lib/python2.7/dist-packages/ReviewBoard-2.0.17-py2.7.egg/reviewboard/diffviewer/managers.py,  
line 287, in _process_files

   copied=f.copied)

  
File /usr/local/lib/python2.7/dist-packages/ReviewBoard-2.0.17-py2.7.egg/reviewboard/scmtools/perforce.py,  
line 332, in parse_diff_revision

   filename, revision = revision_str.rsplit('#', 1)

ValueError: need more than 1 value to unpack


What operating system are you using? What browser?
N/A

Please provide any additional information below.

Looking at revision_str, it doesn't contain any '#' character and is  
actually of the form: 2015-06-16 11:39:35 when called with f.newFile and  
f.newInfo. When called with f.origFile and f.origInfo,  
parse_diff_revision() would work as expected but we bomb out before getting  
there.


--
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: ReviewBot: Problem moving from 0.1 to 0.2

2015-06-16 Thread Steve
Some other information I should have provided:

- I refreshed the installed tools on the extension page
- Here's what's in my .pth files:

root@dory site-packages]# grep -i bot *.pth
easy-install.pth:./Review_Bot_Extension-0.2alpha0-py2.6.egg
easy-install.pth:./ReviewBot-0.3alpha0-py2.6.egg
easy-install.pth:./buildbot-0.8.12-py2.6.egg
[root@dory site-packages]# ls -ld *[Bb]ot*
drwxr-xr-x 4 root root 4096 Jun 15 14:24 buildbot-0.8.12-py2.6.egg
drwxr-xr-x 4 root root 4096 Jan 31  2014 ReviewBot-0.1-py2.6.egg
drwxr-xr-x 4 root root 4096 Jun 15 14:24 ReviewBot-0.3alpha0-py2.6.egg
drwxr-xr-x 4 root root 4096 Sep 25  2013 Review_Bot_Extension-0.1-py2.6.egg
drwxr-xr-x 4 root root 4096 Jun 15 14:23 
Review_Bot_Extension-0.2alpha0-py2.6.egg

On Tuesday, June 16, 2015 at 10:12:58 AM UTC-7, Steve wrote:

 Hi all,

 I'm upgraded a test server from ReviewBoard 1.7.28 to 2.0.15.  That seems 
 to be working fine.  Now I'm trying to upgrade the ReviewBot extension from 
 0.1 to 0.2.  I followed the instructions here:

 https://github.com/reviewboard/ReviewBot

 and everything completed without error.  I then

 - Stopped: reviewbot service, apache and memcached
 - Started: memached, apache, reviewbot


 Then I post a review with a python file that  I know has PEP 8 violations. 
  ReviewBot fails to post a review and the log file complains that

 [2015-06-16 10:08:27,091: DEBUG/MainProcess] Cached response for HTTP GET 
 https://dory/api/extensions/reviewbotext.extension.ReviewBotExtension/ 
 expired and was modified

 Here's the whole output:

 [2015-06-16 09:57:02,650: DEBUG/MainProcess] Making HTTP GET request to 
 https://dory/api/review-requests/75029/diffs/9/files/1698454/patched-file/
 [2015-06-16 09:57:02,819: DEBUG/MainProcess] Added cache entry for HTTP 
 GET request to 
 https://dory/api/review-requests/75029/diffs/9/files/1698454/patched-file/
 [2015-06-16 09:57:02,820: DEBUG/MainProcess] pep8 -r --max-line-length=79 
 --ignore= /tmp/tmpJQJPMa.py
 [2015-06-16 09:57:02,950: INFO/MainProcess] 
 reviewbot.tasks.ProcessReviewRequest[faaa0d58-1dd5-426d-bab7-6b0b5f295be7]: 
 Tool execution completed successfully
 [2015-06-16 09:57:02,951: INFO/MainProcess] 
 reviewbot.tasks.ProcessReviewRequest[faaa0d58-1dd5-426d-bab7-6b0b5f295be7]: 
 Updating tool execution with completed review
 [2015-06-16 09:57:02,951: DEBUG/MainProcess] Making HTTP GET request to 
 https://dory/api/extensions/reviewbotext.extension.ReviewBotExtension/
 [2015-06-16 09:57:03,090: DEBUG/MainProcess] Cached response for HTTP GET 
 https://dory/api/extensions/reviewbotext.extension.ReviewBotExtension/ 
 expired and was modified
 [2015-06-16 09:57:03,093: ERROR/MainProcess] 
 reviewbot.tasks.ProcessReviewRequest[faaa0d58-1dd5-426d-bab7-6b0b5f295be7]: 
 Error updating tool execution:
 [2015-06-16 09:57:03,093: INFO/MainProcess] Task 
 reviewbot.tasks.ProcessReviewRequest[faaa0d58-1dd5-426d-bab7-6b0b5f295be7] 
 succeeded in 1.39496517181s: False

 Is there some other step I need to take to upgrade the ReviewBot extension?

 --Steve



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


ReviewBot: Problem moving from 0.1 to 0.2

2015-06-16 Thread Steve
Hi all,

I'm upgraded a test server from ReviewBoard 1.7.28 to 2.0.15.  That seems 
to be working fine.  Now I'm trying to upgrade the ReviewBot extension from 
0.1 to 0.2.  I followed the instructions here:

https://github.com/reviewboard/ReviewBot

and everything completed without error.  I then

- Stopped: reviewbot service, apache and memcached
- Started: memached, apache, reviewbot


Then I post a review with a python file that  I know has PEP 8 violations. 
 ReviewBot fails to post a review and the log file complains that

[2015-06-16 10:08:27,091: DEBUG/MainProcess] Cached response for HTTP GET 
https://dory/api/extensions/reviewbotext.extension.ReviewBotExtension/ 
expired and was modified

Here's the whole output:

[2015-06-16 09:57:02,650: DEBUG/MainProcess] Making HTTP GET request to 
https://dory/api/review-requests/75029/diffs/9/files/1698454/patched-file/
[2015-06-16 09:57:02,819: DEBUG/MainProcess] Added cache entry for HTTP GET 
request to 
https://dory/api/review-requests/75029/diffs/9/files/1698454/patched-file/
[2015-06-16 09:57:02,820: DEBUG/MainProcess] pep8 -r --max-line-length=79 
--ignore= /tmp/tmpJQJPMa.py
[2015-06-16 09:57:02,950: INFO/MainProcess] 
reviewbot.tasks.ProcessReviewRequest[faaa0d58-1dd5-426d-bab7-6b0b5f295be7]: 
Tool execution completed successfully
[2015-06-16 09:57:02,951: INFO/MainProcess] 
reviewbot.tasks.ProcessReviewRequest[faaa0d58-1dd5-426d-bab7-6b0b5f295be7]: 
Updating tool execution with completed review
[2015-06-16 09:57:02,951: DEBUG/MainProcess] Making HTTP GET request to 
https://dory/api/extensions/reviewbotext.extension.ReviewBotExtension/
[2015-06-16 09:57:03,090: DEBUG/MainProcess] Cached response for HTTP GET 
https://dory/api/extensions/reviewbotext.extension.ReviewBotExtension/ 
expired and was modified
[2015-06-16 09:57:03,093: ERROR/MainProcess] 
reviewbot.tasks.ProcessReviewRequest[faaa0d58-1dd5-426d-bab7-6b0b5f295be7]: 
Error updating tool execution:
[2015-06-16 09:57:03,093: INFO/MainProcess] Task 
reviewbot.tasks.ProcessReviewRequest[faaa0d58-1dd5-426d-bab7-6b0b5f295be7] 
succeeded in 1.39496517181s: False

Is there some other step I need to take to upgrade the ReviewBot extension?

--Steve

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


Re: RbTools 0.7.4 - ERROR: The current directory does not contain a checkout from a supported source code repository.

2015-06-16 Thread David Trowbridge
On Tue, Jun 16, 2015 at 3:33 AM, diwan.b...@gmail.com wrote:

  Running: git version
 ERROR: Could not load SCM Client git: [Error 2] The system cannot find
 the fil
 e specified


This looks like it's unable to find the command-line git in your path.
This is odd, because it should be detecting that you're on windows and
therefore trying git.cmd

I've never seen this error case before, and I don't see anything
immediately wrong with the code, so I can't give you an easy answer.
Perhaps you could help us with some debugging?

-David

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


Re: Issue 3887 in reviewboard: Perforce diff submission is broken in 2.0.17

2015-06-16 Thread reviewboard


Comment #2 on issue 3887 by trowb...@gmail.com: Perforce diff submission is  
broken in 2.0.17

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

Also, what version of perforce on the client and the server?

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


customizing reviewboard emails to include repository name.

2015-06-16 Thread Damien NOZAY
Hi,

I would like to see if it is possible to add the repository name to the 
email subject (see mail_review_request); or allow admins to configure the 
mail subject.

subject = Review Request %d: %s % (review_request.display_id, summary)

subject = [%s] Review Request %d: %s % (review_request.repository.name, 
review_request.display_id, summary)

The rationale is that there are a lot of repositories that developers are 
subscribed to, and being able to skim through review email headers would 
make them more productive.

Thanks,
Damien

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


Re: Issue 3887 in reviewboard: Perforce diff submission is broken in 2.0.17

2015-06-16 Thread reviewboard

Updates:
Status: NeedInfo

Comment #1 on issue 3887 by trowb...@gmail.com: Perforce diff submission is  
broken in 2.0.17

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

How are you creating your diff? Having a diff that contains a timestamp  
rather than a file revision isn't going to work, and isn't a regression.


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


assert index_name in indexes during `rb-site upgrade`

2015-06-16 Thread John Schmitt
http://ur1.ca/muhac


(reviewboard-venv)[root@dyn40-25-179 tintri]# rb-site upgrade 
/var/www/reviewboard
Rebuilding directory structure
Updating database. This may take a while.

The log output below, including warnings and errors,
can be ignored unless upgrade fails.

-- begin log output --
Creating tables ...
There are unapplied evolutions for auth.
There are unapplied evolutions for contenttypes.
There are unapplied evolutions for accounts.
There are unapplied evolutions for changedescs.
There are unapplied evolutions for diffviewer.
There are unapplied evolutions for reviews.
There are unapplied evolutions for scmtools.
Project signature has changed - an evolution is required
Installing custom SQL ...
Installing indexes ...
Installed 0 object(s) from 0 fixture(s)
CRITICAL:root:Error running database evolver function 
change_column_attr_unique: 
Traceback (most recent call last):
  File 
/tintri/reviewboard-venv/lib/python2.7/site-packages/django_evolution-0.7.5-py2.7.egg/django_evolution/db/common.py,
 
line 495, in change_column_attrs
attr_info['new_value'])
  File 
/tintri/reviewboard-venv/lib/python2.7/site-packages/django_evolution-0.7.5-py2.7.egg/django_evolution/db/common.py,
 
line 607, in change_column_attr_unique
self.remove_recorded_index(model, constraint_name, unique=True)
  File 
/tintri/reviewboard-venv/lib/python2.7/site-packages/django_evolution-0.7.5-py2.7.egg/django_evolution/db/common.py,
 
line 822, in remove_recorded_index
index_name, unique=unique)
  File 
/tintri/reviewboard-venv/lib/python2.7/site-packages/django_evolution-0.7.5-py2.7.egg/django_evolution/signature.py,
 
line 186, in remove_index_from_database_sig
assert index_name in indexes
AssertionError
ERROR:root:Unexpected error: 
Traceback (most recent call last):
  File 
/tintri/reviewboard-venv/lib/python2.7/site-packages/django_evolution-0.7.5-py2.7.egg/django_evolution/management/commands/evolve.py,
 
line 65, in handle
self.evolve(*app_labels, **options)
  File 
/tintri/reviewboard-venv/lib/python2.7/site-packages/django_evolution-0.7.5-py2.7.egg/django_evolution/management/commands/evolve.py,
 
line 125, in evolve
sql.extend(self.evolve_app(app))
  File 
/tintri/reviewboard-venv/lib/python2.7/site-packages/django_evolution-0.7.5-py2.7.egg/django_evolution/management/commands/evolve.py,
 
line 165, in evolve_app
app_mutator_sql = app_mutator.to_sql()
  File 
/tintri/reviewboard-venv/lib/python2.7/site-packages/django_evolution-0.7.5-py2.7.egg/django_evolution/mutators.py,
 
line 304, in to_sql
sql.extend(mutator.to_sql())
  File 
/tintri/reviewboard-venv/lib/python2.7/site-packages/django_evolution-0.7.5-py2.7.egg/django_evolution/mutators.py,
 
line 187, in to_sql
return self.evolver.generate_table_ops_sql(self, self._ops)
  File 
/tintri/reviewboard-venv/lib/python2.7/site-packages/django_evolution-0.7.5-py2.7.egg/django_evolution/db/common.py,
 
line 42, in generate_table_ops_sql
prev_sql_result, prev_op)
  File 
/tintri/reviewboard-venv/lib/python2.7/site-packages/django_evolution-0.7.5-py2.7.egg/django_evolution/db/common.py,
 
line 84, in generate_table_op_sql
op['new_attrs']))
  File 
/tintri/reviewboard-venv/lib/python2.7/site-packages/django_evolution-0.7.5-py2.7.egg/django_evolution/db/common.py,
 
line 495, in change_column_attrs
attr_info['new_value'])
  File 
/tintri/reviewboard-venv/lib/python2.7/site-packages/django_evolution-0.7.5-py2.7.egg/django_evolution/db/common.py,
 
line 607, in change_column_attr_unique
self.remove_recorded_index(model, constraint_name, unique=True)
  File 
/tintri/reviewboard-venv/lib/python2.7/site-packages/django_evolution-0.7.5-py2.7.egg/django_evolution/db/common.py,
 
line 822, in remove_recorded_index
index_name, unique=unique)
  File 
/tintri/reviewboard-venv/lib/python2.7/site-packages/django_evolution-0.7.5-py2.7.egg/django_evolution/signature.py,
 
line 186, in remove_index_from_database_sig
assert index_name in indexes
AssertionError
Traceback (most recent call last):
  File /tintri/reviewboard-venv/bin/rb-site, line 9, in module
load_entry_point('ReviewBoard==2.0.17', 'console_scripts', 'rb-site')()
  File 
/tintri/reviewboard-venv/lib/python2.7/site-packages/ReviewBoard-2.0.17-py2.7.egg/reviewboard/cmdline/rbsite.py,
 
line 1885, in main
command.run()
  File 
/tintri/reviewboard-venv/lib/python2.7/site-packages/ReviewBoard-2.0.17-py2.7.egg/reviewboard/cmdline/rbsite.py,
 
line 1694, in run
site.migrate_database()
  File 
/tintri/reviewboard-venv/lib/python2.7/site-packages/ReviewBoard-2.0.17-py2.7.egg/reviewboard/cmdline/rbsite.py,
 
line 438, in migrate_database
self.run_manage_command(evolve, [--noinput, --execute])
  File 
/tintri/reviewboard-venv/lib/python2.7/site-packages/ReviewBoard-2.0.17-py2.7.egg/reviewboard/cmdline/rbsite.py,
 
line 662, in run_manage_command
execute_from_command_line([__file__, cmd] + params)
  File 

Re: RbTools 0.7.4 - ERROR: The current directory does not contain a checkout from a supported source code repository.

2015-06-16 Thread Griffin Myers
I think that error message is a red herring and should already be fixed in 
4678679 
(https://github.com/reviewboard/rbtools/commit/4f344a9efb2c3534e46b117e6bad15856a3a7be9).

On Tuesday, June 16, 2015 at 3:34:59 PM UTC-4, David Trowbridge wrote:


 On Tue, Jun 16, 2015 at 3:33 AM, diwan...@gmail.com javascript: wrote:

  Running: git version
 ERROR: Could not load SCM Client git: [Error 2] The system cannot find 
 the fil
 e specified


 This looks like it's unable to find the command-line git in your path. 
 This is odd, because it should be detecting that you're on windows and 
 therefore trying git.cmd

 I've never seen this error case before, and I don't see anything 
 immediately wrong with the code, so I can't give you an easy answer. 
 Perhaps you could help us with some debugging?

 -David


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


Re: Review Board 2.0.16 is released, with security fixes and more

2015-06-16 Thread Paul Fee
Hi Stephen,

EPEL7 upgrade didn't work for me.  Negative karma and details of haystack 
dependency issue here.
https://admin.fedoraproject.org/updates/FEDORA-EPEL-2015-6685/ReviewBoard-2.0.16-1.el7,python-djblets-0.8.19-1.el7,python-django-evolution-0.7.5-1.el7

Forgive the formatting, I see bodhi doesn't preserve line endings within 
the comment box.

HTH,
Paul


On Friday, June 12, 2015 at 5:55:48 PM UTC+1, Stephen Gallagher wrote:



 On Fri, Jun 12, 2015 at 12:47 AM Christian Hammond chri...@beanbaginc.com 
 javascript: wrote:

 Hey everyone,

 We just put out a release of Review Board 2.0.16. This release contains 
 security fixes, so we recommend that you update!

 There are also some significant performance improvements across the 
 entire site, especially for diff loading, along with a great many bug fixes.

 This is all detailed in the news post: 
 https://www.reviewboard.org/news/2015/06/11/review-board-2-0-16-out-safer-faster-and-more-stable/


 Packages for Fedora 21 and RHEL/CentOS 7 are now available at 
 https://admin.fedoraproject.org/updates/search/ReviewBoard

 Packages for Fedora 22 and 23 (Rawhide) are available in my COPR 
 repository:  
 https://copr.fedoraproject.org/coprs/sgallagh/f22-reviewboard/


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


Re: RbTools 0.7.4 - ERROR: The current directory does not contain a checkout from a supported source code repository.

2015-06-16 Thread diwan . bose
Thanks David. Its the same with 0.7.4 as well.

 RBTools 0.7.4
 Python 2.7.6 (default, Nov 10 2013, 19:24:18) [MSC v.1500 32 bit 
(Intel)]
 Running on Windows-7-6.1.7601-SP1
 Home = C:\Users\diwanc\AppData\Roaming
 Current directory = C:\Program Files (x86)\RBTools\bin
 Running: git version
ERROR: Could not load SCM Client git: [Error 2] The system cannot find 
the fil
e specified
 Checking for a Subversion repository...
 Unable to execute svn help: skipping SVN
 Checking for a Mercurial repository...
 Unable to execute hg --help: skipping Mercurial
 Checking for a CVS repository...
 Unable to execute cvs: skipping CVS
 Checking for a Perforce repository...
 Running: p4 info
 Checking for a Plastic repository...
 Unable to execute cm version: skipping Plastic
 Checking for a ClearCase repository...
 Unable to execute cleartool help: skipping ClearCase
 Checking for a Bazaar repository...
 Unable to execute bzr help: skipping Bazaar
 Checking for a TFS repository...
 Unable to execute tf help: skipping TFS
ERROR: The current directory does not contain a checkout from a supported 
source
 code repository.

On Tuesday, June 16, 2015 at 3:07:29 AM UTC+5:30, David Trowbridge wrote:

 It looks like you're not actually running 0.7.4, but rather some 
 pre-release of 0.7.1. Can you update to the latest and try again?

 -David

 On Mon, Jun 15, 2015 at 4:07 AM, diwan...@gmail.com javascript: wrote:

 I am trying to use RBTools to post review for p4 changesets. Getting 
 following error, even when I try rbt diff -d. What could be the issue? 
 p4 info  works fine.

 C:\Program Files (x86)\RBTools\binrbt diff -d
  RBTools 0.7.1 alpha 0 (dev)
  Python 2.7.6 (default, Nov 10 2013, 19:24:18) [MSC v.1500 32 bit 
 (Intel)]
  Running on Windows-7-6.1.7601-SP1
  Home = C:\Users\diwanc\AppData\Roaming
  Current directory = C:\Program Files (x86)\RBTools\bin
  Checking for a Subversion repository...
  Unable to execute svn help: skipping SVN
  Checking for a Git repository...
  Unable to execute git --help or git.cmd --help: skipping Git
  Checking for a Mercurial repository...
  Unable to execute hg --help: skipping Mercurial
  Checking for a CVS repository...
  Unable to execute cvs: skipping CVS
  Checking for a Perforce repository...
  Running: p4 info
  Checking for a Plastic repository...
  Unable to execute cm version: skipping Plastic
  Checking for a ClearCase repository...
  Unable to execute cleartool help: skipping ClearCase
  Checking for a Bazaar repository...
  Unable to execute bzr help: skipping Bazaar
  Checking for a TFS repository...
  Unable to execute tf help: skipping TFS
 ERROR: The current directory does not contain a checkout from a supported 
 source
  code repository.

 -- 
 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...@googlegroups.com javascript:.
 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.