Re: Remove dependence on gflags (issue 7628044)

0 views
Skip to first unread message

jcgre...@google.com

unread,
Mar 26, 2013, 1:41:17 PM3/26/13
to dhe...@google.com, jcgre...@google.com, joe.gr...@gmail.com, google-api-p...@googlegroups.com, httpli...@googlegroups.com, re...@codereview-hr.appspotmail.com
Great, I fat-fingered the upload and overwrote it with httplib2 changes,
let me see if I can revert.

On 2013/03/26 17:30:26, jcgregorio_google wrote:
> I will update all the rest of the samples in another CL.

> I also took another pass at plus.py to simplify it further.


https://codereview.appspot.com/7628044/diff/12004/oauth2client/tools.py
> File oauth2client/tools.py (right):


https://codereview.appspot.com/7628044/diff/12004/oauth2client/tools.py#newcode45
> oauth2client/tools.py:45: To make this sample run you will need to
populate the
> client_secrets.json file
> On 2013/03/18 20:31:01, dhermes wrote:
> > AFAICT these indents were only there to line up with the return. You
should
> > dedent this line and the following ones by two spaces.

> Done.



https://codereview.appspot.com/7628044/

Joe Gregorio

unread,
Mar 26, 2013, 2:01:10 PM3/26/13
to Danny Hermes, Joseph Gregorio, Joe Gregorio, google-api-p...@googlegroups.com, httpli...@googlegroups.com, re...@codereview-hr.appspotmail.com
OK, fixed.

-joe

Danny Hermes

unread,
Mar 26, 2013, 5:35:29 PM3/26/13
to Joe Gregorio, Joe Gregorio, google-api-p...@googlegroups.com, httplib2-dev, re...@codereview-hr.appspotmail.com
Sorry, having a hard time figuring out what has changed since Patch Set 7 (the last time I looked AFAICT).

What's more, there seem to be a few patch sets in a permanently broken state:
They both just say "Upload in progress."
--
Danny Hermes
Developer Programs Engineer

Joe Gregorio

unread,
Mar 27, 2013, 1:12:29 PM3/27/13
to Danny Hermes, Joe Gregorio, google-api-p...@googlegroups.com, httplib2-dev, re...@codereview-hr.appspotmail.com
Look at the diffs between 8 and 11. The only files to change are
plus.py and tools.py.
--
Joe Gregorio http://bitworking.org
Reply all
Reply to author
Forward
0 new messages