Skip to content

Implement Sync for process::Command #72387

Closed
@jonhoo

Description

@jonhoo

std::process::Command is currently Send, but not Sync. While Sync probably isn't super useful for Command itself, it does mean that any type that wraps a process::Command also isn't Sync even if it might be useful on that type.

Command currently has a manual impl Send following #47760 (issue #47751), though from the discussion on users.r-l.o, it sounds like a manual impl Sync would also be sound. @cuviper not sure if that still holds?

If it does, this seems like a worthwhile impl to add, simply for the auto-trait implications.

Metadata

Metadata

Assignees

No one assigned

    Labels

    C-enhancementCategory: An issue proposing an enhancement or a PR with one.T-libs-apiRelevant to the library API team, which will review and decide on the PR/issue.

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions