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

cygwin-minimal: pip-20.1.1 fails to install #30127

Closed
mkoeppe opened this issue Jul 12, 2020 · 26 comments
Closed

cygwin-minimal: pip-20.1.1 fails to install #30127

mkoeppe opened this issue Jul 12, 2020 · 26 comments

Comments

@mkoeppe
Copy link
Contributor

mkoeppe commented Jul 12, 2020

https://github.com/sagemath/sage/runs/862451271

  [pip-20.1.1]     File "/cygdrive/d/a/sage/sage/local/var/tmp/sage/build/pip-20.1.1/src/src/pip/_vendor/urllib3/__init__.py", line 7, in <module>
  [pip-20.1.1]       from .connectionpool import HTTPConnectionPool, HTTPSConnectionPool, connection_from_url
  [pip-20.1.1]     File "/cygdrive/d/a/sage/sage/local/var/tmp/sage/build/pip-20.1.1/src/src/pip/_vendor/urllib3/connectionpool.py", line 29, in <module>
  [pip-20.1.1]       from .connection import (
  [pip-20.1.1]     File "/cygdrive/d/a/sage/sage/local/var/tmp/sage/build/pip-20.1.1/src/src/pip/_vendor/urllib3/connection.py", line 68, in <module>
  [pip-20.1.1]       class HTTPConnection(_HTTPConnection, object):
  [pip-20.1.1]     File "/cygdrive/d/a/sage/sage/local/var/tmp/sage/build/pip-20.1.1/src/src/pip/_vendor/urllib3/connection.py", line 96, in HTTPConnection
  [pip-20.1.1]       default_socket_options = [(socket.IPPROTO_TCP, socket.TCP_NODELAY, 1)]
  [pip-20.1.1]   AttributeError: module 'socket' has no attribute 'TCP_NODELAY'

The install on cygwin-standard works fine.

Upstream: Fixed upstream, but not in a stable release.

CC: @embray @slel @dimpase

Component: porting: Cygwin

Author: Matthias Koeppe

Branch/Commit: bc8c54f

Reviewer: Dima Pasechnik

Issue created by migration from https://trac.sagemath.org/ticket/30127

@mkoeppe mkoeppe added this to the sage-9.2 milestone Jul 12, 2020
@slel
Copy link
Member

slel commented Aug 1, 2020

comment:1

Note that pip 20.2 was released.

@mkoeppe
Copy link
Contributor Author

mkoeppe commented Aug 7, 2020

comment:2

Let's try with the following updates:

@mkoeppe
Copy link
Contributor Author

mkoeppe commented Aug 7, 2020

@mkoeppe
Copy link
Contributor Author

mkoeppe commented Aug 7, 2020

Changed branch from u/mkoeppe/cygwin_minimal__pip_20_1_1_fails_to_install to none

@mkoeppe
Copy link
Contributor Author

mkoeppe commented Aug 7, 2020

@mkoeppe
Copy link
Contributor Author

mkoeppe commented Aug 7, 2020

@mkoeppe
Copy link
Contributor Author

mkoeppe commented Aug 7, 2020

Commit: 91f21f6

@mkoeppe
Copy link
Contributor Author

mkoeppe commented Aug 7, 2020

New commits:

a22b6aebuild/pkgs/pip: Update to 20.2.1
91f21f6build/pkgs/setuptools: Update to 49.2.1

@mkoeppe
Copy link
Contributor Author

mkoeppe commented Aug 7, 2020

comment:6

Same with pip 20.2.1 - https://github.com/mkoeppe/sage/runs/959258905

@mkoeppe
Copy link
Contributor Author

mkoeppe commented Aug 7, 2020

comment:7

Should probably look at the python3 build log on this system.

@mkoeppe
Copy link
Contributor Author

mkoeppe commented Aug 7, 2020

comment:8

The same error also with pip 20.1.1 and python3 3.8.5 from #27754.

@mkoeppe
Copy link
Contributor Author

mkoeppe commented Aug 7, 2020

comment:9

Apparently caused by a recent change in the Cygwin headers: https://bugs.python.org/issue41374

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Aug 7, 2020

Branch pushed to git repo; I updated commit sha1. This was a forced push. New commits:

bc8c54fbuild/pkgs/python3/patches/cygwin-socket-tcpnodelay-21649.patch: New

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Aug 7, 2020

Changed commit from 91f21f6 to bc8c54f

@mkoeppe
Copy link
Contributor Author

mkoeppe commented Aug 7, 2020

Upstream: Fixed upstream, but not in a stable release.

@mkoeppe
Copy link
Contributor Author

mkoeppe commented Aug 7, 2020

Author: Matthias Koeppe

@mkoeppe
Copy link
Contributor Author

mkoeppe commented Aug 7, 2020

@mkoeppe
Copy link
Contributor Author

mkoeppe commented Aug 8, 2020

comment:14

This worked, as can be seen in https://github.com/mkoeppe/sage/runs/959927471

Needs review

@mkoeppe
Copy link
Contributor Author

mkoeppe commented Aug 10, 2020

@mkoeppe
Copy link
Contributor Author

mkoeppe commented Aug 12, 2020

@mkoeppe
Copy link
Contributor Author

mkoeppe commented Aug 13, 2020

comment:17

Let's get this in please

@dimpase
Copy link
Member

dimpase commented Aug 14, 2020

@dimpase
Copy link
Member

dimpase commented Aug 14, 2020

comment:19

lgtm

@mkoeppe
Copy link
Contributor Author

mkoeppe commented Aug 14, 2020

Changed reviewer from https://github.com/mkoeppe/sage/runs/959927471, Dima Pasechnik to Dima Pasechnik

@mkoeppe
Copy link
Contributor Author

mkoeppe commented Aug 14, 2020

comment:20

Thanks!

@vbraun
Copy link
Member

vbraun commented Aug 15, 2020

Changed branch from u/mkoeppe/cygwin_minimal__pip_20_1_1_fails_to_install to bc8c54f

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants