Skip to content

Comment on the casts in the seek implementations on files#34916

Merged
bors merged 1 commit into
rust-lang:masterfrom
tbu-:pr_comment_on_seek_cast
Aug 6, 2016
Merged

Comment on the casts in the seek implementations on files#34916
bors merged 1 commit into
rust-lang:masterfrom
tbu-:pr_comment_on_seek_cast

Conversation

@tbu-

@tbu- tbu- commented Jul 19, 2016

Copy link
Copy Markdown
Contributor

No description provided.

@rust-highfive

rust-highfive commented Jul 19, 2016

Copy link
Copy Markdown
Contributor

r? @alexcrichton

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

@steveklabnik

steveklabnik commented Jul 19, 2016

Copy link
Copy Markdown
Contributor

@bors: r+ rollup

@bors

bors commented Jul 19, 2016

Copy link
Copy Markdown
Collaborator

📌 Commit de24ae4 has been approved by steveklabnik

@bors

bors commented Jul 20, 2016

Copy link
Copy Markdown
Collaborator

⌛ Testing commit de24ae4 with merge 74d176d...

@bors

bors commented Jul 20, 2016

Copy link
Copy Markdown
Collaborator

💔 Test failed - auto-win-gnu-32-opt-rustbuild

@alexcrichton

alexcrichton commented Jul 20, 2016

Copy link
Copy Markdown
Member

@bors: retry

On Tue, Jul 19, 2016 at 10:05 PM, bors notifications@github.com wrote:

💔 Test failed - auto-win-gnu-32-opt-rustbuild
https://buildbot.rust-lang.org/builders/auto-win-gnu-32-opt-rustbuild/builds/1835

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
#34916 (comment), or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAD95Fwxco4CieYBCBNQ85GJUvS2-9PZks5qXayTgaJpZM4JPbD4
.

@GuillaumeGomez GuillaumeGomez mentioned this pull request Jul 20, 2016
Closed
@GuillaumeGomez

GuillaumeGomez commented Jul 20, 2016

Copy link
Copy Markdown
Member

off64_t import isn't used anymore on android and made homu failed. I r- for the moment.

@bors: r-

@tbu- tbu- force-pushed the pr_comment_on_seek_cast branch from de24ae4 to 291b6f1 Compare August 5, 2016 18:18
@tbu-

tbu- commented Aug 5, 2016

Copy link
Copy Markdown
Contributor Author

Removed the off64_t import on Android.

@GuillaumeGomez

GuillaumeGomez commented Aug 5, 2016

Copy link
Copy Markdown
Member

@bors: r+ rollup

@bors

bors commented Aug 5, 2016

Copy link
Copy Markdown
Collaborator

📌 Commit 291b6f1 has been approved by GuillaumeGomez

sophiajt pushed a commit to sophiajt/rust that referenced this pull request Aug 5, 2016
Rollup merge of rust-lang#34916 - tbu-:pr_comment_on_seek_cast, r=Gui…
e36be6e 
…llaumeGomez

Comment on the casts in the `seek` implementations on files
@sophiajt sophiajt mentioned this pull request Aug 5, 2016
Closed
bors added a commit that referenced this pull request Aug 6, 2016
@bors
Auto merge of #35397 - jonathandturner:rollup, r=jonathandturner
43446fc 
Rollup of 18 pull requests

- Successful merges: #34916, #35287, #35288, #35331, #35353, #35356, #35363, #35364, #35366, #35368, #35370, #35372, #35373, #35374, #35375, #35376, #35380, #35394
- Failed merges: #35395
@eddyb eddyb mentioned this pull request Aug 6, 2016
Merged
eddyb added a commit to eddyb/rust that referenced this pull request Aug 6, 2016
@eddyb
Rollup merge of rust-lang#34916 - tbu-:pr_comment_on_seek_cast, r=Gui…
68f7b26 
…llaumeGomez

Comment on the casts in the `seek` implementations on files
bors added a commit that referenced this pull request Aug 6, 2016
@bors
Auto merge of #35407 - eddyb:rollup, r=eddyb
444ff9f 
Rollup of 21 pull requests

- Successful merges: #33951, #34916, #35287, #35288, #35351, #35353, #35356, #35362, #35363, #35364, #35366, #35368, #35370, #35372, #35373, #35374, #35375, #35376, #35380, #35393, #35394
- Failed merges: #35331, #35395
@bors bors merged commit 291b6f1 into rust-lang:master Aug 6, 2016
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.

6 participants

@tbu- @rust-highfive @steveklabnik @bors @alexcrichton @GuillaumeGomez