-
Notifications
You must be signed in to change notification settings - Fork 1.2k
Haiku: More build fixes #416
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
Conversation
(rust_highfive has picked a reviewer for you, use r? to override) |
This should be just about the last build fixes. I'm far enough std is almost linking:
|
hm. I injected those libraries into one of the library paths.. and now have arrived at the following:
|
following instructions:
|
@bors: r+ |
📌 Commit 573c5fe has been approved by |
☀️ Test successful - status-appveyor, travis |
add unistd::getcwd and unistd::mkdir As a (late) followup of [this withdrawn PR](rust-lang#326) I have added getcwd (wrapper around `libc::getcwd`) and mkdir (wrapper around `libc::mkdir`) and added testing. A few notes: - I'm new to rust so I would appreciate some pair of eyes testing the code, plus I'm open for revision of code or general remarks about my coding style - I have run the tests both on OSX as on Linux (Ubuntu) - I've run `clippy` to see if my code is well formatted, however clippy issues many warnings about the project. I think I didn't add any more warnings - the methods in unistd are not documented so I also left out the documentation of `getcwd` and `mkdir`, although I think it'd probably be good to add some documentation, especially some example code how to use the methods - the base idea of `getcwd` is [taken from std](https://github.com/rust-lang/rust/blob/1.9.0/src/libstd/sys/unix/os.rs#L95-L119), should I mention that somewhere?
No description provided.