ReagentX / imessage-exporter

Export iMessage data + run iMessage Diagnostics
GNU General Public License v3.0
3.08k stars 135 forks source link

fix build.rs #351

Closed freeferg closed 1 month ago

freeferg commented 1 month ago

build.rs was causing the a crash during the installation with cargo :

error[E0432]: unresolved import std::fs::exists --> imessage-database/build.rs:3:16 | 3 | fs::{copy, exists}, | ^^^^^^ no exists i

ReagentX commented 1 month ago

On what platform and cargo version does this crash? I am able to install and run imessage-exporter on Windows and MacOS via cargo 1.81.0 (2dbb1af80 2024-08-20) without this issue. The import definitely exists as well:

freeferg commented 1 month ago

It crashes on MacOS Sequioa 15.0 with cargo 1.80.0.

ReagentX commented 1 month ago

I am unable to replicate this issue using the same platform and cargo version. Can you post the full compiler error?

The import is definitely valid, as you can see from this playground.

ReagentX commented 1 month ago

Closing this pull request.

Upon review, the issue this PR claims to address does not appear to actually exist in imessage-exporter.

If you believe this is in error, please provide additional context or explain the specific issue this PR is meant to address. Otherwise, feel free to open a new PR/issue if you've identified a legitimate problem that needs solving.

Thank you for your contribution.

meap02 commented 1 month ago

Hello! I'm getting the same issue on Arch Linux 6.11.3 with cargo version 1.78.0. Here is my compilation output:

Compiling imessage-database v2.1.1
error[E0432]: unresolved import `std::fs::exists`
 --> /home/meap/.cargo/registry/src/index.crates.io-6f17d22bba15001f/imessage-database-2.1.1/build.rs:3:16
  |
3 |     fs::{copy, exists},
  |                ^^^^^^ no `exists` in `fs`

For more information about this error, try `rustc --explain E0432`.
error: could not compile `imessage-database` (build script) due to 1 previous error
ReagentX commented 1 month ago

Use a newer version of Rust: https://doc.rust-lang.org/src/std/fs.rs.html#2752

meap02 commented 1 month ago

Ah I was not aware that rustup needed to be updated outside of pacman, thank you!