wayland-transpositor / wprs

Apache License 2.0
300 stars 8 forks source link

build failure on dependency divbuf #33

Closed khimaros closed 4 months ago

khimaros commented 4 months ago

here's the build output:

$ cargo build --profile=release-lto
    Updating crates.io index
    Updating git repository `https://github.com/nicolasavru/divbuf.git`
error: failed to get `divbuf` as a dependency of package `wprs v0.1.0 (/home/pawalls/src/github.com/wayland-transpositor/wprs)`

Caused by:
  failed to load source for dependency `divbuf`

Caused by:
  Unable to update https://github.com/nicolasavru/divbuf.git?branch=extract-vec#2976d292

Caused by:
  object not found - no match for id (2976d292a2d87660ab03d7773537b4a5102d23d8); class=Odb (9); code=NotFound (-3)
nicolasavru commented 4 months ago

Weird, it builds fine for me with a clean checkout. Any chance you have some weird cargo config?

The patch wprs needs from divbuf was recently upstreamed (https://github.com/asomers/divbuf), so we can drop the git dependency once another release of divbuf happens.

nicolasavru commented 4 months ago

Some googling suggests clearing cargo cache may help with this.

khimaros commented 4 months ago

i cleared ~/.cargo/ and tried again, same error.

i'm running rust-1.70.0

khimaros commented 4 months ago

i think maybe there was a force-push to the divbuf branch?

it seems that the commit hash no longer matches.

cargo update seemed to fix the lock file.

nicolasavru commented 4 months ago

Ah, I have it pointing to the branch, not a specific commit. Yeah, might be it. Either way, I'll change the dep to be the upstream git repo until divbuf is released.

nicolasavru commented 4 months ago

Heh, by which I mean maxhbooth will get to it first in #34.

khimaros commented 4 months ago

interesting.. it looks like switching to rust@1.74.1 also fixed the error i was seeing with the commit hash. that's odd. i was previously on rust-1.70.0 which ended up being too old for one of the other deps as well. now everything seems to build cleanly. sorry for the noise. it looks like mise doesn't pick up the toolchain in rust-toolchain.toml

maxhbooth commented 4 months ago

Should be fixed now :)

maxhbooth commented 4 months ago

(if the toolchain didn't mysteriously fix it anyway.)