Skip to content

Fix networking setup in language runtime. #500

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

Merged
merged 1 commit into from
Sep 12, 2024
Merged

Conversation

jemc
Copy link
Contributor

@jemc jemc commented Sep 12, 2024

In order for networking stuff to work as desired, we need to pass "true" for init_network in the pony_language_features_init_t passed to the pony_start call that starts the runtime.

Without that, things like the TCP library won't work properly. (for example, on UNIX systems, a SIGPIPE may interrupt the program when writing to a TCP socket whose other ends is known by the kernel to have already closed).

@jemc jemc self-assigned this Sep 12, 2024
In order for networking stuff to work as desired, we need to
pass "true" for `init_network` in the `pony_language_features_init_t`
passed to the `pony_start` call that starts the runtime.

Without that, things like the `TCP` library won't work properly.
(for example, on UNIX systems, a `SIGPIPE` may interrupt the
program when writing to a TCP socket whose other ends is known
by the kernel to have already closed).
@jemc jemc force-pushed the fix/networking-setup-runtime branch from 9e4c22c to f8578b5 Compare September 12, 2024 12:00
@jemc jemc merged commit 3734de8 into main Sep 12, 2024
5 of 8 checks passed
@jemc jemc deleted the fix/networking-setup-runtime branch September 12, 2024 12:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant