rust3ds / ctru-rs

Rust wrapper for libctru
https://rust3ds.github.io/ctru-rs/
Other
116 stars 17 forks source link

[rust target] `std` fails to compile on latest nightly (2024-03-20) #173

Closed patataofcourse closed 4 months ago

patataofcourse commented 4 months ago

Not sure if I should report this here or in the main Rust repo, please let me know how to go about this stuff in the future.

Upgraded from 2024-02-18, project immediately failed to compile.

error[E0308]: mismatched types
   --> /home/patat-mini/.rustup/toolchains/nightly-x86_64-unknown-linux-gnu/lib/rustlib/src/rust/library/std/src/sys/pal/unix/fs.rs:520:29
    |
520 |         Ok(SystemTime::from(self.stat.st_mtim))
    |            ---------------- ^^^^^^^^^^^^^^^^^ expected `SystemTime`, found `timespec`
    |            |
    |            arguments to this function are incorrect
    |
note: associated function defined here
   --> /home/patat-mini/.rustup/toolchains/nightly-x86_64-unknown-linux-gnu/lib/rustlib/src/rust/library/core/src/convert/mod.rs:585:8
    |
585 |     fn from(value: T) -> Self;
    |        ^^^^

error[E0308]: mismatched types
   --> /home/patat-mini/.rustup/toolchains/nightly-x86_64-unknown-linux-gnu/lib/rustlib/src/rust/library/std/src/sys/pal/unix/fs.rs:548:29
    |
548 |         Ok(SystemTime::from(self.stat.st_atim))
    |            ---------------- ^^^^^^^^^^^^^^^^^ expected `SystemTime`, found `timespec`
    |            |
    |            arguments to this function are incorrect
    |
note: associated function defined here
   --> /home/patat-mini/.rustup/toolchains/nightly-x86_64-unknown-linux-gnu/lib/rustlib/src/rust/library/core/src/convert/mod.rs:585:8
    |
585 |     fn from(value: T) -> Self;
    |        ^^^^
ian-h-chamberlain commented 4 months ago

Hi, thanks for the report! In the future issues with compilation are probably best suited to be reported at https://github.com/rust-lang/rust but you can definitely tag us on those (unfortunately I don't think cross-org tagging works so e.g. like @rust3ds/active only works in this repo).

Seems related to https://github.com/rust-lang/rust/pull/123057 so perhaps the fix for that will work for us as well, I'll leave a comment on that PR.

ian-h-chamberlain commented 4 months ago

Believe this is fixed as https://github.com/rust-lang/rust/pull/123057 is merged