Skip to content

Redox - Implement rename using new system call #47042

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Merged
merged 1 commit into from
Jan 2, 2018
Merged

Conversation

jackpot51
Copy link
Contributor

This does the following:

  • Update syscall module to match upstream
  • Implement rename using new system call
  • Make readlink and symlink utilize O_CLOEXEC
  • Make readlink and symlink not leave dangling file handles on failure

Fix readlink and symlink to utilize O_CLOEXEC
@estebank
Copy link
Contributor

estebank commented Jan 2, 2018

@bors r+

@bors
Copy link
Collaborator

bors commented Jan 2, 2018

📌 Commit 5919243 has been approved by estebank

@bors
Copy link
Collaborator

bors commented Jan 2, 2018

⌛ Testing commit 5919243 with merge fd1fe079a26b951f623508cbea3d79b309afa2aa...

@bors
Copy link
Collaborator

bors commented Jan 2, 2018

💔 Test failed - status-appveyor

@kennytm
Copy link
Member

kennytm commented Jan 2, 2018

@bors retry #46903 (3 hour timeout)

bors added a commit that referenced this pull request Jan 2, 2018
Redox - Implement rename using new system call

This does the following:

- Update syscall module to match upstream
- Implement rename using new system call
- Make readlink and symlink utilize O_CLOEXEC
- Make readlink and symlink not leave dangling file handles on failure
@bors
Copy link
Collaborator

bors commented Jan 2, 2018

⌛ Testing commit 5919243 with merge a37126b...

@bors
Copy link
Collaborator

bors commented Jan 2, 2018

☀️ Test successful - status-appveyor, status-travis
Approved by: estebank
Pushing a37126b to master...

@bors bors merged commit 5919243 into rust-lang:master Jan 2, 2018
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants