date:20120806

Dear All,

I am trying to post the review request as below argument.

My perforce id:njaiswal

*I am using --submit-as option.*

*post-review  --bugs-closed=TEST --file-list=/tmp/tmp9Dj3Dt
--branch=DEV_BR
--server=http://rbc.nds.com:8000/rbsvr--submit-as=nileshj
--description-file=/tmp/tmp79D4Hj 1567 --p4-port=
pf-ch.nds.com:1666*

My while posting i am getting below error message.

FOR RB USE DO NOT EDIT
--
p4web link: http://pf-ch.nds.com:5500/1567?ac=10
baseline: 1564
' for review request '3'
 HTTP PUTting to
http://rbc.nds.com:8000/rbsvr/api/review-requests/3/draft/: {'description':
'RB testing for china \n\n\nFOR RB USE DO NOT
EDIT\n--\np4web link:
http://pf-ch.nds.com:5500/1567?ac=10\nbaseline: 1564\n'}
 Uploading diff, size: 159
 HTTP POSTing to
http://rbc.nds.com:8000/rbsvr/api/review-requests/3/diffs/: {}
 Got API Error 105 (HTTP code 400): One or more fields had errors
 Error data: {u'fields': {u'path': [u'[Errno 2] No such file or
directory']}, u'stat': u'fail', u'err': {u'msg': u'One or more fields had
errors', u'code': 105}}
Your review request still exists, but the diff is not attached.

*My question is, *

1. Does my RB server user id requires account in perforce?

2. What should i do make posting of review request if am having one
perforce id with multiple RB server user id.

3. Is it mandatory that perforce id to be in the user list of reviewboard
server


This is very urgent request, i would really appreciate your quick reply
with solution.

Regards,
Nilesh J

-- 
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: Not able to use post-review

Hi Christian,
Thanks for the reply . I an getting following error when i typed 
post-review in cmd

GNU diff is required for Subversion repositories. Make sure it is installed
and in the path.

On Windows, you can install this from:
http://gnuwin32.sourceforge.net/packages/diffutils.htm;

I have already installed GNU Diff and set in the Path . 

For your information,I have also installed both Tortoise SVN as well as 
collabnet subversion client .


Please suggest how to proceed further.



Regards
Vijay

On Saturday, 4 August 2012 00:29:03 UTC+5:30, Christian Hammond wrote:

 Hi,

 Can you describe to me how GNU diff isn't working? What are you seeing?

 Christian



 On Aug 3, 2012, at 2:50, Vijay Kumar Tiwary vijayktiw...@gmail.com 
 wrote:

 Hi ,
 I was trying to use post-review filename but I get following error always :
  
 GNU diff is required for Subversion repositories. Make sure it is 
 installed
 and in the path.
 On Windows, you can install this from:
 http://gnuwin32.sourceforge.net/packages/diffutils.htmd
  
 I have installed GNU Diff and even post-review installation seems to work 
 but some how GNU diff is not working.
  
  
 i debuged and type following command SVN info and it is giving me the 
 configured SVN repo browser value
  
 Please suggest how to configure .
  
  
 For your information I setup Review Board on Linux machine and configured 
 SVN which is hosted on Windows machine . I am using my Windows client 
 machine to use review board and trying to post review but apparently got 
 stuck with GNU diff ...Please help me to come out from here !!!
  
 Regards
 Vijay
  
  
  

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



-- 
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: URGENT: post-review queries

On Sun, Aug 5, 2012 at 11:21 PM, Nilesh Jaiswal nileshj...@gmail.comwrote:

 Dear All,

 I am trying to post the review request as below argument.

 My perforce id:njaiswal

 *I am using --submit-as option.*

 *post-review  --bugs-closed=TEST --file-list=/tmp/tmp9Dj3Dt
 --branch=DEV_BR --server=http://rbc.nds.com:8000/rbsvr--submit-as=nileshj 
 --description-file=/tmp/tmp79D4Hj 1567 --p4-port=
 pf-ch.nds.com:1666*

 My while posting i am getting below error message.

 FOR RB USE DO NOT EDIT


It looks like you're using a custom post-review script. We can't fully
support that, but I'll try to answer your questions.



 --
 p4web link: http://pf-ch.nds.com:5500/1567?ac=10
 baseline: 1564
 ' for review request '3'
  HTTP PUTting to
 http://rbc.nds.com:8000/rbsvr/api/review-requests/3/draft/:
 {'description': 'RB testing for china \n\n\nFOR RB USE DO NOT
 EDIT\n--\np4web link:
 http://pf-ch.nds.com:5500/1567?ac=10\nbaseline: 1564\n'}
  Uploading diff, size: 159
  HTTP POSTing to
 http://rbc.nds.com:8000/rbsvr/api/review-requests/3/diffs/: {}
  Got API Error 105 (HTTP code 400): One or more fields had errors
  Error data: {u'fields': {u'path': [u'[Errno 2] No such file or
 directory']}, u'stat': u'fail', u'err': {u'msg': u'One or more fields had
 errors', u'code': 105}}
 Your review request still exists, but the diff is not attached.

 *My question is, *

 1. Does my RB server user id requires account in perforce?


No.



 2. What should i do make posting of review request if am having one
 perforce id with multiple RB server user id.


All you need is one Perforce ID that's configured in the repository
settings. The Review Board users have nothing to do with Perforce users.



 3. Is it mandatory that perforce id to be in the user list of reviewboard
 server


No. It wouldn't matter in any way.

It sounds like there's just something wrong with your diff. Run that
command with --output-diff and inspect the output. Also, check your
reviewboard.log for errors that correspond to the time you made that
request.

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: Not able to use post-review

Can you run diff --version and show me the output?

Christian

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

On Sun, Aug 5, 2012 at 11:49 PM, Vijay Kumar Tiwary
vijayktiw...@gmail.comwrote:

Hi Christian,
Thanks for the reply . I an getting following error when i typed
post-review in cmd

GNU diff is required for Subversion repositories. Make sure it is
installed
and in the path.

On Windows, you can install this from:
http://gnuwin32.sourceforge.net/packages/diffutils.htm;

I have already installed GNU Diff and set in the Path .

For your information,I have also installed both Tortoise SVN as well as
collabnet subversion client .

Please suggest how to proceed further.

Regards
Vijay

On Saturday, 4 August 2012 00:29:03 UTC+5:30, Christian Hammond wrote:

Hi,

Can you describe to me how GNU diff isn't working? What are you seeing?

Christian

On Aug 3, 2012, at 2:50, Vijay Kumar Tiwary vijayktiw...@gmail.com
wrote:

Hi ,
I was trying to use post-review filename but I get following error always
:

GNU diff is required for Subversion repositories. Make sure it is
installed
and in the path.
On Windows, you can install this from:
http://gnuwin32.sourceforge.**net/packages/diffutils.htmdhttp://gnuwin32.sourceforge.net/packages/diffutils.htm%22d

I have installed GNU Diff and even post-review installation seems to work
but some how GNU diff is not working.

i debuged and type following command SVN info and it is giving me the
configured SVN repo browser value

Please suggest how to configure .

For your information I setup Review Board on Linux machine and configured
SVN which is hosted on Windows machine . I am using my Windows client
machine to use review board and trying to post review but apparently got
stuck with GNU diff ...Please help me to come out from here !!!

Regards
Vijay

--
Want to help the Review Board project? Donate today at
http://www.reviewboard.org/**donate/ http://www.reviewboard.org/donate/
Happy user? Let us know at
http://www.reviewboard.org/**users/http://www.reviewboard.org/users/
-~--~~~~--**~~--~--~---
To unsubscribe from this group, send email to reviewboard+unsubscribe@**
googlegroups.com reviewboard+unsubscr...@googlegroups.com
For more options, visit this group at http://groups.google.com/**
group/reviewboard?hl=enhttp://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: Post Review script fails to submit SVN deleted files

Or try to help improve post-review by submitting a patch.

Christian

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


On Sun, Aug 5, 2012 at 5:03 PM, Ivan Gavryliuk ivan.gavril...@gmail.comwrote:

 This saved my life today, thanks! I may dump post-review in favour of my
 own script.


 On Wednesday, 2 May 2012 14:17:13 UTC+1, carlshen wrote:

 Hi,
 You can use svn diff to generate diff file.
 such as:

 svn diff  text.diff
 Then login Review Board and click ew Review Request by add the Repository
 and text.diff file.

 regards.
 carl shen

 2012/5/1 David Virtser poiso...@gmail.com

 Hi,

 When I try to run manually post review script in order to submit a new
 review request from range of SVN revisions and in some revision was a file
 that was changed and then deleted, it fails with the error below:

 *$ post-review --revision-range=43023:43470 --debug RBTools 0.4.1*
 * Home = /home/dev*
 * HTTP GETting api/*
 * HTTP GETting http://localhost/api/info/*
 * Using the new web API*
 *Failed to execute command: ['svn', 'info', 'Mobile/css/main.css']*
 *['Mobile/css/main.css:  (Not a versioned resource)\n', '\n', 'svn: A
 problem occurred; see other errors for details\n']*



 Is there anything we can do to solve it?
 I tried to look for all the keys that post-review get, but nothing
 really help there..


  Thanks,
 David.

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




 --
 Best regards.
 carl shen (xiangfeng)
 Phone: 13811550669


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


-- 
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: URGENT: post-review queries

Thank for prompt reply. Your reply motivated me. I think you will provide
me solution after looking into the requested diff.


Here is the requested output. The line marked with red have been added to
file.

posting review request...
 RBTools 0.3.4
 Home = /home/nileshj
 svn info
 p4 info
 repository info: Path: pf-ch.nds.com:1666, Base path: None, Supports
changesets: True
 HTTP GETting api/
 HTTP GETting http://rbc.nds.com:8000/rbsvr/api/info/
 Using the new web API
 Generating diff for changenum 1568
 p4 describe -s 1568
 CREATING WHERE FILE FROM /tmp/tmpMhEcPa
 p4 -ztag -x /tmp/tmpMhEcPa where
 Processing edit of //dator/Dev/DEV_BR/Plugins/PostPlugin/SConscript
 WRITING //dator/Dev/DEV_BR/Plugins/PostPlugin/SConscript#1
 to /tmp/tmp_yDlEz
 CREATING OLD PRINT BATCH /tmp/tmp26cwU1
 p4 -s -p pf-ch.nds.com:1666 -x /tmp/tmp_yDlEz print
 CREATING NEW PRINT BATCH /tmp/tmp5APfF-
 p4 -s -p pf-ch.nds.com:1666 -x /tmp/tmpxe11Ch print
 Processing edit of //dator/Dev/DEV_BR/Plugins/PostPlugin/SConscript
 Writing //dator/Dev/DEV_BR/Plugins/PostPlugin/SConscript#1 to
/tmp/tmpnNHcoH
 diff -urNp /tmp/tmpnNHcoH
/home/nileshj/fwei///dator/Dev/DEV_BR/Plugins/PostPlugin/SConscript
--- //dator/Dev/DEV_BR/Plugins/PostPlugin/SConscript
//dator/Dev/DEV_BR/Plugins/PostPlugin/SConscript#1
+++ //dator/Dev/DEV_BR/Plugins/PostPlugin/SConscript  2012-08-06
14:50:29
@@ -18,6 +18,6 @@ env.InstallTree( DeclarativeFiles, ..


 #==
-*
+#=*
 retval = env.ProcessDir(env_base)
 Return('retval')

Thank you again.

Regards,
Nilesh

On Mon, Aug 6, 2012 at 12:52 PM, Christian Hammond chip...@chipx86.comwrote:

 On Sun, Aug 5, 2012 at 11:21 PM, Nilesh Jaiswal nileshj...@gmail.comwrote:

 Dear All,

 I am trying to post the review request as below argument.

 My perforce id:njaiswal

 *I am using --submit-as option.*

 *post-review  --bugs-closed=TEST --file-list=/tmp/tmp9Dj3Dt
 --branch=DEV_BR --server=http://rbc.nds.com:8000/rbsvr--submit-as=nileshj 
 --description-file=/tmp/tmp79D4Hj 1567 --p4-port=
 pf-ch.nds.com:1666*

 My while posting i am getting below error message.

 FOR RB USE DO NOT EDIT


 It looks like you're using a custom post-review script. We can't fully
 support that, but I'll try to answer your questions.



 --
 p4web link: http://pf-ch.nds.com:5500/1567?ac=10
 baseline: 1564
 ' for review request '3'
  HTTP PUTting to
 http://rbc.nds.com:8000/rbsvr/api/review-requests/3/draft/:
 {'description': 'RB testing for china \n\n\nFOR RB USE DO NOT
 EDIT\n--\np4web link:
 http://pf-ch.nds.com:5500/1567?ac=10\nbaseline: 1564\n'}
  Uploading diff, size: 159
  HTTP POSTing to
 http://rbc.nds.com:8000/rbsvr/api/review-requests/3/diffs/: {}
  Got API Error 105 (HTTP code 400): One or more fields had errors
  Error data: {u'fields': {u'path': [u'[Errno 2] No such file or
 directory']}, u'stat': u'fail', u'err': {u'msg': u'One or more fields had
 errors', u'code': 105}}
 Your review request still exists, but the diff is not attached.

 *My question is, *

 1. Does my RB server user id requires account in perforce?


 No.



 2. What should i do make posting of review request if am having one
 perforce id with multiple RB server user id.


 All you need is one Perforce ID that's configured in the repository
 settings. The Review Board users have nothing to do with Perforce users.



 3. Is it mandatory that perforce id to be in the user list of reviewboard
 server


 No. It wouldn't matter in any way.

 It sounds like there's just something wrong with your diff. Run that
 command with --output-diff and inspect the output. Also, check your
 reviewboard.log for errors that correspond to the time you made that
 request.

 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

-- 
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: URGENT: post-review queries

Can you find anything in reviewboard.log that starts with Error uploading
new diff ? That's the most likely trigger for this error.

Christian

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


On Mon, Aug 6, 2012 at 12:33 AM, Nilesh Jaiswal nileshj...@gmail.comwrote:

 Thank for prompt reply. Your reply motivated me. I think you will provide
 me solution after looking into the requested diff.


 Here is the requested output. The line marked with red have been added to
 file.

 posting review request...
  RBTools 0.3.4
  Home = /home/nileshj
  svn info
  p4 info
  repository info: Path: pf-ch.nds.com:1666, Base path: None, Supports
 changesets: True
  HTTP GETting api/
  HTTP GETting http://rbc.nds.com:8000/rbsvr/api/info/
  Using the new web API
  Generating diff for changenum 1568
  p4 describe -s 1568
  CREATING WHERE FILE FROM /tmp/tmpMhEcPa
  p4 -ztag -x /tmp/tmpMhEcPa where
  Processing edit of //dator/Dev/DEV_BR/Plugins/PostPlugin/SConscript
  WRITING //dator/Dev/DEV_BR/Plugins/PostPlugin/SConscript#1
  to /tmp/tmp_yDlEz
  CREATING OLD PRINT BATCH /tmp/tmp26cwU1
  p4 -s -p pf-ch.nds.com:1666 -x /tmp/tmp_yDlEz print
  CREATING NEW PRINT BATCH /tmp/tmp5APfF-
  p4 -s -p pf-ch.nds.com:1666 -x /tmp/tmpxe11Ch print
  Processing edit of //dator/Dev/DEV_BR/Plugins/PostPlugin/SConscript
  Writing //dator/Dev/DEV_BR/Plugins/PostPlugin/SConscript#1 to
 /tmp/tmpnNHcoH
  diff -urNp /tmp/tmpnNHcoH
 /home/nileshj/fwei///dator/Dev/DEV_BR/Plugins/PostPlugin/SConscript
 --- //dator/Dev/DEV_BR/Plugins/PostPlugin/SConscript
 //dator/Dev/DEV_BR/Plugins/PostPlugin/SConscript#1
 +++ //dator/Dev/DEV_BR/Plugins/PostPlugin/SConscript  2012-08-06
 14:50:29
 @@ -18,6 +18,6 @@ env.InstallTree( DeclarativeFiles, ..


  #==
 -*
 +#=*
  retval = env.ProcessDir(env_base)
  Return('retval')

 Thank you again.

 Regards,
 Nilesh

 On Mon, Aug 6, 2012 at 12:52 PM, Christian Hammond chip...@chipx86.comwrote:

 On Sun, Aug 5, 2012 at 11:21 PM, Nilesh Jaiswal nileshj...@gmail.comwrote:

 Dear All,

 I am trying to post the review request as below argument.

 My perforce id:njaiswal

 *I am using --submit-as option.*

 *post-review  --bugs-closed=TEST --file-list=/tmp/tmp9Dj3Dt
 --branch=DEV_BR --server=http://rbc.nds.com:8000/rbsvr--submit-as=nileshj 
 --description-file=/tmp/tmp79D4Hj 1567 --p4-port=
 pf-ch.nds.com:1666*

 My while posting i am getting below error message.

 FOR RB USE DO NOT EDIT


 It looks like you're using a custom post-review script. We can't fully
 support that, but I'll try to answer your questions.



 --
 p4web link: http://pf-ch.nds.com:5500/1567?ac=10
 baseline: 1564
 ' for review request '3'
  HTTP PUTting to
 http://rbc.nds.com:8000/rbsvr/api/review-requests/3/draft/:
 {'description': 'RB testing for china \n\n\nFOR RB USE DO NOT
 EDIT\n--\np4web link:
 http://pf-ch.nds.com:5500/1567?ac=10\nbaseline: 1564\n'}
  Uploading diff, size: 159
  HTTP POSTing to
 http://rbc.nds.com:8000/rbsvr/api/review-requests/3/diffs/: {}
  Got API Error 105 (HTTP code 400): One or more fields had errors
  Error data: {u'fields': {u'path': [u'[Errno 2] No such file or
 directory']}, u'stat': u'fail', u'err': {u'msg': u'One or more fields had
 errors', u'code': 105}}
 Your review request still exists, but the diff is not attached.

 *My question is, *

 1. Does my RB server user id requires account in perforce?


 No.



 2. What should i do make posting of review request if am having one
 perforce id with multiple RB server user id.


 All you need is one Perforce ID that's configured in the repository
 settings. The Review Board users have nothing to do with Perforce users.



 3. Is it mandatory that perforce id to be in the user list of
 reviewboard server


 No. It wouldn't matter in any way.

 It sounds like there's just something wrong with your diff. Run that
 command with --output-diff and inspect the output. Also, check your
 reviewboard.log for errors that correspond to the time you made that
 request.

 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


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

Re: URGENT: post-review queries

here you go:


2012-08-06 00:00:57,711 - DEBUG - DiffParser.parse: Finished parsing diff.
2012-08-06 00:00:57,800 - ERROR - Error uploading new diff: [Errno 2] No
such file or directory
Traceback (most recent call last):
  File
/usr/local/lib/python2.7/site-packages/ReviewBoard-1.6.9-py2.7.egg/reviewboard/webapi/resources.py,
line 1510, in create
request.FILES.get('parent_diff_path'))
  File
/usr/local/lib/python2.7/site-packages/ReviewBoard-1.6.9-py2.7.egg/reviewboard/reviews/forms.py,
line 295, in create
history)
  File
/usr/local/lib/python2.7/site-packages/ReviewBoard-1.6.9-py2.7.egg/reviewboard/diffviewer/forms.py,
line 84, in create
diff_file, basedir, check_existance=(not parent_diff_file)))
  File
/usr/local/lib/python2.7/site-packages/ReviewBoard-1.6.9-py2.7.egg/reviewboard/diffviewer/forms.py,
line 170, in _process_files
not self.repository.get_file_exists(filename, revision))):
  File
/usr/local/lib/python2.7/site-packages/ReviewBoard-1.6.9-py2.7.egg/reviewboard/scmtools/models.py,
line 169, in get_file_exists
return self.get_scmtool().file_exists(path, revision)
  File
/usr/local/lib/python2.7/site-packages/ReviewBoard-1.6.9-py2.7.egg/reviewboard/scmtools/core.py,
line 72, in file_exists
self.get_file(path, revision)
  File
/usr/local/lib/python2.7/site-packages/ReviewBoard-1.6.9-py2.7.egg/reviewboard/scmtools/perforce.py,
line 306, in get_file
return self.client.get_file(path, revision)
  File
/usr/local/lib/python2.7/site-packages/ReviewBoard-1.6.9-py2.7.egg/reviewboard/scmtools/perforce.py,
line 224, in get_file
return self._run_worker(lambda: self._get_file(path, revision))
  File
/usr/local/lib/python2.7/site-packages/ReviewBoard-1.6.9-py2.7.egg/reviewboard/scmtools/perforce.py,
line 154, in _run_worker
result = worker()
  File
/usr/local/lib/python2.7/site-packages/ReviewBoard-1.6.9-py2.7.egg/reviewboard/scmtools/perforce.py,
line 224, in lambda
return self._run_worker(lambda: self._get_file(path, revision))
  File
/usr/local/lib/python2.7/site-packages/ReviewBoard-1.6.9-py2.7.egg/reviewboard/scmtools/perforce.py,
line 201, in _get_file
stderr=subprocess.PIPE)
  File /usr/local/lib/python2.7/subprocess.py, line 679, in __init__
errread, errwrite)
  File /usr/local/lib/python2.7/subprocess.py, line 1228, in
_execute_child
raise child_exception
OSError: [Errno 2] No such file or directory
2012-08-06 00:25:50,140 - DEBUG - Attempting authentication on API for user
nileshj



On Mon, Aug 6, 2012 at 1:07 PM, Christian Hammond chip...@chipx86.comwrote:

 Can you find anything in reviewboard.log that starts with Error uploading
 new diff ? That's the most likely trigger for this error.


 Christian

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


 On Mon, Aug 6, 2012 at 12:33 AM, Nilesh Jaiswal nileshj...@gmail.comwrote:

 Thank for prompt reply. Your reply motivated me. I think you will provide
 me solution after looking into the requested diff.


 Here is the requested output. The line marked with red have been added to
 file.

 posting review request...
  RBTools 0.3.4
  Home = /home/nileshj
  svn info
  p4 info
  repository info: Path: pf-ch.nds.com:1666, Base path: None, Supports
 changesets: True
  HTTP GETting api/
  HTTP GETting http://rbc.nds.com:8000/rbsvr/api/info/
  Using the new web API
  Generating diff for changenum 1568
  p4 describe -s 1568
  CREATING WHERE FILE FROM /tmp/tmpMhEcPa
  p4 -ztag -x /tmp/tmpMhEcPa where
  Processing edit of //dator/Dev/DEV_BR/Plugins/PostPlugin/SConscript
  WRITING //dator/Dev/DEV_BR/Plugins/PostPlugin/SConscript#1
  to /tmp/tmp_yDlEz
  CREATING OLD PRINT BATCH /tmp/tmp26cwU1
  p4 -s -p pf-ch.nds.com:1666 -x /tmp/tmp_yDlEz print
  CREATING NEW PRINT BATCH /tmp/tmp5APfF-
  p4 -s -p pf-ch.nds.com:1666 -x /tmp/tmpxe11Ch print
  Processing edit of //dator/Dev/DEV_BR/Plugins/PostPlugin/SConscript
  Writing //dator/Dev/DEV_BR/Plugins/PostPlugin/SConscript#1 to
 /tmp/tmpnNHcoH
  diff -urNp /tmp/tmpnNHcoH
 /home/nileshj/fwei///dator/Dev/DEV_BR/Plugins/PostPlugin/SConscript
 --- //dator/Dev/DEV_BR/Plugins/PostPlugin/SConscript
 //dator/Dev/DEV_BR/Plugins/PostPlugin/SConscript#1
 +++ //dator/Dev/DEV_BR/Plugins/PostPlugin/SConscript  2012-08-06
 14:50:29
 @@ -18,6 +18,6 @@ env.InstallTree( DeclarativeFiles, ..


  #==
 -*
 +#=*
  retval = env.ProcessDir(env_base)
  Return('retval')

 Thank you again.

 Regards,
 Nilesh

 On Mon, Aug 6, 2012 at 12:52 PM, Christian Hammond 
 chip...@chipx86.comwrote:

 On Sun, Aug 5, 2012 at 11:21 PM, Nilesh Jaiswal nileshj...@gmail.comwrote:

 Dear All,

 I am trying to post the review request as below argument.

 My perforce id:njaiswal

 *I am using --submit-as option.*

 *post-review  --bugs-closed=TEST --file-list=/tmp/tmp9Dj3Dt
 --branch=DEV_BR 
 --server=http://rbc.nds.com:8000/rbsvr--submit-as=nileshj

Need Help with Integrating Clearcase with Review Board

2012-08-06 Thread Peer

Hi,

I'm new to Review Board, i have installed RB 1.6.11 in windows machine 
after long struggle.

I'm now facing issue with Integrating Clearcase with Review Board. and 
upload diffs.

If someone could help me with step by step of the same would be really 
helpful,

Thanks,
Peer

-- 
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: URGENT: post-review queries

You don't have p4 installed in a path that Apache can access. Make sure
it's installed in /usr/bin.

Christian

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


On Mon, Aug 6, 2012 at 12:40 AM, Nilesh Jaiswal nileshj...@gmail.comwrote:

 here you go:


 2012-08-06 00:00:57,711 - DEBUG - DiffParser.parse: Finished parsing diff.
 2012-08-06 00:00:57,800 - ERROR - Error uploading new diff: [Errno 2] No
 such file or directory
 Traceback (most recent call last):
   File
 /usr/local/lib/python2.7/site-packages/ReviewBoard-1.6.9-py2.7.egg/reviewboard/webapi/resources.py,
 line 1510, in create
 request.FILES.get('parent_diff_path'))
   File
 /usr/local/lib/python2.7/site-packages/ReviewBoard-1.6.9-py2.7.egg/reviewboard/reviews/forms.py,
 line 295, in create
 history)
   File
 /usr/local/lib/python2.7/site-packages/ReviewBoard-1.6.9-py2.7.egg/reviewboard/diffviewer/forms.py,
 line 84, in create
 diff_file, basedir, check_existance=(not parent_diff_file)))
   File
 /usr/local/lib/python2.7/site-packages/ReviewBoard-1.6.9-py2.7.egg/reviewboard/diffviewer/forms.py,
 line 170, in _process_files
 not self.repository.get_file_exists(filename, revision))):
   File
 /usr/local/lib/python2.7/site-packages/ReviewBoard-1.6.9-py2.7.egg/reviewboard/scmtools/models.py,
 line 169, in get_file_exists
 return self.get_scmtool().file_exists(path, revision)
   File
 /usr/local/lib/python2.7/site-packages/ReviewBoard-1.6.9-py2.7.egg/reviewboard/scmtools/core.py,
 line 72, in file_exists
 self.get_file(path, revision)
   File
 /usr/local/lib/python2.7/site-packages/ReviewBoard-1.6.9-py2.7.egg/reviewboard/scmtools/perforce.py,
 line 306, in get_file
 return self.client.get_file(path, revision)
   File
 /usr/local/lib/python2.7/site-packages/ReviewBoard-1.6.9-py2.7.egg/reviewboard/scmtools/perforce.py,
 line 224, in get_file
 return self._run_worker(lambda: self._get_file(path, revision))
   File
 /usr/local/lib/python2.7/site-packages/ReviewBoard-1.6.9-py2.7.egg/reviewboard/scmtools/perforce.py,
 line 154, in _run_worker
 result = worker()
   File
 /usr/local/lib/python2.7/site-packages/ReviewBoard-1.6.9-py2.7.egg/reviewboard/scmtools/perforce.py,
 line 224, in lambda
 return self._run_worker(lambda: self._get_file(path, revision))
   File
 /usr/local/lib/python2.7/site-packages/ReviewBoard-1.6.9-py2.7.egg/reviewboard/scmtools/perforce.py,
 line 201, in _get_file
 stderr=subprocess.PIPE)
   File /usr/local/lib/python2.7/subprocess.py, line 679, in __init__
 errread, errwrite)
   File /usr/local/lib/python2.7/subprocess.py, line 1228, in
 _execute_child
 raise child_exception
 OSError: [Errno 2] No such file or directory
 2012-08-06 00:25:50,140 - DEBUG - Attempting authentication on API for
 user nileshj




 On Mon, Aug 6, 2012 at 1:07 PM, Christian Hammond chip...@chipx86.comwrote:

 Can you find anything in reviewboard.log that starts with Error
 uploading new diff ? That's the most likely trigger for this error.


 Christian

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


 On Mon, Aug 6, 2012 at 12:33 AM, Nilesh Jaiswal nileshj...@gmail.comwrote:

 Thank for prompt reply. Your reply motivated me. I think you will
 provide me solution after looking into the requested diff.


 Here is the requested output. The line marked with red have been added
 to file.

 posting review request...
  RBTools 0.3.4
  Home = /home/nileshj
  svn info
  p4 info
  repository info: Path: pf-ch.nds.com:1666, Base path: None,
 Supports changesets: True
  HTTP GETting api/
  HTTP GETting http://rbc.nds.com:8000/rbsvr/api/info/
  Using the new web API
  Generating diff for changenum 1568
  p4 describe -s 1568
  CREATING WHERE FILE FROM /tmp/tmpMhEcPa
  p4 -ztag -x /tmp/tmpMhEcPa where
  Processing edit of //dator/Dev/DEV_BR/Plugins/PostPlugin/SConscript
  WRITING //dator/Dev/DEV_BR/Plugins/PostPlugin/SConscript#1
  to /tmp/tmp_yDlEz
  CREATING OLD PRINT BATCH /tmp/tmp26cwU1
  p4 -s -p pf-ch.nds.com:1666 -x /tmp/tmp_yDlEz print
  CREATING NEW PRINT BATCH /tmp/tmp5APfF-
  p4 -s -p pf-ch.nds.com:1666 -x /tmp/tmpxe11Ch print
  Processing edit of //dator/Dev/DEV_BR/Plugins/PostPlugin/SConscript
  Writing //dator/Dev/DEV_BR/Plugins/PostPlugin/SConscript#1 to
 /tmp/tmpnNHcoH
  diff -urNp /tmp/tmpnNHcoH
 /home/nileshj/fwei///dator/Dev/DEV_BR/Plugins/PostPlugin/SConscript
 --- //dator/Dev/DEV_BR/Plugins/PostPlugin/SConscript
 //dator/Dev/DEV_BR/Plugins/PostPlugin/SConscript#1
 +++ //dator/Dev/DEV_BR/Plugins/PostPlugin/SConscript  2012-08-06
 14:50:29
 @@ -18,6 +18,6 @@ env.InstallTree( DeclarativeFiles, ..


  #==
 -*
 +#=*
  retval = env.ProcessDir(env_base)
  Return('retval')

 Thank you again.

 Regards,
 Nilesh

 On Mon, Aug 6, 2012 at 12:52 PM, Christian Hammond

Re: URGENT: post-review queries

I am having p4 installed at /usr/local/bin/p4

What should i do.?

On Mon, Aug 6, 2012 at 1:19 PM, Christian Hammond chip...@chipx86.comwrote:

 You don't have p4 installed in a path that Apache can access. Make sure
 it's installed in /usr/bin.


 Christian

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


 On Mon, Aug 6, 2012 at 12:40 AM, Nilesh Jaiswal nileshj...@gmail.comwrote:

 here you go:


 2012-08-06 00:00:57,711 - DEBUG - DiffParser.parse: Finished parsing diff.
 2012-08-06 00:00:57,800 - ERROR - Error uploading new diff: [Errno 2] No
 such file or directory
 Traceback (most recent call last):
   File
 /usr/local/lib/python2.7/site-packages/ReviewBoard-1.6.9-py2.7.egg/reviewboard/webapi/resources.py,
 line 1510, in create
 request.FILES.get('parent_diff_path'))
   File
 /usr/local/lib/python2.7/site-packages/ReviewBoard-1.6.9-py2.7.egg/reviewboard/reviews/forms.py,
 line 295, in create
 history)
   File
 /usr/local/lib/python2.7/site-packages/ReviewBoard-1.6.9-py2.7.egg/reviewboard/diffviewer/forms.py,
 line 84, in create
 diff_file, basedir, check_existance=(not parent_diff_file)))
   File
 /usr/local/lib/python2.7/site-packages/ReviewBoard-1.6.9-py2.7.egg/reviewboard/diffviewer/forms.py,
 line 170, in _process_files
 not self.repository.get_file_exists(filename, revision))):
   File
 /usr/local/lib/python2.7/site-packages/ReviewBoard-1.6.9-py2.7.egg/reviewboard/scmtools/models.py,
 line 169, in get_file_exists
 return self.get_scmtool().file_exists(path, revision)
   File
 /usr/local/lib/python2.7/site-packages/ReviewBoard-1.6.9-py2.7.egg/reviewboard/scmtools/core.py,
 line 72, in file_exists
 self.get_file(path, revision)
   File
 /usr/local/lib/python2.7/site-packages/ReviewBoard-1.6.9-py2.7.egg/reviewboard/scmtools/perforce.py,
 line 306, in get_file
 return self.client.get_file(path, revision)
   File
 /usr/local/lib/python2.7/site-packages/ReviewBoard-1.6.9-py2.7.egg/reviewboard/scmtools/perforce.py,
 line 224, in get_file
 return self._run_worker(lambda: self._get_file(path, revision))
   File
 /usr/local/lib/python2.7/site-packages/ReviewBoard-1.6.9-py2.7.egg/reviewboard/scmtools/perforce.py,
 line 154, in _run_worker
 result = worker()
   File
 /usr/local/lib/python2.7/site-packages/ReviewBoard-1.6.9-py2.7.egg/reviewboard/scmtools/perforce.py,
 line 224, in lambda
 return self._run_worker(lambda: self._get_file(path, revision))
   File
 /usr/local/lib/python2.7/site-packages/ReviewBoard-1.6.9-py2.7.egg/reviewboard/scmtools/perforce.py,
 line 201, in _get_file
 stderr=subprocess.PIPE)
   File /usr/local/lib/python2.7/subprocess.py, line 679, in __init__
 errread, errwrite)
   File /usr/local/lib/python2.7/subprocess.py, line 1228, in
 _execute_child
 raise child_exception
 OSError: [Errno 2] No such file or directory
 2012-08-06 00:25:50,140 - DEBUG - Attempting authentication on API for
 user nileshj




 On Mon, Aug 6, 2012 at 1:07 PM, Christian Hammond chip...@chipx86.comwrote:

 Can you find anything in reviewboard.log that starts with Error
 uploading new diff ? That's the most likely trigger for this error.


 Christian

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


 On Mon, Aug 6, 2012 at 12:33 AM, Nilesh Jaiswal nileshj...@gmail.comwrote:

 Thank for prompt reply. Your reply motivated me. I think you will
 provide me solution after looking into the requested diff.


 Here is the requested output. The line marked with red have been added
 to file.

 posting review request...
  RBTools 0.3.4
  Home = /home/nileshj
  svn info
  p4 info
  repository info: Path: pf-ch.nds.com:1666, Base path: None,
 Supports changesets: True
  HTTP GETting api/
  HTTP GETting http://rbc.nds.com:8000/rbsvr/api/info/
  Using the new web API
  Generating diff for changenum 1568
  p4 describe -s 1568
  CREATING WHERE FILE FROM /tmp/tmpMhEcPa
  p4 -ztag -x /tmp/tmpMhEcPa where
  Processing edit of //dator/Dev/DEV_BR/Plugins/PostPlugin/SConscript
  WRITING //dator/Dev/DEV_BR/Plugins/PostPlugin/SConscript#1
  to /tmp/tmp_yDlEz
  CREATING OLD PRINT BATCH /tmp/tmp26cwU1
  p4 -s -p pf-ch.nds.com:1666 -x /tmp/tmp_yDlEz print
  CREATING NEW PRINT BATCH /tmp/tmp5APfF-
  p4 -s -p pf-ch.nds.com:1666 -x /tmp/tmpxe11Ch print
  Processing edit of //dator/Dev/DEV_BR/Plugins/PostPlugin/SConscript
  Writing //dator/Dev/DEV_BR/Plugins/PostPlugin/SConscript#1 to
 /tmp/tmpnNHcoH
  diff -urNp /tmp/tmpnNHcoH
 /home/nileshj/fwei///dator/Dev/DEV_BR/Plugins/PostPlugin/SConscript
 --- //dator/Dev/DEV_BR/Plugins/PostPlugin/SConscript
 //dator/Dev/DEV_BR/Plugins/PostPlugin/SConscript#1
 +++ //dator/Dev/DEV_BR/Plugins/PostPlugin/SConscript  2012-08-06
 14:50:29
 @@ -18,6 +18,6 @@ env.InstallTree( DeclarativeFiles, ..


  #==
 -*
 +#=*
  retval =

Re: URGENT: post-review queries

Move it to /usr/bin so Apache can see it.

Christian

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


On Mon, Aug 6, 2012 at 12:51 AM, Nilesh Jaiswal nileshj...@gmail.comwrote:

 I am having p4 installed at /usr/local/bin/p4

 What should i do.?


 On Mon, Aug 6, 2012 at 1:19 PM, Christian Hammond chip...@chipx86.comwrote:

 You don't have p4 installed in a path that Apache can access. Make sure
 it's installed in /usr/bin.


 Christian

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


 On Mon, Aug 6, 2012 at 12:40 AM, Nilesh Jaiswal nileshj...@gmail.comwrote:

 here you go:


 2012-08-06 00:00:57,711 - DEBUG - DiffParser.parse: Finished parsing
 diff.
 2012-08-06 00:00:57,800 - ERROR - Error uploading new diff: [Errno 2] No
 such file or directory
 Traceback (most recent call last):
   File
 /usr/local/lib/python2.7/site-packages/ReviewBoard-1.6.9-py2.7.egg/reviewboard/webapi/resources.py,
 line 1510, in create
 request.FILES.get('parent_diff_path'))
   File
 /usr/local/lib/python2.7/site-packages/ReviewBoard-1.6.9-py2.7.egg/reviewboard/reviews/forms.py,
 line 295, in create
 history)
   File
 /usr/local/lib/python2.7/site-packages/ReviewBoard-1.6.9-py2.7.egg/reviewboard/diffviewer/forms.py,
 line 84, in create
 diff_file, basedir, check_existance=(not parent_diff_file)))
   File
 /usr/local/lib/python2.7/site-packages/ReviewBoard-1.6.9-py2.7.egg/reviewboard/diffviewer/forms.py,
 line 170, in _process_files
 not self.repository.get_file_exists(filename, revision))):
   File
 /usr/local/lib/python2.7/site-packages/ReviewBoard-1.6.9-py2.7.egg/reviewboard/scmtools/models.py,
 line 169, in get_file_exists
 return self.get_scmtool().file_exists(path, revision)
   File
 /usr/local/lib/python2.7/site-packages/ReviewBoard-1.6.9-py2.7.egg/reviewboard/scmtools/core.py,
 line 72, in file_exists
 self.get_file(path, revision)
   File
 /usr/local/lib/python2.7/site-packages/ReviewBoard-1.6.9-py2.7.egg/reviewboard/scmtools/perforce.py,
 line 306, in get_file
 return self.client.get_file(path, revision)
   File
 /usr/local/lib/python2.7/site-packages/ReviewBoard-1.6.9-py2.7.egg/reviewboard/scmtools/perforce.py,
 line 224, in get_file
 return self._run_worker(lambda: self._get_file(path, revision))
   File
 /usr/local/lib/python2.7/site-packages/ReviewBoard-1.6.9-py2.7.egg/reviewboard/scmtools/perforce.py,
 line 154, in _run_worker
 result = worker()
   File
 /usr/local/lib/python2.7/site-packages/ReviewBoard-1.6.9-py2.7.egg/reviewboard/scmtools/perforce.py,
 line 224, in lambda
 return self._run_worker(lambda: self._get_file(path, revision))
   File
 /usr/local/lib/python2.7/site-packages/ReviewBoard-1.6.9-py2.7.egg/reviewboard/scmtools/perforce.py,
 line 201, in _get_file
 stderr=subprocess.PIPE)
   File /usr/local/lib/python2.7/subprocess.py, line 679, in __init__
 errread, errwrite)
   File /usr/local/lib/python2.7/subprocess.py, line 1228, in
 _execute_child
 raise child_exception
 OSError: [Errno 2] No such file or directory
 2012-08-06 00:25:50,140 - DEBUG - Attempting authentication on API for
 user nileshj




 On Mon, Aug 6, 2012 at 1:07 PM, Christian Hammond 
 chip...@chipx86.comwrote:

 Can you find anything in reviewboard.log that starts with Error
 uploading new diff ? That's the most likely trigger for this error.


 Christian

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


 On Mon, Aug 6, 2012 at 12:33 AM, Nilesh Jaiswal 
 nileshj...@gmail.comwrote:

 Thank for prompt reply. Your reply motivated me. I think you will
 provide me solution after looking into the requested diff.


 Here is the requested output. The line marked with red have been added
 to file.

 posting review request...
  RBTools 0.3.4
  Home = /home/nileshj
  svn info
  p4 info
  repository info: Path: pf-ch.nds.com:1666, Base path: None,
 Supports changesets: True
  HTTP GETting api/
  HTTP GETting http://rbc.nds.com:8000/rbsvr/api/info/
  Using the new web API
  Generating diff for changenum 1568
  p4 describe -s 1568
  CREATING WHERE FILE FROM /tmp/tmpMhEcPa
  p4 -ztag -x /tmp/tmpMhEcPa where
  Processing edit of //dator/Dev/DEV_BR/Plugins/PostPlugin/SConscript
  WRITING //dator/Dev/DEV_BR/Plugins/PostPlugin/SConscript#1
  to /tmp/tmp_yDlEz
  CREATING OLD PRINT BATCH /tmp/tmp26cwU1
  p4 -s -p pf-ch.nds.com:1666 -x /tmp/tmp_yDlEz print
  CREATING NEW PRINT BATCH /tmp/tmp5APfF-
  p4 -s -p pf-ch.nds.com:1666 -x /tmp/tmpxe11Ch print
  Processing edit of //dator/Dev/DEV_BR/Plugins/PostPlugin/SConscript
  Writing //dator/Dev/DEV_BR/Plugins/PostPlugin/SConscript#1 to
 /tmp/tmpnNHcoH
  diff -urNp /tmp/tmpnNHcoH
 /home/nileshj/fwei///dator/Dev/DEV_BR/Plugins/PostPlugin/SConscript
 --- //dator/Dev/DEV_BR/Plugins/PostPlugin/SConscript

Re: URGENT: post-review queries

Hello Mr Life Saver !!

You made my work so easy !! you are fantastic !!

Whole of MSC Software will be proud of you !!

Thank you so much.

On Mon, Aug 6, 2012 at 1:28 PM, Christian Hammond chip...@chipx86.comwrote:

 Move it to /usr/bin so Apache can see it.


 Christian

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


 On Mon, Aug 6, 2012 at 12:51 AM, Nilesh Jaiswal nileshj...@gmail.comwrote:

 I am having p4 installed at /usr/local/bin/p4

 What should i do.?


 On Mon, Aug 6, 2012 at 1:19 PM, Christian Hammond chip...@chipx86.comwrote:

 You don't have p4 installed in a path that Apache can access. Make sure
 it's installed in /usr/bin.


 Christian

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


 On Mon, Aug 6, 2012 at 12:40 AM, Nilesh Jaiswal nileshj...@gmail.comwrote:

 here you go:


 2012-08-06 00:00:57,711 - DEBUG - DiffParser.parse: Finished parsing
 diff.
 2012-08-06 00:00:57,800 - ERROR - Error uploading new diff: [Errno 2]
 No such file or directory
 Traceback (most recent call last):
   File
 /usr/local/lib/python2.7/site-packages/ReviewBoard-1.6.9-py2.7.egg/reviewboard/webapi/resources.py,
 line 1510, in create
 request.FILES.get('parent_diff_path'))
   File
 /usr/local/lib/python2.7/site-packages/ReviewBoard-1.6.9-py2.7.egg/reviewboard/reviews/forms.py,
 line 295, in create
 history)
   File
 /usr/local/lib/python2.7/site-packages/ReviewBoard-1.6.9-py2.7.egg/reviewboard/diffviewer/forms.py,
 line 84, in create
 diff_file, basedir, check_existance=(not parent_diff_file)))
   File
 /usr/local/lib/python2.7/site-packages/ReviewBoard-1.6.9-py2.7.egg/reviewboard/diffviewer/forms.py,
 line 170, in _process_files
 not self.repository.get_file_exists(filename, revision))):
   File
 /usr/local/lib/python2.7/site-packages/ReviewBoard-1.6.9-py2.7.egg/reviewboard/scmtools/models.py,
 line 169, in get_file_exists
 return self.get_scmtool().file_exists(path, revision)
   File
 /usr/local/lib/python2.7/site-packages/ReviewBoard-1.6.9-py2.7.egg/reviewboard/scmtools/core.py,
 line 72, in file_exists
 self.get_file(path, revision)
   File
 /usr/local/lib/python2.7/site-packages/ReviewBoard-1.6.9-py2.7.egg/reviewboard/scmtools/perforce.py,
 line 306, in get_file
 return self.client.get_file(path, revision)
   File
 /usr/local/lib/python2.7/site-packages/ReviewBoard-1.6.9-py2.7.egg/reviewboard/scmtools/perforce.py,
 line 224, in get_file
 return self._run_worker(lambda: self._get_file(path, revision))
   File
 /usr/local/lib/python2.7/site-packages/ReviewBoard-1.6.9-py2.7.egg/reviewboard/scmtools/perforce.py,
 line 154, in _run_worker
 result = worker()
   File
 /usr/local/lib/python2.7/site-packages/ReviewBoard-1.6.9-py2.7.egg/reviewboard/scmtools/perforce.py,
 line 224, in lambda
 return self._run_worker(lambda: self._get_file(path, revision))
   File
 /usr/local/lib/python2.7/site-packages/ReviewBoard-1.6.9-py2.7.egg/reviewboard/scmtools/perforce.py,
 line 201, in _get_file
 stderr=subprocess.PIPE)
   File /usr/local/lib/python2.7/subprocess.py, line 679, in __init__
 errread, errwrite)
   File /usr/local/lib/python2.7/subprocess.py, line 1228, in
 _execute_child
 raise child_exception
 OSError: [Errno 2] No such file or directory
 2012-08-06 00:25:50,140 - DEBUG - Attempting authentication on API for
 user nileshj




 On Mon, Aug 6, 2012 at 1:07 PM, Christian Hammond 
 chip...@chipx86.comwrote:

 Can you find anything in reviewboard.log that starts with Error
 uploading new diff ? That's the most likely trigger for this error.


 Christian

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


 On Mon, Aug 6, 2012 at 12:33 AM, Nilesh Jaiswal 
 nileshj...@gmail.comwrote:

 Thank for prompt reply. Your reply motivated me. I think you will
 provide me solution after looking into the requested diff.


 Here is the requested output. The line marked with red have been
 added to file.

 posting review request...
  RBTools 0.3.4
  Home = /home/nileshj
  svn info
  p4 info
  repository info: Path: pf-ch.nds.com:1666, Base path: None,
 Supports changesets: True
  HTTP GETting api/
  HTTP GETting http://rbc.nds.com:8000/rbsvr/api/info/
  Using the new web API
  Generating diff for changenum 1568
  p4 describe -s 1568
  CREATING WHERE FILE FROM /tmp/tmpMhEcPa
  p4 -ztag -x /tmp/tmpMhEcPa where
  Processing edit of
 //dator/Dev/DEV_BR/Plugins/PostPlugin/SConscript
  WRITING //dator/Dev/DEV_BR/Plugins/PostPlugin/SConscript#1
  to /tmp/tmp_yDlEz
  CREATING OLD PRINT BATCH /tmp/tmp26cwU1
  p4 -s -p pf-ch.nds.com:1666 -x /tmp/tmp_yDlEz print
  CREATING NEW PRINT BATCH /tmp/tmp5APfF-
  p4 -s -p pf-ch.nds.com:1666 -x /tmp/tmpxe11Ch print
  Processing edit of
 //dator/Dev/DEV_BR/Plugins/PostPlugin/SConscript
  Writing

Re: URGENT: post-review queries

Glad it works!

Christian

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


On Mon, Aug 6, 2012 at 1:04 AM, Nilesh Jaiswal nileshj...@gmail.com wrote:

 Hello Mr Life Saver !!

 You made my work so easy !! you are fantastic !!

 Whole of MSC Software will be proud of you !!

 Thank you so much.


 On Mon, Aug 6, 2012 at 1:28 PM, Christian Hammond chip...@chipx86.comwrote:

 Move it to /usr/bin so Apache can see it.


 Christian

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


 On Mon, Aug 6, 2012 at 12:51 AM, Nilesh Jaiswal nileshj...@gmail.comwrote:

 I am having p4 installed at /usr/local/bin/p4

 What should i do.?


 On Mon, Aug 6, 2012 at 1:19 PM, Christian Hammond 
 chip...@chipx86.comwrote:

 You don't have p4 installed in a path that Apache can access. Make sure
 it's installed in /usr/bin.


 Christian

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


 On Mon, Aug 6, 2012 at 12:40 AM, Nilesh Jaiswal 
 nileshj...@gmail.comwrote:

 here you go:


 2012-08-06 00:00:57,711 - DEBUG - DiffParser.parse: Finished parsing
 diff.
 2012-08-06 00:00:57,800 - ERROR - Error uploading new diff: [Errno 2]
 No such file or directory
 Traceback (most recent call last):
   File
 /usr/local/lib/python2.7/site-packages/ReviewBoard-1.6.9-py2.7.egg/reviewboard/webapi/resources.py,
 line 1510, in create
 request.FILES.get('parent_diff_path'))
   File
 /usr/local/lib/python2.7/site-packages/ReviewBoard-1.6.9-py2.7.egg/reviewboard/reviews/forms.py,
 line 295, in create
 history)
   File
 /usr/local/lib/python2.7/site-packages/ReviewBoard-1.6.9-py2.7.egg/reviewboard/diffviewer/forms.py,
 line 84, in create
 diff_file, basedir, check_existance=(not parent_diff_file)))
   File
 /usr/local/lib/python2.7/site-packages/ReviewBoard-1.6.9-py2.7.egg/reviewboard/diffviewer/forms.py,
 line 170, in _process_files
 not self.repository.get_file_exists(filename, revision))):
   File
 /usr/local/lib/python2.7/site-packages/ReviewBoard-1.6.9-py2.7.egg/reviewboard/scmtools/models.py,
 line 169, in get_file_exists
 return self.get_scmtool().file_exists(path, revision)
   File
 /usr/local/lib/python2.7/site-packages/ReviewBoard-1.6.9-py2.7.egg/reviewboard/scmtools/core.py,
 line 72, in file_exists
 self.get_file(path, revision)
   File
 /usr/local/lib/python2.7/site-packages/ReviewBoard-1.6.9-py2.7.egg/reviewboard/scmtools/perforce.py,
 line 306, in get_file
 return self.client.get_file(path, revision)
   File
 /usr/local/lib/python2.7/site-packages/ReviewBoard-1.6.9-py2.7.egg/reviewboard/scmtools/perforce.py,
 line 224, in get_file
 return self._run_worker(lambda: self._get_file(path, revision))
   File
 /usr/local/lib/python2.7/site-packages/ReviewBoard-1.6.9-py2.7.egg/reviewboard/scmtools/perforce.py,
 line 154, in _run_worker
 result = worker()
   File
 /usr/local/lib/python2.7/site-packages/ReviewBoard-1.6.9-py2.7.egg/reviewboard/scmtools/perforce.py,
 line 224, in lambda
 return self._run_worker(lambda: self._get_file(path, revision))
   File
 /usr/local/lib/python2.7/site-packages/ReviewBoard-1.6.9-py2.7.egg/reviewboard/scmtools/perforce.py,
 line 201, in _get_file
 stderr=subprocess.PIPE)
   File /usr/local/lib/python2.7/subprocess.py, line 679, in __init__
 errread, errwrite)
   File /usr/local/lib/python2.7/subprocess.py, line 1228, in
 _execute_child
 raise child_exception
 OSError: [Errno 2] No such file or directory
 2012-08-06 00:25:50,140 - DEBUG - Attempting authentication on API for
 user nileshj




 On Mon, Aug 6, 2012 at 1:07 PM, Christian Hammond chip...@chipx86.com
  wrote:

 Can you find anything in reviewboard.log that starts with Error
 uploading new diff ? That's the most likely trigger for this error.


 Christian

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


 On Mon, Aug 6, 2012 at 12:33 AM, Nilesh Jaiswal nileshj...@gmail.com
  wrote:

 Thank for prompt reply. Your reply motivated me. I think you will
 provide me solution after looking into the requested diff.


 Here is the requested output. The line marked with red have been
 added to file.

 posting review request...
  RBTools 0.3.4
  Home = /home/nileshj
  svn info
  p4 info
  repository info: Path: pf-ch.nds.com:1666, Base path: None,
 Supports changesets: True
  HTTP GETting api/
  HTTP GETting http://rbc.nds.com:8000/rbsvr/api/info/
  Using the new web API
  Generating diff for changenum 1568
  p4 describe -s 1568
  CREATING WHERE FILE FROM /tmp/tmpMhEcPa
  p4 -ztag -x /tmp/tmpMhEcPa where
  Processing edit of
 //dator/Dev/DEV_BR/Plugins/PostPlugin/SConscript
  WRITING //dator/Dev/DEV_BR/Plugins/PostPlugin/SConscript#1
  to /tmp/tmp_yDlEz
  CREATING OLD PRINT BATCH /tmp/tmp26cwU1
  p4 -s -p

Re: Not able to use post-review

Hi ,
Following is the output diff --version.

Usage: diff [OPTIONS] file1 file2

Display the differences between file1 and file2 in unified diff
format. OPTIONS are diff extensions as described by 'svn help diff'.
Use '--' alone to indicate that no more options follow.

Regards
Vijay

On Monday, 6 August 2012 12:52:54 UTC+5:30, Christian Hammond wrote:

Can you run diff --version and show me the output?

Christian

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

On Sun, Aug 5, 2012 at 11:49 PM, Vijay Kumar Tiwary
vijayktiw...@gmail.com wrote:

Hi Christian,
Thanks for the reply . I an getting following error when i typed
post-review in cmd

GNU diff is required for Subversion repositories. Make sure it is
installed
and in the path.

On Windows, you can install this from:
http://gnuwin32.sourceforge.net/packages/diffutils.htm;

I have already installed GNU Diff and set in the Path .

For your information,I have also installed both Tortoise SVN as well as
collabnet subversion client .

Please suggest how to proceed further.

Regards
Vijay

On Saturday, 4 August 2012 00:29:03 UTC+5:30, Christian Hammond wrote:

Hi,

Can you describe to me how GNU diff isn't working? What are you seeing?

Christian

On Aug 3, 2012, at 2:50, Vijay Kumar Tiwary vijayktiw...@gmail.com
wrote:

Hi ,
I was trying to use post-review filename but I get following error
always :

I have installed GNU Diff and even post-review installation seems to
work but some how GNU diff is not working.

i debuged and type following command SVN info and it is giving me the
configured SVN repo browser value

Please suggest how to configure .

For your information I setup Review Board on Linux machine and
configured SVN which is hosted on Windows machine . I am using my Windows
client machine to use review board and trying to post review but apparently
got stuck with GNU diff ...Please help me to come out from here !!!

Regards
Vijay

--
Want to help the Review Board project? Donate today at
http://www.reviewboard.org/**donate/http://www.reviewboard.org/donate/
Happy user? Let us know at
http://www.reviewboard.org/**users/http://www.reviewboard.org/users/
-~--~~~~--**~~--~--~---
To unsubscribe from this group, send email to reviewboard+unsubscribe@**
googlegroups.com reviewboard+unsubscr...@googlegroups.com
For more options, visit this group at http://groups.google.com/**
group/reviewboard?hl=enhttp://groups.google.com/group/reviewboard?hl=en

Re: Not able to use post-review

Hi ,
As a workaround I guess ... I have put all the diff exe to the folder where
I have installed post-review . Then this error goes off .Now I am getting
another error.
if i give command like post-review --server http:\\10.117.8.9 --debug ,i
am getting following error

RBTools 0.4.1
Home = C:\Users\vtiwary\AppData\Roaming
HTTP GETting api/
HTTP GETting http://10.117.8.9/api/info/
Using the new web API
Failed to execute command: ['svn', 'diff', '--diff-cmd=diff']
['Index: .\n',
'
===\n', '--- .\t(revision 206)\n', '+++ .\t(working copy)\n', '\n',
'Property ch
anges on: .\n',
'___
\n', 'Added: reviewboard:url\n', '## -0,0 +1 ##\n',
'+http://10.117.2.8\n',
'\\ No newline at end of property\n', 'Index: IMessageCallback.cs\n',
'=
==\n', 'Usage: diff
[OPT
IONS] file1 file2\n', '\n', 'Display the differences between file1
and fi
le2 in unified diff\n', format. OPTIONS are diff extensions as described
by '
svn help diff'.\n, Use '--' alone to indicate that no more options
follow.\n,
svn: E200012: 'diff' returned 2\n]

How can i get rid of this error .

Thanks
Vijay

On Monday, 6 August 2012 15:51:33 UTC+5:30, Vijay Kumar Tiwary wrote:

Hi ,
Following is the output diff --version.

Usage: diff [OPTIONS] file1 file2

Display the differences between file1 and file2 in unified diff
format. OPTIONS are diff extensions as described by 'svn help diff'.
Use '--' alone to indicate that no more options follow.

Regards
Vijay

On Monday, 6 August 2012 12:52:54 UTC+5:30, Christian Hammond wrote:

Can you run diff --version and show me the output?

Christian

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

On Sun, Aug 5, 2012 at 11:49 PM, Vijay Kumar Tiwary
vijayktiw...@gmail.com wrote:

Hi Christian,
Thanks for the reply . I an getting following error when i typed
post-review in cmd

GNU diff is required for Subversion repositories. Make sure it is
installed
and in the path.

On Windows, you can install this from:
http://gnuwin32.sourceforge.net/packages/diffutils.htm;

I have already installed GNU Diff and set in the Path .

For your information,I have also installed both Tortoise SVN as well as
collabnet subversion client .

Please suggest how to proceed further.

Regards
Vijay

On Saturday, 4 August 2012 00:29:03 UTC+5:30, Christian Hammond wrote:

Hi,

Can you describe to me how GNU diff isn't working? What are you seeing?

Christian

On Aug 3, 2012, at 2:50, Vijay Kumar Tiwary vijayktiw...@gmail.com
wrote:

Hi ,
I was trying to use post-review filename but I get following error
always :

I have installed GNU Diff and even post-review installation seems to
work but some how GNU diff is not working.

i debuged and type following command SVN info and it is giving me the
configured SVN repo browser value

Please suggest how to configure .

For your information I setup Review Board on Linux machine and
configured SVN which is hosted on Windows machine . I am using my Windows
client machine to use review board and trying to post review but
apparently
got stuck with GNU diff ...Please help me to come out from here !!!

Regards
Vijay

--
Want to help the Review Board project? Donate today at
http://www.reviewboard.org/**donate/http://www.reviewboard.org/donate/
Happy user? Let us know at
http://www.reviewboard.org/**users/http://www.reviewboard.org/users/
-~--~~~~--**~~--~--~---
To unsubscribe from this group, send email to reviewboard+unsubscribe@*
*googlegroups.com reviewboard+unsubscr...@googlegroups.com
For more options, visit this group at http://groups.google.com/**
group/reviewboard?hl=enhttp://groups.google.com/group/reviewboard?hl=en

Re: Not able to use post-review

The version of diff it's using is not GNU diff. It sounds like it's some
other diff tool, possibly from collabnet or Tortoise SVN. You need exactly
GNU diff.

Christian

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

On Mon, Aug 6, 2012 at 3:25 AM, Vijay Kumar Tiwary
vijayktiw...@gmail.comwrote:

Hi ,
As a workaround I guess ... I have put all the diff exe to the folder
where I have installed post-review . Then this error goes off .Now I am
getting another error.
if i give command like post-review --server http:\\10.117.8.9 --debug ,i
am getting following error

RBTools 0.4.1
Home = C:\Users\vtiwary\AppData\Roaming
HTTP GETting api/
HTTP GETting http://10.117.8.9/api/info/
Using the new web API
Failed to execute command: ['svn', 'diff', '--diff-cmd=diff']
['Index: .\n',
'
===\n', '--- .\t(revision 206)\n', '+++ .\t(working copy)\n', '\n',
'Property ch
anges on: .\n',
'___
\n', 'Added: reviewboard:url\n', '## -0,0 +1 ##\n', '+
http://10.117.2.8\n',
'\\ No newline at end of property\n', 'Index: IMessageCallback.cs\n',
'=
==\n', 'Usage:
diff [OPT
IONS] file1 file2\n', '\n', 'Display the differences between file1
and fi
le2 in unified diff\n', format. OPTIONS are diff extensions as
described by '
svn help diff'.\n, Use '--' alone to indicate that no more options
follow.\n,
svn: E200012: 'diff' returned 2\n]

How can i get rid of this error .

Thanks
Vijay

On Monday, 6 August 2012 15:51:33 UTC+5:30, Vijay Kumar Tiwary wrote:

Hi ,
Following is the output diff --version.

Usage: diff [OPTIONS] file1 file2

Display the differences between file1 and file2 in unified diff
format. OPTIONS are diff extensions as described by 'svn help diff'.
Use '--' alone to indicate that no more options follow.

Regards
Vijay

On Monday, 6 August 2012 12:52:54 UTC+5:30, Christian Hammond wrote:

Can you run diff --version and show me the output?

Christian

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

On Sun, Aug 5, 2012 at 11:49 PM, Vijay Kumar Tiwary
vijayktiw...@gmail.com wrote:

Hi Christian,
Thanks for the reply . I an getting following error when i typed
post-review in cmd

GNU diff is required for Subversion repositories. Make sure it is
installed
and in the path.

On Windows, you can install this from:
http://gnuwin32.sourceforge.**net/packages/diffutils.htmhttp://gnuwin32.sourceforge.net/packages/diffutils.htm

I have already installed GNU Diff and set in the Path .

For your information,I have also installed both Tortoise SVN as well as
collabnet subversion client .

Please suggest how to proceed further.

Regards
Vijay

On Saturday, 4 August 2012 00:29:03 UTC+5:30, Christian Hammond wrote:

Hi,

Can you describe to me how GNU diff isn't working? What are you seeing?

Christian

On Aug 3, 2012, at 2:50, Vijay Kumar Tiwary vijayktiw...@gmail.com
wrote:

Hi ,
I was trying to use post-review filename but I get following error
always :

GNU diff is required for Subversion repositories. Make sure it is
installed
and in the path.
On Windows, you can install this from:
http://gnuwin32.sourceforge.**ne**t/packages/diffutils.htmdhttp://gnuwin32.sourceforge.net/packages/diffutils.htm%22d

I have installed GNU Diff and even post-review installation seems to
work but some how GNU diff is not working.

i debuged and type following command SVN info and it is giving me the
configured SVN repo browser value

Please suggest how to configure .

For your information I setup Review Board on Linux machine and
configured SVN which is hosted on Windows machine . I am using my Windows
client machine to use review board and trying to post review but
apparently
got stuck with GNU diff ...Please help me to come out from here !!!

Regards
Vijay

--
Want to help the Review Board project? Donate today at
http://www.reviewboard.org/**don**ate/http://www.reviewboard.org/donate/
Happy user? Let us know at
http://www.reviewboard.org/**use**rs/http://www.reviewboard.org/users/
-~--~~~~--~~--~--~---
To unsubscribe from this group, send email to reviewboard+unsubscribe@
**google**groups.com reviewboard+unsubscr...@googlegroups.com
For more options, visit this group at http://groups.google.com/**group
**/reviewboard?hl=enhttp://groups.google.com/group/reviewboard?hl=en

Re: Not able to use post-review

Hi Christian,

But I have installed the GNU Diff . How can I override the property
and use GNU Diff . Do you have idea ?


Regards
Vijay

On Mon, Aug 6, 2012 at 3:59 PM, Christian Hammond chip...@chipx86.com wrote:
 The version of diff it's using is not GNU diff. It sounds like it's some
 other diff tool, possibly from collabnet or Tortoise SVN. You need exactly
 GNU diff.

 Christian

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


 On Mon, Aug 6, 2012 at 3:25 AM, Vijay Kumar Tiwary vijayktiw...@gmail.com
 wrote:

 Hi ,
 As a workaround I guess ... I have put all the diff exe to the folder
 where I have installed post-review . Then this error goes off .Now I am
 getting another error.
  if i give command like post-review --server http:\\10.117.8.9  --debug ,i
 am getting following error

  RBTools 0.4.1
  Home = C:\Users\vtiwary\AppData\Roaming
  HTTP GETting api/
  HTTP GETting http://10.117.8.9/api/info/
  Using the new web API
 Failed to execute command: ['svn', 'diff', '--diff-cmd=diff']
 ['Index: .\n',
 '
 ===\n', '--- .\t(revision 206)\n', '+++ .\t(working copy)\n', '\n',
 'Property ch
 anges on: .\n',
 '___
 \n', 'Added: reviewboard:url\n', '## -0,0 +1 ##\n',
 '+http://10.117.2.8\n',
 '\\ No newline at end of property\n', 'Index: IMessageCallback.cs\n',
 '=
 ==\n', 'Usage:
 diff [OPT
 IONS] file1 file2\n', '\n', 'Display the differences between file1
 and fi
 le2 in unified diff\n', format.  OPTIONS are diff extensions as
 described by '
 svn help diff'.\n, Use '--' alone to indicate that no more options
 follow.\n,
  svn: E200012: 'diff' returned 2\n]

 How can i get rid of this error .

 Thanks
 Vijay


 On Monday, 6 August 2012 15:51:33 UTC+5:30, Vijay Kumar Tiwary wrote:

 Hi ,
 Following is the output  diff --version.

 Usage: diff [OPTIONS] file1 file2

 Display the differences between file1 and file2 in unified diff
 format.  OPTIONS are diff extensions as described by 'svn help diff'.
 Use '--' alone to indicate that no more options follow.

 Regards
 Vijay

 On Monday, 6 August 2012 12:52:54 UTC+5:30, Christian Hammond wrote:

 Can you run diff --version and show me the output?

 Christian

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


 On Sun, Aug 5, 2012 at 11:49 PM, Vijay Kumar Tiwary
 vijayktiw...@gmail.com wrote:

 Hi Christian,
 Thanks for the reply . I an getting following error when i typed
 post-review in cmd


 GNU diff is required for Subversion repositories. Make sure it is
 installed
 and in the path.

 On Windows, you can install this from:
 http://gnuwin32.sourceforge.net/packages/diffutils.htm;

 I have already installed GNU Diff and set in the Path .

 For your information,I have also installed both Tortoise SVN as well as
 collabnet subversion client .


 Please suggest how to proceed further.



 Regards
 Vijay


 On Saturday, 4 August 2012 00:29:03 UTC+5:30, Christian Hammond wrote:

 Hi,

 Can you describe to me how GNU diff isn't working? What are you
 seeing?

 Christian



 On Aug 3, 2012, at 2:50, Vijay Kumar Tiwary vijayktiw...@gmail.com
 wrote:

 Hi ,
 I was trying to use post-review filename but I get following error
 always :

 GNU diff is required for Subversion repositories. Make sure it is
 installed
 and in the path.
 On Windows, you can install this from:
 http://gnuwin32.sourceforge.net/packages/diffutils.htmd

 I have installed GNU Diff and even post-review installation seems to
 work but some how GNU diff is not working.


 i debuged and type following command SVN info and it is giving me the
 configured SVN repo browser value

 Please suggest how to configure .


 For your information I setup Review Board on Linux machine and
 configured SVN which is hosted on Windows machine . I am using my Windows
 client machine to use review board and trying to post review but 
 apparently
 got stuck with GNU diff ...Please help me to come out from here !!!

 Regards
 Vijay




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

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

Re: Not able to use post-review

It's going to be whatever is in the path first. When you run diff --version
with GNU diff, you should see something like:

diff (GNU diffutils) 3.0
Copyright (C) 2010 Free Software Foundation, Inc.
License GPLv3+: GNU GPL version 3 or later http://gnu.org/licenses/gpl.html
.
This is free software: you are free to change and redistribute it.
There is NO WARRANTY, to the extent permitted by law.

Written by Paul Eggert, Mike Haertel, David Hayes,
Richard Stallman, and Len Tower.


All I can say is, make sure you have your path and the executables all set
correctly. There's not much I can really do other than suggest that.

Christian

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


On Mon, Aug 6, 2012 at 3:41 AM, Vijay Kumar Tiwary
vijayktiw...@gmail.comwrote:

 Hi Christian,

 But I have installed the GNU Diff . How can I override the property
 and use GNU Diff . Do you have idea ?


 Regards
 Vijay

 On Mon, Aug 6, 2012 at 3:59 PM, Christian Hammond chip...@chipx86.com
 wrote:
  The version of diff it's using is not GNU diff. It sounds like it's some
  other diff tool, possibly from collabnet or Tortoise SVN. You need
 exactly
  GNU diff.
 
  Christian
 
  --
  Christian Hammond - chip...@chipx86.com
  Review Board - http://www.reviewboard.org
  VMware, Inc. - http://www.vmware.com
 
 
  On Mon, Aug 6, 2012 at 3:25 AM, Vijay Kumar Tiwary 
 vijayktiw...@gmail.com
  wrote:
 
  Hi ,
  As a workaround I guess ... I have put all the diff exe to the folder
  where I have installed post-review . Then this error goes off .Now I am
  getting another error.
   if i give command like post-review --server http:\\10.117.8.9  --debug
 ,i
  am getting following error
 
   RBTools 0.4.1
   Home = C:\Users\vtiwary\AppData\Roaming
   HTTP GETting api/
   HTTP GETting http://10.117.8.9/api/info/
   Using the new web API
  Failed to execute command: ['svn', 'diff', '--diff-cmd=diff']
  ['Index: .\n',
  '
  ===\n', '--- .\t(revision 206)\n', '+++ .\t(working copy)\n', '\n',
  'Property ch
  anges on: .\n',
  '___
  \n', 'Added: reviewboard:url\n', '## -0,0 +1 ##\n',
  '+http://10.117.2.8\n',
  '\\ No newline at end of property\n', 'Index: IMessageCallback.cs\n',
  '=
  ==\n', 'Usage:
  diff [OPT
  IONS] file1 file2\n', '\n', 'Display the differences between file1
  and fi
  le2 in unified diff\n', format.  OPTIONS are diff extensions as
  described by '
  svn help diff'.\n, Use '--' alone to indicate that no more options
  follow.\n,
   svn: E200012: 'diff' returned 2\n]
 
  How can i get rid of this error .
 
  Thanks
  Vijay
 
 
  On Monday, 6 August 2012 15:51:33 UTC+5:30, Vijay Kumar Tiwary wrote:
 
  Hi ,
  Following is the output  diff --version.
 
  Usage: diff [OPTIONS] file1 file2
 
  Display the differences between file1 and file2 in unified diff
  format.  OPTIONS are diff extensions as described by 'svn help diff'.
  Use '--' alone to indicate that no more options follow.
 
  Regards
  Vijay
 
  On Monday, 6 August 2012 12:52:54 UTC+5:30, Christian Hammond wrote:
 
  Can you run diff --version and show me the output?
 
  Christian
 
  --
  Christian Hammond - chip...@chipx86.com
  Review Board - http://www.reviewboard.org
  VMware, Inc. - http://www.vmware.com
 
 
  On Sun, Aug 5, 2012 at 11:49 PM, Vijay Kumar Tiwary
  vijayktiw...@gmail.com wrote:
 
  Hi Christian,
  Thanks for the reply . I an getting following error when i typed
  post-review in cmd
 
 
  GNU diff is required for Subversion repositories. Make sure it is
  installed
  and in the path.
 
  On Windows, you can install this from:
  http://gnuwin32.sourceforge.net/packages/diffutils.htm;
 
  I have already installed GNU Diff and set in the Path .
 
  For your information,I have also installed both Tortoise SVN as well
 as
  collabnet subversion client .
 
 
  Please suggest how to proceed further.
 
 
 
  Regards
  Vijay
 
 
  On Saturday, 4 August 2012 00:29:03 UTC+5:30, Christian Hammond
 wrote:
 
  Hi,
 
  Can you describe to me how GNU diff isn't working? What are you
  seeing?
 
  Christian
 
 
 
  On Aug 3, 2012, at 2:50, Vijay Kumar Tiwary vijayktiw...@gmail.com
 
  wrote:
 
  Hi ,
  I was trying to use post-review filename but I get following error
  always :
 
  GNU diff is required for Subversion repositories. Make sure it is
  installed
  and in the path.
  On Windows, you can install this from:
  http://gnuwin32.sourceforge.net/packages/diffutils.htmd
 
  I have installed GNU Diff and even post-review installation seems to
  work but some how GNU diff is not working.
 
 
  i debuged and type following command SVN info and it is giving me
 the
  configured SVN repo browser value
 
  Please suggest how to configure .
 
 
  For your information I setup Review Board on Linux

Re: Not able to use post-review

2012-08-06 Thread Michał Muszyński

Hi Vijay,

Let's try few things.

1. Check if cmd finds 'diff' command. Send us the output of 'where
diff'. This should basically point to your GNU diff installation
directory ('C:\Program Files (x86)\GnuWin32\bin\diff.exe' in my
system). If it doesn't, add this path at the beginning of system
variable 'Path', run a new instance of cmd and execute 'where diff'
again.

2. Go to %appdata%\Subversion and open 'config' file in text editor.
Check if diff tool is specified in this file (line should look like:
diff-cmd=something).


On 6 August 2012 12:41, Vijay Kumar Tiwary vijayktiw...@gmail.com wrote:
 Hi Christian,

 But I have installed the GNU Diff . How can I override the property
 and use GNU Diff . Do you have idea ?

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

URGENT: Review Board still not building (was Re: Review Board 1.7 beta 1 released)

On Fri, 2012-08-03 at 15:56 -0400, Stephen Gallagher wrote:
 On Fri, 2012-08-03 at 12:31 -0700, Christian Hammond wrote:
  Yeah, the pipeline (django_pipeline) module. Guessing we left it out of the 
  dependency list for Djblets (though RB has it).
  
  I can fix this in a couple hours, but I imagine that's not fully necessary 
  on fedora once you have a package for it?
 
 Yeah, looks like I'm going to have to package that up first, since it
 doesn't appear to be available in Fedora yet.
 
 Ah well, one step at a time.
 

Ok, finally got django_pipeline accepted into Fedora today.
Unfortunately, now I'm having a compilation problem with ReviewBoard:

Note: TypeError: compile_file() got an unexpected keyword argument
'outdated'

This is with the ReviewBoard 1.7beta1 tarball and django_pipeline
1.2.14.

Any help would be appreciated. I need to get this in today to make the
Fedora 18 deadline :(


Full traceback:

Traceback (most recent call last):
  File ./contrib/internal/build-media.py, line 19, in module
ret = call_command('collectstatic', interactive=False, verbosity=2)
  File
/usr/lib/python2.7/site-packages/django/core/management/__init__.py,
line 150, in call_command
return klass.execute(*args, **defaults)
  File
/usr/lib/python2.7/site-packages/django/core/management/base.py, line
232, in execute
output = self.handle(*args, **options)
  File
/usr/lib/python2.7/site-packages/django/core/management/base.py, line
371, in handle
return self.handle_noargs(**options)
  File
/usr/lib/python2.7/site-packages/django/contrib/staticfiles/management/commands/collectstatic.py,
 line 163, in handle_noargs
collected = self.collect()
  File
/usr/lib/python2.7/site-packages/django/contrib/staticfiles/management/commands/collectstatic.py,
 line 119, in collect
dry_run=self.dry_run)
  File /usr/lib/python2.7/site-packages/pipeline/storage.py, line 30,
in post_process
packager.pack_stylesheets(package)
  File /usr/lib/python2.7/site-packages/pipeline/packager.py, line 90,
in pack_stylesheets
variant=package.variant, **kwargs)
  File /usr/lib/python2.7/site-packages/pipeline/packager.py, line 99,
in pack
paths = self.compile(package.paths, force=True)
  File /usr/lib/python2.7/site-packages/pipeline/packager.py, line 93,
in compile
return self.compiler.compile(paths, force=force)
  File
/usr/lib/python2.7/site-packages/pipeline/compilers/__init__.py, line
41, in compile
compiler.compile_file(infile, outfile, outdated=outdated,
force=force)
TypeError: compile_file() got an unexpected keyword argument 'outdated'
Traceback (most recent call last):
  File setup.py, line 182, in module
Topic :: Software Development :: Quality Assurance,
  File /usr/lib64/python2.7/distutils/core.py, line 152, in setup
dist.run_commands()
  File /usr/lib64/python2.7/distutils/dist.py, line 953, in
run_commands
self.run_command(cmd)
  File /usr/lib64/python2.7/distutils/dist.py, line 972, in
run_command
cmd_obj.run()
  File /usr/lib/python2.7/site-packages/setuptools/command/install.py,
line 53, in run
return _install.run(self)
  File /usr/lib64/python2.7/distutils/command/install.py, line 575, in
run
self.run_command(cmd_name)
  File /usr/lib64/python2.7/distutils/cmd.py, line 326, in run_command
self.distribution.run_command(command)
  File /usr/lib64/python2.7/distutils/dist.py, line 972, in
run_command
cmd_obj.run()
  File
/usr/lib/python2.7/site-packages/setuptools/command/install_egg_info.py, line 
29, in run
self.run_command('egg_info')
  File /usr/lib64/python2.7/distutils/cmd.py, line 326, in run_command
self.distribution.run_command(command)
  File /usr/lib64/python2.7/distutils/dist.py, line 972, in
run_command
cmd_obj.run()
  File setup.py, line 63, in run
self.run_command('build_media')
  File /usr/lib64/python2.7/distutils/cmd.py, line 326, in run_command
self.distribution.run_command(command)
  File /usr/lib64/python2.7/distutils/dist.py, line 972, in
run_command
cmd_obj.run()
  File setup.py, line 81, in run
raise RuntimeError('Failed to build media files')
RuntimeError: Failed to build media files
error: Bad exit status from /var/tmp/rpm-tmp.dvLLKA (%install)


-- 
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: URGENT: Review Board still not building (was Re: Review Board 1.7 beta 1 released)

There's a few things that it seems are broken with newer Pipelines. I know
for sure that 1.2.1 works. Any way you could just package that for now?
Looking into the remaining things, but there's some ugly issue I'm not
completely sure about yet.

Christian

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


On Mon, Aug 6, 2012 at 1:31 PM, Stephen Gallagher step...@gallagherhome.com
 wrote:

 On Fri, 2012-08-03 at 15:56 -0400, Stephen Gallagher wrote:
  On Fri, 2012-08-03 at 12:31 -0700, Christian Hammond wrote:
   Yeah, the pipeline (django_pipeline) module. Guessing we left it out
 of the dependency list for Djblets (though RB has it).
  
   I can fix this in a couple hours, but I imagine that's not fully
 necessary on fedora once you have a package for it?
 
  Yeah, looks like I'm going to have to package that up first, since it
  doesn't appear to be available in Fedora yet.
 
  Ah well, one step at a time.
 

 Ok, finally got django_pipeline accepted into Fedora today.
 Unfortunately, now I'm having a compilation problem with ReviewBoard:

 Note: TypeError: compile_file() got an unexpected keyword argument
 'outdated'

 This is with the ReviewBoard 1.7beta1 tarball and django_pipeline
 1.2.14.

 Any help would be appreciated. I need to get this in today to make the
 Fedora 18 deadline :(


 Full traceback:

 Traceback (most recent call last):
   File ./contrib/internal/build-media.py, line 19, in module
 ret = call_command('collectstatic', interactive=False, verbosity=2)
   File
 /usr/lib/python2.7/site-packages/django/core/management/__init__.py,
 line 150, in call_command
 return klass.execute(*args, **defaults)
   File
 /usr/lib/python2.7/site-packages/django/core/management/base.py, line
 232, in execute
 output = self.handle(*args, **options)
   File
 /usr/lib/python2.7/site-packages/django/core/management/base.py, line
 371, in handle
 return self.handle_noargs(**options)
   File
 /usr/lib/python2.7/site-packages/django/contrib/staticfiles/management/commands/collectstatic.py,
 line 163, in handle_noargs
 collected = self.collect()
   File
 /usr/lib/python2.7/site-packages/django/contrib/staticfiles/management/commands/collectstatic.py,
 line 119, in collect
 dry_run=self.dry_run)
   File /usr/lib/python2.7/site-packages/pipeline/storage.py, line 30,
 in post_process
 packager.pack_stylesheets(package)
   File /usr/lib/python2.7/site-packages/pipeline/packager.py, line 90,
 in pack_stylesheets
 variant=package.variant, **kwargs)
   File /usr/lib/python2.7/site-packages/pipeline/packager.py, line 99,
 in pack
 paths = self.compile(package.paths, force=True)
   File /usr/lib/python2.7/site-packages/pipeline/packager.py, line 93,
 in compile
 return self.compiler.compile(paths, force=force)
   File
 /usr/lib/python2.7/site-packages/pipeline/compilers/__init__.py, line
 41, in compile
 compiler.compile_file(infile, outfile, outdated=outdated,
 force=force)
 TypeError: compile_file() got an unexpected keyword argument 'outdated'
 Traceback (most recent call last):
   File setup.py, line 182, in module
 Topic :: Software Development :: Quality Assurance,
   File /usr/lib64/python2.7/distutils/core.py, line 152, in setup
 dist.run_commands()
   File /usr/lib64/python2.7/distutils/dist.py, line 953, in
 run_commands
 self.run_command(cmd)
   File /usr/lib64/python2.7/distutils/dist.py, line 972, in
 run_command
 cmd_obj.run()
   File /usr/lib/python2.7/site-packages/setuptools/command/install.py,
 line 53, in run
 return _install.run(self)
   File /usr/lib64/python2.7/distutils/command/install.py, line 575, in
 run
 self.run_command(cmd_name)
   File /usr/lib64/python2.7/distutils/cmd.py, line 326, in run_command
 self.distribution.run_command(command)
   File /usr/lib64/python2.7/distutils/dist.py, line 972, in
 run_command
 cmd_obj.run()
   File
 /usr/lib/python2.7/site-packages/setuptools/command/install_egg_info.py,
 line 29, in run
 self.run_command('egg_info')
   File /usr/lib64/python2.7/distutils/cmd.py, line 326, in run_command
 self.distribution.run_command(command)
   File /usr/lib64/python2.7/distutils/dist.py, line 972, in
 run_command
 cmd_obj.run()
   File setup.py, line 63, in run
 self.run_command('build_media')
   File /usr/lib64/python2.7/distutils/cmd.py, line 326, in run_command
 self.distribution.run_command(command)
   File /usr/lib64/python2.7/distutils/dist.py, line 972, in
 run_command
 cmd_obj.run()
   File setup.py, line 81, in run
 raise RuntimeError('Failed to build media files')
 RuntimeError: Failed to build media files
 error: Bad exit status from /var/tmp/rpm-tmp.dvLLKA (%install)


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

Re: URGENT: Review Board still not building (was Re: Review Board 1.7 beta 1 released)

On Mon, 2012-08-06 at 15:18 -0700, Christian Hammond wrote:
 There's a few things that it seems are broken with newer Pipelines. I
 know for sure that 1.2.1 works. Any way you could just package that
 for now? Looking into the remaining things, but there's some ugly
 issue I'm not completely sure about yet.

Ugh, unfortunately not. It's a real pain to revert a version in Fedora.
(It can be done, such as when regressions necessitate it), but it's
never done lightly. It requires mangling the version numbers in a
particularly ugly way to ensure that anyone who happened to install the
previous package will still update to the new one (despite the version
being lower).

I don't suppose you could point me at specific patches that I could
revert in the django_pipeline 1.2.14 release instead that would get it
working in the meantime?

-- 
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: URGENT: Review Board still not building (was Re: Review Board 1.7 beta 1 released)

I have no idea. I fixed a few things (they changed the API around a bit,
breaking non-bundled media compilers), but even after fixing that, I'm
getting a much more concerning error. I'll try to track it down and see if
we can fix it, but if not, it'll need to be fixed in pipeline.

Given the API changes, I'd be concerned about reverting code in pipeline
for the package.

I'll keep you informed as I learn more.

Christian

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


On Mon, Aug 6, 2012 at 4:32 PM, Stephen Gallagher step...@gallagherhome.com
 wrote:

 On Mon, 2012-08-06 at 15:18 -0700, Christian Hammond wrote:
  There's a few things that it seems are broken with newer Pipelines. I
  know for sure that 1.2.1 works. Any way you could just package that
  for now? Looking into the remaining things, but there's some ugly
  issue I'm not completely sure about yet.

 Ugh, unfortunately not. It's a real pain to revert a version in Fedora.
 (It can be done, such as when regressions necessitate it), but it's
 never done lightly. It requires mangling the version numbers in a
 particularly ugly way to ensure that anyone who happened to install the
 previous package will still update to the new one (despite the version
 being lower).

 I don't suppose you could point me at specific patches that I could
 revert in the django_pipeline 1.2.14 release instead that would get it
 working in the meantime?

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


-- 
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: URGENT: Review Board still not building (was Re: Review Board 1.7 beta 1 released)

This branch has what we need on the djblets side:

https://github.com/chipx86/djblets/tree/pipeline-fix

Unless there's just something messed up here, you should hit problems with
pipeline not finding files in the PipelineCachedStorage. I traced the first
occurrence to revision dd2eed6e51bcfa4352b95a12e8f43c0feaca8819, but even
if I revert that on master, I still hit some issues. Can you confirm?

Christian

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


On Mon, Aug 6, 2012 at 4:46 PM, Stephen Gallagher step...@gallagherhome.com
 wrote:

 On Mon, 2012-08-06 at 16:33 -0700, Christian Hammond wrote:
  I have no idea. I fixed a few things (they changed the API around a
  bit, breaking non-bundled media compilers), but even after fixing
  that, I'm getting a much more concerning error. I'll try to track it
  down and see if we can fix it, but if not, it'll need to be fixed in
  pipeline.
 
  Given the API changes, I'd be concerned about reverting code in
  pipeline for the package.
 
  I'll keep you informed as I learn more.

 Thanks, I appreciate it. I'm attacking it on my end as well. Do you have
 a github with your in-progress changes I could sync to, so we don't
 duplicate effort?

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


-- 
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: URGENT: Review Board still not building (was Re: Review Board 1.7 beta 1 released)

On Mon, 2012-08-06 at 16:54 -0700, Christian Hammond wrote:
 This branch has what we need on the djblets side:
 
 https://github.com/chipx86/djblets/tree/pipeline-fix
 
 Unless there's just something messed up here, you should hit problems
 with pipeline not finding files in the PipelineCachedStorage. I traced
 the first occurrence to revision
 dd2eed6e51bcfa4352b95a12e8f43c0feaca8819, but even if I revert that on
 master, I still hit some issues. Can you confirm?
 

I get the following traceback with your bless patch (as expected):

Traceback (most recent call last):
  File ./contrib/internal/build-media.py, line 19, in module
ret = call_command('collectstatic', interactive=False, verbosity=2)
  File
/usr/lib/python2.7/site-packages/django/core/management/__init__.py,
line 150, in call_command
return klass.execute(*args, **defaults)
  File
/usr/lib/python2.7/site-packages/django/core/management/base.py, line
232, in execute
output = self.handle(*args, **options)
  File
/usr/lib/python2.7/site-packages/django/core/management/base.py, line
371, in handle
return self.handle_noargs(**options)
  File
/usr/lib/python2.7/site-packages/django/contrib/staticfiles/management/commands/collectstatic.py,
 line 163, in handle_noargs
collected = self.collect()
  File
/usr/lib/python2.7/site-packages/django/contrib/staticfiles/management/commands/collectstatic.py,
 line 119, in collect
dry_run=self.dry_run)
  File /usr/lib/python2.7/site-packages/pipeline/storage.py, line 30,
in post_process
packager.pack_stylesheets(package)
  File /usr/lib/python2.7/site-packages/pipeline/packager.py, line 90,
in pack_stylesheets
variant=package.variant, **kwargs)
  File /usr/lib/python2.7/site-packages/pipeline/packager.py, line
100, in pack
content = compress(paths, **kwargs)
  File
/usr/lib/python2.7/site-packages/pipeline/compressors/__init__.py,
line 72, in compress_css
css = self.concatenate_and_rewrite(paths, output_filename, variant)
  File
/usr/lib/python2.7/site-packages/pipeline/compressors/__init__.py,
line 135, in concatenate_and_rewrite
content = self.read_file(path)
  File
/usr/lib/python2.7/site-packages/pipeline/compressors/__init__.py,
line 208, in read_file
file = default_storage.open(path, 'rb')
  File /usr/lib/python2.7/site-packages/django/core/files/storage.py,
line 33, in open
return self._open(name, mode)
  File /usr/lib/python2.7/site-packages/pipeline/storage.py, line 114,
in _open
storage = self.find_storage(name)
  File /usr/lib/python2.7/site-packages/pipeline/storage.py, line 111,
in find_storage
raise ValueError(The file '%s' could not be found with %r. %
(name, self))
ValueError: The file 'rb/css/diffviewer.css' could not be found with
pipeline.storage.PipelineFinderStorage object at 0x1aeaad0.
Traceback (most recent call last):
  File setup.py, line 182, in module
Topic :: Software Development :: Quality Assurance,
  File /usr/lib64/python2.7/distutils/core.py, line 152, in setup
dist.run_commands()
  File /usr/lib64/python2.7/distutils/dist.py, line 953, in
run_commands
self.run_command(cmd)
  File /usr/lib64/python2.7/distutils/dist.py, line 972, in
run_command
cmd_obj.run()
  File /usr/lib/python2.7/site-packages/setuptools/command/install.py,
line 53, in run
return _install.run(self)
  File /usr/lib64/python2.7/distutils/command/install.py, line 575, in
run
self.run_command(cmd_name)
  File /usr/lib64/python2.7/distutils/cmd.py, line 326, in run_command
self.distribution.run_command(command)
  File /usr/lib64/python2.7/distutils/dist.py, line 972, in
run_command
cmd_obj.run()
  File
/usr/lib/python2.7/site-packages/setuptools/command/install_egg_info.py, line 
29, in run
self.run_command('egg_info')
  File /usr/lib64/python2.7/distutils/cmd.py, line 326, in run_command
self.distribution.run_command(command)
  File /usr/lib64/python2.7/distutils/dist.py, line 972, in
run_command
cmd_obj.run()
  File setup.py, line 63, in run
self.run_command('build_media')
  File /usr/lib64/python2.7/distutils/cmd.py, line 326, in run_command
self.distribution.run_command(command)
  File /usr/lib64/python2.7/distutils/dist.py, line 972, in
run_command
cmd_obj.run()
  File setup.py, line 81, in run
raise RuntimeError('Failed to build media files')
RuntimeError: Failed to build media files
error: Bad exit status from /var/tmp/rpm-tmp.pmGIf8 (%install)



Looking at the pipeline commit you referenced, I can understand why it's
still having issue. That patch is just an exception-catcher for an
underlying issue that they must have introduced earlier. I'm going to
see if I can spot an earlier patch that may have introduced the issue
with writing to a static root (unless that's an assumption that Review
Board is making that it should not be).





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

Re: URGENT: Review Board still not building (was Re: Review Board 1.7 beta 1 released)

On Mon, Aug 6, 2012 at 6:38 PM, Stephen Gallagher step...@gallagherhome.com
 wrote:

 On Mon, 2012-08-06 at 20:50 -0400, Stephen Gallagher wrote:
  On Mon, 2012-08-06 at 17:27 -0700, Christian Hammond wrote:
   Going further, if that change is reverted, the next failure seems to
   come from when they redid the compiler API. What's odd is that the
   code doesn't look like it should be an issue, but the bisects keep
   pointing to it. This is revision
   96b356d7c9a8e5626b4ce018577984b89c7f2178.
 
  From further investigation, it looks like that patch was completely
  broken until they added both c1ff4867fe57a9fffbddae38b51d888c85304a87
  and fb51fba3f391a9685f4ab65757e7511e3f927df8 to fix it properly. So
  we're probably getting a false-positive from git bisect because those
  two patches are also not included. Maybe a quick 'git rebase -i' to
  merge those into 96b356d7c9a8e5626b4ce018577984b89c7f2178 will allow git
  bisect to find the real culprit?
 
  The only other thing I noticed was that they did change behavior in one
  place in that code. In Compiler.compile(), they were previously passing
  output_path to the compilers as the output argument, but this patch
  changes that to be output_file as determined by finders.find(). Being
  unfamiliar with the code and the return type of finders.find(), I can't
  say whether this is relevant.
 

 Thanks very much for your help on this Christian, but at this point
 (with my deadline running close and the hour growing late), I'm just
 going to go the downgrade route and treat this as a life-lesson: always
 test version compatibility before making a release.

 Thanks again, and if we find a solution for this later on, I can still
 upgrade django_pipeline back up to a recent release. I'll just be
 carrying the annoying epoch version-mangle forever. C'est la vie.


Sorry man :( That's my life-lesson. I had thought our servers and nightly
builds upgraded to django-pipeline along with the release, but no,
apparently not.

I'll keep poking at this and talk to the developer. He's a good guy and has
helped with these regressions before, so we'll get it figured out.

Thanks for all your help on this. Really sorry for the trouble.

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: URGENT: Review Board still not building (was Re: Review Board 1.7 beta 1 released)

On Mon, 2012-08-06 at 18:48 -0700, Christian Hammond wrote:
 
 
 
 Sorry man :( That's my life-lesson. I had thought our servers and
 nightly builds upgraded to django-pipeline along with the release, but
 no, apparently not.
 
 I'll keep poking at this and talk to the developer. He's a good guy
 and has helped with these regressions before, so we'll get it figured
 out.
 
 Thanks for all your help on this. Really sorry for the trouble.


Actually, in a lucky comedy of errors, an unrelated glitch in the Fedora
servers meant that my 1.2.14 package never actually made it out to any
of the Fedora mirrors. So I got lucky and was able to just yank it out
of the queue and I'll be able to avoid the version-mangling for now.

So all's well that ends well, I suppose. If you and the pipeline
upstream do sort things out though, I'd be appreciative of a heads-up so
I can rev the version back up.

-- 
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: URGENT: Review Board still not building (was Re: Review Board 1.7 beta 1 released)

On Mon, 2012-08-06 at 22:00 -0400, Stephen Gallagher wrote:
 On Mon, 2012-08-06 at 18:48 -0700, Christian Hammond wrote:
  
  
  
  Sorry man :( That's my life-lesson. I had thought our servers and
  nightly builds upgraded to django-pipeline along with the release, but
  no, apparently not.
  
  I'll keep poking at this and talk to the developer. He's a good guy
  and has helped with these regressions before, so we'll get it figured
  out.
  
  Thanks for all your help on this. Really sorry for the trouble.
 
 
 Actually, in a lucky comedy of errors, an unrelated glitch in the Fedora
 servers meant that my 1.2.14 package never actually made it out to any
 of the Fedora mirrors. So I got lucky and was able to just yank it out
 of the queue and I'll be able to avoid the version-mangling for now.
 
 So all's well that ends well, I suppose. If you and the pipeline
 upstream do sort things out though, I'd be appreciative of a heads-up so
 I can rev the version back up.
 



One more problem :(

Now I'm seeing this traceback (failure to import jsmin.py). I assume
there's another package I need, but Google is failing me here.

Traceback (most recent call last):
  File ./contrib/internal/build-media.py, line 19, in module
ret = call_command('collectstatic', interactive=False, verbosity=2)
  File
/usr/lib/python2.7/site-packages/django/core/management/__init__.py,
line 150, in call_command
return klass.execute(*args, **defaults)
  File
/usr/lib/python2.7/site-packages/django/core/management/base.py, line
232, in execute
output = self.handle(*args, **options)
  File
/usr/lib/python2.7/site-packages/django/core/management/base.py, line
371, in handle
return self.handle_noargs(**options)
  File
/usr/lib/python2.7/site-packages/django/contrib/staticfiles/management/commands/collectstatic.py,
 line 163, in handle_noargs
collected = self.collect()
  File
/usr/lib/python2.7/site-packages/django/contrib/staticfiles/management/commands/collectstatic.py,
 line 119, in collect
dry_run=self.dry_run)
  File /usr/lib/python2.7/site-packages/pipeline/storage.py, line 30,
in post_process
output_file = packager.pack_javascripts(package)
  File /usr/lib/python2.7/site-packages/pipeline/packager.py, line
106, in pack_javascripts
return self.pack(package, self.compressor.compress_js,
js_compressed, templates=package.templates, **kwargs)
  File /usr/lib/python2.7/site-packages/pipeline/packager.py, line
100, in pack
content = compress(paths, **kwargs)
  File
/usr/lib/python2.7/site-packages/pipeline/compressors/__init__.py,
line 65, in compress_js
js = getattr(compressor(verbose=self.verbose), 'compress_js')(js)
  File
/usr/lib/python2.7/site-packages/pipeline/compressors/jsmin/__init__.py, line 
10, in compress_js
from jsmin import jsmin
ImportError: No module named jsmin
Traceback (most recent call last):
  File setup.py, line 182, in module
Topic :: Software Development :: Quality Assurance,
  File /usr/lib64/python2.7/distutils/core.py, line 152, in setup
dist.run_commands()
  File /usr/lib64/python2.7/distutils/dist.py, line 953, in
run_commands
self.run_command(cmd)
  File /usr/lib64/python2.7/distutils/dist.py, line 972, in
run_command
cmd_obj.run()
  File /usr/lib/python2.7/site-packages/setuptools/command/install.py,
line 53, in run
return _install.run(self)
  File /usr/lib64/python2.7/distutils/command/install.py, line 575, in
run
self.run_command(cmd_name)
  File /usr/lib64/python2.7/distutils/cmd.py, line 326, in run_command
self.distribution.run_command(command)
  File /usr/lib64/python2.7/distutils/dist.py, line 972, in
run_command
cmd_obj.run()
  File
/usr/lib/python2.7/site-packages/setuptools/command/install_egg_info.py, line 
29, in run
self.run_command('egg_info')
  File /usr/lib64/python2.7/distutils/cmd.py, line 326, in run_command
self.distribution.run_command(command)
  File /usr/lib64/python2.7/distutils/dist.py, line 972, in
run_command
cmd_obj.run()
  File setup.py, line 63, in run
self.run_command('build_media')
  File /usr/lib64/python2.7/distutils/cmd.py, line 326, in run_command
self.distribution.run_command(command)
  File /usr/lib64/python2.7/distutils/dist.py, line 972, in
run_command
cmd_obj.run()
  File setup.py, line 81, in run
raise RuntimeError('Failed to build media files')
RuntimeError: Failed to build media files
error: Bad exit status from /var/tmp/rpm-tmp.tnVeed (%install)


-- 
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: URGENT: Review Board still not building (was Re: Review Board 1.7 beta 1 released)

On Mon, 2012-08-06 at 22:07 -0400, Stephen Gallagher wrote:
 On Mon, 2012-08-06 at 22:00 -0400, Stephen Gallagher wrote:
  On Mon, 2012-08-06 at 18:48 -0700, Christian Hammond wrote:
   
   
   
   Sorry man :( That's my life-lesson. I had thought our servers and
   nightly builds upgraded to django-pipeline along with the release, but
   no, apparently not.
   
   I'll keep poking at this and talk to the developer. He's a good guy
   and has helped with these regressions before, so we'll get it figured
   out.
   
   Thanks for all your help on this. Really sorry for the trouble.
  
  
  Actually, in a lucky comedy of errors, an unrelated glitch in the Fedora
  servers meant that my 1.2.14 package never actually made it out to any
  of the Fedora mirrors. So I got lucky and was able to just yank it out
  of the queue and I'll be able to avoid the version-mangling for now.
  
  So all's well that ends well, I suppose. If you and the pipeline
  upstream do sort things out though, I'd be appreciative of a heads-up so
  I can rev the version back up.
  
 
 
 
 One more problem :(
 
 Now I'm seeing this traceback (failure to import jsmin.py). I assume
 there's another package I need, but Google is failing me here.
 
 Traceback (most recent call last):
   File ./contrib/internal/build-media.py, line 19, in module
 ret = call_command('collectstatic', interactive=False, verbosity=2)
   File
 /usr/lib/python2.7/site-packages/django/core/management/__init__.py,
 line 150, in call_command
 return klass.execute(*args, **defaults)
   File
 /usr/lib/python2.7/site-packages/django/core/management/base.py, line
 232, in execute
 output = self.handle(*args, **options)
   File
 /usr/lib/python2.7/site-packages/django/core/management/base.py, line
 371, in handle
 return self.handle_noargs(**options)
   File
 /usr/lib/python2.7/site-packages/django/contrib/staticfiles/management/commands/collectstatic.py,
  line 163, in handle_noargs
 collected = self.collect()
   File
 /usr/lib/python2.7/site-packages/django/contrib/staticfiles/management/commands/collectstatic.py,
  line 119, in collect
 dry_run=self.dry_run)
   File /usr/lib/python2.7/site-packages/pipeline/storage.py, line 30,
 in post_process
 output_file = packager.pack_javascripts(package)
   File /usr/lib/python2.7/site-packages/pipeline/packager.py, line
 106, in pack_javascripts
 return self.pack(package, self.compressor.compress_js,
 js_compressed, templates=package.templates, **kwargs)
   File /usr/lib/python2.7/site-packages/pipeline/packager.py, line
 100, in pack
 content = compress(paths, **kwargs)
   File
 /usr/lib/python2.7/site-packages/pipeline/compressors/__init__.py,
 line 65, in compress_js
 js = getattr(compressor(verbose=self.verbose), 'compress_js')(js)
   File
 /usr/lib/python2.7/site-packages/pipeline/compressors/jsmin/__init__.py, 
 line 10, in compress_js
 from jsmin import jsmin
 ImportError: No module named jsmin
 Traceback (most recent call last):
   File setup.py, line 182, in module
 Topic :: Software Development :: Quality Assurance,
   File /usr/lib64/python2.7/distutils/core.py, line 152, in setup
 dist.run_commands()
   File /usr/lib64/python2.7/distutils/dist.py, line 953, in
 run_commands
 self.run_command(cmd)
   File /usr/lib64/python2.7/distutils/dist.py, line 972, in
 run_command
 cmd_obj.run()
   File /usr/lib/python2.7/site-packages/setuptools/command/install.py,
 line 53, in run
 return _install.run(self)
   File /usr/lib64/python2.7/distutils/command/install.py, line 575, in
 run
 self.run_command(cmd_name)
   File /usr/lib64/python2.7/distutils/cmd.py, line 326, in run_command
 self.distribution.run_command(command)
   File /usr/lib64/python2.7/distutils/dist.py, line 972, in
 run_command
 cmd_obj.run()
   File
 /usr/lib/python2.7/site-packages/setuptools/command/install_egg_info.py, 
 line 29, in run
 self.run_command('egg_info')
   File /usr/lib64/python2.7/distutils/cmd.py, line 326, in run_command
 self.distribution.run_command(command)
   File /usr/lib64/python2.7/distutils/dist.py, line 972, in
 run_command
 cmd_obj.run()
   File setup.py, line 63, in run
 self.run_command('build_media')
   File /usr/lib64/python2.7/distutils/cmd.py, line 326, in run_command
 self.distribution.run_command(command)
   File /usr/lib64/python2.7/distutils/dist.py, line 972, in
 run_command
 cmd_obj.run()
   File setup.py, line 81, in run
 raise RuntimeError('Failed to build media files')
 RuntimeError: Failed to build media files
 error: Bad exit status from /var/tmp/rpm-tmp.tnVeed (%install)
 
 


*sigh* Okay, apparently django_pipeline has an undeclared runtime
dependency on http://pypi.python.org/pypi/jsmin/

Looks like I'll have to try to package that in the morning. Review Board
is probably not going to make it into the Fedora Alpha release.
Hopefully they'll let me sneak it in for Beta (since 1.6.x is FTBFS with

Re: URGENT: Review Board still not building (was Re: Review Board 1.7 beta 1 released)

On Aug 6, 2012, at 19:16, Stephen Gallagher step...@gallagherhome.com wrote:

 On Mon, 2012-08-06 at 22:07 -0400, Stephen Gallagher wrote:
 On Mon, 2012-08-06 at 22:00 -0400, Stephen Gallagher wrote:
 On Mon, 2012-08-06 at 18:48 -0700, Christian Hammond wrote:
 
 
 
 Sorry man :( That's my life-lesson. I had thought our servers and
 nightly builds upgraded to django-pipeline along with the release, but
 no, apparently not.
 
 I'll keep poking at this and talk to the developer. He's a good guy
 and has helped with these regressions before, so we'll get it figured
 out.
 
 Thanks for all your help on this. Really sorry for the trouble.
 
 
 Actually, in a lucky comedy of errors, an unrelated glitch in the Fedora
 servers meant that my 1.2.14 package never actually made it out to any
 of the Fedora mirrors. So I got lucky and was able to just yank it out
 of the queue and I'll be able to avoid the version-mangling for now.
 
 So all's well that ends well, I suppose. If you and the pipeline
 upstream do sort things out though, I'd be appreciative of a heads-up so
 I can rev the version back up.
 
 
 
 
 One more problem :(
 
 Now I'm seeing this traceback (failure to import jsmin.py). I assume
 there's another package I need, but Google is failing me here.
 
 Traceback (most recent call last):
  File ./contrib/internal/build-media.py, line 19, in module
ret = call_command('collectstatic', interactive=False, verbosity=2)
  File
 /usr/lib/python2.7/site-packages/django/core/management/__init__.py,
 line 150, in call_command
return klass.execute(*args, **defaults)
  File
 /usr/lib/python2.7/site-packages/django/core/management/base.py, line
 232, in execute
output = self.handle(*args, **options)
  File
 /usr/lib/python2.7/site-packages/django/core/management/base.py, line
 371, in handle
return self.handle_noargs(**options)
  File
 /usr/lib/python2.7/site-packages/django/contrib/staticfiles/management/commands/collectstatic.py,
  line 163, in handle_noargs
collected = self.collect()
  File
 /usr/lib/python2.7/site-packages/django/contrib/staticfiles/management/commands/collectstatic.py,
  line 119, in collect
dry_run=self.dry_run)
  File /usr/lib/python2.7/site-packages/pipeline/storage.py, line 30,
 in post_process
output_file = packager.pack_javascripts(package)
  File /usr/lib/python2.7/site-packages/pipeline/packager.py, line
 106, in pack_javascripts
return self.pack(package, self.compressor.compress_js,
 js_compressed, templates=package.templates, **kwargs)
  File /usr/lib/python2.7/site-packages/pipeline/packager.py, line
 100, in pack
content = compress(paths, **kwargs)
  File
 /usr/lib/python2.7/site-packages/pipeline/compressors/__init__.py,
 line 65, in compress_js
js = getattr(compressor(verbose=self.verbose), 'compress_js')(js)
  File
 /usr/lib/python2.7/site-packages/pipeline/compressors/jsmin/__init__.py, 
 line 10, in compress_js
from jsmin import jsmin
 ImportError: No module named jsmin
 Traceback (most recent call last):
  File setup.py, line 182, in module
Topic :: Software Development :: Quality Assurance,
  File /usr/lib64/python2.7/distutils/core.py, line 152, in setup
dist.run_commands()
  File /usr/lib64/python2.7/distutils/dist.py, line 953, in
 run_commands
self.run_command(cmd)
  File /usr/lib64/python2.7/distutils/dist.py, line 972, in
 run_command
cmd_obj.run()
  File /usr/lib/python2.7/site-packages/setuptools/command/install.py,
 line 53, in run
return _install.run(self)
  File /usr/lib64/python2.7/distutils/command/install.py, line 575, in
 run
self.run_command(cmd_name)
  File /usr/lib64/python2.7/distutils/cmd.py, line 326, in run_command
self.distribution.run_command(command)
  File /usr/lib64/python2.7/distutils/dist.py, line 972, in
 run_command
cmd_obj.run()
  File
 /usr/lib/python2.7/site-packages/setuptools/command/install_egg_info.py, 
 line 29, in run
self.run_command('egg_info')
  File /usr/lib64/python2.7/distutils/cmd.py, line 326, in run_command
self.distribution.run_command(command)
  File /usr/lib64/python2.7/distutils/dist.py, line 972, in
 run_command
cmd_obj.run()
  File setup.py, line 63, in run
self.run_command('build_media')
  File /usr/lib64/python2.7/distutils/cmd.py, line 326, in run_command
self.distribution.run_command(command)
  File /usr/lib64/python2.7/distutils/dist.py, line 972, in
 run_command
cmd_obj.run()
  File setup.py, line 81, in run
raise RuntimeError('Failed to build media files')
 RuntimeError: Failed to build media files
 error: Bad exit status from /var/tmp/rpm-tmp.tnVeed (%install)
 
 
 
 
 *sigh* Okay, apparently django_pipeline has an undeclared runtime
 dependency on http://pypi.python.org/pypi/jsmin/
 
 Looks like I'll have to try to package that in the morning. Review Board
 is probably not going to make it into the Fedora Alpha release.
 Hopefully they'll let me sneak it in for Beta (since 1.6.x is FTBFS with
 Django 1.4)

It's

Re: Issue 1388 in reviewboard: Visual Studio plugin/extension to run post-review/RBTools



Comment #5 on issue 1388 by ivan.gav...@gmail.com: Visual Studio  
plugin/extension to run post-review/RBTools

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

OMG sorry but this is terribly ugly and doesn't 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 2108 in reviewboard: Cannot run rbssh: No such file or directory; /usr/local/bin not in env path



Comment #2 on issue 2108 by ews...@gmail.com: Cannot run rbssh: No such  
file or directory;  /usr/local/bin not in env path

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

I had a similar problem with reviewboard errors on adding a private GIT  
repository. I'd see rbssh errors in httpd's error_log such as:



[Mon Aug 06 15:13:20 2012] [error] ERROR:root:Git: Failed to find valid  
repository ssh://g...@github.com/ourname/Common.git: Traceback (most  
recent call last):

[Mon Aug 06 15:13:20 2012] [error]   File /usr/bin/rbssh, line 5, in ?
[Mon Aug 06 15:13:20 2012] [error] from pkg_resources import  
load_entry_point
[Mon Aug 06 15:13:20 2012] [error]
File /usr/lib/python2.4/site-packages/setuptools-0.6c12dev_r88846-py2.4.egg/pkg_resources.py,  
line 2603, in ?

[Mon Aug 06 15:13:20 2012] [error] working_set.require(__requires__)
[Mon Aug 06 15:13:20 2012] [error]
File /usr/lib/python2.4/site-packages/setuptools-0.6c12dev_r88846-py2.4.egg/pkg_resources.py,  
line 666, in require
[Mon Aug 06 15:13:20 2012] [error] needed =  
self.resolve(parse_requirements(requirements))
[Mon Aug 06 15:13:20 2012] [error]
File /usr/lib/python2.4/site-packages/setuptools-0.6c12dev_r88846-py2.4.egg/pkg_resources.py,  
line 565, in resolve
[Mon Aug 06 15:13:20 2012] [error] raise DistributionNotFound(req)  #  
XXX put more info here
[Mon Aug 06 15:13:20 2012] [error] pkg_resources.DistributionNotFound:  
Django==1.3.2
[Mon Aug 06 15:13:20 2012] [error] fatal: The remote end hung up  
unexpectedly

[Mon Aug 06 15:13:20 2012] [error]

at first we thought it was an SSH problem, but a peek at the above error  
log and the above post reminded me that we have a Python 2.4.3 install on  
this CENTOS 5 machine, and a recently installed Python 2.7 install after  
which I installed setuptools, easy_install Reviewboard, etc And thus  
there was an rbssh at /usr/bin/rbssh which was invoking Python 2.4 (hence  
the python 2.4 calls, should have been 2.7) and another at  
/usr/local/bin/rbssh which was invoking Python 2.7. So I linked the  
/usr/local/bin/rbssh to /usr/bin/rbssh (first mv /usr/bin/rbssh to  
/usr/bin/rbssh.py2.4), and git repository was now being added no problem.


--
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 1388 in reviewboard: Visual Studio plugin/extension to run post-review/RBTools



Comment #6 on issue 1388 by p...@swooby.com: Visual Studio plugin/extension  
to run post-review/RBTools

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

?
Please be more descriptive.
What do you mean by is terribly ugly and doesn't work.
Neither one of those two statements is helpful or constructive to solving  
any problem.


--
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 2108 in reviewboard: Cannot run rbssh: No such file or directory; /usr/local/bin not in env path