Re: post-review problem on Windows

2012-07-15 Thread whatever
So I figured out what was happening.  We use SVN via cygwin.  So when SVN 
is launched, it gets launched in a cygwin shell even if you run it from a 
windows command prompt.

So this causes the path to be set (not sure how) to the cygwin path 
"/cygdrive/c/trunk" and then the path passed in "C:\\trunk" gets appended 
to the cygwin path.

The only way I found to get this to work was to remove the path variable 
from the execute command in svn.py as follows:

def get_url_prop(path):
url = execute(["svn", "propget", "reviewboard:url", 
path]).strip()
return url or None

def get_url_prop(path):
url = execute(["svn", "propget", "reviewboard:url", ""]).strip()
return url or None

Obviously this will only work with the setup I'm using, but I just wanted 
to post this in case anyone else is using SVN via cygwin and runs into the 
same problem.


On Thursday, July 12, 2012 2:56:31 PM UTC-4, whatever wrote:
>
> Thanks for the reply!
>
> So I tried your suggestion, but no dice.  Still the same issue.
>
> On Thursday, July 12, 2012 2:35:27 PM UTC-4, superakuma wrote:
>>
>> I had this problem before. You probably have cygwin in the PATH. I didn't 
>> know how to fix it so I just took out cgywin in the PATH to use post-review.
>>
>> On Thursday, July 12, 2012 11:21:29 AM UTC-7, whatever wrote:
>>>
>>> So I'm trying to run post-review on a windows box with cygwin installed 
>>> and I'm getting the following error:
>>>
>>> C:\trunk>post-review
>>> Failed to execute command: ['svn', 'propget', 'reviewboard:url', 
>>> 'C:\\trunk']
>>> cygwin warning:
>>>   MS-DOS style path detected: /cygdrive/c/trunk/C:\trunk
>>>   Preferred POSIX equivalent is: /cygdrive/c/trunk/C:/trunk
>>>   CYGWIN environment variable option "nodosfilewarning" turns off this 
>>> warning.
>>>   Consult the user's guide for more details about POSIX paths:
>>> http://cygwin.com/cygwin-ug-net/using.html#using-pathnames
>>> svn: E25: '/cygdrive/c/trunk/C:\trunk' is not under version control
>>>
>>>
>>> Obviously the problem is that post-review is adding "/cygdrive/c/trunk/" 
>>> to the path.  How do I avoid this happening?
>>>
>>> Thanks!
>>>
>>

-- 
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 problem on Windows

2012-07-12 Thread whatever
Thanks for the reply!

So I tried your suggestion, but no dice.  Still the same issue.

On Thursday, July 12, 2012 2:35:27 PM UTC-4, superakuma wrote:
>
> I had this problem before. You probably have cygwin in the PATH. I didn't 
> know how to fix it so I just took out cgywin in the PATH to use post-review.
>
> On Thursday, July 12, 2012 11:21:29 AM UTC-7, whatever wrote:
>>
>> So I'm trying to run post-review on a windows box with cygwin installed 
>> and I'm getting the following error:
>>
>> C:\trunk>post-review
>> Failed to execute command: ['svn', 'propget', 'reviewboard:url', 
>> 'C:\\trunk']
>> cygwin warning:
>>   MS-DOS style path detected: /cygdrive/c/trunk/C:\trunk
>>   Preferred POSIX equivalent is: /cygdrive/c/trunk/C:/trunk
>>   CYGWIN environment variable option "nodosfilewarning" turns off this 
>> warning.
>>   Consult the user's guide for more details about POSIX paths:
>> http://cygwin.com/cygwin-ug-net/using.html#using-pathnames
>> svn: E25: '/cygdrive/c/trunk/C:\trunk' is not under version control
>>
>>
>> Obviously the problem is that post-review is adding "/cygdrive/c/trunk/" 
>> to the path.  How do I avoid this happening?
>>
>> Thanks!
>>
>

-- 
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 problem on Windows

2012-07-12 Thread superakuma
I had this problem before. You probably have cygwin in the PATH. I didn't 
know how to fix it so I just took out cgywin in the PATH to use post-review.

On Thursday, July 12, 2012 11:21:29 AM UTC-7, whatever wrote:
>
> So I'm trying to run post-review on a windows box with cygwin installed 
> and I'm getting the following error:
>
> C:\trunk>post-review
> Failed to execute command: ['svn', 'propget', 'reviewboard:url', 
> 'C:\\trunk']
> cygwin warning:
>   MS-DOS style path detected: /cygdrive/c/trunk/C:\trunk
>   Preferred POSIX equivalent is: /cygdrive/c/trunk/C:/trunk
>   CYGWIN environment variable option "nodosfilewarning" turns off this 
> warning.
>   Consult the user's guide for more details about POSIX paths:
> http://cygwin.com/cygwin-ug-net/using.html#using-pathnames
> svn: E25: '/cygdrive/c/trunk/C:\trunk' is not under version control
>
>
> Obviously the problem is that post-review is adding "/cygdrive/c/trunk/" 
> to the path.  How do I avoid this happening?
>
> Thanks!
>

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

2012-05-27 Thread naga murali
Hi Eric,

This problem occuring only for a specific machine, I set the proxy
also, I am using ReviewBaord 1.6.3, right now I am trying myself to
solve

Thanks,
Nagamurali


On 5/28/12, Eric Johnson  wrote:
> Hi K Nagamurali,
>
> I'm just a reviewboard user, but have you solved your problem, or diagnosed
> it further? What version of the ReviewBoard server are you running?
>
> From the command line, can you ping the server that you're trying to reach?
> Is it running ReviewBoard successfully? Is this failure specific to the
> machine in question, or do you have this problem from any machine from
> which you've tried?
>
> -Eric.
>
> On Thu, May 24, 2012 at 10:34 AM, murali  wrote:
>
>>
>>
>> Hi,
>>
>> While I am doing the post-review from command line, the following
>> errors were getting
>>
>> Traceback (most recent call last):
>>  File "/usr/local/bin/post-review", line 9, in 
>>load_entry_point('RBTools==0.4.1', 'console_scripts', 'post-
>> review')()
>>  File "/usr/local/lib/python2.6/dist-packages/RBTools-0.4.1-py2.6.egg/
>> rbtools/postreview.py", line 1222, in main
>>if not server.check_api_version():
>>  File "/usr/local/lib/python2.6/dist-packages/RBTools-0.4.1-py2.6.egg/
>> rbtools/postreview.py", line 226, in check_api_version
>>root_resource = self.api_get('api/')
>>  File "/usr/local/lib/python2.6/dist-packages/RBTools-0.4.1-py2.6.egg/
>> rbtools/postreview.py", line 669, in api_get
>>return self.process_json(self.http_get(path))
>>  File "/usr/local/lib/python2.6/dist-packages/RBTools-0.4.1-py2.6.egg/
>> rbtools/postreview.py", line 639, in http_get
>>rsp = urllib2.urlopen(url).read()
>>  File "/usr/lib/python2.6/urllib2.py", line 126, in urlopen
>>return _opener.open(url, data, timeout)
>>  File "/usr/lib/python2.6/urllib2.py", line 391, in open
>>response = self._open(req, data)
>>  File "/usr/lib/python2.6/urllib2.py", line 409, in _open
>>'_open', req)
>>  File "/usr/lib/python2.6/urllib2.py", line 369, in _call_chain
>>result = func(*args)
>>  File "/usr/lib/python2.6/urllib2.py", line 1161, in http_open
>>return self.do_open(httplib.HTTPConnection, req)
>>  File "/usr/lib/python2.6/urllib2.py", line 1134, in do_open
>>r = h.getresponse()
>>  File "/usr/lib/python2.6/httplib.py", line 986, in getresponse
>>response.begin()
>>  File "/usr/lib/python2.6/httplib.py", line 391, in begin
>>version, status, reason = self._read_status()
>>  File "/usr/lib/python2.6/httplib.py", line 349, in _read_status
>>line = self.fp.readline()
>>  File "/usr/lib/python2.6/socket.py", line 397, in readline
>>data = recv(1)
>> KeyboardInterrupt
>> velocix@velocix-desktop:/home/svn/js$ post-review aplication.js
>> Unable to access the root /api/ URL on the server.
>>
>> Please provide a Solution for post-review to work
>>
>> Thanks & Regards,
>> K Nagamurali
>>
>> --
>> 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

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

2012-05-27 Thread Eric Johnson
Hi K Nagamurali,

I'm just a reviewboard user, but have you solved your problem, or diagnosed
it further? What version of the ReviewBoard server are you running?

>From the command line, can you ping the server that you're trying to reach?
Is it running ReviewBoard successfully? Is this failure specific to the
machine in question, or do you have this problem from any machine from
which you've tried?

-Eric.

On Thu, May 24, 2012 at 10:34 AM, murali  wrote:

>
>
> Hi,
>
> While I am doing the post-review from command line, the following
> errors were getting
>
> Traceback (most recent call last):
>  File "/usr/local/bin/post-review", line 9, in 
>load_entry_point('RBTools==0.4.1', 'console_scripts', 'post-
> review')()
>  File "/usr/local/lib/python2.6/dist-packages/RBTools-0.4.1-py2.6.egg/
> rbtools/postreview.py", line 1222, in main
>if not server.check_api_version():
>  File "/usr/local/lib/python2.6/dist-packages/RBTools-0.4.1-py2.6.egg/
> rbtools/postreview.py", line 226, in check_api_version
>root_resource = self.api_get('api/')
>  File "/usr/local/lib/python2.6/dist-packages/RBTools-0.4.1-py2.6.egg/
> rbtools/postreview.py", line 669, in api_get
>return self.process_json(self.http_get(path))
>  File "/usr/local/lib/python2.6/dist-packages/RBTools-0.4.1-py2.6.egg/
> rbtools/postreview.py", line 639, in http_get
>rsp = urllib2.urlopen(url).read()
>  File "/usr/lib/python2.6/urllib2.py", line 126, in urlopen
>return _opener.open(url, data, timeout)
>  File "/usr/lib/python2.6/urllib2.py", line 391, in open
>response = self._open(req, data)
>  File "/usr/lib/python2.6/urllib2.py", line 409, in _open
>'_open', req)
>  File "/usr/lib/python2.6/urllib2.py", line 369, in _call_chain
>result = func(*args)
>  File "/usr/lib/python2.6/urllib2.py", line 1161, in http_open
>return self.do_open(httplib.HTTPConnection, req)
>  File "/usr/lib/python2.6/urllib2.py", line 1134, in do_open
>r = h.getresponse()
>  File "/usr/lib/python2.6/httplib.py", line 986, in getresponse
>response.begin()
>  File "/usr/lib/python2.6/httplib.py", line 391, in begin
>version, status, reason = self._read_status()
>  File "/usr/lib/python2.6/httplib.py", line 349, in _read_status
>line = self.fp.readline()
>  File "/usr/lib/python2.6/socket.py", line 397, in readline
>data = recv(1)
> KeyboardInterrupt
> velocix@velocix-desktop:/home/svn/js$ post-review aplication.js
> Unable to access the root /api/ URL on the server.
>
> Please provide a Solution for post-review to work
>
> Thanks & Regards,
> K Nagamurali
>
> --
> 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: post-review problem

2012-05-25 Thread Christian Hammond
You're getting this particular error because you hit Control-C. I assume it
was just waiting forever and not actually accessing the server, though?

If you're behind a proxy server, you'll need to have your proxy set up
correctly.

Christian

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


On Thu, May 24, 2012 at 10:39 PM, naga murali  wrote:

> Hi,
>
> I installed ReviewBoard 1.6.3 on Ubuntu 10.04 in my local VMWare
> workstation, its working fine and I can able to post-review the file
> from command line like post-review , after that Inorder to
> access ReviewBoard all I copied OS Image  from my local VMWare
> workstation to my server  and after that everybody can able to open
> the ReviewBoard GUI but from that server If I am doing the post-review
> the following errors were getting
>
> Traceback (most recent call last):
>  File "/usr/local/bin/post-review", line 9, in 
>load_entry_point('RBTools==0.4.1', 'console_scripts', 'post-review')()
>  File
> "/usr/local/lib/python2.6/dist-packages/RBTools-0.4.1-py2.6.egg/rbtools/postreview.py",
> line 1222, in main
>if not server.check_api_version():
>  File
> "/usr/local/lib/python2.6/dist-packages/RBTools-0.4.1-py2.6.egg/rbtools/postreview.py",
> line 226, in check_api_version
>root_resource = self.api_get('api/')
>  File
> "/usr/local/lib/python2.6/dist-packages/RBTools-0.4.1-py2.6.egg/rbtools/postreview.py",
> line 669, in api_get
>return self.process_json(self.http_get(path))
>  File
> "/usr/local/lib/python2.6/dist-packages/RBTools-0.4.1-py2.6.egg/rbtools/postreview.py",
> line 639, in http_get
>rsp = urllib2.urlopen(url).read()
>  File "/usr/lib/python2.6/urllib2.py", line 126, in urlopen
>return _opener.open(url, data, timeout)
>  File "/usr/lib/python2.6/urllib2.py", line 391, in open
>response = self._open(req, data)
>  File "/usr/lib/python2.6/urllib2.py", line 409, in _open
>'_open', req)
>  File "/usr/lib/python2.6/urllib2.py", line 369, in _call_chain
>result = func(*args)
>  File "/usr/lib/python2.6/urllib2.py", line 1161, in http_open
>return self.do_open(httplib.HTTPConnection, req)
>  File "/usr/lib/python2.6/urllib2.py", line 1134, in do_open
>r = h.getresponse()
>  File "/usr/lib/python2.6/httplib.py", line 986, in getresponse
>response.begin()
>  File "/usr/lib/python2.6/httplib.py", line 391, in begin
>version, status, reason = self._read_status()
>  File "/usr/lib/python2.6/httplib.py", line 349, in _read_status
>line = self.fp.readline()
>  File "/usr/lib/python2.6/socket.py", line 397, in readline
>data = recv(1)
> KeyboardInterrupt
>
> Please Suggest any solution for this
>
> Thanks,
> Murali
> l
>
>
>
>
>
>
>
>
>
> On 5/25/12, Christian Hammond  wrote:
> > Hi Dan,
> >
> > It is looking for 1.3.1 specifically. RB 1.6.x won't work with Django
> 1.4.
> >
> > This looks like some temporary downtime issue with their server. I can
> > access www.djangoproject.com now. Can you try again?
> >
> > Christian
> >
> > --
> > Christian Hammond - chip...@chipx86.com
> > Review Board - http://www.reviewboard.org
> > VMware, Inc. - http://www.vmware.com
> >
> >
> > On Thu, May 24, 2012 at 3:35 AM, Dan Miller
> > wrote:
> >
> >> My installation was crusing along until it got to www.djangoproject.com
> .
> >> I'm not sure if it's that easy_install is looking for version 1.3.1
> >> and the latest version is 1.4 (perhaps something moved - I'm not very
> >> familiar with how the internals work here).
> >>
> >> Anyways, any ideas of how I can move past this?  I tried downloading
> >> and installing Django 1.3.1 manually but when I go back to
> >> easy_install ReviewBoard it still hangs at the same point.
> >>
> >> Thanks!
> >>
> >> Logs below:
> >>
> >> [root@slc01ozq reviewboard]# easy_install ReviewBoard
> >> Searching for ReviewBoard
> >> Reading http://pypi.python.org/simple/ReviewBoard/
> >> Reading http://www.reviewboard.org/
> >> Reading http://downloads.reviewboard.org/releases/ReviewBoard/1.5/
> >> Reading http://downloads.reviewboard.org/releases/ReviewBoard/1.6/
> >> Reading http://www.review-board.org/
> >> Reading http://downloads.review-board.org/releases/ReviewBoard/1.0/
> >> Reading http://downloads.review-board.org/releases/
> >> Best match: ReviewBoard 1.6.6
> >> Downloading
> >>
> http://downloads.reviewboard.org/releases/ReviewBoard/1.6/ReviewBoard-1.6.6-py2.6.egg
> >> Processing ReviewBoard-1.6.6-py2.6.egg
> >> creating /usr/lib/python2.6/site-packages/ReviewBoard-1.6.6-py2.6.egg
> >> Extracting ReviewBoard-1.6.6-py2.6.egg to /usr/lib/python2.6/site-
> >> packages
> >> Adding ReviewBoard 1.6.6 to easy-install.pth file
> >> Installing rb-site script to /usr/bin
> >> Installing rbssh script to /usr/bin
> >>
> >> Installed /usr/lib/python2.6/site-packages/ReviewBoard-1.6.6-py2.6.egg
> >> Reading http://downloads.reviewboard.org/mirror/
> >> Processing dependencies for ReviewBoard
> >> Searching for recap

Re: post-review problem

2012-05-07 Thread Juhani Tali

Sorry about that.

Background, this is a customized reviewboard 1.0 upgrade to 1.6.6.
Previous installation post-review had in python a VERSION = "0.8"

Now I get:

>>> RBTools 0.4.1
>>> Home = /var/www
>>> HTTP GETting api/
>>> HTTP GETting https://127.0.0.1/reviewboard/api/info/
>>> Using the new web API
There don't seem to be any diffs!

The
raw_outgoing = execute(['hg', '-q', 'outgoing', '--template',
'b:{branches}\nr:{rev}\n\n', remote],
   env=self._hg_env,
   extra_ignore_errors=(1,))

seems to give only https certificate errors.

Juhani

On May 4, 9:12 pm, Christian Hammond  wrote:
> That latest error is from doing --server=--server=
>
> Christian
>
> On May 4, 2012, at 6:07, Juhani Tali  wrote:
>
>
>
>
>
>
>
> > Hi,
>
> > I am not very comfortable with Python.
>
> > review-script is called from mercurial hook.
>
> > This is what I have found so far.
>
> > After fix for mercurial certificate warning messages
> > near error  File "/usr/local/lib/python2.6/dist-packages/rbtools/
> > clients/
> > mercurial.py", line 248, in _get_outgoing_changesets
> >    branch, rev = pair.strip().split('\n')
> >            try:
> >                branch, rev = pair.strip().split('\n')
> >            except ValueError:
> >                if 'warning: ' in pair:
> >                    continue
>
> > Got a new error message:
> > www-data@squeeze:~/hg/.hg$ python /var/www/hg/.hg/review-script
> > 0477768532d98c2a8cb25d92b2a95ab1e14a73e6
>  RBTools 0.4.1
>  Home = /var/www
>  HTTP GETting api/
>  HTTP GETtinghttps://127.0.0.1:443/reviewboard/api/info/
>  Using the new web API
> > There don't seem to be any diffs!
>
> > Mercurial sees:
> > www-data@squeeze:~/hg/.hg$ hg log -v -r
> > 0477768532d98c2a8cb25d92b2a95ab1e14a73e6
>
> > changeset:   28890:0477768532d9
> > branch:      t7702
> > parent:      28735:70bb78288793
> > parent:      28889:e169b5daf95b
> > user:        username
> > date:        Thu May 03 11:18:34 2012 +0800
> > files:       application...
> > description:
> > sync ...
>
> > when running post-review, I got:
> > www-data@squeeze:~/hg/.hg$ post-review --server=--server=https://
> > 127.0.0.1/reviewboard --submit-as=username --username=user  --
> > password=password --publish --target-groups=USER_GROUPS --
> > summary='28890:0477768532d9' --description='long description' --
> > branch='t7702' --debug 0477768532d98c2a8cb25d92b2a95ab1e14a73e6
>  RBTools 0.4.1
>  Home = /var/www
>  HTTP GETting api/
> > Traceback (most recent call last):
> >  File "/usr/local/bin/post-review", line 9, in 
> >    load_entry_point('RBTools==0.4.1', 'console_scripts', 'post-
> > review')()
> >  File "/usr/local/lib/python2.6/dist-packages/rbtools/postreview.py",
> > line 1222, in main
> >    if not server.check_api_version():
> >  File "/usr/local/lib/python2.6/dist-packages/rbtools/postreview.py",
> > line 226, in check_api_version
> >    root_resource = self.api_get('api/')
> >  File "/usr/local/lib/python2.6/dist-packages/rbtools/postreview.py",
> > line 669, in api_get
> >    return self.process_json(self.http_get(path))
> >  File "/usr/local/lib/python2.6/dist-packages/rbtools/postreview.py",
> > line 639, in http_get
> >    rsp = urllib2.urlopen(url).read()
> >  File "/usr/lib/python2.6/urllib2.py", line 126, in urlopen
> >    return _opener.open(url, data, timeout)
> >  File "/usr/lib/python2.6/urllib2.py", line 391, in open
> >    response = self._open(req, data)
> >  File "/usr/lib/python2.6/urllib2.py", line 409, in _open
> >    '_open', req)
> >  File "/usr/lib/python2.6/urllib2.py", line 369, in _call_chain
> >    result = func(*args)
> >  File "/usr/lib/python2.6/urllib2.py", line 1170, in http_open
> >    return self.do_open(httplib.HTTPConnection, req)
> >  File "/usr/lib/python2.6/urllib2.py", line 1116, in do_open
> >    h = http_class(host, timeout=req.timeout) # will parse host:port
> >  File "/usr/lib/python2.6/httplib.py", line 661, in __init__
> >    self._set_hostport(host, port)
> >  File "/usr/lib/python2.6/httplib.py", line 686, in _set_hostport
> >    raise InvalidURL("nonnumeric port: '%s'" % host[i+1:])
> > httplib.InvalidURL: nonnumeric port: ''
>
> > On May 4, 5:27 am, Christian Hammond  wrote:
> >> Hi,
>
> >> Looks like some issue with the interpretation of the data coming from hg.
>
> >> How comfortable are you with Python? It'd be helpful to see what data it's 
> >> operating on.
>
> >> Christian
>
> >> On May 3, 2012, at 0:41, Juhani Tali  wrote:
>
> >>> Hi,
>
> >>> I am a bit stuck with post-review errors.
> >>> What could be the problem?
>
> >>> review-script starts post-review with comments etc.
> >>> Version control is mercurial.
>
> >>> www-data@squeeze:~/hg/.hg$  python /var/www/hg/.hg/review-script
> >>> 0477768532d98c2a8cb25d92b2a95ab1e14a73e6
> >> RBTools 0.4.1
> >> Home = /var/www
> >> HTTP GETting api/
> >> HTTP GETtinghttps://127.0.0.1:443/r

Re: post-review problem

2012-05-04 Thread Christian Hammond
That latest error is from doing --server=--server=

Christian



On May 4, 2012, at 6:07, Juhani Tali  wrote:

> Hi,
> 
> I am not very comfortable with Python.
> 
> review-script is called from mercurial hook.
> 
> This is what I have found so far.
> 
> 
> After fix for mercurial certificate warning messages
> near error  File "/usr/local/lib/python2.6/dist-packages/rbtools/
> clients/
> mercurial.py", line 248, in _get_outgoing_changesets
>branch, rev = pair.strip().split('\n')
>try:
>branch, rev = pair.strip().split('\n')
>except ValueError:
>if 'warning: ' in pair:
>continue
> 
> Got a new error message:
> www-data@squeeze:~/hg/.hg$ python /var/www/hg/.hg/review-script
> 0477768532d98c2a8cb25d92b2a95ab1e14a73e6
 RBTools 0.4.1
 Home = /var/www
 HTTP GETting api/
 HTTP GETting https://127.0.0.1:443/reviewboard/api/info/
 Using the new web API
> There don't seem to be any diffs!
> 
> Mercurial sees:
> www-data@squeeze:~/hg/.hg$ hg log -v -r
> 0477768532d98c2a8cb25d92b2a95ab1e14a73e6
> 
> changeset:   28890:0477768532d9
> branch:  t7702
> parent:  28735:70bb78288793
> parent:  28889:e169b5daf95b
> user:username
> date:Thu May 03 11:18:34 2012 +0800
> files:   application...
> description:
> sync ...
> 
> when running post-review, I got:
> www-data@squeeze:~/hg/.hg$ post-review --server=--server=https://
> 127.0.0.1/reviewboard --submit-as=username --username=user  --
> password=password --publish --target-groups=USER_GROUPS --
> summary='28890:0477768532d9' --description='long description' --
> branch='t7702' --debug 0477768532d98c2a8cb25d92b2a95ab1e14a73e6
 RBTools 0.4.1
 Home = /var/www
 HTTP GETting api/
> Traceback (most recent call last):
>  File "/usr/local/bin/post-review", line 9, in 
>load_entry_point('RBTools==0.4.1', 'console_scripts', 'post-
> review')()
>  File "/usr/local/lib/python2.6/dist-packages/rbtools/postreview.py",
> line 1222, in main
>if not server.check_api_version():
>  File "/usr/local/lib/python2.6/dist-packages/rbtools/postreview.py",
> line 226, in check_api_version
>root_resource = self.api_get('api/')
>  File "/usr/local/lib/python2.6/dist-packages/rbtools/postreview.py",
> line 669, in api_get
>return self.process_json(self.http_get(path))
>  File "/usr/local/lib/python2.6/dist-packages/rbtools/postreview.py",
> line 639, in http_get
>rsp = urllib2.urlopen(url).read()
>  File "/usr/lib/python2.6/urllib2.py", line 126, in urlopen
>return _opener.open(url, data, timeout)
>  File "/usr/lib/python2.6/urllib2.py", line 391, in open
>response = self._open(req, data)
>  File "/usr/lib/python2.6/urllib2.py", line 409, in _open
>'_open', req)
>  File "/usr/lib/python2.6/urllib2.py", line 369, in _call_chain
>result = func(*args)
>  File "/usr/lib/python2.6/urllib2.py", line 1170, in http_open
>return self.do_open(httplib.HTTPConnection, req)
>  File "/usr/lib/python2.6/urllib2.py", line 1116, in do_open
>h = http_class(host, timeout=req.timeout) # will parse host:port
>  File "/usr/lib/python2.6/httplib.py", line 661, in __init__
>self._set_hostport(host, port)
>  File "/usr/lib/python2.6/httplib.py", line 686, in _set_hostport
>raise InvalidURL("nonnumeric port: '%s'" % host[i+1:])
> httplib.InvalidURL: nonnumeric port: ''
> 
> On May 4, 5:27 am, Christian Hammond  wrote:
>> Hi,
>> 
>> Looks like some issue with the interpretation of the data coming from hg.
>> 
>> How comfortable are you with Python? It'd be helpful to see what data it's 
>> operating on.
>> 
>> Christian
>> 
>> On May 3, 2012, at 0:41, Juhani Tali  wrote:
>> 
>> 
>> 
>> 
>> 
>> 
>> 
>>> Hi,
>> 
>>> I am a bit stuck with post-review errors.
>>> What could be the problem?
>> 
>>> review-script starts post-review with comments etc.
>>> Version control is mercurial.
>> 
>>> www-data@squeeze:~/hg/.hg$  python /var/www/hg/.hg/review-script
>>> 0477768532d98c2a8cb25d92b2a95ab1e14a73e6
>> RBTools 0.4.1
>> Home = /var/www
>> HTTP GETting api/
>> HTTP GETtinghttps://127.0.0.1:443/reviewboard/api/info/
>> Using the new web API
>>> Traceback (most recent call last):
>>>  File "/usr/local/bin/post-review", line 9, in 
>>>load_entry_point('RBTools==0.4.1', 'console_scripts', 'post-
>>> review')()
>>>  File "/usr/local/lib/python2.6/dist-packages/rbtools/postreview.py",
>>> line 1248, in main
>>>diff, parent_diff = tool.diff(args)
>>>  File "/usr/local/lib/python2.6/dist-packages/rbtools/clients/
>>> mercurial.py", line 151, in diff
>>>return self._get_outgoing_diff(files)
>>>  File "/usr/local/lib/python2.6/dist-packages/rbtools/clients/
>>> mercurial.py", line 205, in _get_outgoing_diff
>>>self._get_outgoing_changesets(current_branch, remote)
>>>  File "/usr/local/lib/python2.6/dist-packages/rbtools/clients/
>>> mercurial.py", line 248, in _get_outgoing_changesets
>>>branch, rev = pair.strip().

Re: post-review problem

2012-05-04 Thread Juhani Tali
Hi,

I am not very comfortable with Python.

review-script is called from mercurial hook.

This is what I have found so far.


After fix for mercurial certificate warning messages
near error  File "/usr/local/lib/python2.6/dist-packages/rbtools/
clients/
mercurial.py", line 248, in _get_outgoing_changesets
branch, rev = pair.strip().split('\n')
try:
branch, rev = pair.strip().split('\n')
except ValueError:
if 'warning: ' in pair:
continue

Got a new error message:
www-data@squeeze:~/hg/.hg$ python /var/www/hg/.hg/review-script
0477768532d98c2a8cb25d92b2a95ab1e14a73e6
>>> RBTools 0.4.1
>>> Home = /var/www
>>> HTTP GETting api/
>>> HTTP GETting https://127.0.0.1:443/reviewboard/api/info/
>>> Using the new web API
There don't seem to be any diffs!

Mercurial sees:
www-data@squeeze:~/hg/.hg$ hg log -v -r
0477768532d98c2a8cb25d92b2a95ab1e14a73e6

changeset:   28890:0477768532d9
branch:  t7702
parent:  28735:70bb78288793
parent:  28889:e169b5daf95b
user:username
date:Thu May 03 11:18:34 2012 +0800
files:   application...
description:
sync ...

when running post-review, I got:
www-data@squeeze:~/hg/.hg$ post-review --server=--server=https://
127.0.0.1/reviewboard --submit-as=username --username=user  --
password=password --publish --target-groups=USER_GROUPS --
summary='28890:0477768532d9' --description='long description' --
branch='t7702' --debug 0477768532d98c2a8cb25d92b2a95ab1e14a73e6
>>> RBTools 0.4.1
>>> Home = /var/www
>>> HTTP GETting api/
Traceback (most recent call last):
  File "/usr/local/bin/post-review", line 9, in 
load_entry_point('RBTools==0.4.1', 'console_scripts', 'post-
review')()
  File "/usr/local/lib/python2.6/dist-packages/rbtools/postreview.py",
line 1222, in main
if not server.check_api_version():
  File "/usr/local/lib/python2.6/dist-packages/rbtools/postreview.py",
line 226, in check_api_version
root_resource = self.api_get('api/')
  File "/usr/local/lib/python2.6/dist-packages/rbtools/postreview.py",
line 669, in api_get
return self.process_json(self.http_get(path))
  File "/usr/local/lib/python2.6/dist-packages/rbtools/postreview.py",
line 639, in http_get
rsp = urllib2.urlopen(url).read()
  File "/usr/lib/python2.6/urllib2.py", line 126, in urlopen
return _opener.open(url, data, timeout)
  File "/usr/lib/python2.6/urllib2.py", line 391, in open
response = self._open(req, data)
  File "/usr/lib/python2.6/urllib2.py", line 409, in _open
'_open', req)
  File "/usr/lib/python2.6/urllib2.py", line 369, in _call_chain
result = func(*args)
  File "/usr/lib/python2.6/urllib2.py", line 1170, in http_open
return self.do_open(httplib.HTTPConnection, req)
  File "/usr/lib/python2.6/urllib2.py", line 1116, in do_open
h = http_class(host, timeout=req.timeout) # will parse host:port
  File "/usr/lib/python2.6/httplib.py", line 661, in __init__
self._set_hostport(host, port)
  File "/usr/lib/python2.6/httplib.py", line 686, in _set_hostport
raise InvalidURL("nonnumeric port: '%s'" % host[i+1:])
httplib.InvalidURL: nonnumeric port: ''

On May 4, 5:27 am, Christian Hammond  wrote:
> Hi,
>
> Looks like some issue with the interpretation of the data coming from hg.
>
> How comfortable are you with Python? It'd be helpful to see what data it's 
> operating on.
>
> Christian
>
> On May 3, 2012, at 0:41, Juhani Tali  wrote:
>
>
>
>
>
>
>
> > Hi,
>
> > I am a bit stuck with post-review errors.
> > What could be the problem?
>
> > review-script starts post-review with comments etc.
> > Version control is mercurial.
>
> > www-data@squeeze:~/hg/.hg$  python /var/www/hg/.hg/review-script
> > 0477768532d98c2a8cb25d92b2a95ab1e14a73e6
>  RBTools 0.4.1
>  Home = /var/www
>  HTTP GETting api/
>  HTTP GETtinghttps://127.0.0.1:443/reviewboard/api/info/
>  Using the new web API
> > Traceback (most recent call last):
> >  File "/usr/local/bin/post-review", line 9, in 
> >    load_entry_point('RBTools==0.4.1', 'console_scripts', 'post-
> > review')()
> >  File "/usr/local/lib/python2.6/dist-packages/rbtools/postreview.py",
> > line 1248, in main
> >    diff, parent_diff = tool.diff(args)
> >  File "/usr/local/lib/python2.6/dist-packages/rbtools/clients/
> > mercurial.py", line 151, in diff
> >    return self._get_outgoing_diff(files)
> >  File "/usr/local/lib/python2.6/dist-packages/rbtools/clients/
> > mercurial.py", line 205, in _get_outgoing_diff
> >    self._get_outgoing_changesets(current_branch, remote)
> >  File "/usr/local/lib/python2.6/dist-packages/rbtools/clients/
> > mercurial.py", line 248, in _get_outgoing_changesets
> >    branch, rev = pair.strip().split('\n')
> > ValueError: too many values to unpack
>
> > Juhani Tali
>
> > --
> > Want to help the Review Board project? Donate today 
> > athttp://www.reviewboard.org/donate/
> > Happy user? Let us know athttp://www.reviewboard.org/users/
> > -~--~~~~--

Re: post-review problem

2012-05-03 Thread Christian Hammond
Hi,

Looks like some issue with the interpretation of the data coming from hg.

How comfortable are you with Python? It'd be helpful to see what data it's 
operating on.

Christian


On May 3, 2012, at 0:41, Juhani Tali  wrote:

> Hi,
> 
> I am a bit stuck with post-review errors.
> What could be the problem?
> 
> review-script starts post-review with comments etc.
> Version control is mercurial.
> 
> www-data@squeeze:~/hg/.hg$  python /var/www/hg/.hg/review-script
> 0477768532d98c2a8cb25d92b2a95ab1e14a73e6
 RBTools 0.4.1
 Home = /var/www
 HTTP GETting api/
 HTTP GETting https://127.0.0.1:443/reviewboard/api/info/
 Using the new web API
> Traceback (most recent call last):
>  File "/usr/local/bin/post-review", line 9, in 
>load_entry_point('RBTools==0.4.1', 'console_scripts', 'post-
> review')()
>  File "/usr/local/lib/python2.6/dist-packages/rbtools/postreview.py",
> line 1248, in main
>diff, parent_diff = tool.diff(args)
>  File "/usr/local/lib/python2.6/dist-packages/rbtools/clients/
> mercurial.py", line 151, in diff
>return self._get_outgoing_diff(files)
>  File "/usr/local/lib/python2.6/dist-packages/rbtools/clients/
> mercurial.py", line 205, in _get_outgoing_diff
>self._get_outgoing_changesets(current_branch, remote)
>  File "/usr/local/lib/python2.6/dist-packages/rbtools/clients/
> mercurial.py", line 248, in _get_outgoing_changesets
>branch, rev = pair.strip().split('\n')
> ValueError: too many values to unpack
> 
> Juhani Tali
> 
> -- 
> 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