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 denied unused import: pid_t in raw #24988

Merged
merged 1 commit into from
May 1, 2015
Merged

remove denied unused import: pid_t in raw #24988

merged 1 commit into from
May 1, 2015

Conversation

semarie
Copy link
Contributor

@semarie semarie commented Apr 30, 2015

  • unbreak the build under openbsd
  • while here, apply same modification to dragonfly, freebsd, ios (pid_t
    imported, but not used in raw.rs)

r? @alexcrichton
cc @wg @mneumann @vhbit

@semarie
remove denied unused import: pid_t in raw
27bcd2e 
- unbreak the build under openbsd
- while here, apply same modification to dragonfly, freebsd, ios (pid_t
  imported, but not used in raw.rs)
@alexcrichton
Copy link
Member

@bors: r+ 27bcd2e rollup

@semarie
Copy link
Contributor Author

semarie commented Apr 30, 2015

oops, it will conflict with #24983

Manishearth added a commit to Manishearth/rust that referenced this pull request May 1, 2015
@Manishearth
Rollup merge of rust-lang#24988 - semarie:unused-pid_t, r=alexcrichton
5cb5d27 
- unbreak the build under openbsd
- while here, apply same modification to dragonfly, freebsd, ios (pid_t
  imported, but not used in raw.rs)

r? @alexcrichton 
cc @wg @mneumann @vhbit
@Manishearth Manishearth mentioned this pull request May 1, 2015
Merged
Manishearth added a commit to Manishearth/rust that referenced this pull request May 1, 2015
@Manishearth
Rollup merge of rust-lang#24988 - semarie:unused-pid_t, r=alexcrichton
a2b3585 
- unbreak the build under openbsd
- while here, apply same modification to dragonfly, freebsd, ios (pid_t
  imported, but not used in raw.rs)

r? @alexcrichton
cc @wg @mneumann @vhbit
Manishearth added a commit to Manishearth/rust that referenced this pull request May 1, 2015
@Manishearth
Rollup merge of rust-lang#24988 - semarie:unused-pid_t, r=alexcrichton
84a23be 
- unbreak the build under openbsd
- while here, apply same modification to dragonfly, freebsd, ios (pid_t
  imported, but not used in raw.rs)

r? @alexcrichton
cc @wg @mneumann @vhbit
bors added a commit that referenced this pull request May 1, 2015
@bors
Auto merge of #25025 - Manishearth:rollup, r=Manishearth
1320c29 
- Successful merges: #24979, #24980, #24981, #24982, #24983, #24987, #24988, #24991, #24992, #24994, #24998, #25002, #25010, #25014, #25020, #25021
- Failed merges:
@vhbit
Copy link
Contributor

vhbit commented May 1, 2015

oops, it will conflict with #24983

@semarie, hmm, I don't think that would be the case - usually it's smart
enough to not produce conflict error. As I can see rollup proves that.
But if there is any issue I'm gladly will close mine in favor of this.

@bors bors merged commit 27bcd2e into rust-lang:master May 1, 2015
@semarie semarie deleted the unused-pid_t branch May 2, 2015 04:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@alexcrichton alexcrichton

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@semarie @alexcrichton @vhbit @bors