Closed ZhongRuoyu closed 9 months ago
Update: This is reproducible on Fedora 38 (x86_64), which ships Rust 1.72.0.
This fails with the recursion error:
docker run --rm -it fedora:38 sh -c "
dnf install -y rust cargo git pkgconf-pkg-config libsodium libsodium-devel sqlite-libs sqlite-devel libseccomp libseccomp-devel
git clone https://github.com/kpcyrd/sn0int.git
cd sn0int
git checkout v0.25.0
cargo build --release --verbose -p sn0int
"
But this works:
docker run --rm -it fedora:38 sh -c "
dnf install -y rust cargo git pkgconf-pkg-config libsodium libsodium-devel sqlite-libs sqlite-devel libseccomp libseccomp-devel
git clone https://github.com/kpcyrd/sn0int.git
cd sn0int
git checkout v0.25.0
sed -i 's/geo = \"0.23\"/geo = \"0.25\"/' sn0int-std/Cargo.toml
cargo update -p geo
cargo build --release --verbose -p sn0int
"
Sorry for the delay, I've released a new version.
Thanks for the notification!
No worries; thank you, @kpcyrd!
Hello (again)! While packaging Rust for Homebrew at Homebrew/homebrew-core#140367, we discovered that sn0int 0.25.0 (precisely, its dependency
geo
0.23.1) failed to build on x86_64 macOS with Rust 1.72.0, due to the following error (logs taken from CI run):This error only occurred on x86_64 macOS; not on arm64 macOS or x86_64 Linux. Interestingly, when building with the
rustup
-provided toolchain (rustc 1.72.0 (5680fa18f 2023-08-23)
), the build finished fine. I am unsure about the underlying reason, though.Nevertheless, I believe this is an issue with the
geo
dependency, as discussed in georust/geo#1010, and has been fixed bygeo
0.25. When I bumped thegeo
version insn0int-std/Cargo.toml
to 0.25, the build was successful. In that regard, would you consider updating thegeo
dependency?Thanks!
Versions
rustc 1.72.0 (5680fa18f 2023-08-23) (built from a source tarball)
cargo 1.72.0 (26bba4830 2023-08-26)
sn0int 0.25.0
Darwin 88665a5040f1 22.6.0 Darwin Kernel Version 22.6.0: Wed Jul 5 22:21:56 PDT 2023; root:xnu-8796.141.3~6/RELEASE_X86_64 x86_64
Environment