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

Question re requests-unixsocket dependency #598

Closed
lazka opened this issue Oct 30, 2021 · 4 comments · Fixed by #599
Closed

Question re requests-unixsocket dependency #598

lazka opened this issue Oct 30, 2021 · 4 comments · Fixed by #599
Labels

Comments

@lazka
Copy link

lazka commented Oct 30, 2021

setup.cfg lists a requests-unixsocket dependency which doesn't work on Windows but is still installed there.

requests-unixsocket

We are trying to package this on Windows and were wondering why this dependency is needed: msys2/MINGW-packages#9922

Looking at the the PR that added it: #525 it seems like it was added by accident and isn't actually used, unless I'm missing something.

@lazka lazka added the bug label Oct 30, 2021
@welcome
Copy link

welcome bot commented Oct 30, 2021

Thank you for opening your first issue in this project! Engagement like this is essential for open source projects! 🤗

If you haven't done so already, check out Jupyter's Code of Conduct. Also, please try to follow the issue template as it helps other other community members to contribute more effectively.
welcome
You can meet the other Jovyans by joining our Discourse forum. There is also an intro thread there where you can stop by and say Hi! 👋

Welcome to the Jupyter community! 🎉

@Zsailer
Copy link
Member

Zsailer commented Nov 1, 2021

Thank you for reporting, @lazka.

You're right. As @kevin-bates mentions in #599, we removed the need for this dependency by refactoring the tests a bit.

I'll merge #599 and cut a patch release today.

@lazka
Copy link
Author

lazka commented Nov 1, 2021

Thanks!

@Zsailer
Copy link
Member

Zsailer commented Nov 1, 2021

v1.11.2 is released with requests-unixsocket removed.

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

Successfully merging a pull request may close this issue.

2 participants