Skip to content

add setfsgid/setfsuid on Linux #802

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 1 commit into from
Oct 19, 2017
Merged

add setfsgid/setfsuid on Linux #802

merged 1 commit into from
Oct 19, 2017

Conversation

Mic92
Copy link
Contributor

@Mic92 Mic92 commented Oct 12, 2017

No description provided.

@alexcrichton
Copy link
Member

@bors: r+

@bors
Copy link
Contributor

bors commented Oct 12, 2017

📌 Commit e859f49 has been approved by alexcrichton

@bors
Copy link
Contributor

bors commented Oct 12, 2017

⌛ Testing commit e859f49 with merge 91293d3...

bors added a commit that referenced this pull request Oct 12, 2017
add setfsgid/setfsuid on Linux
@bors
Copy link
Contributor

bors commented Oct 13, 2017

💔 Test failed - status-travis

@bors
Copy link
Contributor

bors commented Oct 18, 2017

⌛ Testing commit e859f49 with merge f77c534...

bors added a commit that referenced this pull request Oct 18, 2017
add setfsgid/setfsuid on Linux
@alexcrichton
Copy link
Member

@bors: r- retry

@Mic92
Copy link
Contributor Author

Mic92 commented Oct 18, 2017

Interesting. Actually setfsuid/setfsgid is defined here, but seems to be missing in the headers used for th build: https://github.com/aosp-mirror/platform_bionic/blob/master/libc/include/sys/fsuid.h#L37

@@ -212,6 +211,7 @@ fn main() {
cfg.header("sched.h");
cfg.header("sys/epoll.h");
cfg.header("sys/eventfd.h");
cfg.header("sys/fsuid.h");
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Now it is hopefully also in android available.

@@ -240,6 +239,8 @@ fn main() {
}

if linux || android {
cfg.header("sys/fsuid.h");
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

turns out not to be available in emscripten.

@alexcrichton
Copy link
Member

@bors: r+

@bors
Copy link
Contributor

bors commented Oct 19, 2017

📌 Commit 3406fe9 has been approved by alexcrichton

@bors
Copy link
Contributor

bors commented Oct 19, 2017

⌛ Testing commit 3406fe9 with merge 2172bb4...

bors added a commit that referenced this pull request Oct 19, 2017
add setfsgid/setfsuid on Linux
@bors
Copy link
Contributor

bors commented Oct 19, 2017

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

@bors bors merged commit 3406fe9 into rust-lang:master Oct 19, 2017
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.

3 participants