Skip to content

fix: Update setup.py to install httplib2>=0.15.0#1050

Merged
busunkim96 merged 2 commits into
googleapis:masterfrom
tomchop:tomchop-patch-1
Sep 29, 2020
Merged

fix: Update setup.py to install httplib2>=0.15.0#1050
busunkim96 merged 2 commits into
googleapis:masterfrom
tomchop:tomchop-patch-1

Conversation

@tomchop

@tomchop tomchop commented Sep 28, 2020

Copy link
Copy Markdown
Contributor

I ran into the same issue described here: #1047

  File "/usr/local/lib/python3.6/dist-packages/googleapiclient/_helpers.py", line 134, in positional_wrapper
    return wrapped(*args, **kwargs)
  File "/usr/local/lib/python3.6/dist-packages/googleapiclient/discovery.py", line 300, in build
    discovery_http.close()
AttributeError: 'Http' object has no attribute 'close'

From that same issue:

Having a look at httplib2 changelog, close() method appeared in v0.15.0

@tomchop
tomchop requested a review from a team September 28, 2020 09:17
@google-cla google-cla Bot added the cla: yes This human has signed the Contributor License Agreement. label Sep 28, 2020
@tomchop
tomchop marked this pull request as draft September 28, 2020 09:20
@tomchop tomchop changed the title Update setup.py Update setup.py to install httplib2>=0.15.0 Sep 28, 2020
@tomchop
tomchop marked this pull request as ready for review September 28, 2020 09:31
Comment thread setup.py
# currently upgrade their httplib2 version.
# Please see https://github.com/googleapis/google-api-python-client/pull/841
"httplib2>=0.9.2,<1dev",
"httplib2>=0.15.0,<1dev",

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@aaltay Is it alright to bump this now?

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I believe it can be bumped now. Beam has 'httplib2>=0.8,<0.18.0', as the new range now. (https://github.com/apache/beam/blob/8bedfd402ccd1336f2b40fad58cd5833cecb9638/sdks/python/setup.py#L150)

Do you remember what was the specific problem?

(/cc @tvalentyn in case he remembers.)

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, no need to restrict this for Beam reasons anymore.
https://issues.apache.org/jira/browse/BEAM-9819 has some context for prior restrictions.

@busunkim96 busunkim96 Sep 28, 2020

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@tomchop Could you delete the comments on 38-40 about httplib2?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@busunkim96 Done!

@tomchop tomchop changed the title Update setup.py to install httplib2>=0.15.0 fix: Update setup.py to install httplib2>=0.15.0 Sep 28, 2020

@busunkim96 busunkim96 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you @tomchop!

@busunkim96 busunkim96 added automerge Merge the pull request once unit tests and other checks pass. kokoro:force-run Add this label to force Kokoro to re-run the tests. labels Sep 28, 2020
@yoshi-kokoro yoshi-kokoro removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Sep 28, 2020
@gcf-merge-on-green

Copy link
Copy Markdown

Your PR has attempted to merge for 3 hours. Please check that all required checks have passed, you have an automerge label, and that all your reviewers have approved the PR

@busunkim96
busunkim96 merged commit c00f70d into googleapis:master Sep 29, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

automerge Merge the pull request once unit tests and other checks pass. cla: yes This human has signed the Contributor License Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants