Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Pin dateutil to 2.8.0 in requirements #29464

Closed
wants to merge 2 commits into from
Closed

Pin dateutil to 2.8.0 in requirements #29464

wants to merge 2 commits into from

Conversation

mzjp2
Copy link

@mzjp2 mzjp2 commented Nov 7, 2019

botocore has pinned dateutil to <=2.8.0, which conflicts with our dateutil here and leads to

ERROR: botocore 1.13.12 has requirement python-dateutil<2.8.1,>=2.1; python_version >= "2.7", but you'll have python-dateutil 2.8.1 which is incompatible.

See boto/botocore@e87e7a7

Closes #29465

mzjp2 added 2 commits November 7, 2019 15:27
botocore has pinned dateutil to <=2.8.0, which conflicts with our dateutil here
@TomAugspurger
Copy link
Contributor

Is the error you posted from pip or conda?

Shouldn't the solver know to pick something like 2.8, which is compatible with both?

@mzjp2
Copy link
Author

mzjp2 commented Nov 7, 2019

From pip. I don't know why the solver doesn't work. It might only work with currently installed packages rather than having collected them all together, before installing them. Anyway, the issue that I duplicated seems to say we should just wait for botocore to fix this, so I'm happy to close this PR.

@TomAugspurger
Copy link
Contributor

I asked over in boto/botocore#1872 when they think it'll be fixed.

@gfyoung gfyoung added Build Library building on various platforms Dependencies Required and optional dependencies labels Nov 9, 2019
@jbrockmendel
Copy link
Member

i think theres consensus that this needs to be fixed in botocore, not here?

@jbrockmendel
Copy link
Member

Closing, needs fix in botocore. If I'm wrong on this, please correct me and re-open.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Build Library building on various platforms Dependencies Required and optional dependencies
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Pandas dev requirements broken
4 participants