Skip to content
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

Fix compilation on ARM (for Rust 1.5) #33

Merged
merged 1 commit into from
Feb 4, 2016
Merged

Fix compilation on ARM (for Rust 1.5) #33

merged 1 commit into from
Feb 4, 2016

Conversation

MagaTailor
Copy link

No description provided.

hannobraun added a commit that referenced this pull request Feb 4, 2016
Fix compilation on ARM (for Rust 1.5)
@hannobraun hannobraun merged commit 638cd67 into hannobraun:master Feb 4, 2016
@hannobraun
Copy link
Owner

I currently have no way to test on ARM, but I trust that this is correct.

Thanks a lot!

@MagaTailor
Copy link
Author

No worries, I've been tracking (and fixing from time to time) this issue since October.

Otherwise, you'd get this error:

Compiling inotify v0.2.2 (file:///tmp/inotify-rs-master)
src/wrapper.rs:60:17: 60:31 error: mismatched types:
 expected `*const u8`,
    found `*const i8`
(expected u8,
    found i8) [E0308]
src/wrapper.rs:60                 c_str.as_ptr(),
                                  ^~~~~~~~~~~~~~
src/wrapper.rs:60:17: 60:31 help: run `rustc --explain E0308` to see a detailed explanation
error: aborting due to previous error
Could not compile `inotify`.

@hannobraun
Copy link
Owner

I'm not that worried, or I wouldn't have merged :)

Thanks for the explanation though!

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.

None yet

3 participants