Python hacker emergency on git-cl (was: git-cl error message)

[David Kastrup]

"Trevor Daniels" <address@hidden> writes:

> Aleksandr Andreev wrote Friday, November 30, 2012 9:22 PM
>
>> I am getting the following message when I try to run git cl upload
>> origin/master:
>> urllib2.HTTPError: HTTP Error 301: Moved Permanently
>
> I get the same error.
>
> See http://www.checkupdown.com/status/E301.html
> which says:
>
> "The 301 response from the Web server should always include an
> alternative URL to which redirection should occur. If it does, a Web
> browser will immediately retry the alternative URL. So you never
> actually see a 301 error in a Web browser, unless perhaps you have a
> corrupt redirection chain e.g. URL A redirects to URL B which in turn
> redirects back to URL A. If your client is not a Web browser, it
> should behave in the same way as a Web browser i.e. immediately retry
> the alternative URL.
>
> "If the Web server does not return an alternative URL with the 301
> response, then either the Web server sofware itself is defective or
> the Webmaster has not set up the URL redirection correctly."
>  
>> Traceback (most recent call last):
>>  File "/usr/bin/git-cl", line 888, in <module>
>>    sys.exit(main(sys.argv))
>>  File "/usr/bin/git-cl", line 882, in main
>>    return func(argv[2:])
>>  File "/usr/bin/git-cl", line 607, in CmdUpload
>>    issue, patchset = upload.RealMain(['upload'] + upload_args + args)
>>  File "/home/sasha/git-cl/upload.py", line 1467, in RealMain
>>    response_body = rpc_server.Send("/upload", body, content_type=ctype)
>>  File "/home/sasha/git-cl/upload.py", line 342, in Send
>>    f = self.opener.open(req)
>>  File "/usr/lib/python2.7/urllib2.py", line 407, in open
>>    response = meth(req, response)
>>  File "/usr/lib/python2.7/urllib2.py", line 520, in http_response
>>    'http', request, response, code, msg, hdrs)
>>  File "/usr/lib/python2.7/urllib2.py", line 445, in error
>>    return self._call_chain(*args)
>>  File "/usr/lib/python2.7/urllib2.py", line 379, in _call_chain
>>    result = func(*args)
>>  File "/usr/lib/python2.7/urllib2.py", line 528, in http_error_default
>>    raise HTTPError(req.get_full_url(), code, msg, hdrs, fp)
>> urllib2.HTTPError: HTTP Error 301: Moved Permanently
>> 
>> Is there something wrong with my setup or is this a network issue?

Same here.

My interpretation is that Google has set codereview.appspot.com so that
any http request will automatically get redirected to https, and git-cl
is unable to deal with that.

It is unlikely that this is going to change on Google's side, so until
somebody gets git-cl able to deal with that, our workflow is hosed.

Any Python hackers willing or able to take this on?

In the mean time, what are we going to do?  Create patches manually and
upload to Rietveld manually?  Or route stuff through the mailing list
using git format-patch?

-- 
David Kastrup