-
Notifications
You must be signed in to change notification settings - Fork 43
Version 0.2.0 doesn't compile, missing sys.rs? #10
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
Comments
There is no Can you give me more info about your platform, rustc version, etc.? I haven't been able to reproduce that error, whether on my machine or on CI. |
Ah hah, interesting.
Looks like I'm using rustc 1.10.0 (cfcb716cf 2016-07-03) . I'm using Ubuntu
16.04, 64-bit. Perhaps my rustc is too old? Looks like 1.13.0 is the latest
one. I'll try that out...
…On Mon, Dec 5, 2016 at 2:36 PM, Chris Wong ***@***.***> wrote:
There is no sys.rs, because the code uses the #[cfg_attr]
<https://doc.rust-lang.org/book/conditional-compilation.html#cfg_attr>
attribute to select either unix.rs or windows.rs depending on the
platform.
Can you give me more info about your platform, rustc version, etc.? I
haven't been able to reproduce that error, whether on my machine or on CI.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#10 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AAXV8X4h0gwqm0pI1OVYIdyMYevdNGL3ks5rFJIGgaJpZM4LEwMI>
.
|
Aha! It's probably rust-lang/rust#25544, which was fixed after 1.10 was released. If the issue is what I think it is, then upgrading should fix the problem. |
I did some testing. Rust 1.12 accepts the code without error, but Rust 1.11 fails with the error you describe. I'll refactor the code so that it works on both. |
Released as 0.2.1. Thanks for reporting! |
Thank you for your help! :)
- David
…On Tue, Dec 6, 2016 at 12:12 AM, Chris Wong ***@***.***> wrote:
Released as 0.2.1 <https://crates.io/crates/errno/0.2.1>. Thanks for
reporting!
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#10 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AAXV8TW4euhPx8xhXKqQCEqWq-bYuvMwks5rFRjagaJpZM4LEwMI>
.
|
Commit 45c353d introduces a line in lib.rs:9 "mod sys;", but there is no sys.rs or sys/mod.rs committed. Is this a missing file?
Related: When I try to compile errno 0.2.0, I get:
/home/dko/.cargo/registry/src/github.com-1ecc6299db9ec823/errno-0.2.0/src/lib.rs:9:5: 9:8 error: file not found for module
sys
/home/dko/.cargo/registry/src/github.com-1ecc6299db9ec823/errno-0.2.0/src/lib.rs:9 mod sys;
^~~
/home/dko/.cargo/registry/src/github.com-1ecc6299db9ec823/errno-0.2.0/src/lib.rs:9:5: 9:8 help: name the file either sys.rs or sys/mod.rs inside the directory "/home/dko/.cargo/registry/src/github.com-1ecc6299db9ec823/errno-0.2.0/src"
error: aborting due to previous error
Build failed, waiting for other jobs to finish...
error: Could not compile
errno
.To learn more, run the command again with --verbose.
The text was updated successfully, but these errors were encountered: