Re: post-review tool issues
Sorry, Small typo.. C:\python2.6 %C --p4-client $c --p4-port $p I can see in the P4V command window the arguments getting populated properly : C:\python post-review.py 8624 --p4-client New_DepotView --p4-port xxx.xxx.xxx.xxx:1999 C:\python2.6\post-review.py %C --p4-client $c --p4-port $p I can see in the P4V command window the arguments getting populated properly : C:\python\post-review.py 8624 --p4-client New_DepotView --p4-port xxx.xxx.xxx.xxx:1999 On Dec 9, 9:21 am, roshanpius [EMAIL PROTECTED] wrote: Hi, I am having a small issue integrating post-review with P4V in the Windows systems. I've installed python2.6, setuptools, simplejson and GNU diffutils. Here is the problem I've renamed post-review to post-review.py(so that it gets associated automatically) and when i run it in command line with the appropriate arguments, it works perfectly well. But when I import it into P4V and run the same command, it says not able to execute the command: C:\python2.6 %C --p4-client $c --p4-port $p I can see in the P4V command window the arguments getting populated properly : C:\python post-review.py 8624 --p4-client New_DepotView --p4-port xxx.xxx.xxx.xxx:1999 What am I missing here.. ? --~--~-~--~~~---~--~~ You received this message because you are subscribed to the Google Groups reviewboard group. To post to this group, send email to reviewboard@googlegroups.com To unsubscribe from this group, send email to [EMAIL PROTECTED] For more options, visit this group at http://groups.google.com/group/reviewboard?hl=en -~--~~~~--~~--~--~---
Re: post-review tool issues
With that error, I'd definitely try setting Application to be the Python binary, and then use the script as the first parameter. P4V probably requires that the application itself be an actual binary, rather than assuming it will be opened with something else by file association. Christian -- Christian Hammond - [EMAIL PROTECTED] VMware, Inc. On Mon, Dec 8, 2008 at 9:55 PM, roshan pius [EMAIL PROTECTED]wrote: Hi Christian, It show the code being executed in their P4V command Window and then it pops up a message saying Unable to execute the command Error: Unable to run C:\Python26\post-review.py But the same command works on the command line... There is some issue here I'm attaching a screenshot of the configuration... Maybe someone with a knowledge of how P4V works can help... Thanks, Roshan Pius On Tue, Dec 9, 2008 at 11:01 AM, Christian Hammond [EMAIL PROTECTED] wrote: There's no error messages or any output? It's just doing nothing? I don't know a thing about how P4V works. What happens if you specify the python binary before the script name? Christian -- Christian Hammond - [EMAIL PROTECTED] VMware, Inc. On Mon, Dec 8, 2008 at 8:33 PM, roshanpius [EMAIL PROTECTED] wrote: Sorry, Small typo.. C:\python2.6 %C --p4-client $c --p4-port $p I can see in the P4V command window the arguments getting populated properly : C:\python post-review.py 8624 --p4-client New_DepotView --p4-port xxx.xxx.xxx.xxx:1999 C:\python2.6\post-review.py %C --p4-client $c --p4-port $p I can see in the P4V command window the arguments getting populated properly : C:\python\post-review.py 8624 --p4-client New_DepotView --p4-port xxx.xxx.xxx.xxx:1999 On Dec 9, 9:21 am, roshanpius [EMAIL PROTECTED] wrote: Hi, I am having a small issue integrating post-review with P4V in the Windows systems. I've installed python2.6, setuptools, simplejson and GNU diffutils. Here is the problem I've renamed post-review to post-review.py(so that it gets associated automatically) and when i run it in command line with the appropriate arguments, it works perfectly well. But when I import it into P4V and run the same command, it says not able to execute the command: C:\python2.6 %C --p4-client $c --p4-port $p I can see in the P4V command window the arguments getting populated properly : C:\python post-review.py 8624 --p4-client New_DepotView --p4-port xxx.xxx.xxx.xxx:1999 What am I missing here.. ? --~--~-~--~~~---~--~~ You received this message because you are subscribed to the Google Groups reviewboard group. To post to this group, send email to reviewboard@googlegroups.com To unsubscribe from this group, send email to [EMAIL PROTECTED] For more options, visit this group at http://groups.google.com/group/reviewboard?hl=en -~--~~~~--~~--~--~---
Re: post-review tool issues
You have a space before the script name. Christian -- Christian Hammond - [EMAIL PROTECTED] VMware, Inc. On Mon, Dec 8, 2008 at 10:17 PM, roshan pius [EMAIL PROTECTED]wrote: Hi Now I've made the changes i'm getting a new error now.. I've posted the config and the error screenshots. Thanks, Roshan Pius On Tue, Dec 9, 2008 at 11:35 AM, Christian Hammond [EMAIL PROTECTED] wrote: With that error, I'd definitely try setting Application to be the Python binary, and then use the script as the first parameter. P4V probably requires that the application itself be an actual binary, rather than assuming it will be opened with something else by file association. Christian -- Christian Hammond - [EMAIL PROTECTED] VMware, Inc. On Mon, Dec 8, 2008 at 9:55 PM, roshan pius [EMAIL PROTECTED] wrote: Hi Christian, It show the code being executed in their P4V command Window and then it pops up a message saying Unable to execute the command Error: Unable to run C:\Python26\post-review.py But the same command works on the command line... There is some issue here I'm attaching a screenshot of the configuration... Maybe someone with a knowledge of how P4V works can help... Thanks, Roshan Pius On Tue, Dec 9, 2008 at 11:01 AM, Christian Hammond [EMAIL PROTECTED] wrote: There's no error messages or any output? It's just doing nothing? I don't know a thing about how P4V works. What happens if you specify the python binary before the script name? Christian -- Christian Hammond - [EMAIL PROTECTED] VMware, Inc. On Mon, Dec 8, 2008 at 8:33 PM, roshanpius [EMAIL PROTECTED] wrote: Sorry, Small typo.. C:\python2.6 %C --p4-client $c --p4-port $p I can see in the P4V command window the arguments getting populated properly : C:\python post-review.py 8624 --p4-client New_DepotView --p4-port xxx.xxx.xxx.xxx:1999 C:\python2.6\post-review.py %C --p4-client $c --p4-port $p I can see in the P4V command window the arguments getting populated properly : C:\python\post-review.py 8624 --p4-client New_DepotView --p4-port xxx.xxx.xxx.xxx:1999 On Dec 9, 9:21 am, roshanpius [EMAIL PROTECTED] wrote: Hi, I am having a small issue integrating post-review with P4V in the Windows systems. I've installed python2.6, setuptools, simplejson and GNU diffutils. Here is the problem I've renamed post-review to post-review.py(so that it gets associated automatically) and when i run it in command line with the appropriate arguments, it works perfectly well. But when I import it into P4V and run the same command, it says not able to execute the command: C:\python2.6 %C --p4-client $c --p4-port $p I can see in the P4V command window the arguments getting populated properly : C:\python post-review.py 8624 --p4-client New_DepotView --p4-port xxx.xxx.xxx.xxx:1999 What am I missing here.. ? --~--~-~--~~~---~--~~ You received this message because you are subscribed to the Google Groups reviewboard group. To post to this group, send email to reviewboard@googlegroups.com To unsubscribe from this group, send email to [EMAIL PROTECTED] For more options, visit this group at http://groups.google.com/group/reviewboard?hl=en -~--~~~~--~~--~--~---
Re: post-review tool issues
Hi. I just ran some of the code through for that diff header and it appears to be working okay here. Perhaps there's something subtle we're not accounting for. Can you e-mail me (privately if you like) the actual debug output from post-review? You can strip anything you need to strip, but the diff header parts are pretty important. Output to a file instead of copying and pasting, because I'll need to check tabs vs. spaces and such. Christian -- Christian Hammond - [EMAIL PROTECTED] VMware, Inc. On Sun, Dec 7, 2008 at 2:26 AM, roshanpius [EMAIL PROTECTED]wrote: Hi guys, I have recently installed ReviewBoard for Code Reviews on a Linux server in my Company. We're running a Perforce SCM. Even though i had initially run into some issues regarding installation, it is finally up and running. But now i've hit another roadblock with the post-review tool. We've a development machine on which all of us have our P4 Workspaces. So I've to generate the diff from this development server to the other server on which reviewboard is running. But when I try to run the post-review tool on a changelist, I get an error with debug flag on: Unable to parse diff header: --- /tmp/tmpHMpfDv 2008-12-07 15:43:46.0 +0530 +++ /home/rpius/perforce/evros-dev1.6patch-rpius/nlg_rel1.6_patch/ common/evlog/evl_log.c 1970-01-01 05:30:00.0 +0530 @@ -1,731 +0,0 @@ The second file on my workspace has the time stamp wrong as well (01-01-1970). When I manually do a diff the time-stamp is correct (21-11-2008) diff -urNp /tmp/tmpHMpfDv /home/rpius/perforce/evros-dev1.6patch-rpius/ nlg_rel1.6_patch/common/evlog/evl_log.c --- /tmp/tmpH6xa0M 2008-12-07 15:31:23.0 +0530 +++ /home/rpius/perforce/evros-dev1.6patch-rpius/nlg_rel1.6_patch/ common/evlog/evl_log.c2008-11-21 22:00:53.0 +0530 What is happening here ?? I tried going through the post-review code(with no knowledge of python). What I found was that the post-review tool expects the time stamp to be the first thing on the diff(That is not the case as seen above) . Also the post-review tool appends --- and +++ after the diff is generated, but in my case this is already present in the diff command itself (as seen above). So the post-review tool doesn't seem to be getting the pattern it expects. So what do i do? Even when I deleted the portion of the code checking for the time stamps initially(line number 990-1023)...It still gives me another error when it tries to post the review - something about the arguments being messed up(obviously!!!) in the diff and thus unable to post it... I also read somewhere that this could be some problem with the older diffutils-2.8.1 But I've diffutils-2.8.7-80 On OpenSuse11.. Any Help would be greatly appreciated. --~--~-~--~~~---~--~~ You received this message because you are subscribed to the Google Groups reviewboard group. To post to this group, send email to reviewboard@googlegroups.com To unsubscribe from this group, send email to [EMAIL PROTECTED] For more options, visit this group at http://groups.google.com/group/reviewboard?hl=en -~--~~~~--~~--~--~---