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

Added setpgrp #457

Closed
wants to merge 1 commit into from
Closed

Added setpgrp #457

wants to merge 1 commit into from

Conversation

tinywombat765
Copy link
Contributor

No description provided.

@rust-highfive
Copy link

r? @alexcrichton

(rust_highfive has picked a reviewer for you, use r? to override)

@alexcrichton
Copy link
Member

Thanks for the PR! Looks like the CI failures are vaild?

@tinywombat765
Copy link
Contributor Author

Not sure what those CI errors about. Those systems don't seem to be able to find the function. Any suggestions?

@alexcrichton
Copy link
Member

It looks like this API may not exist on those platforms, perhaps it could be moved farther down in the sourc ehierarchy?

@semarie
Copy link
Contributor

semarie commented Nov 26, 2016

additionnally, not all platforms follows strictly POSIX.

under OpenBSD, setpgrp() is identical to setpgid(), and is retained for calling convention compatibility with historical versions of BSD (see http://man.openbsd.org/2/setpgrp).

Susurrus pushed a commit to Susurrus/libc that referenced this pull request Mar 26, 2017
Add CLONE_NEWCGROUP

This is a new option to clone added earlier this year.  It was already added to rust/libc.
Susurrus pushed a commit to Susurrus/libc that referenced this pull request Mar 26, 2017
Add note about CLONE_NEWCGROUP from pull request rust-lang#457 to CHANGELOG.md.

Signed-off-by: Dylan Reid <dgreid@chromium.org>
Susurrus pushed a commit to Susurrus/libc that referenced this pull request Mar 26, 2017
Update CHANGELOD.md to include CLONE_NEWCGROUP

Add note about CLONE_NEWCGROUP from pull request rust-lang#457 to CHANGELOG.md.

Signed-off-by: Dylan Reid <dgreid@chromium.org>
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.

4 participants