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.

Sign up for GitHub

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

Remove maelstrom_util::process::clone_into_pid_and_user_namespace's dependency on tokio #430

Open
nfachan opened this issue Jan 14, 2025 · 0 comments
Open

Remove maelstrom_util::process::clone_into_pid_and_user_namespace's dependency on tokio #430

nfachan opened this issue Jan 14, 2025 · 0 comments
Labels
code-improvement A desired code change for improved readability, testability, or performance.

Comments

@nfachan
Copy link
Collaborator

nfachan commented Jan 14, 2025

We have all of the tools we need in Linux and in maelstrom-linux. Just use those instead of adding the Tokio dependency.

This has less to do with the code requirement, and more to do with the conceptual simplicity of this module.
@nfachan nfachan added the code-improvement A desired code change for improved readability, testability, or performance. label Jan 14, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
code-improvement A desired code change for improved readability, testability, or performance.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@nfachan