Skip to content

Define missing F_RDLCK, F_WRLCK and F_UNLCK constants on linux-mips -and s390x #1186

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

Merged
merged 2 commits into from
Dec 26, 2018

Conversation

glaubitz
Copy link
Contributor

These are defined in the fcntl.h glibc header on MIPS and s390x
systems on Linux but missing in the libc crate, so add them as they
are required for the file locking API in rustc.

These are defined in the fcntl.h glibc header on MIPS systems
on Linux but missing in the libc crate, so add them as they are
required for the file locking API in rustc.
These are defined in the fcntl.h glibc header on s390x systems
on Linux but missing in the libc crate, so add them as they are
required for the file locking API in rustc.
@rust-highfive
Copy link

r? @alexcrichton

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

@alexcrichton
Copy link
Member

@bors: r+

@bors
Copy link
Contributor

bors commented Dec 26, 2018

📌 Commit 63d1a8c has been approved by alexcrichton

@bors
Copy link
Contributor

bors commented Dec 26, 2018

⌛ Testing commit 63d1a8c with merge 16b6c0e...

bors added a commit that referenced this pull request Dec 26, 2018
Define missing F_RDLCK, F_WRLCK and F_UNLCK constants on linux-mips -and s390x

These are defined in the fcntl.h glibc header on MIPS and s390x
systems on Linux but missing in the libc crate, so add them as they
are required for the file locking API in rustc.
@bors
Copy link
Contributor

bors commented Dec 26, 2018

☀️ Test successful - status-appveyor, status-travis
Approved by: alexcrichton
Pushing 16b6c0e to master...

@bors bors merged commit 63d1a8c into rust-lang:master Dec 26, 2018
@glaubitz
Copy link
Contributor Author

@alexcrichton Could you make a new release so we can use the updated libc in rustc for rust-lang/rust#57121?

@alexcrichton
Copy link
Member

Sure thing, release instructions are in the README!

@glaubitz
Copy link
Contributor Author

glaubitz commented Jan 2, 2019

Ah, thanks for the pointer. Never done this before :).

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