Skip to content

Conversation

sfackler
Copy link
Member

I have this as a Unix-only API since it seems like Windows doesn't have
a similar API.

r? @alexcrichton

@kennytm kennytm added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. T-libs-api Relevant to the library API team, which will review and decide on the PR/issue. labels Nov 19, 2017
@alexcrichton
Copy link
Member

Seems plausible to me! The concept of a parent id I believe exists on Windows, although I don't think that it's as trivial as just a function call.

@alexcrichton
Copy link
Member

Want to fill in the issue number here?

I have this as a Unix-only API since it seems like Windows doesn't have
a similar API.
@sfackler
Copy link
Member Author

Updated

@alexcrichton
Copy link
Member

@bors: r+

@bors
Copy link
Collaborator

bors commented Nov 20, 2017

📌 Commit 1e42d5f has been approved by alexcrichton

@kennytm kennytm added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Nov 20, 2017
kennytm added a commit to kennytm/rust that referenced this pull request Nov 20, 2017
Add process::parent_id

I have this as a Unix-only API since it seems like Windows doesn't have
a similar API.

r? @alexcrichton
bors added a commit that referenced this pull request Nov 20, 2017
Rollup of 9 pull requests

- Successful merges: #46082, #46088, #46092, #46107, #46119, #46121, #46122, #46124, #46128
- Failed merges:
@bors bors merged commit 1e42d5f into rust-lang:master Nov 21, 2017
@sfackler sfackler deleted the ppid branch January 7, 2018 21:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. T-libs-api Relevant to the library API team, which will review and decide on the PR/issue.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants