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

Jump to bottom

gh-95174: WASI: skip missing sockets functions (GH-95179) #95179

Merged
merged 2 commits into from Jul 27, 2022
Merged

gh-95174: WASI: skip missing sockets functions (GH-95179) #95179

merged 2 commits into from Jul 27, 2022

Conversation

tiran
Copy link
Member

@tiran tiran commented Jul 23, 2022 •
edited by brettcannon

WASI can only create sockets from an existing file descriptor. Hostname
lookups are not available.

  • check for missing sys/socket.h functions
  • handle missing netdb.h functions

@tiran tiran requested a review from a team as a code owner Jul 23, 2022
@tiran tiran added the 🔨 test-with-buildbots Test PR w/ buildbots; report in status section label Jul 23, 2022
@bedevere-bot
Copy link

bedevere-bot commented Jul 23, 2022

🤖 New build scheduled with the buildbot fleet by @tiran for commit d391cfad9a4ee14a2efc228bc70cdb8c0d62db02 🤖

If you want to schedule another build, you need to add the "🔨 test-with-buildbots" label again.

@bedevere-bot bedevere-bot removed the 🔨 test-with-buildbots Test PR w/ buildbots; report in status section label Jul 23, 2022
@tiran tiran added the 🔨 test-with-buildbots Test PR w/ buildbots; report in status section label Jul 24, 2022
@bedevere-bot
Copy link

bedevere-bot commented Jul 24, 2022

🤖 New build scheduled with the buildbot fleet by @tiran for commit 08c0117068395faa4a588a8202338861d842543c 🤖

If you want to schedule another build, you need to add the "🔨 test-with-buildbots" label again.

@bedevere-bot bedevere-bot removed the 🔨 test-with-buildbots Test PR w/ buildbots; report in status section label Jul 24, 2022
@tiran tiran force-pushed the gh-95174-socket branch 2 times, most recently from bb3692b to 3b8efe3 Compare Jul 24, 2022
@tiran tiran added the 🔨 test-with-buildbots Test PR w/ buildbots; report in status section label Jul 24, 2022
@bedevere-bot
Copy link

bedevere-bot commented Jul 24, 2022

🤖 New build scheduled with the buildbot fleet by @tiran for commit 3b8efe3374513544b7a34a686531f1ec3f7aea45 🤖

If you want to schedule another build, you need to add the "🔨 test-with-buildbots" label again.

@bedevere-bot bedevere-bot removed the 🔨 test-with-buildbots Test PR w/ buildbots; report in status section label Jul 24, 2022
@tiran
pythongh-95174: WASI: skip missing sockets functions
98f53d2 
WASI can only create sockets from an existing file descriptor. Hostname
looks are not available.

- check for missing ``sys/socket.h`` functions
- handle missing ``netdb.h`` functions
@tiran tiran mentioned this pull request Jul 25, 2022
Merged
@tiran
Copy link
Member Author

tiran commented Jul 25, 2022

I have moved some changes into GH-95229 to have a clean split between socket-related changes and other changes.

@tiran tiran requested a review from brettcannon Jul 25, 2022
@tiran tiran added the 🔨 test-with-buildbots Test PR w/ buildbots; report in status section label Jul 25, 2022
@bedevere-bot
Copy link

bedevere-bot commented Jul 25, 2022

🤖 New build scheduled with the buildbot fleet by @tiran for commit 98f53d2 🤖

If you want to schedule another build, you need to add the "🔨 test-with-buildbots" label again.

@bedevere-bot bedevere-bot removed the 🔨 test-with-buildbots Test PR w/ buildbots; report in status section label Jul 25, 2022
@tiran tiran changed the title gh-95174: WASI: skip missing sockets functions gh-95174: WASI: skip missing sockets functions (GH-95179) Jul 27, 2022
@tiran tiran merged commit 8b24d60 into python:main Jul 27, 2022
14 checks passed
@tiran tiran deleted the gh-95174-socket branch Jul 27, 2022
@miss-islington
Copy link
Contributor

miss-islington commented Jul 27, 2022

Thanks @tiran for the PR 🌮🎉.. I'm working now to backport this PR to: 3.11.
🐍🍒⛏🤖

@miss-islington
Copy link
Contributor

miss-islington commented Jul 27, 2022

Sorry, @tiran, I could not cleanly backport this to 3.11 due to a conflict.
Please backport using cherry_picker on command line.
cherry_picker 8b24d60f1b7725d526ea5b5d3904b72e8b41ae0d 3.11

tiran added a commit to tiran/cpython that referenced this pull request Jul 27, 2022
@tiran
[3.11] pythongh-95174: WASI: skip missing sockets functions (pythonGH…
0788456 
…-95179).

(cherry picked from commit 8b24d60)

Co-authored-by: Christian Heimes <christian@python.org>
@bedevere-bot
Copy link

bedevere-bot commented Jul 27, 2022

GH-95308 is a backport of this pull request to the 3.11 branch.

tiran added a commit that referenced this pull request Jul 31, 2022
@tiran @pablogsal
[3.11] gh-95174: WASI: skip missing sockets functions (GH-95179) (GH-…
06b5f78 
…95308)

Co-authored-by: Christian Heimes <christian@python.org>
Co-authored-by: Pablo Galindo Salgado <Pablogsal@gmail.com>
@tiran tiran mentioned this pull request Aug 26, 2022
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@brettcannon brettcannon

@python/windows-team windows-team

Assignees

@tiran tiran

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
4 participants
@tiran @bedevere-bot @miss-islington @brettcannon